JavaScript检测到字符串含空格时如何正确返回false

原代码问题分析 #

原代码运行不符合预期的核心原因有3点：

• 存在无效逻辑：写了for循环但循环内没有用到遍历索引i，第一次迭代就会触发返回，循环完全没有作用
• 判断顺序颠倒：先执行子串包含判断，传入' '和' '时，单空格本身是双空格的子串，indexOf会直接匹配成功返回true，后续的空格判断分支根本不会执行
• 空白检测覆盖不全：仅判断了字符串是否全等于单个半角空格，既检测不到字符串首尾/中间夹带空格的场景，也覆盖不了多个空格、制表符\t、换行\n、全角空格等其他常见空白字符

空白字符的正确检测方式 #

JavaScript中最稳妥的通用空白检测方案是使用正则/\s/，这个正则元字符可以匹配所有标准空白字符，比单独判断' '的覆盖范围全。如果业务场景只需要检测半角空格，把正则替换为/ /即可。

修正后代码 #

按照「任意字符串包含空白就返回false，否则不区分大小写判断两个字符串是否互为子串」的需求，修正后的代码如下：

function checkSubstring(text, part) {
  // 优先检测空白，任意参数含空白直接返回false
  if (/\s/.test(text) || /\s/.test(part)) {
    return false;
  }
  // 统一转小写后做双向子串匹配
  const textLower = text.toLowerCase();
  const partLower = part.toLowerCase();
  return textLower.includes(partLower) || partLower.includes(textLower);
}

// 测试验证
console.log(checkSubstring(' ', '  ')); // false 符合预期
console.log(checkSubstring('Hello', 'ell')); // true
console.log(checkSubstring('test', 'TEST')); // true 不区分大小写匹配
console.log(checkSubstring('hello world', 'hello')); // false 第一个参数含空格

注：如果需要兼容更老的不支持includes方法的JS环境，可以把includes判断替换为原写法的indexOf(xxx) !== -1，效果完全一致。

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