Promise 解决但没有触发

Question

我创建了一个脚本，它遍历一堆文件夹并将它们分别处理到 webpack 包中。这很好用，只是我不知道为什么循环周围的Promise 没有解决。

我尝试过的一些事情：

• 如果我在resolve() 之前放置一个console.log("hello world")，在} else { ... } 中，它会输出日志。
• 如果我将 resolve() 移出 } else { ... }，它会解决，但我的 gulp 任务的其余部分不会继续（单独但相关的问题）。

如果您能帮我解决这个问题，我将不胜感激。最相关的代码块在下面，其余的在下面的链接中。

// process all the script folders
const process_script_folders = () => {
   return new Promise((resolve) => {
       const FOLDER = script_folders.shift();

       // lint all scripts, except for critical
       if (FOLDER !== "critical") {
           const linted = lint_scripts(js_directory, FOLDER + ".js", source_directory + "/" + FOLDER + "/**/*");
           merged_streams.add(linted);
       }

       process_scripts(js_directory, FOLDER + ".js", source_directory + "/" + FOLDER + "/**/*").then((processed) => {
           merged_streams.add(processed);

           if (script_folders.length > 0) {
               process_script_folders();
           } else {
               // @TODO figure out why this isn't resolving
               resolve();
           }
       });
   });
};

return process_script_folders().then(() => {
    // ... do stuff
    console.log("Testing"); // currently never output
});

https://github.com/JacobDB/new-site/blob/dfeeb3260ab1b314e7562ef313c181adf2ef7f9c/gulp-tasks/scripts.js#L86-L89

Answer 1

您正在递归调用您的 Promise，但是除了最后一个 Promise 之外，这些 Promise 都不会解决，因为您已将您的 resolve 放入您的 resolve 子句中。我认为如果您将解析放在 else 之外并保持其余部分相同，则它应该在完成递归后解析。

if (script_folders.length > 0) {
    process_script_folders();
} 
resolve();

你可以这样试试吗？

编辑：@Bergi 是对的。我认为这样做应该可以正常工作。首先是我建议你做的一个简单的例子：

let i = 0
const example_processing = () => {
    return new Promise((resolve) => {
        i++
        setTimeout(resolve, 1000);
    }).then(() => {
        console.log(i);
        return i < 10 ? example_processing() : "done"
    });
};
example_processing().then(console.log);

关于你的代码，我猜它看起来更像这样：

const process_script_folders = () => {
    return new Promise((resolve) => {
        const FOLDER = script_folders.shift();

        // lint all scripts, except for critical
        if (FOLDER !== "critical") {
            const linted = lint_scripts(js_directory, FOLDER + ".js", source_directory + "/" + FOLDER + "/**/*");
            merged_streams.add(linted);
        }

        process_scripts(js_directory, FOLDER + ".js", source_directory + "/" + FOLDER + "/**/*").then((processed) => {
            merged_streams.add(processed);
            resolve();
        });
    }).then(() => script_folder.length > 0 ? process_script_folders() : "done");
 };

Answer 2

我认为问题与您进行递归调用的方式有关。

尝试一下

if (script_folders.length > 0) {
    process_script_folders().then(resolve);
} else {
    resolve();
}