如何使用JavaScript校验ATM机规则的PIN码？现有代码仅能通过部分测试

你代码的问题不是if语句太多，核心是逻辑顺序、规则校验、返回值三个层面都有错误：

具体问题清单 #

• 执行逻辑断裂：第一段长度判断逻辑只要符合4/6位就直接return true，函数直接终止执行，后续的字符校验代码完全不会运行，所有长度符合但含非数字的测试用例（比如"a123"、"12 4"）都会被误判为合法。
• 正则规则错误：你用的\w元字符不仅匹配0-9的数字，还会匹配大小写字母、下划线，完全不符合「只能是数字」的要求；同时正则没有限定匹配整个字符串，无法过滤中间夹杂非数字的情况。
• 返回值不符合要求：第二段分支里非法情况你返回的是pin本身，题目要求所有非法情况都要返回false。

修复方案 #

最简单的实现是用单个正则同时校验长度和字符规则，不需要多分支判断：

function validatePIN (pin) {
  return /^(\d{4}|\d{6})$/.test(pin)
}

正则规则说明：

• ^ 匹配字符串开头，$ 匹配字符串结尾，确保整个字符串完全符合规则，不会出现前后带空格、换行或其他多余字符的情况
• \d 仅匹配0-9的阿拉伯数字
• \d{4} 要求恰好4位数字，\d{6} 要求恰好6位数字，| 表示两种情况满足其一即可

如果非要用你原本的分支写法调整，也可以改成下面的逻辑，先校验字符全是数字，再校验长度：

function validatePIN (pin) {
  // 先判断是不是全数字，再判断长度是否符合要求
  if (/^\d+$/.test(pin)) {
    return pin.length === 4 || pin.length === 6
  }
  return false
}

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