javascript - 使用 promise.All 和 await 循环遍历文件操作列表的正确方法

Question

好吧，我迷失在等待和异步 hell 中。下面的代码应该遍历文件列表，检查它们是否存在并返回存在的文件。但我得到的是零长度列表。

Node V8 代码:调用者:

await this.sourceList()
if (this.paths.length == 0) {
    this.abort = true
    return
}

调用的函数:(我去掉了不相关的东西)

const testPath = util.promisify(fs.access)

class FMEjob {
    constructor(root, inFiles, layerType, ticket) {
        this.paths = []
        this.config = global.app.settings.config
        this.sourcePath = this.config.SourcePath
    }
    async sourceList() {
        return await Promise.all(this.files.map(async (f) => {
            let source = path.join(this.sourcePath, f.path)
            return async () => {
               if (await checkFile(source)) {
                   this.paths.push(source)
               }
            }
        }))
    }
    async checkFile(path) {
        let result = true
        try {
            await testPath(path, fs.constants.R_OK)
        }
        catch (err) {
            this.errors++
            result = false
            logger.addLog('info', 'FMEjob.checkFile(): File Missing Error: %s', err.path)
        }
        return result
    }

Answer 1

您的sourceList 函数真的很奇怪。它返回一个异步 函数 数组的 promise ，但它从不调用这些函数。删除箭头函数包装器。

我还建议永远不要在 async 方法中改变实例属性，这会在同时执行多个方法时导致疯狂的错误。

this.paths = await this.sourceList()
if (this.abort = (this.paths.length == 0)) {
    return
}

async sourceList() {
    let paths = []
    await Promise.all(this.files.map(async (f) => {
        const source = path.join(this.sourcePath, f.path)
        // no function here, no return here!
        if (await this.checkFile(source)) {
            paths.push(source)
        }
    }))
    return paths
}
async checkFile(path) {
    try {
        await testPath(path, fs.constants.R_OK)
        return true
    } catch (err) {
        logger.addLog('info', 'FMEjob.checkFile(): File Missing Error: %s', err.path)
        this.errors++ // questionable as well - better let `sourceList` count these
    }
    return false
}