JQuery延遲解決其他延遲

Question

我有一個功能，推動來自其他功能，也解決承諾數組承諾。我只是想知道這個代碼是否可以。

這裏的主要功能是

/// <summary>Process the validation results (invalid IDR records will have the iop=4)</summary> 
this.processValidationResults = function() 
{ 
    var promises = []; 
    var count = (self.idrValidationList.length - 1); 

    $.each(self.idrValidationList, function (index, idrValidationItem) 
    { 
     _onProgress(self.utils.formatCounterMessage(index, count, 'Processing Validation Items')); 

     if (idrValidationItem.is_valid = 0) 
     { 
      //there is a problem with this IDR record 
      //update the idr_insp table 
      promises.push(self.updateInvalidEntity(self.configEntities.idr_insp, idrValidationItem.idr_insp_id)); 
      promises.push(self.updateInvalidChildren(self.configEntities.idr_insp, idrValidationItem.idr_insp_id)); 
     } 
     else 
     { 
      //push resolved promise 
      promise.push($.when()); 
     }  
    }); 

    return ($.when.apply($, promises)); 
} 

這裏是由上述功能

/// <summary>Update the invalid record, sets the IOP field to 4 [Cannot sync due to issue]</summary> 
/// <param name="entity" type="Object">GLobal entity definiton </param> 
/// <param name="tabletId" type="Int">Primary Key on the tablet to change</param> 
this.updateInvalidEnity = function (entity, tabletId) 
{ 
    //update the record with the new ID and IOP status 
    var updateSql = 'UPDATE ' + entity.name + ' SET iop=? WHERE ' + entity.key_field + '=?'; 

    //update the record 
    return (self.db.executeSql(updateSql, [4, tabletId])); 
} 

/// <summary>Update the invalid child records, sets the IOP field to 4 [Cannot sync due to issue]</summary> 
/// <param name="entity" type="Object">GLobal entity definiton </param> 
/// <param name="keyId" type="Int">Foreign Key on the tablet to change</param> 
this.updateInvalidChildren= function (parentEntity, keyId) 
{ 
    var promises = []; 
    $.each(parentEntity.child_entities, function (index, child) 
    { 
     var def = new $.Deferred(); 
     var updateSql = 'UPDATE ' + child.child_name + ' SET iop=? WHERE ' + child.key_field + '=?'; 

     promises.push(self.db.executeSql(updateSql, [4, keyId])); 
    }); 

    return ($.when.apply($, promises)); 
} 

而且所有上述方法是推動下面的遞延調用的函數。

/* Executes the sql statement with the parameters provided and returns a deffered jquery object */ 
this.executeSql = function (sql, params) 
{ 
    params = params || []; 

    var def = new $.Deferred(); 

    self.db.transaction(function (tx) 
    { 
     tx.executeSql(sql, params, function (itx, results)// On Success 
     { 
      // Resolve with the results and the transaction. 
      def.resolve(itx, results); 
     }, 
     function (etx, err)// On Error 
     { 
      // Reject with the error and the transaction. 
      def.reject(etx, err); 
     }); 
    }); 

    return (def.promise()); 
} 

這是鏈條的聲音？還沒有測試，但我認爲這是確定的。只是想一些其他的眼睛在這之前，我繼續......

Answer 1

這確實應該在Code Review，事實並非如此，但在這裏不用...

幾個意見：

• 書面，.processValidationResults()中的進度消息表示發出的請求，而不是收到的響應。因此它會直接跳到「計數」。

• 在.processValidationResults(),else { promise.push($.when()); }是不必要的。

• 在.updateInvalidChildren(),var def = $.Deferred()是不必要的。

• jQuery Deferred's resolve和reject方法是「可拆卸的」。在this.executeSql()中，您只需簡單地傳遞參數，就可以簡化tx.executeSql(...)調用tx.executeSql(sql, params, def.resolve, def.reject)。

• 假設self.db.executeSql執行AJAX（並且tx.executeSql()不會以某種方式對請求進行排隊），但代碼有可能在同時發生請求時很難打到服務器。確保它可以處理此代碼將生成的同時請求的數量。否則，採取措施使請求順序進行。