Node.js使用Promises:Promise什么都不返回

时间:2015-11-29 18:43:32

标签: node.js promise cheerio

我正在制作一个简单的网络刮刀,我刮掉了演员/女演员的名字,但我收到了一个错误。请你看看,我觉得我做错了。

返回[ReferenceError: options2 is not defined]  错误。

非工作版(使用Promises)

var entities = require("entities");
var request = require('request');
var cheerio = require('cheerio');

// create promisified version of request()
function requestPromise(options) {
  return new Promise(function (resolve, reject) {
    request(options2, function (err, resp, body) {
      if (err) return reject(err);
      resolve(body);
    });
  });
}

app.get('/fetch', function (req, res) {
  var promises = [];
  var headers = {
    'User-Agent': req.headers['user-agent'],
    'Content-Type': 'application/json; charset=utf-8'
  };
  for (var i = 1; i < 10; i++) {
    promises.push(requestPromise({url: "http://example.com/person/" + i + "/personname.html", headers: headers}));
  }
  Promise.all(promises).then(function (data) {
    // iterate through all the data here
    for (var i = 0; i < data.length; i++) {
      if ($ = cheerio.load(data[i])) {
        var links = $("#container");
        var name = links.find('span[itemprop="name"]').html(); // name
        if (name == null) {
          console.log("null name returned, do nothing");
        } else {
          name = entities.decodeHTML(name);
          console.log(name); // it doesn't echo the name, WHY?
        }
      } else {
        console.log("can't open");
      }
    }
  }, function (err) {
    // error occurred here
       console.log(err);
  });
});

工作版本(丑陋但不按顺序,这就是我试图进入Promise的原因)

var entities = require("entities");
var request = require('request');
var cheerio = require('cheerio');

app.get('/back', function (req, res) {
  for (var y = 1; y < 10; y++) {
    (function () {
          const id = y;
          var url = "http://example.com/person/" + id + "/personname.html";
          var options2 = {
          url: url,
        headers: {
          'User-Agent': req.headers['user-agent'],
          'Content-Type': 'application/json; charset=utf-8'
        }
      };
      request(options2, function (err, resp, body) {
        if (err) {
          console.log(err);
        } else {
          if ($ = cheerio.load(body)) {
                var links = $('#container');
            var name = links.find('span[itemprop="name"]').html(); // name
            if (name == null) {
              console.log("null name returned, do nothing");
            } else {
              name = entities.decodeHTML(name);
              console.log(name); // it echoes name, so it works.
            }
          }
          else {
            console.log("can't open");
          }
        }
      });
    }());
    }
});

1 个答案:

答案 0 :(得分:2)

你在这里犯了错误:

// create promisified version of request()
function requestPromise(options) {
  return new Promise(function (resolve, reject) {
    request(options2, function (err, resp, body) {
      if (err) return reject(err);
      resolve(body);
    });
  });
}

您指的是options2,但参数名为options

将其更改为:

// create promisified version of request()
function requestPromise(options) {
  return new Promise(function (resolve, reject) {
    request(options, function (err, resp, body) {
      if (err) return reject(err);
      resolve(body);
    });
  });
}

从承诺中学到的一件事就是它们会为你捕获错误,因此如果你有一个错误处理程序并且你的代码没有按照需要运行,你应该总是记录错误路径以查看是否有人受到攻击。如果您的代码中没有定义错误处理程序,则应设置一个并记录它。

异步操作中的日志记录错误对于调试至关重要,因为异步代码中的错误(甚至异常)通常不会在控制台中自动显示。因此,您经常需要自己记录它们才能知道它们正在发生。