значение массива не обновляется в цикле for в nodejs - PullRequest
Новая
       11

значение массива не обновляется в цикле for в nodejs

0 голосов
/ 11 апреля 2019

Я пытаюсь поместить данные в массив после загрузки изображения из функции загрузки.Это проблема в обещании nodejs.Как я могу исправить эту проблему.

токовый выход: 

[
    {
        sku: 99104942591,
        retailer: 'JCREWFCT',
        images: []
    }
]

ожидаемый выход: 

[
    {
        sku: 99103497136,
        retailer: 'JCREWFCT',
        images: [
            "http://localhost:4001/JCREWFCT/99103497136.png",
            "http://localhost:4001/JCREWFCT/99103497136_1.png"
        ]
    }
]

inoutputArr Я пытаюсь сохранить данные 

 var downloadImages = async function() {
  var outputArr = [];
  for(var i=0; i<excelJsonArr.length; i++) {
    var d = excelJsonArr[i];
    var out = {
      sku : d.sku,
      retailer : d.retailer,
      images : []
    }
    if(d.image_link_1) {
      var saveDir = path.join('./public', d.retailer, d.sku+'.png');
      var imgUrl = properties.get('protocol') + '://' + properties.get('hostname') + ':' + properties.get('port') + '/' + d.retailer + '/' + d.sku + '.png';
      await download(d.image_link_1, saveDir, function(){
        out.images.push(imgUrl);
      });
    }

    if(d.image_link_2) {
      var saveDir = path.join('./public', d.retailer, d.sku+'_2.png');
      await download(d.image_link_1, saveDir, function(){
        var imgUrl = properties.get('protocol') + '://' + properties.get('hostname') + ':' + properties.get('port') + '/' + d.retailer + '/' + d.sku + '_2.png';
        out.images.push(imgUrl);
      });
    }

    outputArr.push(out);
  }
  console.log(outputArr);
}

var download = async function(uri, filename, callback){
  await request.head(uri, function(err, res, body){
    console.log('content-type:', res.headers['content-type']);
    console.log('content-length:', res.headers['content-length']);

    request(uri).pipe(fs.createWriteStream(filename)).on('close', callback);
  });
};

Ответы [ 2 ]

1 голос
/ 11 апреля 2019

Поскольку вы используете async await, вам не нужно использовать функцию обратного вызова.Просто вызовите нужные функции после await

var downloadImages = async function () {
    var outputArr = [];
    for (var i = 0; i < excelJsonArr.length; i++) {
        var d = excelJsonArr[i];
        var out = {
            sku: d.sku,
            retailer: d.retailer,
            images: []
        }
        if (d.image_link_1) {
            var saveDir = path.join('./public', d.retailer, d.sku + '.png');
            var imgUrl = properties.get('protocol') + '://' + properties.get('hostname') + ':' + properties
                .get('port') + '/' + d.retailer + '/' + d.sku + '.png';
            await download(d.image_link_1, saveDir, function () {
                // out.images.push(imgUrl);// <-- not here
            });
            out.images.push(imgUrl); // <-- here
        }

        if (d.image_link_2) {
            var saveDir = path.join('./public', d.retailer, d.sku + '_2.png');
            await download(d.image_link_1, saveDir, function () {
                /* var imgUrl = properties.get('protocol') + '://' + properties.get('hostname') + ':' +
                    properties.get('port') + '/' + d.retailer + '/' + d.sku + '_2.png';
                out.images.push(imgUrl); */ // <-- not here
            });
            var imgUrl = properties.get('protocol') + '://' + properties.get('hostname') + ':' +
                properties.get('port') + '/' + d.retailer + '/' + d.sku + '_2.png';
            out.images.push(imgUrl);  // <-- here
        }

        outputArr.push(out);
    }
    console.log(outputArr);
}
1 голос
/ 11 апреля 2019

Я не знаю, что на самом деле делает ваша функция загрузки, но обычно, при работе с asnyc, вы бы сделали это: 

await download(d.image_link_1, saveDir);
out.images.push(imgUrl);

, и я должен попытаться работать с try до catch любые ошибки, возникающие при загрузке, см .: Правильная попытка… Поймать синтаксис с использованием Async / Await

К вашему сведению, в следующий раз поделитесь большим количеством вашего кода и, если возможно, воспроизводимым кодом или примером GitHubрепо, чтобы мы могли сами увидеть ошибку.

...