node.jsの非同期関数の再帰呼び出し

Question

データベースレベルでデータを更新する1つの再帰クエリを作成しました。

私はこのコードを書かれている：

let updateInvitationRewards = function(parent, level, callback) { 

    Service.customer.getOneCustomer({ 
    _id: parent 
    }, {}, {}, function(err, data) { 

    console.log(data.email); 
    level--; 
    if (data.parent) { 
     parent = data.parent.id; 
    } else if (level == 0 || data.parent == null) 
     callback("end"); 

    updateInvitationRewards(parent, level--, callback); 
    }); 

}; 

updateInvitationRewards(parent, level, function(string) { 
    console.log(string); 
    cb(); 
}); 

このクエリは、このようなエラーを与えている：

Error: Callback was already called. 

私はコールバックのこれらの種類に新しいです、私を助けてください。すべてのブラケット付き

Answer 1

、我々は持っている：

if (data.parent) { 
    parent = data.parent.id; 
} else if (level == 0 || data.parent == null) { 
    callback("end"); 
} 

updateInvitationRewards(parent, level--, callback); 

levelが0またはdata.parent nullのに等しい場合、コールバックは、その後、トリガ、同じコールバックを持つupdateInvitationRewardsです。

それを修正するには、次のコールバックは、一度だけ呼ばれるべき任意の関数で

if (data.parent) { 
    parent = data.parent.id; 
} else if (level == 0 || data.parent == null) { 
    return callback("end"); // the execution stops here 
} 

updateInvitationRewards(parent, level--, callback); 

Answer 2

を。コールバックが複数回呼び出された場合、Error: Callback was already called.が来るでしょう。

コードサンプルから、関数が再帰的に呼び出されることがわかります。したがって、レベルが0になるか、親がnullのときにエラーが表示される可能性があります。

次のシナリオを考えてください。レベルがnullの場合、またはコールバックが初めて呼び出されるときに親がnullの場合。しかし、再帰呼び出しは引き続き発生します。次回の実行で、親がnullの場合、2回目にコールバックが呼び出され、エラーがスローされます。

レベルからの使用方法はわかりませんでしたが、次のコードを実装することをお勧めします。これにより、コールバックが次回呼び出されないことが保証されます。

let updateInvitationRewards = function(parent, level, callback) { 

    Service.customer.getOneCustomer({ 
     _id: parent 
    }, {}, {}, function(err, data) { 
     console.log(data.email); 
     level--; 
     if (data.parent) { 
      parent = data.parent.id; 
      updateInvitationRewards(parent, level--, callback); 
     } else if (level == 0 || data.parent == null) 
      callback("end"); 
    }); 
}; 

updateInvitationRewards(parent, level, function(string) { 
    console.log(string); 
    cb(); 
});