0
async.forEachOf(allFlights, function (flight, key, callback) {
    Airport.findOne({_id: flight.airportFrom}, function(err, airport) {
        if (err) return next(err);
        flight.airportFrom = airport; // Тут почему то не перезаписывает
        callback();
    });
}, function (err) {
    if (err) return next(err);
    res.json(allFlights);
});

2 ответа 2

1

А чем вам async.map не угодил? Можно же, например, так:

async.map(allFlights, function (flight, callback) {
    Airport.findOne({_id: flight.airportFrom}, function(err, airport) {
        if (err) {
            return callback(err);
        }
        flight.airportFrom = airport;
        callback(null, flight);
    });
}, function (err, flights) {
    if (err) {
        return next(err);
    }
    res.json(flights);
});
-1

Попробуйте так

function array_async_action(input,action,callback,resolve_all){
    var promises =[];
    for(var i in input){
        promises.push(new Promise(function(resolve,reject){
            var key=this;
            action(input[key],function(err,res){
                if(err && !resolve_all){
                    reject(err);
                }else{
                    resolve(res||err);
                }
            });
        }.bind(i)))
    }
    Promise.all(promises).then(function(res){
        callback(null,res);
    }).catch(callback);
}

array_async_action(allFlights,function(flight,callback){
    Airport.findOne({_id: flight.airportFrom}, function(err, airport) {
        if (err) return next(err);
        flight.airportFrom = airport; // Тут почему то не перезаписывает
        callback(null,flight);
    });
},function(err,allFlights){
    //
    if (err) return next(err);
    res.json(allFlights);
});
11
  • Вы повторили async.map на Promise. Зачем?) 28 ноя 2015 в 11:39
  • Потому что 1) не слыхал про async 2) Не люблю лишние зависимости. Считаю, что подтягивать целый модуль/библиотеку из за одной фичи, которую можно реализовать в 10 строк - удел дилетантов, которые не могут написать эти 10 строк 3) Вследствие 2 имею набор сниппетов, которые реализуют подобные фичи. Я мог бы сделать свою бибилотеку, но см. пункт 2
    – Darth
    28 ноя 2015 в 11:57
  • Вот когда вы каждый из своих "снипетов" покроете хотя бы половиной тестов, используемых в сторонних решениях, вот тогда и будете рассуждать про дилетантов. Да и подход с повторным использованием кода не просто так придумали и пропагандируют. 28 ноя 2015 в 12:05
  • Вы кажется не в курсе, что (юнит)тесты пишутся вместе с кодом, и что "тесты, используемые в сторонних решениях" покрывают только эти самые сторонние решения.
    – Darth
    28 ноя 2015 в 12:09
  • Эти тесты гарантируют, что сторонние решения работают. Так же как и комьюнити, использующее эти решения. А вот при использовании самописных велосипедов такой гарантии нет ;) 28 ноя 2015 в 12:10

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.