如何正确迭代数组并处理作用域？数组元素相除找最高分异常

首先咱们来拆解你代码里的几个核心问题：

1. arrayCombi 的创建方式错误 #

你在每次外层循环里都重新创建一个只包含当前元素除法结果的数组：let arrayCombi = [arrayOne[i] / arrayTwo[i]];，这意味着arrayCombi永远只有一个元素，既没法存储所有的除法结果，后续也没法正常调用。而且每次循环都会覆盖这个变量，完全达不到你想要的“存储所有对应索引相除结果”的目的。

2. 找最高分的逻辑位置错误 #

你把找最高分的内层循环放在了外层循环里面，而且每次外层循环都会把topScore重置为0。这就导致每次计算完一个除法结果后，都会和0比较——只要结果是正数（你的数组元素都是正数），就会触发if条件，输出当前结果作为topScore，这就是为什么你看到每个值都被当成topScore输出的原因。

3. 变量未正确声明 #

你的循环变量i和a没有用let声明，会变成全局变量，可能导致循环逻辑混乱，出现意想不到的bug。

修正后的代码 #

let arrayOne = [2,7,5,89,4,67,8,5,3,8]; 
let arrayTwo = [5,87,5,3,7,89,5,34,7,98]; 

// 把arrayCombi移到外层，初始化空数组存储所有除法结果
let arrayCombi = [];

// 先遍历数组，计算所有对应索引的除法结果，存入arrayCombi
for(let i = 0; i < arrayOne.length; i++){ 
  arrayCombi.push(arrayOne[i] / arrayTwo[i]); 
}

// 单独计算最高分，此时arrayCombi已经有所有结果了
let topScore = 0;
for(let a = 0; a < arrayCombi.length; a++){ 
  if (arrayCombi[a] > topScore){ 
    topScore = arrayCombi[a]; 
    console.log("Top Score: " + topScore); 
  } 
}

// 现在可以随时调用arrayCombi，比如查看所有结果
console.log("所有相除结果：", arrayCombi);

额外优化（可选） #

如果想简化代码，还可以用数组的map方法生成arrayCombi，用Math.max直接找最高分：

let arrayOne = [2,7,5,89,4,67,8,5,3,8]; 
let arrayTwo = [5,87,5,3,7,89,5,34,7,98]; 

// 用map生成除法结果数组
const arrayCombi = arrayOne.map((num, index) => num / arrayTwo[index]);
// 直接找最高分
const topScore = Math.max(...arrayCombi);

console.log("所有相除结果：", arrayCombi);
console.log("最终最高分：", topScore);

这样代码更简洁，可读性也更高～

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