ネストされたasync.each： "エラー：コールバックは既に呼び出されていました"

Question

async.eachのネストされた問題が発生しています。なぜ私は正しくコールバックを配置しています。

checkDefaultOverlap: function(default_shifts, done) { 
    async.each(default_shifts, function(default_shift, next) { 
     var subarray = default_shifts.slice(default_shifts.indexOf(default_shift) + 1, default_shifts.length - 1); 
     async.each(subarray, function(default_shift2, next) { 
      default_shift.week_days.map(function(day1) { 
       default_shift2.week_days.map(function(day2) { 
        if (day1 === day2 && 
         default_shift.start <= default_shift2.end && default_shift2.start <= default_shift.end) 
         next({error: 'The shifts overlap!'}); 
       }); 
      }); 
      next(); 
     }, function(err) { 
      if (err) next(err); 
      else next(null); 
     }); 
    }, function(err) { 
     if (err) return done(err); 
     else return done(null); 
    }); 
    } 
} 

本当にありがとうございます。

Answer 1

条件が満たされた場合は、default_shiftをループした後に、再度next();を呼び出しています。

default_shift.week_days.map(function(day1) { 
      default_shift2.week_days.map(function(day2) { 
       if (condition) 
        next({error: 'The shifts overlap!'}); //The problem is here. 
      }); 
}); 
next(); //If shifts overlap, next was already called. 

簡単な方法は、フラグを追加して、シフトが重なった場合には2番目の次を無視することです。

var nextCalled = false; 
default_shift.week_days.map(function(day1) { 
    default_shift2.week_days.map(function(day2) { 
     if (condition && !nextCalled){ 
      next({error: 'The shifts overlap!'}); 
      nextCalled = true; 
     } 
    }); 
}); 

if(!nextCalled) 
    next();