使用try/catch块验证邮箱用户名与密码的问题排查

你的代码有几个关键错误导致无论输入什么都会输出注册成功，下面逐个说明并修复：

1. 从未调用验证方法 #

你定义了validateUsername和validatePassword验证方法，但在main里只是读取了用户名和密码，完全没调用这两个方法，所以根本不会触发任何异常判断，直接执行后续的成功提示。

2. 成功提示的位置错误 #

System.out.println("Successfully signed up.");写在try/catch块外面，就算验证时抛出异常被catch捕获，执行完异常提示后还是会走到这句代码，导致无论验证通不通过都会输出成功。

3. 用户名验证逻辑完全颠倒 #

当前validateUsername的逻辑是：如果用户名匹配.+@.+\\.com格式（符合邮箱规则），就抛出"无效用户名"的异常，这和你的需求完全相反——应该是不匹配格式时才抛异常。

4. 密码验证逻辑严重错误 #

• 循环里要求每个字符同时是大写、小写、数字、特殊字符，这不可能实现（比如一个字符是大写字母，就不可能同时是小写或数字），会直接抛出异常。正确逻辑应该是遍历所有字符，标记是否出现过每种类型，最后检查所有标记是否为true。
• password.charAt(ch)是错误用法：ch是字符类型，charAt需要传入索引（int类型），应该直接用ch判断是否为数字/字母。
• 你的需求是密码需包含至少一个大写、小写、特殊字符，但代码里额外加了数字判断，下面修复时保留该逻辑，如果你不需要数字可以自行删除对应检查。

修复后的完整代码 #

import java.util.Scanner;

public class UserSignUp {
    public static void main(String[] args) {
        // 只用一个Scanner即可，避免资源浪费
        Scanner input = new Scanner(System.in);
        String username;
        String password;

        try {
            System.out.print("Enter your username: ");
            username = input.nextLine();
            // 调用用户名验证
            validateUsername(username);
            
            System.out.print("Enter your password: ");
            password = input.nextLine();
            // 调用密码验证
            validatePassword(password);
            
            // 只有所有验证通过才执行成功提示
            System.out.println("Successfully signed up.");
        } catch (IllegalArgumentException e) {
            // 输出验证失败的具体原因
            System.out.println("Sign up failed: " + e.getMessage());
        } finally {
            // 关闭Scanner释放资源
            input.close();
        }
    }

    public static void validateUsername(String username) throws IllegalArgumentException {
        // 更通用的邮箱正则，也可以保留你原来的.com规则
        String pattern = "^[A-Za-z0-9+_.-]+@[A-Za-z0-9.-]+$";
        // 不匹配邮箱格式时抛出异常
        if (!username.matches(pattern)) {
            throw new IllegalArgumentException("Invalid username: must be a valid email address.");
        }
    }

    public static void validatePassword(String password) throws IllegalArgumentException {
        boolean hasUppercase = false;
        boolean hasLowercase = false;
        boolean hasDigit = false;
        boolean hasSpecialChar = false;

        for (int i = 0; i < password.length(); i++) {
            char ch = password.charAt(i);
            if (Character.isUpperCase(ch)) {
                hasUppercase = true;
            } else if (Character.isLowerCase(ch)) {
                hasLowercase = true;
            } else if (Character.isDigit(ch)) {
                hasDigit = true;
            } else {
                // 非字母数字则判定为特殊字符
                hasSpecialChar = true;
            }
        }

        // 逐一检查所有条件，不满足则抛出对应异常
        if (!hasUppercase) {
            throw new IllegalArgumentException("Password must contain at least one uppercase letter.");
        }
        if (!hasLowercase) {
            throw new IllegalArgumentException("Password must contain at least one lowercase letter.");
        }
        if (!hasDigit) {
            throw new IllegalArgumentException("Password must contain at least one digit.");
        }
        if (!hasSpecialChar) {
            throw new IllegalArgumentException("Password must contain at least one special character.");
        }
    }
}

修复后逻辑说明 #

• 读取输入后立即调用验证方法，任何验证失败都会抛出异常并进入catch块，不会执行成功提示。
• 用户名验证改为不匹配格式时抛异常，符合需求。
• 密码验证遍历所有字符，标记每种类型是否出现，最后统一检查所有条件，确保密码满足要求。
• 只用一个Scanner并在finally块关闭，规范资源管理。

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