Node.js http.request in loop

Question

Am using a function to save data. Using the http.request option. It is working fine. If I call the same function in a loop, some of the data not saving in database. Also getting the parse error message for some response.

How Can I call the function of http.request in a loop?

for (var i = 1; i <= 23; i++) {
    turn_timer(i);
}

function turn_timer(nos) {
    try {
        var str = "num=" + nos;
        var len = str.length;

        win_settings.headers = {
            'Content-length': len,
            'Content-Type': 'application/x-www-form-urlencoded'
        }

        var request = http.request(win_settings, function(response) {
            response.on('data', function(data) {

            });

            response.on('error', function(err) {

            });

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

            });
        });

        request.on('error', function(err) {

        });

        request.write(str + "\0");
        request.end();

    } catch (err) {
        console.log(err);
    }
}

Have you read this stackoverflow.com/questions/21320159/… ? — Brian Lewis
– Brian Lewis, Commented Nov 19, 2014 at 4:30
added the code. it accepts the first five correctly and then next 5 error. next few ok and again error. is it due to socket hanging while waiting for response ? how to fix it? — swaraj
– swaraj, Commented Nov 19, 2014 at 4:36

Dongho Yoo · Accepted Answer · 2014-11-19 04:31:23Z

0

Check the scope of your variable that stores message because async function call may overwrite it.

answered Nov 19, 2014 at 4:31

Dongho Yoo

1,49116 silver badges16 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

Community · Accepted Answer · 2017-05-23 12:27:26Z

0

I believe your problem is because you are using a for loop, instead of going with a more asynchronous approach. Here is a quick attempt to solve your problem. I have omitted some of your code, as it seemed to be incomplete. I have left the important parts and added a few things based on an answer to a similar question.

var http = require('http'),
    async = require('async');

function turn_timer(n, callback) {
    var var str = "num=" + n,
        len = str.length,
        req;

    win_settings.headers = {
        'Content-length': len,
        'Content-Type': 'application/x-www-form-urlencoded'
    };

    req = http.request(options, function(response) {
        ...
        callback(null);
    });

    req.on('error', function(err) {
        ...
        callback(err);
    });

    req.end();
}

async.timesSeries(23, function (n, next) {
    turn_timer(options, function(err) {
        next(err);
    });
});

For more information, you can read more about async.timesSeries here: https://github.com/caolan/async#timesseriesn-callback

edited May 23, 2017 at 12:27

CommunityBot

11 silver badge

answered Nov 19, 2014 at 5:09

Brian Lewis

5,7191 gold badge23 silver badges28 bronze badges

2 Comments

swaraj Over a year ago

how this will execute the same function with different values ? like a loop am expecting

Brian Lewis Over a year ago

It will receive the number as the first argument. You can use that to calculate any values that you need.

Collectives™ on Stack Overflow

Node.js http.request in loop

2 Answers 2

Comments

2 Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

2 Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related