Exercise 9 - Juggling Async (Alternate Solution)

[SidShenoy]

I thought that the exercise involved using 'callbacks' as you guys had stressed on the fact that callbacks lie at the core of nodeJS. Here is my alternate solution:

var http = require("http");
var bl = require('bl');
var arg_index = 2;
var count = 3;
function http_get(callback)
{
	http.get(process.argv[arg_index++], function(resp){
				resp.setEncoding("utf8");
				resp.pipe(bl(function(err, data){
						if(err) console.error(err);
						var str = data.toString();
						console.log(str);				
				}));
											
				if(arg_index!=2+count)			
				callback(callback);
	}).on('error',console.error);
}
						
http_get(http_get);

[th-santos]

I have done something similar. But after seeing the "official solution", I realized my approach was not taking advantage of the asynchronous operation.

I mean, the "2nd get" has to wait for a response from "1st get", and the "3rd get" has to wait for a response from the "2nd get".

[PRANAY100]

can anyone tell why my solution doesn't work? while its some what similar to official solution ..
var http = require('http');
var bl = require('bl');
let count = 0;
const results = [];

for (let i = 0; i < 3; i++) { // what makes the difference by adding a function here, instead of for loop?
http.get(process.argv[i + 2], function (res) {
res.setEncoding('utf8');
res.pipe(bl(function (err, data) {
if (err){
return console.error(err);
}
results[i] = data.toString();
count ++;
if (count == 3) {
printResults()
}
}));
});

}

function printResults() {
for (let i = 0; i < 3; i++) {
console.log(results[i]);
}
}

[AnshulMalik]

@sidorares You are logging as soon as the network request is complete console.log(str); which might not always be in order.