執行承諾遞歸的NodeJS

Question

以下功能通過XML-RPC我的服務器上創建新的文件夾

var createFolder = function(folder_name) { 
    var defer = Q.defer(); 
    client.methodCall('create_folder', [sessionID, folder_name], function(err, resp) { 
    if (err) { 
     if (err.responseString && err.responseString.match('already exist')) { 
     //call the same function recursively with folder_name+Math.round(Math.random()*100) 
     } else { 
     defer.reject(err); 
     } 
    } else { 
     defer.resolve(folder_name); 
    } 
    }); 
    return defer.promise; 
} 

功能成功地創建一個新的文件夾 但是，如果文件夾已經存在，我想觸發此功能再次遞歸地與新的文件夾命名並返回它的承諾，以便每當這個函數被調用，它會返回文件夾名稱並不重要是如何執行的次數

像

createFolder('directory').then(function(resp){ 
console.log(resp);// may return directory || directory1 .... etc 
}); 

**編輯** 所以我manged通過將延遲對象來實現這一 讓我知道是否有實現這一

var createFolder = function(folder_name,defer) { 
    defer =defer || Q.defer(); 
    client.methodCall('create_folder', [sessionID, folder_name], function(err, resp) { 
    if (err) { 
     if (err.responseString && err.responseString.match('already exist')) { 
     return createFolder(folder_name+Math.round(Math.random()*100,defer) 
     } else { 
     defer.reject(err); 
     } 
    } else { 
     defer.resolve(folder_name); 
    } 
    }); 
    return defer.promise; 
} 

Answer 1

這裏的更優雅的方式是解決的壞簡單的方法你的問題：

var createFolder = function(folder_name) { 
    var defer = Q.defer(); 
    client.methodCall('create_folder', [sessionID, folder_name], function(err, resp) { 
    if (err) { 
     if (err.responseString && err.responseString.match('already exist')) { 
     //call the same function recursively with folder_name+Math.round(Math.random()*100) 
     defer.resolve(createFolder(folder_name+Math.round(Math.random()*100))); 
     } else { 
     defer.reject(err); 
     } 
    } else { 
     defer.resolve(folder_name); 
    } 
    }); 
    return defer.promise; 
} 

然而，defer被認爲是不好的做法。這是一個very nice article about promises。

你應該贊成這樣的：

var createFolder = function(folder_name) { 
    return Q.Promise(function(resolve, reject){ 
    client.methodCall('create_folder', [sessionID, folder_name], function(err, resp) { 
     if (err) { 
      if (err.responseString && err.responseString.match('already exist')) { 
      //call the same function recursively with folder_name+Math.round(Math.random()*100) 
      resolve(createFolder(folder_name+Math.round(Math.random()*100))); 
      } else { 
      reject(err); 
      } 
     } else { 
      resolve(folder_name); 
     } 
     }); 
    }); 
} 

編輯：由@Bergi指出，這仍然是不對的，難以調試。從methodCall的回調中拋出的任何潛在錯誤都不會實際上拒絕承諾並很可能會被吞下（儘管此回調似乎很少有錯誤傾向，但可能會發生變化）。請參閱his answer以獲取更好的方法。請參閱the official Q doc here。

Answer 2

絕對不要在普通（非承諾）回調中做任何邏輯。 Promisify在最低水平：

var defer = Q.defer(); 
client.methodCall('create_folder', [sessionID, folder_name], function(err, resp) { 
    if (err) defer.reject(err); 
    else defer.resolve(folder_name); 
}); 
return defer.promise; 

或者更加簡單與Q.ninvoke：

return Q.ninvoke(client, 'methodCall', 'create_folder', [sessionID, folder_name]); 

---

現在我們可以開始實施我們的遞歸。這是一個很簡單的回調，您可以從中返回另一個承諾then。在你的情況下：

function createFolder(folder_name) { 
    return Q.ninvoke(client, 'methodCall', 'create_folder', [sessionID, folder_name]) 
    .catch(function(err) { 
     if (err.responseString && err.responseString.match('already exist')) { 
     return createFolder(folder_name+Math.floor(Math.random()*100)); 
     } else { 
     throw err; 
     } 
    }); 
}