JS不使用replace/regex/includes实现元音大写的代码问题排查
问题根因
- 核心错误是两层循环的逻辑错误:元音列表有
aeiou共5个字符,外层每遍历1个原字符串的字符,内层会循环5次,每次循环都会往newString里拼接1次字符,最终每个原字符会被重复拼接5次,这就是你输出字符数是预期5倍的原因。 - 其次判断逻辑错误:即使字符是元音,也只会在内层循环匹配到对应元音的那1次拼接大写,其余4次还是会拼接小写,导致同一个字符同时出现大小写混合的重复结果。
修复方案
调整逻辑为内层循环仅负责判断当前字符是否为元音,待判断完成后每个字符仅拼接1次,完全符合你要求的不使用replace、正则、includes方法的限制,修改后代码如下:
flipCase(); function flipCase() { var string = prompt("Please enter a string:"); var vowels = "aeiou"; var newString = ""; for (var i = 0; i < string.length; i++) { var currentChar = string.charAt(i); var isVowel = false; // 内层循环仅做元音判断,不直接拼接 for (var j = 0; j < vowels.length; j++) { if (currentChar.toLowerCase() === vowels.charAt(j)) { isVowel = true; break; // 匹配到元音直接跳出内层循环,减少冗余遍历 } } // 每个字符仅拼接1次 if (isVowel) { newString += currentChar.toUpperCase(); } else { newString += currentChar.toLowerCase(); } } console.log(newString); }
输入open时运行上述代码,输出结果为预期的OpEn。
内容的提问来源于stack exchange,提问作者NewbieAeg
相关产品推荐
相关产品推荐

