Node.js构造函数中如何将频道名称添加到调用者对象数组？

嘿，别担心，新手遇到this指向问题太正常了！我来一步步帮你理清问题，然后给你几个简单易懂的解决方案~

首先先点明你代码里的两个核心问题：

• 问题1：遍历数组时修改原数组：你把server.channels.array()赋值给了局部变量channels，然后在forEach里又往这个channels数组push元素——这会导致forEach永远遍历不完，因为数组长度一直在增加，相当于触发了无限循环！
• 问题2：forEach匿名函数里的this指向错误：普通函数作为forEach的回调时，它的this默认指向全局对象（Node.js里是global），而不是你的server构造函数实例，所以直接用this.channels会找不到你想要的那个实例数组。

接下来给你几个适合新手的解决方案：

方案1：提前保存this的引用（经典新手友好写法） #

在forEach执行前，把构造函数里的this存到一个变量里（比如叫self或者that），这样在回调函数里就能通过这个变量访问到实例的this.channels了，同时我们要新建临时数组来存频道列表，避免修改正在遍历的数组：

function Server(server) { 
  // 初始化实例的channels数组
  this.channels = [];
  // 保存当前实例的this引用，方便回调函数里使用
  const self = this;
  // 用临时变量存所有频道，避免遍历过程中修改原数组
  const allChannels = server.channels.array(); 
  allChannels.forEach(function(selectedchannel){ 
    if (selectedchannel.constructor.name === "TextChannel") { 
      // 通过self访问实例的channels数组，添加频道名称
      self.channels.push(selectedchannel.name); 
    } 
  }); 
}

方案2：使用箭头函数（ES6+，更简洁） #

箭头函数不会绑定自己的this，它会继承外层作用域的this，所以直接在forEach里用箭头函数的话，this就会自动指向构造函数的实例：

function Server(server) { 
  this.channels = [];
  const allChannels = server.channels.array(); 
  allChannels.forEach((selectedchannel) => { 
    if (selectedchannel.constructor.name === "TextChannel") { 
      // 这里的this就是构造函数的实例，直接访问即可
      this.channels.push(selectedchannel.name); 
    } 
  }); 
}

方案3：利用forEach的第二个参数（指定this指向） #

forEach方法本身支持第二个参数，用来指定回调函数里的this指向，把构造函数的this传进去就能解决问题：

function Server(server) { 
  this.channels = [];
  const allChannels = server.channels.array(); 
  allChannels.forEach(function(selectedchannel){ 
    if (selectedchannel.constructor.name === "TextChannel") { 
      this.channels.push(selectedchannel.name); 
    } 
  }, this); // 这里的this就是要绑定给回调函数的上下文
}

最后给你个小建议：判断对象类型时，用selectedchannel instanceof TextChannel会比constructor.name更可靠哦，能避免一些特殊场景下的判断错误~

内容的提问来源于stack exchange，提问作者leumasme