JS字符串处理函数hasNoneLetters主测试用例失败排查求助
问题分析与修复
你的hasNoneLetters函数存在三个关键问题,导致测试用例失败:
1. 小写转换未生效
你已经将phrase和blacklist转成了小写的x和y,但循环中依然使用原blacklist[i]获取字符。如果黑名单包含大写字母,会无法匹配到phrase里的小写字符,导致漏判。
2. lastIndexOf的判断逻辑错误
lastIndexOf找到字符时返回对应索引(>=0),未找到时返回-1。你当前的判断if(x.lastIndexOf(z))会把索引0(即字符在短语开头的情况)当成false,不触发返回false,这会导致本该判定为"存在黑名单字符"的情况被误判为不存在。
3. 循环条件冗余
i <= blacklist.length-1完全等价于i < blacklist.length,前者写法没必要,还容易出错。
修复后的代码
function hasNoneLetters(phrase, blacklist) { const lowerPhrase = phrase.toLowerCase(); const lowerBlacklist = blacklist.toLowerCase(); for (let i = 0; i < lowerBlacklist.length; i++) { const char = lowerBlacklist[i]; if (lowerPhrase.lastIndexOf(char) !== -1) { return false; } } return true; }
关键修改说明:
- 统一使用转换后的小写字符串进行匹配,避免大小写问题
- 把判断条件改为
lastIndexOf(char) !== -1,准确识别字符是否存在 - 简化循环条件,提升代码可读性
比如测试用例phrase="Hello", blacklist="H",原函数会返回true(错误),修复后的函数会正确返回false。
内容的提问来源于stack exchange,提问作者Luís Henrique
相关产品推荐
相关产品推荐

