JavaScript循环练习求助：变量作用域与嵌套循环问题调试

你的代码存在多个逻辑错误，导致无法正确计算元音与长度的比值，以下是具体问题和修正方案：

原代码核心错误 #

• 重复调用gets()导致取值异常：循环内每次调用两次gets()，但测试输入只有4个食物，第三次循环时gets()会返回undefined，直接导致for...of循环报错。
• 计数器作用域与拼写错误：全局计数器counter未在每次处理单个食物时重置，且最后重置时拼写错误为couner，导致计数混乱。
• 变量拼写错误：retio1、retio2应为ratio1、ratio2，拼写错误导致变量未正确赋值。
• 比较逻辑混乱：当前的比值比较逻辑完全颠倒，没有正确跟踪当前最高比值的食物，反而把变量互相覆盖，导致结果错误。
• 未初始化变量：vowels、ratio初始为undefined，直接参与运算会引发NaN错误。

修正后的代码 #

const getGets = (arr) => {
  let index = 0;
  return () => {
    const toReturn = arr[index];
    index += 1;
    return toReturn;
  };
};

// Test input
const testInput = [
  '4', 'pizza', 'macaroni', 'kiufte', 'banica'
];

const gets = this.gets || getGets(testInput);
const n = +gets();

// 初始化最高比值相关变量
let maxRatio = 0;
let bestFood = '';
let bestVowelCount = 0;

// 辅助函数：计算单个字符串中的元音数量
const countVowels = (str) => {
  const vowels = ['a', 'o', 'u', 'e', 'i'];
  let count = 0;
  for (let char of str) {
    if (vowels.includes(char.toLowerCase())) { // 兼容大写情况，可选
      count++;
    }
  }
  return count;
};

// 遍历每个食物
for (let i = 0; i < n; i++) {
  const currentFood = gets();
  const vowelCount = countVowels(currentFood);
  const currentRatio = vowelCount / currentFood.length;

  // 比较当前比值与最高比值，更新最高值
  if (currentRatio > maxRatio || (currentRatio === maxRatio && currentFood.length > bestFood.length)) {
    maxRatio = currentRatio;
    bestFood = currentFood;
    bestVowelCount = vowelCount;
  }
}

// 输出结果
console.log(`${bestFood} ${bestVowelCount}/${bestFood.length}`);

代码说明 #

1. 正确获取食物：循环n次，每次调用一次gets()，确保每个食物都被正确读取，避免undefined问题。
2. 独立元音计数函数：将元音计数逻辑抽离为辅助函数，代码更清晰，避免嵌套循环的混乱。
3. 跟踪最高比值：用maxRatio、bestFood、bestVowelCount三个变量跟踪当前最优结果，每次处理完一个食物就进行比较更新。
4. 处理平局情况：如果两个食物比值相同，可选保留长度更长的食物（可根据需求调整逻辑）。

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