Java实现邮箱注册重复校验时插入逻辑不生效问题排查

代码无法按预期执行的核心原因有3个：

• 变量作用域错误：boolean isValid是定义在while(rs.next())循环内部的局部变量，循环体外的if/else逻辑根本访问不到这个变量，正常编译阶段就会抛出「找不到符号」的错误。而且当待注册邮箱没有被占用时，SQL查询不会返回任何结果，rs.next()直接返回false，循环根本不会执行，isValid连赋值的机会都没有，这也是插入逻辑永远无法触发的直接原因。
• 判断逻辑完全写反：业务预期是邮箱未被占用时执行插入，但原代码if条件写的!isValid分支打印的日志居然是"Email is Exists"，和else分支的"Email is already Exists"语义完全冲突，判断条件和业务需求完全错位。
• 冗余设计：编写的SQL已经用传入的邮箱做了精确匹配，只要ResultSet有返回行就说明邮箱已被注册，根本不需要在循环里再调用emailCheck方法做二次比对，多余的逻辑反而增加了出错概率。

// 提前定义重复校验标记，默认值为false（邮箱未被占用）
boolean emailIsDuplicate = false;
PreparedStatement st1 = conn.prepareStatement("SELECT * FROM customer where email = ?");
st1.setString(1, email);
ResultSet rs = st1.executeQuery();
checkDuplicateEmail checkDuplicateEmail = new checkDuplicateEmail();

while(rs.next()) {
    String emailcounter = rs.getString("email");
    // 匹配到重复邮箱时更新标记
    emailIsDuplicate = checkDuplicateEmail.emailCheck(emailcounter, email);
}

if(!emailIsDuplicate) {
    System.out.println("Email validation passed, insert new user data");
    PreparedStatement st = conn.prepareStatement("insert into customer(name, email, mobileno, username, password) values(?, ?, ?, ?, ?)");
    st.setString(1, name);
    st.setString(2, email);
    st.setInt(3, mobileNo);
    st.setString(4, username);
    String encoded = enc.encodeToString(password.getBytes());
    st.setString(5, encoded);
    st.executeUpdate();
    // 插入完成后关闭所有数据库资源
    st.close();
    st1.close();
    rs.close();
    conn.close();
} else {
    System.out.println("Email already registered");
    PrintWriter out = response.getWriter();
    out.write("<html><body>");
    out.write("<h1>Registration cannot be sucessful because email is already registered!</h1>");
    out.write("</body></html>");
    // 异常分支也要关闭资源，避免连接泄漏
    st1.close();
    rs.close();
    conn.close();
}

• 把校验布尔变量提前到循环外声明初始化，解决作用域问题，保证不管有没有查询到重复邮箱，后续的分支判断都能拿到合法值。
• 对齐判断逻辑和业务语义：!emailIsDuplicate对应邮箱无重复，走插入流程；值为true时走重复提示流程，和预期完全一致。
• 补全了所有分支下的资源关闭逻辑，原代码只在插入成功时关闭数据库连接，重复报错场景下会直接泄漏连接。
• 修正了原代码里分支和日志不匹配的问题，避免后续排查被错误日志误导。

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