麻烦地在Node / Javascript

时间:2015-08-19 18:53:46

标签: javascript node.js http asynchronous callback

我正在尝试重新创建一个我在Node / js中编写的Python脚本,而我在绕过异步/回调方式时遇到了麻烦。

该脚本非常简单,并且对eBay的API使用了两个基本的HTTP请求。第一个请求获取结果项ID的列表,然后第二个请求获取每个项的特定信息(描述/列表信息等)。在Python中,这非常简单,我使用了一个简单的循环。无论如何它并不快,但它确实有效。

然而,在javascript中,我很难获得相同的功能。我现在的代码如下:

var ebay = require('ebay-api'); 
var params ={};

params.keywords = ["pS4"];
var pages = 2;
var perPage = 2;


ebay.paginateGetRequest({

serviceName: 'FindingService',
opType: 'findItemsAdvanced',
appId: '',
params: params,
pages: pages,
perPage: perPage,
parser: ebay.parseItemsFromResponse
},

function allItemsCallback(error,items){
    if(error) throw error;

    console.log('FOUND', items.length, 'items from', pages, 'pages');

    for (var i=0; i<items.length; i++){

        getSingle(items[i].itemId);

    }
}   

);

function getSingle(id){

console.log("hello");
ebay.ebayApiGetRequest({

    'serviceName': 'Shopping',
    'opType': 'GetSingleItem',
    'appId': '',     

    params: {
    'ItemId': id ,
     'includeSelector': 'Description' 
    }



},
    function(error, data) {
        if (error) throw error;
        console.dir(data);  //single item data I want

    }
);
}

这是许多人的一次尝试,但我收到了“检测到可能的EventEmitter内存泄漏”警告,并最终打破了“错误:错误'确认'代码未定义errorMessage?null”。我很确定这与正确使用回调有关,但我不确定如何正确使用它。任何答案或帮助将不胜感激。如果这不是一个好问题,我道歉,如果是这样,请让我知道如何正确地询问。

1 个答案:

答案 0 :(得分:1)

Node.js的异步事件链建立在回调上。而不是:

getSingle(items[i].itemId);

您需要在父函数完成后执行的函数中写回调:

getSingle(items[i].itemId, function(err, data) {
     // now you can access the data   
});

因为ebay.ebayApiGetRequest是一个冗长的函数,所以告诉其父函数完成它的回调必须在之后调用,如下所示:

ebay.ebayApiGetRequest({
        //
    },
    function(error, data) {
        callback(error, data);
    }
);

但是,当然,如果父函数getSingle不支持回调,那么它就不会去任何地方。所以你也需要在那里支持一个回调参数。这是完整的脚本,使用事件驱动的回调模型重写:

var ebay = require('ebay-api');
var async = require('async');
var params = {};

params.keywords = ["pS4"];
var pages = 2;
var perPage = 2;


ebay.paginateGetRequest({
        serviceName: 'FindingService',
        opType: 'findItemsAdvanced',
        appId: '',
        params: params,
        pages: pages,
        perPage: perPage,
        parser: ebay.parseItemsFromResponse
    },
    function allItemsCallback(error, items) {
        if (error) throw error;

        console.log('FOUND', items.length, 'items from', pages, 'pages');

        async.each(items, function(item, callback) {
            getSingle(item.itemId, function(err, data) {
                callback(err, data);
            });
        }, function(err, results) {
            // now results is an array of all the data objects
        });
    }
);

function getSingle(id, callback) {
    console.log("hello");
    ebay.ebayApiGetRequest({
            'serviceName': 'Shopping',
            'opType': 'GetSingleItem',
            'appId': '',
            params: {
                'ItemId': id,
                'includeSelector': 'Description'
            }
        },
        function(error, data) {
            if (error) throw error;
            console.dir(data); //single item data I want
            callback(error, data);
        }
    );
}