Question

我刚刚开始使用nodeJS并且在过去8小时内一直在干扰它。我以前做过J2EE和.NET，也想进入NodeJS。

我在尝试从Data.js文件中获取响应时遇到了问题。我可以成功地对数据进行字符串化，但出于某种原因我无法将其返回。它最终在index.js路由器中为null或空字符串（其默认值）。我在下面的Data.js代码中解释了注释。

提前感谢您，请告诉我如何改进我的代码:) 我还有很多要学习的概念

index.js

var express = require('express');
var router = express.Router();

/* GET home page. */
router.get('/', function(req, res, next) {
var Data = require('../public/javascripts/Data');
var responseString = Data.test;

  res.render('index', { d: responseString });
  console.log(responseString)
});

module.exports = router;

Data.js

var request = require('request'),
cheerio = require('cheerio'),
responseTitles = [],
responseUrls = [],
responseJSONArray = [],
responseTotalNumber = 0,
responseString = "";


function getData(){
request('URL',function(err,response,body){
    console.log("requesting..")
    if(!err && response.statusCode == 200){
        console.log("success..")
        var $ = cheerio.load(body);
        var data = [];
        data = $('span.titletext');
        data.each(function(i){
            responseTitles.push(data[i].children[0].data);
            responseUrls.push(data[i].parent.attribs.url);
            responseTotalNumber = i;

            responseJSONArray.push([{index: i, Title: data[i].children[0].data, url: data[i].parent.attribs.url}])
        });
        responseString = JSON.stringify(responseJSONArray);
        //At this point, responseString has all the data but
        //if I put the return here, it doesn't return anything
    }
  });
 //At this point responseString returns "". And so it returns an empty string
 return responseString;
}


module.exports= {
   test: getData()
};

Answer 1

您的getData函数异步调用request，然后立即返回responseString的值，该值始终为''。

想想代码：

module.exports= {
    test: getData()
};

这里发生了什么？你是

立即执行getData功能，其中：
- 执行request和
- 返回''
使用包含此结果的单个字段test导出您的模块（''）。

所以你基本上是这样做的：

module.exports = {
    test: ''
};

我认为这不是你的意图。你可能想要更接近这个的东西：

module.exports = {
    test: getData
};

排除parens意味着您将返回函数本身而不是调用函数的结果。

当您异步工作时，您可能希望包含一个回调参数或从getData函数返回一个promise以访问请求的结果。也许是这样的：

Data.js：

function getData (cb) {
    request('URL', function (err, response, body) {

       ...

       responseString = JSON.stringify(responseJSONArray);
       cb(responseString);
    });
}

module.exports = {
    test: getData
};

然后像这样使用它：

Index.js

var data = require('../public/javascripts/Data');
router.get('/', function (req, res, next) {
    data.test(function (responseString) {
        res.render('index', { d: responseString });
    });
});

初学者Node.JS，没有按预期返回

1 个答案: