Using for loop with async in nodejs

Question

I have connected with mongodb and nodejs.

I have question about sav query results to list.

results = [];

for(i = 0; i<query.length; i++){
    db.collection(collectionName).find(query[i]).toArray(function(err, result){
        results[i] = result;
    });
}

I used async.waterfall but it doesn't work very well... If you fixed this problem like me, would you please how to fix this?

query like this :

{
location:{
  $geoWithin : {
    $center: [[lng, lat], radian]}}, 
time : "time value" 
}

lng, lat, time are list..

What do you mean, "doesn't work very well"? What is unclear to you and needs to be clarified? — Sergio Tulentsev, Jul 12 '18 at 09:21
I do not understand "async" yet. I will look into asyn-await.. — mings, Jul 12 '18 at 09:27
As a side node: For the most cases, querying database in a loop is not a good idea. — Anand Undavia, Jul 12 '18 at 09:36

score 1 · Answer 1 · answered Jul 12 '18 at 09:42

Depending on what the query array holds, you are better off running a single query which uses the $or operator instead of looping through the query array and firing server requests for each query:

db.collection(collectionName).find({ '$or': query }).toArray((err, results) => {
    console.log(results);
});

Using for loop with async in nodejs

1 Answers1