重构：从值或现有承诺返回承诺

本文介绍了重构：从值或现有承诺返回承诺的处理方法，对大家解决问题具有一定的参考价值，需要的朋友们下面随着小编来一起学习吧！ 问题描述

我曾经使用完成了一些 node.js 的实现回调但我现在正在重构我的代码以使用 Promises - 使用 Q 模块。我有以下 update（）函数，其中内部 _update（）函数已经返回承诺：

I used to have some node.js implementation done using callbacks but I am now refactoring my code to use Promises instead - using Q module. I have the following update() function where the inner _update() function already returns a Promise:

exports.update = function(id, template, callback) { if (!_isValid(template)){ return callback(new Error('Invalid data', Error.INVALID_DATA)); } _update(id, template) // this already returns a promise .then(function() { console.log('UPDATE was OK!'); callback(); }, function(err) { console.log('UPDATE with ERRORs!'); callback(err); }); };

我的问题

我想要实现类似以下内容：

exports.update = function(id, template) { if (!_isValid(template)){ // how could I make it return a valid Promise Error? return reject(new Error('Invalid data', Error.INVALID_DATA)); } return _update(id, template) // return the promise .done(); };

因为 _update（）已经返回承诺，我想用这种方式改变它就足够了（不会？）：

Because _update() already returns a promise, I guess changing it this way would be enough (wouldn't be?):

return _update(id, template) .done();

如果条件在 if子句中等于 true ？我怎么能重构

And... what about if the condition inside the if-clause equals true? How could I refactor

返回回调（新错误（'无效数据'，BaboonError.INVALID_DATA））;

抛出错误以避免将回调传递给 update（）并处理该错误（或者错误可能会返回 _update（））？

to throw an error to avoid passing the callback into update() and handling that error (or what ever error could ever be returning _update())?

此外，致电更新（）：

myModule.update(someId, someTemplate) .then(function() { /* if the promise returned ok, let's do something */ }) .catch(function(err) { /* wish to handle errors here if there was any */});

我的代码中的其他地方：

somewhere else in my code:

• 如果承诺传播期间出错 - 它应该处理它，
• 或者，如果有这不是一个错误 - 它应该做一些其他的事情

• if there is an error during the promise propagation - it should handle it,
• or, if there wasn't an error - it should do some other things

我接近我所期待的吗？我怎么能最终实现呢？

Am I close to what I am expecting? How could I finally achieve it?

推荐答案

我只看到两个问题。

如果您想显式返回带有值的被拒绝的承诺，您应该使用 Q.reject 。

致电 .done（） 意味着承诺在那里结束。它不能被进一步链接。

Calling .done() on promise means that the promise ends there. It cannot be chained further.

所以，你的代码看起来像这样

So, your code would look like this

exports.update = function (id, template) { if (!_isValid(template)) { return Q.reject(new Error('Invalid data', Error.INVALID_DATA)); } return _update(id, template); };

现在，更新函数只返回一个永远承诺。由呼叫者连接成功或失败处理程序。

Now, the update function just returns a promise always. Its up to the callers to attach the success or failure handlers to it.