从数组随机选取索引始终返回同一值？JavaScript代码问题求助

嘿，我来帮你揪出问题根源！你的代码里有几个小疏漏导致每次都返回同一个结果，咱们挨个拆解来看：

1. 数组名称拼写失误 #

你定义的数组是var letters = ["a", "b", "c", "d", "e", "f", "g", "h"]，但访问元素时写成了letter[Math.round(...)]——这里少了一个s，应该是letters[...]。如果letter是之前定义过的某个固定值（比如刚好是"g"），那每次都会取这个变量的“索引”，自然得不到随机结果。

2. 引用了错误的数组长度 #

你用了quotes.length，但目标数组明明是letters，这里必须替换成letters.length。如果quotes是另一个长度固定的数组（比如长度为7），那计算出来的索引范围会和letters不匹配，进而导致结果固定。

3. 随机索引的计算逻辑有问题 #

就算前面两个错误改了，Math.round(Math.random()*(letters.length))也不是获取数组随机索引的正确方式：

• Math.random()返回[0, 1)区间的数，乘以letters.length（也就是8）后得到[0, 8)；
• Math.round()会把0-0.5转为0，0.5-1.5转为1……7.5-8转为8，但你的数组索引只有0到7，得到8时会取到undefined；
• 而且这种方式的随机分布不均匀，0和8的概率只有其他索引的一半。

正确的做法是用Math.floor(Math.random() * letters.length)，它会把[0,8)区间的数向下取整，得到0到7的整数，刚好覆盖数组所有有效索引，且分布均匀。

修复后的代码 #

var letters = ["a", "b", "c", "d", "e", "f", "g", "h"];
var randomIndex = Math.floor(Math.random() * letters.length);
var letter = letters[randomIndex];
console.log(letter); // 现在会随机返回数组里的字母啦

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