JavaScript随机生成器频繁返回undefined问题排查求助

嘿，我一眼就抓住你描述里的关键线索了——80%概率返回undefined、数组长度相同就没问题、调换长度乘法顺序就消失，这几乎肯定是随机索引生成时搞混了对应数组的长度！

问题根源分析 #

你大概率是在生成第一个数组（one）的随机索引时，错误地用了第二个数组（two）的长度来计算。举个完全匹配你场景的例子：

• 假设one只有2个元素，two有10个元素
• 你写了类似 var index = Math.floor(Math.random() * two.length) 来取one的元素
• 这时候Math.random()*10会有80%的概率生成≥2的数值，用这个当索引去访问长度为2的one，自然就返回undefined——完美对应你说的80%概率！

其他现象也能对应上：

• 数组规模增大时：比如one的长度变成8，超出范围的概率就降到20%，问题出现频率自然降低
• 两个数组长度相同时：乘哪个长度结果都一样，索引不会越界，所以无问题
• 调换* two.length和* one.length的顺序：相当于把错误的长度换成了对应数组的正确长度，索引合法了，问题就消失了

修复后的代码示例 #

假设你原来的错误逻辑是从两个数组取随机元素，正确的写法应该是每个数组的索引用自己的长度计算：

var one = ['Abashed', 'Abnormal', ...]; // 你的第一个数组
var two = ['Xyz', 'Yyy', ...]; // 你的第二个数组

// 正确生成随机元素对的函数
function getRandomPair() {
  // 用one的长度生成one的合法索引
  var indexOne = Math.floor(Math.random() * one.length);
  // 用two的长度生成two的合法索引
  var indexTwo = Math.floor(Math.random() * two.length);
  return [one[indexOne], two[indexTwo]];
}

关键注意点 #

Math.random()返回的是0（包含）到1（不包含）的浮点数，乘以数组长度后用Math.floor()取整，能确保得到0到数组长度-1的合法索引，绝对不会出现越界返回undefined的情况。

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