async/awaitでasync functionのfunctionに対してunexpected tokenのエラーがでる

Question

firebaseを利用したサーバサイドの開発をしており、非同期処理を同期的に直す過程でエラーが出ました。

データをajaxで受け取る→それをfirebaseのデータベースと突合→合致する場合はresponse(500)を帰す
というバックエンドの処理をしようとしています。

「データを格納→完了後に突合」としたく、async/awaitの非同期処理の最後の例にならって、
promise,async,awaitを用いた非同期処理の待ちを実装しましたが、下記のエラーが出ます。

65:9 error Parsing error: Unexpected token function

/////////////////////////////////////////index.js//////////////////////////////////

const functions = require('firebase-functions');
// // Create and Deploy Your First Cloud Functions
// // https://firebase.google.com/docs/functions/write-firebase-functions
// The Firebase Admin SDK to access the Firebase Realtime Database.

// Import Admin SDK
const admin = require('firebase-admin');
admin.initializeApp(functions.config().firebase);

// Get a database reference to our posts
var db = admin.database();
var ref  = db.ref("/product");

//chromeExtensionからのデータを受信
exports.product = functions.https.onRequest((request, response) => {
    //POSTを受けると発火
    if (request.method === "POST") {
        const body = request.body;
        const pushRef = admin.database().ref("/product").push();
        pushRef.set({
          //薬処方時のデータ
          patientId: body["patientId"],
          doctor:    body["doctor"],
          idisease:  body["idisease"],
          medicine:  body["medicine"],
          medicineID:body["medicineID"],
        }, error => {
            if (error) {
                console.log("save error", error.message);
            } else {
                console.log("save Success!!");
            }
        }
      );
  }//ajaxの受け取り

////////////////////////////////////////////////////////////////////　　問題個所はここ以降
  //medicineIDを薬マスタと突合

  //薬のIDを配列に格納
  function getInMedId() {
    const p = new Promise((resolve,reject) => {
      var inMedId = body.medicineID;
      resolve(inMedId);
    });
    return p;
  }

  //マスター側の薬のIDを配列に格納
  function getPreMedId() {
    const q = new Promise((resolve,reject) => {
       var preMedId = [];
       for(j=0; j<(90+1); j++){
         var m = db.ref("/"+j+"/medicineId");
         m.on("value", function(snapshot){
           preMedId = snapshot.val();
         });
       }
       resolve(preMedId);
     });
     return q;
   }

  //薬のIDを突合
  async function checkMedId() {         <============================================ ※エラー箇所※
   for(i=0; i<inMedId.length+1; i++){
     for(j=0; j<(90+1); j++){
       var a = await getInMedId();
       var b = await getPreMedId();
       if(a[0][i] === b[j]){
         console.log('This medicine is skeptical:'+a);
         response.status(500).send(error.message).end();
       }
     }
   }
 }
 checkMedId();
  //for回して無事だったら普通のレスポンス
 response.status(200).end();

})

Answer 1

というか、そもそも構文エラーで質問する以前の問題です。
インデントもぐちゃぐちゃなのでまずはESLint等を導入してエラーを取り除いてください。
正しく括弧や波括弧の整合性があってるのか確認してください。

もしコードが正しければNode.jsのバージョンが古い可能性があります。
http://node.green/#ES2017-features-async-functions

async / awaitはバージョン7.10以降に実装された書き方ですので、
下記のコマンドでNode.jsのバージョンを確認し、古ければ新しいバージョンをインストールしてください。
基本的にECMAScriptは下位互換を強く意識しているので、酷いバグでも無い限り出来るだけ最新を入れたほうが良いです。

$ node -v

---

さて、コードが動くようになれば恐らく次の問題が出てくるでしょう。
なので先受けして書いておきます。
（今回のエラーはあくまで構文エラーなので、asyncがどうのとかそういう以前の問題です）

async function () {}を同期処理と勘違いしていませんか？
この形式で書かれた場合、必ずPromiseが戻り値になります。

もしreturn 123;というに正常終了していれば、resolve(123)が実行された事になります。
throw new Error('エラー理由')という風に例外を投げれば、reject(new Error('エラー理由'))が実行された事になります。

つまり、構文の上で取り繕うだけで、
いくら取り繕ったとしても非同期は非同期であり、単なるPromiseです。

---

これを元にコードを見ていきましょう。

  //薬のIDを突合
  async function checkMedId() {
   for(i=0; i<inMedId.length+1; i++){
     for(j=0; j<(90+1); j++){
       var a = await getInMedId();
       var b = await getPreMedId();
       if(a[0][i] === b[j]){
         console.log('This medicine is skeptical:'+a);
         response.status(500).send(error.message).end();
       }
     }
   }
 }
 checkMedId();
  //for回して無事だったら普通のレスポンス
 response.status(200).end();

まずcheckMedId()が実行されます。
中身はPromiseな上二重ループの中で何度もawait getInMedId()等が呼ばれているので、
非同期でありその場での処理を諦めています。

なのでどうあがこうが先にresponse.status(200).end();が叩かれ、ステータスコード200が確約されます。
その後、checkMedId内の変数aとbが確定され、response.status(500).send(error.message).end();を実行しようとします。
それ以前にerrorが宣言されてませんが

これはExpressですか？
Expressはステータスコードを二重に宣言しようとしたり、.end後になにかを返そうとすればエラーになります。
こういった流れでエラーが発生しているようです。

ではどうすればよかったのか。
Promiseを叩いて受けるようにコードを修正しましょう。

  // 薬のIDを突合
  async function checkMedId() {
    for(i=0; i<inMedId.length+1; i++){
      for(j=0; j<(90+1); j++){
        var a = await getInMedId();
        var b = await getPreMedId();
        if(a[0][i] === b[j]){
          throw new Error('This medicine is skeptical:' + a)
        }
      }
    }
  }
  // 普通のPromiseだと思って対応する
  checkMedId()
    .then(() => {
      response.status(200).end();
    })
    .catch(err => {
      console.log(err.message);
      response.status(500).send(err.message).end();
    })

Comments

[TarouKotani]

もろもろありがとうございます。。。
バージョンを確認し、最後の通りに直しましたがやはり同じエラーが出ます。。

[miyabi-sun]

質問文を私のローカルで構文解析したところエラーは出ませんでした。
従って転記ミスの可能性が高いです。

またさらっとですがコードを一通り読みました。
・getPreMedIdの中身が明らかにおかしいです。
　DBに接続してpreMedIdよこせって言ってるのに、待たずに即返してるので絶対にUndefindが帰ります。
・for文で90回もループしてますが、これ本当に合ってますか？
　90回もループしつつその中で90回また固定値でループし始めるプログラムってかなりおかしいですよ。

Answer 2

firebaseと言ってもたくさんのプロダクトがありますが、コードを拝見するにFirebase Cloud Functionsのこととお見受けします。

で、現時点(2018.05.15)におけるFirebase Cloud FunctionsのNode環境はv.6.11.5となります。

ローカルがv7.10以降でも、firebaseクラウド環境がNode v.6.11.5ですのでasync/awaitは一工夫しないと使えません。そのためasyncキーワードでunexpected tokenエラーが発生したものと思われます。

---

それと、少々耳の痛い話かもしれませんが、仮にasync/awaitが使えたとしてもコードの内容に突っ込みどころが多く想定している動作をしないように思います。

面倒かもしれませんが要素技術ごとに分解して簡略化した最小コードを書いてみることをお勧めします。