Android注册表单指定学校邮箱域名校验问题及优化咨询

嘿，我来帮你搞定这个邮箱校验导致崩溃的问题！你的代码崩溃大概率是因为正则表达式写得有问题，还有逻辑里的一些小坑（比如未定义的regEmail变量可能引发空指针异常）。咱们来重构一个更简洁、可靠的版本：

先分析原代码的核心问题 #

• 正则表达式错误：原代码里的[aou.edu.sa]是字符集合，意思是匹配a/o/u/./e/d/s这些单个字符，不是匹配完整的域名后缀，这会导致校验完全不符合预期
• 代码冗余重复：分别编译两个正则、做两次匹配，没必要，合并成一个正则就能搞定
• 潜在空指针风险：代码里的regEmail没有定义，调用isInEditMode()会直接引发崩溃

优化后的简洁实现 #

// 把正则定义为类的静态常量，只编译一次，提升性能
private static final String UNIVERSITY_EMAIL_REGEX = "^[\\w-+]+(\\.[\\w-]+)*@(aou\\.edu\\.sa|arabou\\.edu\\.sa)$";

public void validateEmail(EditText anEmail) {
    // 先去除输入前后的空格，避免空格导致校验失败
    String inputEmail = anEmail.getText().toString().trim();
    
    Pattern emailPattern = Pattern.compile(UNIVERSITY_EMAIL_REGEX);
    Matcher matcher = emailPattern.matcher(inputEmail);

    if (matcher.matches()) {
        // 邮箱符合本校域名要求，启用提交按钮并执行提交逻辑
        signUp.setEnabled(true);
        submitForm();
        
        // 如果需要区分学生/导师邮箱，可以在这里加判断
        if (inputEmail.endsWith("@aou.edu.sa")) {
            // 学生邮箱的额外逻辑（比如标记用户类型）
        } else if (inputEmail.endsWith("@arabou.edu.sa")) {
            // 导师邮箱的额外逻辑
        }
    } else {
        // 不符合要求，禁用按钮并提示错误
        signUp.setEnabled(false);
        anEmail.setError("Please enter your university email only.");
        
        // 原代码的regEmail应该是指当前输入框，这里替换为anEmail避免空指针
        if (anEmail.isInEditMode()) {
            signUp.setEnabled(true);
        }
    }
}

额外优化建议 #

1. 把正则的编译放在类的初始化代码里（比如onCreate方法中），而不是每次调用validateEmail都编译，进一步提升性能
2. 可以在输入框的TextWatcher中实时触发校验，让用户即时得到反馈，而不是等到点击提交时才校验
3. 确保signUp按钮已经在布局中正确初始化，避免空指针异常

内容的提问来源于stack exchange，提问作者Somaru-chan