Home / Questions / 24135
Accepted
Bryan Romero
Asked: 2020-09-18 03:12:17 +0800 CST 2020-09-18 03:12:17 +0800 CST

在方法中使用多个 'if - else' 或 'switch'

  • 772

我在 Java 中创建了一个允许我“检查”几个的方法JTextFields，这个想法是用户输入他的个人数据（姓名、姓氏、电子邮件、密码等）

并且这种方法可以让我检查JTextfields它们是否不NULL正确，电子邮件的格式是否正确，最后密码是否正确NULL。

public boolean checkRegistrationData(ArrayList<JTextField> jTxts, JPasswordField pass, JTextField theEmail) {

    if (isJtextNotNull(jTxts) && isJPasswordNotNull(pass) && isValidEmailAddress(theEmail)) {
        JOptionPane.showMessageDialog(null, "A confirmation Code has been sent to your Email adress");
        return true;

    } else if (isJtextNotNull(jTxts) && !isJPasswordNotNull(pass) && isValidEmailAddress(theEmail)) {
        JOptionPane.showMessageDialog(null, "The password field is empty");
        return false;

    } else if (isJtextNotNull(jTxts) && isJPasswordNotNull(pass) && !isValidEmailAddress(theEmail)) {
        JOptionPane.showMessageDialog(null, "The email address is not valid");
        return false;
    }

    else {

        JOptionPane.showMessageDialog(null, "Some data is empty");
        return false;
    }
}

它ArrayList<JTextField> jTxts是大多数带有用户数据的 JTextField 被“存储”的地方。

有什么办法可以改进这种方法吗？使用switch会更好吗？

java

3 Answers

  1. 为了避免为每种类型的错误显示一条消息，这对用户来说可能很麻烦，您可以在列表的帮助下对所有错误进行分组。那是：

    ✍ 代码

    public boolean checkRegistrationData(ArrayList<JTextField> jTxts, JPasswordField pass, JTextField theEmail) {
    List<String> errors = new ArrayList<>();
    if (!isJtextNotNull(jTxts)) {
        errors.add("Some data is empty");
    }
    if (!isJPasswordNotNull(pass)) {
        errors.add("The password field is empty");
    }
    if (!isValidEmailAddress(theEmail)) {
        errors.add("The email address is not valid");
    }
    if (errors.isEmpty()) {
        JOptionPane.showMessageDialog(null, "A confirmation Code has been sent to your Email adress");
    } else {
        JOptionPane.showMessageDialog(null, String.join("\n", errors));
    }
    return errors.isEmpty();
}

    ✍ 退出

    捕获

    通过这种方式，您可以显示输入数据中存在的所有错误，并且用户有机会一次更正所有错误。

    • 4
  2. Best Answer

    一种常见的模式（Guard 子句/ Early exit）是在方法的开头检查参数是否有效，如果不是则退出方法。这样，该方法的真正目标就与之前的验证分离了。使用该模式，该方法将如下所示：

    public boolean checkRegistrationData(ArrayList<JTextField> jTxts, JPasswordField pass, JTextField theEmail) {

    if (!isJtextNotNull(jTxts)) {
        JOptionPane.showMessageDialog(null, "Some data is empty");
        return false;
    }

    if (!isJPasswordNotNull(pass)) {
        JOptionPane.showMessageDialog(null, "The password field is empty");
        return false;
    }

    if (!isValidEmailAddress(theEmail)) {
        JOptionPane.showMessageDialog(null, "The email address is not valid");
        return false;
    }

    JOptionPane.showMessageDialog(null, "A confirmation Code has been sent to your Email adress");
    return true;
}
    • 2

  3. switch评估同一表达式的不同结果会很有用。开关总是比 if、elseif...else 的串联更有效。但在您的情况下，您评估 4 个不同的表达式：

    1. isJtex​​tNotNull(jTxts) && isJPasswordNotNull(pass) && isValidEmailAddress(theEmail)
    2. isJtex​​tNotNull(jTxts) && !isJPasswordNotNull(pass) && isValidEmailAddress(theEmail)
    3. isJtex​​tNotNull(jTxts) && isJPasswordNotNull(pass) && !isValidEmailAddress(theEmail)
    4. 如果两个表达式都不为真

    我认为在这种情况下使用 switch 没有多大意义，除非您使用额外的变量，例如form_state，并且它的值对应于表单的每个可能状态。

    • 1