承諾不會返回對象數組

Question

不定值：

我使用Q管理與Node.js的承諾，我使用easyImage處理圖像。

這段代碼工作正常，它加載，保存，剪切tmp文件夾中的圖像並將其粘貼到用戶文件夾中。我唯一的問題是將最終數據保存到數據庫。我得到了一個未定義值的數組中......

exports.postAccountImages = function(req, res, next) { 
    User.findById(req.user.id, function(err, user) { 
    var path = __dirname + '/../public/images/u/' + user._id + '/'; 
    var max = 800; 
    fs.exists(path, function(exists) { 
     if (!exists) fs.mkdirSync(path); 
    }); 

    var promises = []; 
    for (var key in req.files) { 
     if (req.files.hasOwnProperty(key)) { 
     (function(file) { 
      q().then(function() { 
      return promises.push(easyimg.info(file.path).then(function(image) { 
       easyimg.resize({ 
       src: file.path, 
       dst: path + file.name, 
       width: (image.width >= max) ? max : image.width, 
       height: (image.height >= max) ? max : image.height, 
       quality: 80 
       }).then(function(image) { 
       fs.remove('./tmp-uploads/' + image.name); 
       return { 
        src: image.name, 
        main: false 
       }; 
       }); 
      })); 
      }); 
     })(req.files[key]); 
     } 
    } 

    q.all(promises).then(function(result) { 
     console.log(result); // [undefined, undefined, undefined, ...] 
     // Here I should push result to the DB 
    }); 
    }); 
}; 

Answer 1

下面是如何做到這一點的一般概念（未經測試）：

var promises = []; 
for (var key in req.files) { 
    if (req.files.hasOwnProperty(key)) { 
     (function(file) { 
      promises.push(easyimg.info(file.path).then(function(image) { 
       return easyimg.resize({ 
        src: file.path, 
        dst: path + file.name, 
        width: Math.max(image.width, 800), 
        height: Math.max(image,height, 800), 
        quality: 80 
       }).then(function(image) { 
        fs.remove('./tmp-uploads/' + image.name, function(err) { 
         if (err) { 
          // logging error, but not stopping execution 
          // since this is a non-fatal error 
          console.log("err removing temp upload: ", err); 
         } 
        }); 
        return {src: file.name, main: false}; 
       }); 
      })); 
     })(req.files[key]); 
    } 
} 
// now wait for all promises to finish 
// assumes you want to call next() no matter what when all image processing is done 
Promise.all(promises).then(function(results) { 
    // all results are in the results array here 
    // do whatever processing of the results array you want to do here 
    res.sendStatus(200); 
    next(); 
}, function() { 
    // set some status to send when there's an error 
    res.sendStatus(xxx); 
    next(); 
}); 

我糾正一些問題，並提出了一些改進：

1. 當處理多個文件時，您的變量file被覆蓋，因爲您嘗試在多個異步回調中使用相同的變量。我把它放到一個閉包中，因此它被分別保存到每個正在處理的圖像中。這可能是使其不能處理多個圖像的主要問題。

2. 你沒有在所有正確的時間呼叫next()（有時稱它太多次）。

3. 你的錯誤處理有多個問題，因爲你不能只是返回一個異步方法來停止處理。

4. 我決定，如果你不能刪除處理應該繼續而不是中止的臨時文件，因爲這不是一個致命的問題。

5. 此代碼使用Promise.all()在完成所有操作而不是使用手動計數器時獲得回調。這也使錯誤處理更簡單。