Загрузите файл из Интернета, используя Node js и loop

Question

Загрузите файл из Интернета, используя Node js и loop

1

Я хочу загрузить несколько файлов из Интернета, используя этот код:

    var fs = require('fs');
    var http = require('http');
    var request = require('request');

    var file;
    for(var i = 1; i <= 5; i++) {
         //CHECK IF REMOTE FILE EXISTS
         request('http://webaddress.com/filename' + i + '.jar', function (err, resp) {

         //IF EXISTS DO
         if (resp.statusCode == 200) {
             //DOWNLOAD DATA AND CREATE A NEW .JAR FILE
             file = fs.createWriteStream('D:\\filename' + i + '.jar');
             http.get('http://webaddress.com/filename' + i + '.jar', function(response) {
                  response.pipe(file);
                  file.on('finish', function() {
                      file.close();
                  });
             });
         }

         //FILE DOES NOT EXIST
         });
     }

В результате я хочу: несколько файлов, загруженных с именами файлов filename1-5.jar. Результатом, который я получаю, является всего лишь 1 файл с именем файла filename5.jar (или последним значением я var в цикле). Что я делаю неправильно?

lupu51nfactum N778 16 июль 2018, в 13:48

Источник

1

Ваш запрос асинхронный, и он будет выполняться только после того, как ваш цикл завершится, следовательно, 5 из имени файла.
Ionut 16 июль 2018, в 11:47
0

Я так и подозревал. Благодарю. Что я могу сделать, чтобы это работало?
lupu51nfactum N778 16 июль 2018, в 11:48
0

Посмотрите этот ответ, он может помочь вам решить вашу проблему stackoverflow.com/a/19911657/8221063
Lucas Duarte 16 июль 2018, в 11:52
0

Спасибо @LucasDuarte, я проверю это.
lupu51nfactum N778 16 июль 2018, в 11:56
0

@ lupu51nfactumN778, я добавил для тебя ответ.
Ionut 16 июль 2018, в 11:57

Показать ещё 3 комментария

Теги:

javascript

node.js

2 ответа

Ещё вопросы

Ваш запрос асинхронный, и он будет выполняться только после того, как ваш цикл завершится, следовательно, 5 из имени файла.
Я так и подозревал. Благодарю. Что я могу сделать, чтобы это работало?
Посмотрите этот ответ, он может помочь вам решить вашу проблему stackoverflow.com/a/19911657/8221063
@ lupu51nfactumN778, я добавил для тебя ответ.

Ionut · Answer 1 · 2018-07-16T09-38-00.000Z

Ваш запрос является асинхронным, и он будет выполняться только после завершения цикла, следовательно, 5 из имени файла. Решением для этого является угроза для вашего кода отдельно, создавая новую функцию и вызывая ее внутри цикла:

var fs = require('fs');
var http = require('http');
var request = require('request');

var file;

function customRequest(i){

  //CHECK IF REMOTE FILE EXISTS
  return request('http://webaddress.com/filename' + i + '.jar', function(err, resp) {

    //IF EXISTS DO
    if (resp.statusCode == 200) {
      //DOWNLOAD DATA AND CREATE A NEW .JAR FILE
      file = fs.createWriteStream('D:\\filename' + i + '.jar');
      http.get('http://webaddress.com/filename' + i + '.jar', function(response) {
        response.pipe(file);
        file.on('finish', function() {
          file.close();
        });
      });
    }

    //FILE DOES NOT EXIST
  });

}

for (var i = 1; i <= 5; i++) {
   customRequest(i)
}

Спасибо, ваш код делает 5 отдельных файлов. У меня сейчас проблема в том, что все данные отправляются только в 1 случайный файл, остальные - пустые. Может ли быть так, что функция выполняется слишком быстро? Я попробовал setTimeout без особой удачи.

BraveButter · Answer 2 · 2018-07-16T12-45-00.000Z

Как @Ionut сказал, что ваши запросы асинхронны, поэтому вам нужно подождать

let fs = require('fs');
let request = require('request');

let download = (uri, filename) => {
    return new Promise ((resolve, reject) => {
        request.head(uri, function(err, res) {

            if (res.statusCode === 200) {
                request(uri).pipe(fs.createWriteStream(filename)).on('close', resolve);
            } else {
                reject(res.statusCode);
            }
        });
    });
};

let promises = [];

for(let i = 1; i <= 5; i++) {
    promises.push(download('http://webaddress.com/filename' + i + '.jar', 'D:\\filename' + i + '.jar'));
}

Promise.all(promises).then(() => {
    process.exit(0);
});