咨询isValidPassword()方法验证失效原因及合并返回可行性

一、验证失效的可能原因 #

先看你提供的isValidPassword()方法逻辑，理论上密码长度小于6或包含非字母数字字符时应该返回false，但你说验证失效，大概率是以下情况之一：

• 传入的password为null：如果password是null，调用password.length()会直接抛出NullPointerException，如果你的代码没有捕获这个异常，程序会跳过if(!SoccerUtils.isValidPassword(password))的判断，直接进入else块，看起来就像“验证未生效”。这是最常见的原因，建议先在调用前判断password是否为null。
• 调用前对password做了额外处理：比如你在传入方法前给password做了trim()或者自动填充字符的操作，导致原本长度不足6的密码被拉长，或者特殊字符被替换掉了。可以在调用方法前打印一下password的实际值确认。
• 代码版本不一致：你贴出来的方法代码和实际运行的代码可能不一样，比如不小心把return false写成了return true，这种低级错误也有可能发生，可以再检查一下实际运行的方法实现。

二、合并两个验证条件的实现方式 #

当然可以把两个条件合并成一处，用正则表达式会更简洁优雅，而且逻辑更清晰。这里给你两种实现方式：

方式1：正则表达式直接匹配（推荐） #

public static boolean isValidPassword(String password) {
    // 先判断非null，然后正则匹配：长度>=6，且全为字母或数字
    return password != null && password.matches("^[a-zA-Z0-9]{6,}$");
}

解释一下正则规则：

• ^ 标记字符串的开始位置
• [a-zA-Z0-9] 匹配任意大小写字母或数字
• {6,} 表示长度至少为6位
• $ 标记字符串的结束位置

方式2：优化原有循环逻辑（保留原生判断） #

如果你不想用正则，也可以在原有逻辑基础上优化，同时补上null判断避免异常：

public static boolean isValidPassword(String password) {
    // 先统一处理null和长度不足的情况
    if (password == null || password.length() < 6) {
        return false;
    }
    // 再遍历检查每个字符
    for (int i = 0; i < password.length(); i++) {
        char c = password.charAt(i);
        if (!Character.isLetterOrDigit(c)) {
            return false;
        }
    }
    return true;
}

另外，你的调用代码里，当验证不通过时只提示了“密码长度至少6位”，但如果是因为包含特殊字符导致的验证失败，用户会看到错误的提示信息，建议优化一下：

if (password == null) {
    out.print("<p style=\"color:red\">Password cannot be empty</p>");
} else if (!SoccerUtils.isValidPassword(password)) {
    String errorMsg = password.length() < 6 
        ? "Password must be at least 6 characters long" 
        : "Password can only contain letters and digits";
    out.print(String.format("<p style=\"color:red\">%s</p>", errorMsg));
    RequestDispatcher rd = request.getRequestDispatcher("register.jsp");
    rd.include(request, response);
} else {
    //Do some action here
}

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