javascript – 结束Promise链

我正在遇到一个关于未终止的Promise链的Promise警告(‘一个承诺是在一个处理程序中创建的,但是没有从它返回’).我是Promises的新手,并怀疑我不应该使用非事件驱动的思维.但我不知道该怎么办.这都在nodejs项目中.

我正在与ZWave服务器交互以打开和关闭灯.该交互采取以下形式：向控制ZWave网络的服务器发出http请求.我正在使用Promises,因为通过http进行交互的异步性质.

在我的程序的一个级别,我定义了以下类方法：

ZWave.prototype.onOff = function (nodeNumber,turnOn) {
var self = this;
var level = turnOn ? 255 : 0;

return new Promise(function (resolve,reject) {
    self.requestAsync(sprintf('/Run/devices[%d].instances[0].commandClasses[0x20].Set(%d)',nodeNumber,level))
    .then(function (value) {
        resolve(value == 'null');
    })
    .catch(function (error) {
        reject(error);
    });
});

};

requestAsync类方法实际上处理与ZWave服务器的交互.从概念上讲,在onOff()中,我试图打开或关闭由this.nodeNumber标识的特定光,然后返回该请求的结果.

从Switch类方法调用onOff(),表示特定的灯光,如下所示：

this.onOff = function( turnOn ) {
    var state = turnOn ? 'ON' : 'OFF';

    var self = this;

    zWave.onOff( this.nodeNumber,turnOn )
    .then( function() {
        winston.info( sprintf( '%s: turned %s',self.displayName,state ) );
        return true;
    } )
    .catch( function( error ) {
        winston.info( sprintf( '%s: error while turning %s => %s',state,error ) );
        return false;
    } );
}

‘return true’和’return false’语句是我试图结束Promise链的尝试.但它不起作用,我仍然收到警告.

这让我觉得这是一个更普遍的问题的具体例子：你如何从基于Promise的模型转向传统的阻塞代码？

编辑

从评论中回答几个问题……

我正在使用蓝鸟.

zWave.onOff()返回一个promise.

Switch.onOff()和zWave.onOff()是不同类中的不同函数.

所有代码都是javascript.

编辑2

我相信我已经实现了jfriend00的建议,虽然我在zWave.onOff()函数中包含了一个.catch()处理程序,但我仍然得到相同的未处理的promise错误：

ZWave.prototype.onOff = function (nodeNumber,turnOn) {
    var self = this;
    var level = turnOn ? 255 : 0;

    return self.requestAsync( sprintf( '/Run/devices[%d].instances[0].commandClasses[0x20].Set(%d)',level ) )
        .then( function( value ) {
            resolve( value == 'null' );
        } )
        .catch( function( error ) {
            reject( error );
        } );
};

和

// function inside Switch class
this.onOff = function( turnOn ) {
    var state = turnOn ? 'ON' : 'OFF';
    var self = this;

    return zWave.onOff( this.nodeNumber,turnOn ).reflect()
        .then( function() {
            winston.info( sprintf( '%s: turned %s',state ) );
            return true;
        } )
       .catch( function( error ) {
            winston.info( sprintf( '%s: error while turning %s => %s',error ) );
            return false;
        } );
}

以下是警告的内容：

Warning: a promise was created in a handler but was not returned from it
at ZWave.requestAsync (/home/mark/XmasLights/zWaveRequest.js:19:12)
at ZWave.onOff (/home/mark/XmasLights/zWaveRequest.js:93:17)
at onOff (/home/mark/XmasLights/switch.js:42:22)
at updateCron (/home/mark/XmasLights/switch.js:80:18)
at dailyUpdate (/home/mark/XmasLights/app.js:196:21)
at /home/mark/XmasLights/app.js:134:58
at processImmediate [as _immediateCallback] (timers.js:383:17)

抱歉有关格式化警告的运行,我似乎无法获得stackoverflow来正确分隔行.

解决方法

根据Bluebird文档,当您在promise范围内创建promise时会发生此警告,但您不会从该Promise范围返回任何内容. Bluebird检测到你在promise处理程序中创建了一个promise,但是没有从该处理程序返回任何东西,当它应该链接到其他promises时,它可能是一个未链接的promise.您可以阅读他们对问题 here的看法.

在您公开的代码中,您没有显示任何类似Bluebird在其文档中显示的示例,但我猜这里有一个问题：

ZWave.prototype.onOff = function (nodeNumber,turnOn) {
    var self = this;
    var level = turnOn ? 255 : 0;
    return new Promise(function (resolve,reject) {
        self.requestAsync(sprintf('/Run/devices[%d].instances[0].commandClasses[0x20].Set(%d)',level)).then(function (value) {
            resolve(value == 'null');
        }).catch(function (error) {
            reject(error);
        });
    });
};

你最好避免反模式并做到这一点：

ZWave.prototype.onOff = function (nodeNumber,turnOn) {
    var level = turnOn ? 255 : 0;
    return this.requestAsync(sprintf('/Run/devices[%d].instances[0].commandClasses[0x20].Set(%d)',level)).then(function (value) {
        return(value == 'null');
    });
};

这里没有必要创建一个新的承诺,因为你已经拥有了一个可以返回的承诺.您可以阅读有关避免使用here的反模式类型的更多信息.

并且,另一个可能的问题是,您在此处创建返回承诺值,但不要从方法this.onOff返回它.你明确地做了一个返回true;或者返回false;来自你的.then()处理程序,但是你永远不会对该返回值做任何事情,因为没有后续的.then()处理程序,并且不会返回promise本身.

this.onOff = function( turnOn ) {
    var state = turnOn ? 'ON' : 'OFF';

    var self = this;

    zWave.onOff( this.nodeNumber,error ) );
        return false;
    } );
}

我建议改变它以返回这样的承诺：

this.onOff = function( turnOn ) {
    var state = turnOn ? 'ON' : 'OFF';

    var self = this;

    // ===== add return statement here
    return zWave.onOff( this.nodeNumber,error ) );
        return false;
    } );
}

然后,它向.onOff()的调用者提供true或false返回的promise值.或者,如果您不需要获取这些返回的值,那么您可以删除返回true并返回false语句,因此没有promise值.

This strikes me as a specific example of a more general issue: how do
you move from a Promise-based model to Traditional blocking code?

您不会为异步操作编写传统的阻塞代码.异步操作是异步的,永远不会是传统的阻塞代码. Promise为管理和协调异步操作提供了更多的结构,但您仍然必须以异步方式为它们编写代码.使用promises编写异步代码更容易.

JS语言正在添加更多功能,例如生成器和等待,您可以使用它们来帮助解决这个问题.这是关于该主题的简短文章：ES7 async functions.

原文地址：https://www.jb51.cc/js/158103.html

javascript – 结束Promise链

解决方法

相关推荐