Очистка кода спагетта с обещаниями

Question

В одной из моих функций beforeSave мне нужно проверить ряд условий, прежде чем отвечать на них с успехом или ошибкой.

Проблема заключается в том, мой код кажется очень грязный, а иногда успех/ошибка не называется:

Parse.Cloud.beforeSave("Entry", function(request, response) { 
    var entry = request.object; 
    var contest = request.object.get("contest"); 

    entry.get("user").fetch().then(function(fetchedUser) { 
     contest.fetch().then(function(fetchedContest) { 
      if (fetchedUser.get("fundsAvailable") < fetchedContest.get("entryFee")) { 
       response.error('Insufficient Funds.'); 
      } else { 
       fetchedContest.get("timeSlot").fetch().then(function(fetchedTimeSlot) { 
        var now = new Date(); 
        if (fetchedTimeSlot.get("startDate") < now) { 
         response.error('This contest has already started.'); 
        } else { 
         contest.increment("entriesCount"); 
         contest.increment("entriesLimit", 0); //have to do this, otherwise entriesLimit is undefined in save callback (?) 

         contest.save().then(function(fetchedContest) { 
          if (contest.get("entriesCount") > contest.get("entriesLimit")) { 
           response.error('The contest is full.'); 
          } else { 
           response.success(); 
          } 
         }); 
        } 
       }); 
      } 
     }); 
    }); 
}); 

Я пытался узнать обещания навести порядок в этом, и здесь был мой (не удалось) попытка:

Parse.Cloud.beforeSave("Entry", function(request, response) { 
    var entry = request.object; 
    var contest = request.object.get("contest"); 

    entry.get("user").fetch().then(function(fetchedUser) { 
     contest.fetch().then(function(fetchedContest) { 
      if (fetchedUser.get("fundsAvailable") < fetchedContest.get("entryFee")) { 
       response.error('Insufficient Funds.'); 
      } 
      return fetchedContest; 
     }); 
    }).then(function(result) { 
     result.get("timeSlot").fetch().then(function(fetchedTimeSlot) { 
      var now = new Date(); 

      if (fetchedTimeSlot.get("startDate") < now) { 
       response.error('This contest has already started.'); 
      } 
     }); 
    }).then(function() { 
     contest.increment("entriesCount"); 
     contest.increment("entriesLimit", 0); 
     contest.save().then(function(fetchedContest) { 
      if (contest.get("entriesCount") > contest.get("entriesLimit")) { 
       response.error('The contest is full.'); 
      } 
     }); 
    }).then(function() { 
     response.success(); 
    }, function(error) { 
     response.error(error); 
    }); 
}); 

Любая помощь или примеры того, как использовать обещания для этого, будут высоко оценены. Ясно, что я не совсем понимаю, как они работают синтаксически.

Answer 1

Я немного почистил его, предварительно собрав выбранные переменные, используя цепочку обещаний. Это следует за правила пару стилей, которые делают его более удобным для чтения (для меня во всяком случае) ...

Parse.Cloud.beforeSave("Entry", function(request, response) { 
    var entry = request.object; 
    var contest = request.object.get("contest"); 

    var fetchedUser, fetchedContest; 
    var errorMessage; 

    entry.get("user").fetch().then(function(result) { 
     fetchedUser = result; 
     return contest.fetch(); 
    }).then(function(result) { 
     fetchedContest = result; 
     return fetchedContest.get("timeSlot").fetch(); 
    }).then(function(fetchedTimeSlot) { 
     // now we have all the variables we need to determine validity 
     var now = new Date(); 
     var hasSufficientFunds = fetchedUser.get("fundsAvailable") >= fetchedContest.get("entryFee"); 
     var contestNotStarted = fetchedTimeSlot.get("startDate") >= now; 
     if (hasSufficientFunds && contestNotStarted) { 
      contest.increment("entriesCount"); 
      contest.increment("entriesLimit", 0); //have to do this, otherwise entriesLimit is undefined in save callback (?) 
      return contest.save(); 
     } else { 
      errorMessage = (hasSufficientFunds)? 'This contest has already started.' : 'Insufficient Funds.'; 
      return null; 
     } 
    }).then(function(result) { 
     if (!result) { 
      response.error(errorMessage); 
     } else { 
      if (contest.get("entriesCount") > contest.get("entriesLimit")) { 
       response.error('The contest is full.'); 
      } else { 
       response.success(); 
      } 
     } 
    }, function(error) { 
     response.error(error); 
    }); 
}); 

Обратите внимание, как мы никогда не оставить response.anything() болтается, мы всегда ясно, что течет к следующему обещанию через возвращение.

Answer 2

Вы делаете новобранец ошибку, не возвращая обещаний внутри .then в то время как сцепление их, кроме того, что ваша цепь будет еще продолжаться даже тогда, когда вы звоните response.error(.., простейший способ разорвать цепь бросает ошибку, ваш код может быть плоской, как :

Parse.Cloud.beforeSave("Entry", (request, response) => { 
    var entry = request.object, 
    , contest = request.object.get("contest") 
    , fetchedUser 
    , fetchedContest; 

    entry.get("user").fetch() 
    .then(_fetchedUser => { 
     fetchedUser = _fetchedUser; 
     return contest.fetch(); 
    }).then(_fetchedContest => { 
     fetchedContest = _fetchedContest; 
     if (fetchedUser.get("fundsAvailable") < fetchedContest.get("entryFee")) 
     return Promise.reject('Insufficient Funds.');  
     return fetchedContest.get("timeSlot").fetch(); 
    }).then(fetchedTimeSlot => { 
     var now = new Date(); 
     if (fetchedTimeSlot.get("startDate") < now) 
     return Promise.reject('This contest has already started.');  
     contest.increment("entriesCount"); 
     contest.increment("entriesLimit", 0); //have to do this, otherwise entriesLimit is undefined in save callback (?) 
     return contest.save(); 
    }).then(fetchedContest => { 
     if (contest.get("entriesCount") > contest.get("entriesLimit")) 
     return Promise.reject('The contest is full.');   
     response.success();   
    }).catch(response.error.bind(response)); 

});