[英]Node.js downloading files - not all requests resolve, Promise.all doesn't fire
[英]Promise.all in Node.js doesn't call the then function
我学习了如何使用Node和Ecmascript 6。
我的脚本的目的是重命名目录中的png文件列表。 我只想重命名png文件(假设还有jpg),并在最后显示重命名的文件数。 由于Node的非阻塞性,它并不是那么明显,我决定利用这次机会来发现ES6承诺。
'use strict';
const fs = require('fs-extra');
const dir = '/Users/toto/Desktop/png/';
const suffix = '_IMAGE.';
const regex = /(.*)\.(png)$/;
var nbFiles = 0;
// Rename a png file with a suffix
var renameFile = (filename) => {
return new Promise((resolve, reject) => {
if(regex.test(filename)){
let newFileName = filename.replace(regex, '$1' + suffix + '$2');
fs.rename(dir + filename, dir + newFileName, (err) => {
let msg = filename + ' => ' + newFileName;
if (err) {
console.log('KO : rename of ' + msg);
reject(err);
}
console.log('OK : rename of ' + msg);
resolve(nbFiles++);
});
}
});
};
// Read files in a directory and call renameFile + display number of files renamed
fs.readdir(dir, (err, files) => {
if(err) return console.error(err);
var promise = Promise.all(
files.map(renameFile)
).then(function(nb){
console.log('Number of files renamed : ', nb);
}).catch(function(err){
console.log('Error ', err);
});
});
预期的结果是要重命名文件并查看消息重命名文件数。
我将文件重命名,但是看不到任何消息(then或catch呼叫的消息)。 某处有问题,但是调试会话无法帮助我。
谢谢你的帮助!
PS:我的环境是Node 5.10和OS X 10.11。
问题是您正在创建一些承诺,这些承诺将永远待定(永远不会解决):当regex
与filename
不匹配时。 Promise.all
将无限期地等待它们。
你应该总是promisify在尽可能低的水平-你的情况fs.rename
和fs.readdir
-并把没有其他的代码,功能与“老”回调API涉及内。 没有应用程序逻辑,没有字符串连接,没有日志记录,什么都没有。
function rename(from, to) {
return new Promise((resolve, reject) => {
fs.rename(from, to, (err, res) => {
if (err) reject(err);
else resolve(res);
});
});
}
function readdir(from, to) {
return new Promise((resolve, reject) => {
fs.readdir(from, to, (err, res) => {
if (err) reject(err);
else resolve(res);
});
});
}
(如果这看起来是重复的,那就是-编写一个辅助函数,或使用Promise库中的一个)
有了这些,您现在可以正确(并且更容易)实现脚本:
const dir = '/Users/toto/Desktop/png/';
const suffix = '_IMAGE.';
const regex = /(.*)\.(png)$/;
readdir(dir).then(files =>
Promise.all(files.map(filename => ({
from: dir + filename,
to: dir + filename.replace(regex, '$1' + suffix + '$2')
})).filter(r => r.from != r.to).map(r => {
let msg = r.from + " => " + r.to;
return rename(r.from, r.to).then(() => {
console.log("OK: " + msg);
}, err => {
console.log("KO: " + msg);
throw err;
});
}));
).then(function(res) {
console.log('Number of files renamed : ', res.length);
}).catch(function(err) {
console.error('Error ', err);
});
声明:本站的技术帖子网页,遵循CC BY-SA 4.0协议,如果您需要转载,请注明本站网址或者原文地址。任何问题请咨询:yoyou2525@163.com.