node with express js for each loop

Question

I'm using express with node for my rest api, I need to run a for loop to determine the output json. my route file looks

var Redis = require('ioredis')
var redis = new Redis({
    port: 6379,
    host: '127.0.0.1',
    family: 4,
    password: 'password',
    db: 0
});

var Jsonresult = {};
var process = function(lat,lon,dist,unit)
{
    Jsonresult.result = 'success';
    var vehicle_type = new Array('small','medium');  
    vehicle_type.forEach(function(vehicle, index, arr) 
    {
        redis.georadius ( vehicle,lat,lon ,dist,unit,'WITHCOORD','WITHDIST',function( ERR , Result ) 
        {
            if (ERR) 
            {
                console.log(ERR);
            }

            Jsonresult[vehicle] = Result;
        }) ;
    })

    return Jsonresult;
}

router.get('/:lat/:lon/:dist/:unit', function(req, res, next) {
    var lat = req.params.lat;
    var lon = req.params.lon; 
    var dist = req.params.dist;
    var unit = req.params.unit;
    res.json(process(lat,lon,dist,unit));
});

module.exports = router;

and my expected json output is

{"result":"success","small":[["driver_1","0.2779",["56.507199704647064","-0.12500104133338397"]],["driver_2","0.2782",["56.50730162858963","-0.12500104133338397"]]],"medium":[]}

but i'm getting only

{"result":"success"}

whats wrong in the code ?

What do you pass when you pass a GET request? can you provide an example? — Vicky Gonsalves
– Vicky Gonsalves, Commented Feb 8, 2016 at 19:52
I have updated the question and my get parameters are /vehicles/56.5072/-0.1275/5/km — Vigikaran
– Vigikaran, Commented Feb 8, 2016 at 19:59

narainsagar · Accepted Answer · 2016-02-08 19:34:54Z

6

this way:

vehicle_type.forEach(function(vehicle, index, arr) {
   Jsonresult[vehicle] = true;
})

and pass result in response as well when calling res.json()..

res.json(Jsonresult);

answered Feb 8, 2016 at 19:34

narainsagar

1,1292 gold badges13 silver badges29 bronze badges

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

5 Comments

Vigikaran Over a year ago

Hi i have tried this also. but when i refresh the page the expected result is coming. but first time not coming.

Felix Kling Over a year ago

Maybe you can explain what you changed and why.

narainsagar Over a year ago

then, i guess, use res.render(Jsonresult) instead of res.json(Jsonresult);. let me know if this works.

Vigikaran Over a year ago

@FelixKling : i added res.json(Jsonresult);

Felix Kling Over a year ago

That's not the only thing you've changed as far as I can see.

Community · Accepted Answer · 2017-05-23 10:29:28Z

0

vehicle_type.forEach(function(vehicle) 
{
    calls.push(function(callback){
        redis.georadius ( vehicle,lat,lon ,dist,unit,'WITHCOORD','WITHDIST',  function  ( ERR , Result ) 
        {
            if (ERR) 
            return callback(ERR);  

            Jsonresult[vehicle] = Result;
            callback(null, vehicle);


        });


      });
});

async.parallel(calls, function(err, result) {
if (err)
    return console.log(err);
res.json(Jsonresult);
 });

This finaly solve my problem: answer here

edited May 23, 2017 at 10:29

CommunityBot

11 silver badge

answered Feb 8, 2016 at 22:42

Vigikaran

7251 gold badge10 silver badges28 bronze badges

Collectives™ on Stack Overflow

node with express js for each loop

2 Answers 2

5 Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

5 Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related