为何多次迭代后代码输出出现"undefined"？

让我来帮你排查这个问题！从你描述的现象（初始能正常输出3个随机城市，迭代不到10次就出现undefined）和给出的代码片段来看，最可能的成因和你遗漏的细节主要有这些：

1. 原数组元素被耗尽，导致随机索引指向空值 #

你大概率在未展示的函数逻辑里，做了「从coolWords中随机选元素，同时把选中的元素从原数组里移除」的操作（比如用splice()方法）。

举个最典型的错误场景：假设你的函数是这样写的：

function pickRandomCities() {
  newList = [];
  for (let i = 0; i < 3; i++) {
    const randomIdx = Math.floor(Math.random() * coolWords.length);
    newList.push(coolWords[randomIdx]);
    coolWords.splice(randomIdx, 1); // 移除已选中的元素
  }
  console.log(newList);
}

第一次调用时，coolWords有9个元素，能正常取3个；第二次剩6个，没问题；第三次剩3个，刚好取完；第四次调用时coolWords已经是空数组了，此时Math.random() * 0结果为0，访问coolWords[0]自然会得到undefined。

2. 随机索引计算未动态适配数组长度 #

如果你的随机索引计算写死了固定值（比如Math.floor(Math.random() * 9)），而不是用coolWords.length动态获取当前数组长度，当coolWords元素被移除、长度变短后，就可能生成超出数组最大索引的数值，访问这个位置的元素就会返回undefined。

3. 全局数组newList未正确重置（概率较低） #

如果newList是全局变量，且每次迭代前没有清空，或者在某些分支逻辑中不小心把undefined推进了数组，也可能出现这个问题，但结合你“初始正常、几次后出问题”的现象，这个原因的可能性相对小一些。

• 允许重复选城市：直接复用原数组，不要修改它。每次都从完整的coolWords里取随机元素，永远不会出现索引越界：

function pickRandomCities() {
  newList = [];
  for (let i = 0; i < 3; i++) {
    const randomIdx = Math.floor(Math.random() * coolWords.length);
    newList.push(coolWords[randomIdx]);
  }
  console.log(newList);
}

• 需要选不重复的城市，但要保留原数组：每次迭代前复制一份原数组到临时变量，从临时变量里操作，不要动原数组：

function pickRandomCities() {
  const tempWords = [...coolWords]; // 复制原数组的副本
  newList = [];
  for (let i = 0; i < 3; i++) {
    const randomIdx = Math.floor(Math.random() * tempWords.length);
    newList.push(tempWords[randomIdx]);
    tempWords.splice(randomIdx, 1);
  }
  console.log(newList);
}

• 确保索引计算动态化：永远用array.length来计算随机索引，不要写死固定的数字，避免数组长度变化后出现索引越界。

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