如何循环res.on('end',function()){..}以及res.on('end)如何真正起作用?

时间:2015-10-03 09:53:01

标签: javascript node.js express callback

我试图查看所有类似的问题,但没有一个能完全解答我的问题。

我正在尝试使用rottentomatoes api来检索与一般标题搜索相对应的电影列表,并为每个电影显示其导演。

现在,导演必须通过一个单独的json(与电影本身对应的那个)获取,因此我必须做一个http.request来获取电影列表,然后,对于他们每个人,制作另一个http.request获取相应的导演。

问题是当我尝试在第一个res.on('end' , function(){..})内循环时,所以对于每个检索到的电影,我可以做另一个http.request。 我知道所有请求都是异步的,但这不应该是一个问题,相反,第一部电影的导演附加到包含同一首部电影的导演的json对象,当我尝试解析这个对象时,我得到一个错误

undefined:1
.rottentomatoes.com/api/public/v1.0/movies/770687943/similar.json"}}{"id":9272
                                                                    ^

由于两个json响应附加到自身,它们不再是json,因此错误。

就好像循环的第一次迭代(获取控制器的那次)被执行两次。我认为它不应该是favicon等的问题。

我通过反复试验发现这种循环方式有效:

function httpGetMovies(response) {
var fullRequestQueryMovie = partialQueryMovie + apiKeyMovie + '&' + queryTermMovie + '&page_limit=2'+'&_prettyprint=true';
console.log('Calling -> ' + fullRequestQueryMovie);

var headersMovie = {
        'Content-Type': 'application/json'
};


var optionsMovie = {
        host: hostMovie,
        path: fullRequestQueryMovie,
        method: 'GET',
        headers: headersMovie
};

var jsonStringResponseMovie = '';

var req = http.request(optionsMovie, function(res) {
    console.log("statusCode: ", res.statusCode);
    console.log("headers: ", res.headers);

    res.on('data', function(piece) {
        jsonStringResponseMovie += piece;
        //console.log("-----------------\n"+jsonStringResponseMovie);
    });

    var i=0;
    res.on('end', function addDirector() {

        var jsonObj = JSON.parse(jsonStringResponseMovie);

        var moviesBag = [];

            console.log("film n°: "+i+" of "+jsonObj.movies.length+" "+jsonObj.movies[i].title);


//              /-----begin nested http.request--------
            var singleMovieDirectors = [];

            var fullRequestSingleMovie = jsonObj.movies[i].links.self + '?apikey=' + apiKeyMovie ;

            var headersSingleMovie = {
                    'Content-Type': 'application/json'
            };

            var optionsSingleMovie = {
                    host: hostMovie,
                    path: fullRequestSingleMovie,
                    method: 'GET',
                    headers: headersSingleMovie
            };

            var jsonStringResponseSingleMovie = '';

            var req2 = http.request(optionsSingleMovie, function(res2) {
                console.log("request for "+optionsSingleMovie.path);
//                  console.log("statusCode: ", res.statusCode);
//                  console.log("headers: ", res.headers);

                res2.on('data', function(piece2) {
                    jsonStringResponseSingleMovie += piece2;
                });

                res2.on('end', function() {

                    var jsonObjSingleMovie = JSON.parse(jsonStringResponseSingleMovie);
                    console.log("directors > "+JSON.stringify(jsonObjSingleMovie.abridged_directors));
                    // response.send(blabla);

                if (i++ < jsonObj.movies.length-1){
                    addDirector()
                }
                    else{
                        console.log ("finished adding directors");
                    }

                });
            });

            req2.end();

            req2.on('error', function(e) {
                console.error(e);
            });

//              -------end of nested http.request-----------------




    });

});

req.end();

req.on('error', function(e) {
    console.error(e);
});


};

你可以看到主i=0的{​​{1}}之前有一个res.on('end'),里面有一个嵌套的请求,其末尾有一个检查和增量,如果还有剩下的电影,我会再次调用回调函数但是会调用http.request,从而转移到下一部电影。 请注意,此调用i++乘以回调函数

这样,相反,不工作,我不知道为什么

#movies

如果您已经找到解决方案,您可能会问,问题是什么? 首先,我想分享一下,因为它是由我的一个朋友建议的,并且iIhonestly从未想过循环回调函数本身。 其次,我不知道为什么我的选择不起作用。问题在哪里?

对不起,我试着解释一切。如果这是一个转贴,请随意指出我去天堂的路。

最大

1 个答案:

答案 0 :(得分:0)

您可能会遇到此处引用的问题closure inside a for loop - callback with loop variable as parameter ,这很常见。

调试/跟踪的一种简单方法是使用大量console.logs输出变量。这样你就可以看到发生了什么。

另一件有用的事情是使用request-promise-json模块,可能还有async/await babel。它基本上可能是这样的:

  async function checkMovies(movies) {
    let result = [];
    for (let title of movies) {
      let movie = await request(`${base}?${movie}.json`);
      let movie.director = await request(`${base}${movie.director}.json`);
      result.push(movie);
    }
    return result;
  }