电话号码格式匹配函数失效，寻求高效实现方案

原代码的问题 #

你的代码逻辑存在多处错误，导致无法正确验证电话号码格式：

• 遍历方式错误：使用for...in遍历数组，它会遍历数组的所有可枚举属性，包括非索引属性，不适合用来遍历数组元素。
• 提前返回：在循环的第一次迭代中就执行了return false，根本没完成所有字符的处理，也无法正确判断最终格式。
• 逻辑运算符错误：判断条件中用了位或运算符|，应该用逻辑或运算符||，位或会对数值进行位运算，完全不符合你的判断需求。
• 核心思路错误：把非5的数字替换成5再匹配固定格式的思路完全偏离了“验证格式”的目标，你需要的是检查字符串的结构是否符合规则，而不是把数字改成特定值。

高效实现方法 #

验证这类固定格式的字符串，最高效的方式是使用正则表达式。下面是符合你需求的实现：

function telephoneCheck(str) {
  const regex = /^(1\s?)?(\(\d{3}\)|\d{3})([\s-]?)\d{3}\3\d{4}$/;
  return regex.test(str);
}

console.log(telephoneCheck("1 456 789 4444")); // 返回true
console.log(telephoneCheck("555-555-5555")); // 返回true
console.log(telephoneCheck("(555)555-5555")); // 返回true
console.log(telephoneCheck("(555) 555-5555")); // 返回true
console.log(telephoneCheck("555 555 5555")); // 返回true
console.log(telephoneCheck("5555555555")); // 返回true
console.log(telephoneCheck("123456789")); // 返回false，格式不符

正则表达式解释 #

• ^：匹配字符串开头
• (1\s?)?：可选的国家码"1"，后面可以跟一个空格或者没有空格
• (\(\d{3}\)|\d{3})：匹配3位区号，格式可以是带括号的(555)或者不带括号的555
• ([\s-]?)：可选的分隔符，空格或者短横线，用括号捕获这个分隔符，后面可以引用
• \d{3}：匹配中间3位数字
• \3：引用前面捕获的分隔符，保证中间和最后的分隔符一致（比如前面用空格，后面也必须用空格；前面用短横线，后面也必须用短横线；或者都没有）
• \d{4}：匹配最后4位数字
• $：匹配字符串结尾

这样就能准确匹配你列出的所有合法格式，同时拒绝不符合规则的字符串。

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