Node.js async.each throw "コールバックは既に呼び出されています"

Question

この問題は他の多くのものと同じように見えるかもしれませんが、私は間違いを見つけることができません。

問題は、async.eachが "コールバックが既に呼び出されています"ということです。ここでスニペットが（それは私のコード内の他のコールバックと混乱しないように、私は、非同期コールバックdoneの名前）です。

async.each(this.requirements, (requirement, done) => { 
    // That thing here passes the result as a callback 
    requirement.callback((result) => { 
    if (!result) { 
     // requirement not passed -> return error 
     done(true); // LINE 42 
    } else { 
     done(); // LINE 44 
    } 
    }, data, params, bot); 
}, (err) => { // 'done' callback 
    log.info('handler', 
    `Handler '${this.label}' ${err ? 'failed' : 'succeeded'}`); 
    // if any requirement did not pass, do not execute handler callback 
    if (!err) this.callback(data, params, bot); 
}); 

ここでのスタックトレースです：

C:\Users\samuel\Code\node\sk22tgjs\node_modules\async\dist\async.js:837 
      if (fn === null) throw new Error("Callback was already called."); 
         ^

Error: Callback was already called. 
    at C:\Users\samuel\Code\node\sk22tgjs\node_modules\async\dist\async.js:837:34 
    at requirement.callback (C:\Users\samuel\Code\node\sk22tgjs\node_modules\telegramjs\core\handler.js:44:11) 
    at Requirement.exports.command.Requirement.callback (C:\Users\samuel\Code\node\sk22tgjs\node_modules\telegramjs\telegram\requires.js:21:5) 
    at async.each (C:\Users\samuel\Code\node\sk22tgjs\node_modules\telegramjs\core\handler.js:39:19) 
    at C:\Users\samuel\Code\node\sk22tgjs\node_modules\async\dist\async.js:2953:18 
    at replenish (C:\Users\samuel\Code\node\sk22tgjs\node_modules\async\dist\async.js:872:19) 
    at C:\Users\samuel\Code\node\sk22tgjs\node_modules\async\dist\async.js:878:27 
    at C:\Users\samuel\Code\node\sk22tgjs\node_modules\async\dist\async.js:840:18 
    at requirement.callback (C:\Users\samuel\Code\node\sk22tgjs\node_modules\telegramjs\core\handler.js:44:11) 
    at Requirement.callback (C:\Users\samuel\Code\node\sk22tgjs\node_modules\telegramjs\core\requires.js:19:5) 

面白いです問題はdone(true)が呼び出された場合にのみ発生するということです。事前にhttps://github.com/22sk/telegramjs

ありがとう：それにもかかわらず、エラーはまた、GitHubの上で破損したコード、特にtestingブランチを見ることができない42

、44行で発生します。

Answer 1

コマンドが見つからない場合、失敗ではなく成功と失敗の両方が呼び出されています。二回呼ばれ得ることから `result`を防ぐために

https://github.com/22sk/telegramjs/blob/5b85f04fe890a8fd32b373edb97bfebc923156b1/bot/telegram/requires.js

exports.command = new Requirement({ 
    label: 'command', 
    requires: requires.has('message', 'text'), 
    callback: (result, data, params, bot) => { 
    const command = new Command(data.message.text); 
    if (!command.valid || command.bot && bot.me.username !== command.bot) { 
     // command is not valid or not meant to be handled by this bot 
     result(false); // <--- ### Missing return ### ---> 
    } 
    // command is valid and should be handled by this bot 
    // write command into data 
    params.command = command; 
    result(true); 
    } 
});