它的发送数量是会员数量的 4 倍

Question

所以我希望某个服务器中的机器人每 10 分钟向随机成员发送一个 dm。 当我的机器人从服务器向每个人发送 dm 时，它会发送一条完整的消息。

但是当我启动机器人时，它会发送 4 倍的成员数量

if (message.content.startsWith(botconfig.prefix + 'dmall')) {
    console.log("demo");
    var list = message.guild.members.array();
    sendMessage(list);
  }
});


function sendMessage(list) {
  setTimeout(function () {
    for (i = 0; i < list.length; i++) {
      console.log(list.length);
    }

    console.log("I'm done, mate!");
    sendMessage(list);
  }, 10 * 1000);
}

安慰：

demo
4 (is the amount of the members)
4
4
4
I'm done mate!

Answer 1

这部分代码：

for (i = 0; i < list.length; i++) {
  console.log(list.length);
}

告诉 Javascript 运行语句：

  console.log(list.length);

list.length次。 如果list.length是4 ，它似乎在这里，那么您将在控制台中看到

4
4
4
4

这就是代码被指示做的事情。

除非您想分别输出每个数组元素，否则我看不出有任何理由将其放入循环中。 所以，如果你只想输出一次长度，那么替换这个：

for (i = 0; i < list.length; i++) {
  console.log(list.length);
}

有了这个：

console.log(list.length);

---

此外，如果您要使用for循环，则必须声明您使用的所有变量。 所以这：

for (i = 0; i < list.length; i++) {

非常危险。 它依赖于更高范围的i ，它很容易与其他更高范围的i变量发生冲突并产生难以解决的错误。 每个for循环都应该声明它自己的循环变量，如下所示：

for (let i = 0; i < list.length; i++) {

如果你在严格模式下运行你的代码（你应该这样做），上面的 for 循环声明可能会导致一个错误（这是一件好事，因为你会立即看到编码错误并修复它）。

Answer 2

这里有一些错误。 对于：JavaScript 变量作用域让你一团糟（ 这里有不同的例子

其次：setTimeout 与 setInterval 不同。

完全解决您的问题将是：

 // Use the discord API let users = ['justme'] function getUsers() { // Mutate just to display a new one each time users = [...users, users.length.toString()]; return users } function notifyUsers() { // Retrieve all users now const currentUsers = getUsers() // Send a message to each user currentUsers.forEach(user => sendMessage(user)) console.log('I have sent everyone a message') } function sendMessage(user) { console.log(`Sending message to ${user}`) } // Start the application by looping every <whatever> setInterval(() => notifyUsers(), 1000)