Map.for创造的promise每个人都在第一次处理拒绝时停下来

Question

我正在尝试使用节点在Discord上创建一个机器人。js模块不一致。js，在它的一个命令中，它遍历公会（服务器）中的成员地图，并将所有成员都禁用。以下是我使用的代码：

const Discord = require('discord.js');

const client = new Discord.Client();

const map = new Map();

client.once('ready', () => {
    // This is just for demonstration purposes. In the actual code, the values are all different
    // members and the keys are their respective IDs. This is inside the "ready" event because
    // otherwise the client's cache would be empty.
    const hjtunfgb = client.guilds.cache.get('myServerID').members.cache.get('myID');
    map.set(0, hjtunfgb);
    map.set(1, hjtunfgb);
    map.set(2, hjtunfgb);
    map.set(3, hjtunfgb);
    map.set(4, hjtunfgb);
    map.set(5, hjtunfgb);
    map.set(6, hjtunfgb);
    map.set(7, hjtunfgb);
    map.set(8, hjtunfgb);
    map.set(9, hjtunfgb);

    console.log(`Logged in as ${client.user.tag}!`);
});

client.on('message', message => {
    // Here I have two commands: "foo", which mutes everyone in the Map, and "bar", which does other
    // stuff that is not relevant to the problem
    if (message.content.toLowerCase() === 'foo') {
        map.forEach((member, n) => {
            // member.voice.setMute(true) is how the bot mutes people. It returns a promise, that
            // rejects if the member is not connected to any voice channel in that server.
            member.voice.setMute(true)
                .then(() => console.log(n + ' muted!'))
                .catch(() => console.error(n + ' error!'));
        });
    } else if (message.content.toLowerCase() === 'bar') {
        console.log('baz');
    }
});

client.login('myBotsSecretLoginKey');

但是，如果一个成员没有连接到任何语音通道，setMute（）方法创建的promise将被拒绝，并且在这里通过记录错误消息来处理它（同样，在实际代码中，拒绝被正确处理）。问题是，在拒绝之后，forEach会停止并不会执行，直到我再次执行“foo”，然后它才会执行。例如，如果我在聊天中发送“foo”，并在“4静音”后立即断开连接如果已记录，则输出如下所示：

0 muted!
1 muted!
2 muted!
3 muted!
4 muted!
5 error!

如果我连接或断开与语音频道的连接，则不会发生任何事情，如果我写入“bar”，则程序正常输出“baz”。然后，如果我加入一个语音频道并再次发送“foo”，则输出以下内容：

0 muted!
6 muted!
7 muted!
8 muted!
9 muted!
1 muted!
2 muted!
3 muted!
4 muted!
5 muted!
6 muted!
7 muted!
8 muted!
9 muted!

因此，在Map上调用了第一个foreach命令，这触发了之前的foreach恢复操作。相反，如果我调用foo，然后加入并第三次调用foo，则前一次调用0错误！输出，后者的输出是

0 muted!
6 muted!
...
9 muted!
1 muted!
...
9 muted!
1 muted!
...
9 muted!

因此，即使拒绝得到处理，forEach也会停止拒绝promise。但是我需要forEach立即恢复，因为bot将不断地静音和取消静音，并且一个断开连接的成员不能阻止其他成员静音。我怎样才能解决这个问题？

编辑：使回调异步并在尝试中包装setMute。。。抓块并没有解决问题。而且，正如有人指出的，forEach确实在任何promise被拒绝之前运行到完成阶段——我通过控制台检查了这一点。运行前记录日志。setMute（）。所以，问题在于node discord如何处理promise。js，或不和谐本身。有人能提供信息说明为什么会发生这种情况，以及如何避免这种情况？

我还试着用。。。使用数组而不是映射，但它们都产生了相同的问题。

编辑2：我可以通过更改“message”事件侦听器中的回调来解决此问题，如下所示：

client.on('message', async message => {
    if (message.content.toLowerCase() === 'foo') {
        for (const [n, member] of map) {
            try {
                await member.voice.setMute(true);
                console.log(n + ' muted!');
            } catch (error) {
                console.error(n + ' error!');
            }
        }
    } else if (message.content.toLowerCase() === 'bar') {
        console.log('baz');
    }
});

这样，代码在创建下一个promise之前会等待该promise得到解决。这并不是最理想的方式，因为这些promise可以并行运行，但不一致性会一次处理一个promise，因此在实践中不会影响性能。我尝试使用以下设置在不使用forEach的情况下并行运行promise：

client.on('message', message => {
    if (message.content.toLowerCase() === 'foo') {
        Promise.all(Array.from(map.entries()).map(async ([n, member]) => {
            try {
                await member.voice.setMute(true);
                console.log(n + ' muted!');
            } catch (error) {
                console.error(n + ' error!');
            }
        }));
    } else if (message.content.toLowerCase() === 'bar') {
        console.log('baz');
    }
});

但它也有同样的问题。问题似乎在于节点如何，不协调。js和/或Discord同时处理多个请求的promise。我能做些什么来解决这个问题？同步运行异步代码目前还可以，但从长远来看，并行运行它们会更好。我是否应该将此报告为discord的bug。js和/或Discord，或者有一个我不知道的目的（和修复）？

Answer 1

在没有看到整个过程的情况下，我无法肯定地告诉您出了什么问题，但人们强烈怀疑setMute有时不会拒绝，而是抛出异常。在这种情况下，forEach将被终止。同样：您可以使用catch处理promise拒绝，但如果发生异常，则不处理在setMute中引发异常（通常，返回promise的函数不应编码为引发异常）。

现在，如果您将forEach内部的函数更改为async，那么您可以在代码内部使用wait，更重要的是，使用try/catch捕获promise拒绝和抛出setMute，如下所示：

client.on("message", (message) => {
    if (message.content.toLowerCase() === "foo") {
        map.forEach(async (member, n) => {
            try {
                await member.voice.setMute(true);
                console.log(n + " muted!");
            } catch (err) {
                console.error(n + " error! " + err);
            }
        });
    } else if (message.content.toLowerCase() === "bar") {
        console.log("baz");
    }
});

Answer 2

在forEach循环中不能使用promise（wait）。

看看这个问题。

在forEach循环中使用async/await

干杯