nodejs loop async вызывает

Question

в этой функции nodejs, я создаю несколько записей в mongodb с использованием мангуста. счетчик подсчитывает, сколько.

export function createTimeslots(req, res){ 
    var startDate = req.body.startDate; 
    var Duration = req.body.Duration; 
    var repeat = req.body.repeat; 
    var counter=0; 

    for(var i=0; i<repeat; i++){ 
    var startTime = ... calculation 
    var endTime = ...calculation 

    var ts = { 
     startTime: startTime, 
     endTime: endTime, 
    }; 

    TimeSlot.create(ts, function(err, timeslot){ 
     counter = counter+1; 
    }); 
    } 

    res.status(200).json(counter + ' timeslots created'); 

} 

проблема заключается в том, что в результате всегда создается «0 временных интервалов». Я подозреваю, что функция create async, и к тому времени, когда это будет сделано, нам больше не нужно обновлять счетчик. Итак, как я могу считать эти записи? (я знаю, что я могу использовать счетчик Петли, но я планирую сделать валидации , а в некоторых случаях, чтобы не создавать запись. Спасибо :)

Answer 1

Promises поможет здесь:

export function createTimeslots(req, res){ 
    var startDate = req.body.startDate; 
    var Duration = req.body.Duration; 
    var repeat = req.body.repeat; 
    var counter=0; 

    var promises = []; 

    for(var i=0; i<repeat; i++){ 
    var startTime = ... calculation 
    var endTime = ...calculation 

    var ts = { 
     startTime: startTime, 
     endTime: endTime, 
    }; 

    promises.push(new Promise(function(resolve) { 
     TimeSlot.create(ts, function(err, timeslot) { 
     counter = counter+1; 
     resolve(); 
     }); 
    })); 
    } 

    Promise.all(promises).then(function() { 
     res.status(200).json(counter + ' timeslots created'); 
    }); 
} 

Это будет ждать завершения всех асинхронных операций и только затем продолжить и создать сообщение возврата.

Answer 2

счетчика устанавливается на 0 каждый раз, когда у позвонить createTimeslots, так что это модель varible

var counter=0; 
 
export function createTimeslots(req, res){ 
 
    var startDate = req.body.startDate; 
 
    var Duration = req.body.Duration; 
 
    var repeat = req.body.repeat; 
 
    
 
    for(var i=0; i<repeat; i++){ 
 
    var startTime = ... calculation 
 
    var endTime = ...calculation 
 

 
    var ts = { 
 
     startTime: startTime, 
 
     endTime: endTime, 
 
    }; 
 

 
    TimeSlot.create(ts, function(err, timeslot){ 
 
     counter = counter+1; 
 
    }); 
 
    } 
 
    
 
    res.status(200).json(counter + ' timeslots created'); 
 

 
}

Answer 3

Переместите счетчик вне тела обратного вызова async, который вы передаете методу создания TimeSlot.

export function createTimeslots(req, res){ 
    var startDate = req.body.startDate; 
    var Duration = req.body.Duration; 
    var repeat = req.body.repeat; 
    var counter=0; 


    for(var i=0; i<repeat; i++){ 
    var startTime = ... calculation 
    var endTime = ...calculation 

    var ts = { 
     startTime: startTime, 
     endTime: endTime, 
    }; 

    counter = counter+1; 

    TimeSlot.create(ts, function(err, timeslot){ 
     // do stuff with counter 

    }); 
    } 

    res.status(200).json(counter + ' timeslots created'); 

}