1. ホーム
  2. javascript

[解決済み] await は非同期関数でのみ有効です。

2022-01-30 03:37:13

質問

でこのコードを書きました。 lib/helper.js

var myfunction = async function(x,y) {
   ....
   return [variableA, variableB]
}
exports.myfunction = myfunction;

で、それを別のファイルで使おうとすると

 var helper = require('./helper.js');   
 var start = function(a,b){
     ....
     const result = await helper.myfunction('test','test');
 }
 exports.start = start;

エラーが発生しました

awaitは非同期関数でのみ有効です。

何が問題なのか?

どのように解決するのですか?

を指すエラーではありません。 myfunction でなく start .

async function start() {
   ....

   const result = await helper.myfunction('test', 'test');
}


// My function
const myfunction = async function(x, y) {
  return [
    x,
    y,
  ];
}

// Start function
const start = async function(a, b) {
  const result = await myfunction('test', 'test');
  
  console.log(result);
}

// Call start
start();



この質問をきっかけに、以下を使った既知のアンチパターンについてアドバイスします。 await ということです。 return await .


WRONG

async function myfunction() {
  console.log('Inside of myfunction');
}

// Here we wait for the myfunction to finish
// and then returns a promise that'll be waited for aswell
// It's useless to wait the myfunction to finish before to return
// we can simply returns a promise that will be resolved later

// useless async here
async function start() {
  // useless await here
  return await myfunction();
}

// Call start
(async() => {
  console.log('before start');

  await start();
  
  console.log('after start');
})();


正解

async function myfunction() {
  console.log('Inside of myfunction');
}

// Here we wait for the myfunction to finish
// and then returns a promise that'll be waited for aswell
// It's useless to wait the myfunction to finish before to return
// we can simply returns a promise that will be resolved later

// Also point that we don't use async keyword on the function because
// we can simply returns the promise returned by myfunction
function start() {
  return myfunction();
}

// Call start
(async() => {
  console.log('before start');

  await start();
  
  console.log('after start');
})();


また、以下のような特殊なケースもあることを知っておいてください。 return await が正しくて重要です : (try/catchを使用)

return await`でパフォーマンスの懸念はありますか?