javascript - Node.js & co - 避免混合 promise 和事件回调

标签 javascript node.js callback promise phantomjs-node

最近,我一直在尝试 phantomjs-node图书馆。我想要实现的基本上是创建一个动态网页模板,使用 phantomjs-node 库来“运行”它,最后从渲染的页面中提取一些数据。

在最简单的设置中,第一次尝试解决这个问题看起来像这样(在下面的示例中,模板只是静态的,但原则上它可能包含一些利用外部库等的进一步逻辑):

var phantom = require('phantom');
var co = require('co');
var sleep = require('system-sleep');
var winston = require('winston');

const logger = new winston.Logger({
    level: 'debug',
    transports: [new winston.transports.Console({
        json: false, timestamp: () => (new Date()).toLocaleString()
    })]
});

co(function*() {
    logger.info('start');
    var instance = yield phantom.create();   
    try {
        const html = `
                <!DOCTYPE html>
                <html>
                    <head>
                        <title>Page title</title>
                    </head>
                    <body>
                        <div id='results'>Page data</div>
                    </body>
                </html>
            `;

        var page = yield instance.createPage();    

        yield page.on('onLoadFinished', function(){
            logger.info('onLoadFinished');

            page.evaluate(function(){
                return document.getElementById('results').textContent;    
            }).then(function(val){
                logger.info(`RESULT = ${val}`);    
            }).catch(function(val){
                logger.error(val.message);    
            });
        });

        yield page.setContent(html, 'http://localhost');

    }catch (e){
        logger.error(e.message);       
    }finally{
        instance.exit();
    }
    logger.info('done');
});

但是,这会失败并输出:

12/18/2017, 2:44:32 PM - info: start
12/18/2017, 2:44:33 PM - info: done
12/18/2017, 2:44:33 PM - info: onLoadFinished
12/18/2017, 2:44:33 PM - error: Phantom process stopped with exit code 0

很可能是因为当最终调用page.evaluate返回的promise的then回调时,主幻影进程已经退出。

为了“解决”这个问题,我采用了以下即兴策略(省略了下面示例的其余部分):

    var page = yield instance.createPage();

    var resolver;
    var P = new Promise(function(resolve, reject){ resolver = resolve; });

    yield page.on('onLoadFinished', function(){
        logger.info('onLoadFinished');

        resolver(page.evaluate(function(){
            return document.getElementById('results').textContent;
        }));
    });

    yield page.setContent(html, 'http://localhost');

    const val = yield P;
    logger.info(`RESULT = ${val}`);

这实际上创建了一个新的 Promise,该 Promise 通过从 page.evaluate 返回的 Promise 进行“外部”解析。然后,co block 末尾的 yield P 语句会阻塞,直到所需的结果准备就绪,因此输出符合预期:

12/18/2017, 2:53:47 PM - info: start
12/18/2017, 2:53:48 PM - info: onLoadFinished
12/18/2017, 2:53:48 PM - info: RESULT = .....
12/18/2017, 2:53:48 PM - info: done

虽然这似乎有效,但感觉相当“hacky”(例如,在调用 resolver 之前回调中抛出的异常不会在主 try/catch block ),所以我想知道什么是更干净的方法,以便将控制从 onLoadFinished 回调“转移”回由 co 管理的领域?

最佳答案

  • 不要再使用 co + 生成器函数。 async/await 在这里。
  • 是的,您应该将(最多)触发一次的所有事件回调转换为 Promise。
  • 不,永远不要创建这样的 promise 并“在外部解决它们”。只需将解决它们的内容放入 Promise 构造函数中即可。

(async function() {
    logger.info('start');
    var instance = await phantom.create();   
    try {
        const html = `…`;
        const page = await instance.createPage();    

        await new Promise((resolve, reject) => {
            page.on('loadFinished', resolve);
            page.on('resourceError', reject); // or something like that?
            page.setContent(html, 'http://localhost'); // this doesn't appear to return a promise
        })
        logger.info('onLoadFinished');

        try { // this extra inner try looks superfluous
            const val = await page.evaluate(function(){
                return document.getElementById('results').textContent;    
            });
            logger.info(`RESULT = ${val}`);
        } catch(e) {
            logger.error(e.message);    
        }
    } catch(e) {
        logger.error(e.message);       
    } finally {
        instance.exit();
    }
    logger.info('done');
}());

关于javascript - Node.js & co - 避免混合 promise 和事件回调,我们在Stack Overflow上找到一个类似的问题: https://stackoverflow.com/questions/47870195/

相关文章:

javascript - 在 Node.js 中传播错误回调并避免所有额外的麻烦?

javascript - 图像 map 浏览器兼容性和 imagemapster

javascript - 使用 Javascript 添加的 HTML 表单隐藏字段未发布

javascript - MarkerWithLabel 缩放时移动

javascript - 条形图上的反转序列,数据从表中加载

node.js - 我是否必须在需要使用 npm 命令的 bitbucket 管道中的每一步中进行 npm install

javascript - 无法使用 Socket.IO 发送到 Node 服务器

jquery - 回调内绑定(bind)/取消绑定(bind)

javascript - 将回调函数从任何子框架传递到顶部窗口的最有效方法?

javascript - Node.JS - 无法使用 try/catch block 获得异步抛出