承诺不会返回对象数组

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()在完成所有操作而不是使用手动计数器时获得回调。这也使错误处理更简单。