Как вывести переменные из функции async.eachSeries?

Question

[nezzard]

Добрый день, есть 2 цикла с запросами, которые формируют 2 массива, я вывожу их в два разных коллбека, но мне нужно их обьединить в один после успешной обработки одного и другого, но никак не получается вывести

async.eachSeries(result, function(item, callback2) {
		  request.get("http://ws.audioscrobbler.com/2.0/?method=artist.getinfo&artist="+encodeURIComponent(item['artist'])+"&api_key=xxxx&format=json", function(err, res, body)
		  {
		    if(res.statusCode == 200 ) {     
			    if(JSON.parse(body)['artist'] !== undefined) {
			    	temp2.push({name: count2, number: JSON.parse(body)['artist']['image'][2]['#text']});		       
			  	} 
			  	else {
			    	temp2.push({name: count2, number: false});
			  	}
		 			callback2();
		 			count2++;
		  	}
	 			else {
		      callback2('Error');
		    }
		        
		  });    
		}, 
		function(err) {
		  if( err ) {
		    console.log('Ошибка случилась');
		  } else {
		    console.log('Все хорошо едем дальше');
		    console.log(temp2);
		  }
		});

И есть такое же, только для ютуба, нужно в итоге вывести два массива из двух циклов, чтобы сформировать 1 массив, подскажите пожалуйста в каком направлении двигаться

Answer 1

[Aves]

В направлении Promise.all
Например как-то так.

Answer 2

[Антон Л]

Вы все таки выбрали async. Это будет выглядеть ужасно, но можно использовать caolan.github.io/async/docs.html#parallel. Заснуть два этих eachSeries в parallel, а там где выводится 'едем дальше' вызывать parallel'евский callbak с вашим массивом. В конце, когда отработает parallel, он будет содержать оба ваших массива.
Написано ужасно, а в коде будет выглядеть ещё хуже. Категорически не рекомендую вам пользоваться этим древним пятилетней давности подходом. На дворе 2017 год, пора набраться смелости и вызубрить Promise и другие плюшки современного javascript. Благо тут помогут советом.

Comments

[nezzard]

Я пытался через request-promise но у меня никак не получалось вывести переменные из самой функции promise а как вы заметили у меня два запроса к разным сервисам, а потом это все дело нужно обьеденить и делать запрос в wordpress

[nezzard]

К тому же, мне еще нужен цикл, т.е. нужно сделать несколько request так как у меня массив из 7 значений

[nezzard]

В общем переделал на promise все кажется работает как надо, но всеровно проблема осталась в последовательности, т.е. если ответ от сервера пришел немного позже, то массив уже будет в другом порядке, а мне нужен строго в том порядке, в каком отправился, это возможно исправить?

[Антон Л]

Ок, Вы молодец. Напишите как переделали и я попробую вам помочь. если есть что-то конфиденциальное замените звездочками

[nezzard]

Антон Л: Вот так написал, в общем этот скрипт если выполняется с if то mysql падает моментально
https://toster.ru/answer?answer_id=935309#answers_...

[nezzard]

Антон Л: Подозреваю, что в моем случае нужно использовать bluebird с задержкой

[Антон Л]

Отлично, что Вы начали задумываться (по-моему в правильном направлении). Я отвечу скорее всего не раньше вторника. И кстати если есть какие-то вопросы напишите мне лично [e-mail]

Answer 3

[nezzard]

Вот так написал, в общем этот скрипт если выполняется с if то mysql падает моментально

result.forEach(function(item){
	wp.song().search( item['artist']+' - '+item['song'] ).then(function( posts ) {
		if(!posts[0]){

			var myRequests = [];
			myRequests.push(rp({uri: "http://ws.audioscrobbler.com/2.0/?method=artist.getinfo&artist="+encodeURIComponent(item['artist'])+"&api_key=*****&format=json", json: true}));
			myRequests.push(rp({uri: "https://www.googleapis.com/youtube/v3/search?part=snippet&order=viewCount&q="+encodeURIComponent(item['artist']+'+'+item['song'])+"&type=video&key=***", json: true}));
			Promise.all(myRequests)
			  .then((arrayOfHtml) => {
			  	console.log(arrayOfHtml[0]['artist']['image'][2]['#text']);
			    console.log(arrayOfHtml[1]['items'][0]['id'].videoId);


					wp.posts().create({
					    title: item['artist']+' - '+item['song'],
					    content: 'Your post content',
					    status: 'publish'
					}).then(function( response ) {
					    console.log( response.id );
					})

			    
			  })
			  .catch(/* handle error */);


		}
	});
})`