时间戳

首页 > 解决方案 > JS：尽管函数为空，但在断开连接时超出了最大调用堆栈大小

javascript - JS：尽管函数为空，但在断开连接时超出了最大调用堆栈大小

问题描述

我对 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 ......事实证明，微软再次把它的拳头放在了我的后端。它在 Linux 发行版上运行良好 100。我会将赌注押在 Windows 防火墙更改或我不同意的其他更新上。谢谢微软。你强迫我迁移到 Linux。

编辑 5：断开时的错误仅在 Windows 上引发，但 stackoverflow 是由写入客户端的错误事件引起的，由于客户端不再连接，因此引发了错误。

标签： javascriptnode.jstcpmodule

解决方案

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

每当您写信给客户时，您都会使用以下循环writeAll()：

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

在这里，我们看到你做到了client.write(...)。该命令可能会检测到这个特定client的关闭了它的连接。这意味着emit('error')将要发生并且您相应的回调将被调用。

在该回调中，您正在调用dc(c)预期从数组中删除客户端：

clients.splice(index, 1)

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

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

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

请注意，我不会首先将 保存clients.length在变量中，因为它可能会随着时间而改变！

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

下一个最好的办法是使用循环向后。为此 awhile()非常适合：

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()从代码中删除所有调用并复制数组（对象）之后，我不确定它们是否不会在我的脚下被修改，我的代码工作得更好。

推荐阅读