javascript - JS : Maximum call stack size exceeded on disconnect despite function being empty

Question

我对 js 有点陌生，正在尝试使用 net 模块。

我有一个简单的服务器在运行，它将包分发给所有已知的客户端，但是一旦一个客户端断开连接，服务器就会在“结束”事件上崩溃，尽管这个功能只包括一个日志命令:

var clients = [];

const server = net.createServer((c) => {

  // 'connection' listener
  console.log("client connected")
  clients.push(c);

  c.on('end', () => {
    console.log('client disconnected');
    //dc(c)
  });

  c.on('error', () => {
    c.write("400");
  })

  c.on('data', (data) => {

    console.log(data.toString())
    writeAll(c, data);

  });

});

Write all 是下面的一个函数，它将传入的包分发给所有已知的客户端，不包括 c:

function writeAll(exclude, buffer) {

  clients.forEach((client) => {
    if(client != exclude) {
      client.write(buffer);
    }
  });

}

dc() 是一个函数，它应该从我之前定义的数组列表中删除客户端，但被注释掉了，所以它应该无关紧要。 一切正常，直到一个客户端断开连接，即当我收到以下错误消息时:

internal/errors.js:207
function getMessage(key, args) {
                   ^

RangeError: Maximum call stack size exceeded
    at getMessage (internal/errors.js:207:20)
    at new NodeError (internal/errors.js:153:13)
    at doWrite (_stream_writable.js:411:19)
    at writeOrBuffer (_stream_writable.js:399:5)
    at Socket.Writable.write (_stream_writable.js:299:11)
    at Socket.c.on (C:\Users\...\server.js:17:7)
    at Socket.emit (events.js:197:13)
    at errorOrDestroy (internal/streams/destroy.js:98:12)
    at onwriteError (_stream_writable.js:430:5)
    at onwrite (_stream_writable.js:461:5)

引用的第 17 行是您在上面看到的 on-error 事件。 我已经尝试将最大堆栈长度扩展到 2000，但它仍然不起作用。最奇怪的是，抛出错误消息的部分是从另一个我知道有效的代码复制粘贴过来的。

如果你们中的一位专业人士愿意接受这个，我将不胜感激，因为我完全迷失了。

编辑:完整文件可以在这里找到:https://ghostbin.com/paste/knm3f

编辑 2:我重新安装了 net 模块，现在错误发生了变化，让我更加困惑:

internal/errors.js:222
  const expectedLength = (msg.match(/%[dfijoOs]/g) || []).length;
                              ^
RangeError: Maximum call stack size exceeded
    at String.match (<anonymous>)
    at getMessage (internal/errors.js:222:31)
    at new NodeError (internal/errors.js:153:13)
    at doWrite (_stream_writable.js:411:19)
    at writeOrBuffer (_stream_writable.js:399:5)
    at Socket.Writable.write (_stream_writable.js:299:11)
    at Socket.c.on (C:\Users\...\server.js:17:7)
    at Socket.emit (events.js:197:13)
    at errorOrDestroy (internal/streams/destroy.js:98:12)
    at onwriteError (_stream_writable.js:430:5)

编辑 3:确实有效的代码已经过测试，由于某种原因，即使在编写它的机器上以及我的笔记本电脑上也会遇到同样的问题。我感觉这不适合我。

编辑 4:SOOOOOOOOOOO... 事实证明，Microsoft 再次将拳头放在了我的后端。它在 Linux 发行版上运行良好 100。我会把赌注押在 Windows 防火墙更改或我不同意的其他更新上。谢谢微软。你是在逼我迁移到 Linux。

编辑 5:断开连接的错误仅在 Windows 上抛出，但 stackoverflow 是由写入客户端的错误事件引起的，它本身抛出错误，因为客户端不再连接。

Answer 1

您的代码中有两个错误，它们绝对不是太明显。

无论何时向客户端写入，都使用 writeAll() 中的循环:

clients.forEach((client) => {
  if(client != exclude) {
    client.write(buffer);
  }
});

在这里我们看到您执行了 client.write(...)。该命令可能会检测到此特定 client 已关闭其连接。这意味着 emit('error') 将会发生，您的相应回调将被调用。

在那个回调中，你正在调用 dc(c)，它应该从数组中删除客户端:

clients.splice(index, 1)

换句话说，您在执行 clients.forEach() 时正在修改名为 clients 的数组。这是一个很大的禁忌!实际上，我试图完全忘记 forEach() 函数。它给我带来了很多问题!有人说 forEach() 也比 for() 或 while() 循环慢。但这个问题可能已经得到改善。

至少，您要做的是使用 for() 循环:

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

请注意，我没有先将 clients.length 保存在变量中，因为它可能会随时间变化!

这种方法的问题在于，无论何时发生 splice()，您都会错过一些客户。

下一个最好的办法是使用向后循环。为此，while() 非常适用:

let i = clients.length
while(i > 0)
{
    --i
    ...
}

只要满足以下条件，这个循环就会工作得更好:(1) 只有一个项目被 splice() 输出，(2) 没有新的客户端被添加到列表中。

我建议的最后一个强有力的解决方案是首先复制数组:

const client_list = clients.slice()
client_list.forEach((client) => { ... })    // if you really want to use the forEach()?!

副本不会改变:它是本地的并且const。

但为什么堆栈会因此变得困惑？可能是因为错误导致您对客户端执行了 write() ......好吧，这与 forEach() 无关，但此代码循环:

c.on('error', () => {
    c.write("400")     // are you missing a "\n", btw?
});

如您所见，c.write() 是产生错误的罪魁祸首，您将在这个 on() 中永远循环回调...您需要检查 c 是否有效或捕获错误:

c.on('error', () => {
    try
    {
        c.write("400")     // are you missing a "\n", btw?
    }
    catch(err)
    {
        // maybe log the error?
        console.log(err)
    }
});

也就是说，我从经验中知道，在我从我的代码中删除所有 forEach() 调用并制作数组(对象)的副本后，我无法确定它们是否不会得到在我的脚下修改后，我的代码运行得更好。