无法找到正确的方法来执行循环，直到输入有效

Question

我仍在学习java编程，请原谅我的知识不足。这可能是有史以来最简单的事情，但我一辈子都想不出更好的方法来循环checkTaskDescription方法，直到用户输入的描述低于字符限制。到目前为止，这是我能够做到的唯一方法，但它显然会重复输入对话框两次。

这是执行它的部分

 do{
     checkTaskDescription();
    }
       while (checkTaskDescription() == false);

这是正在执行的checkTaskDescription方法：

public boolean checkTaskDescription() {
        taskDesc = JOptionPane.showInputDialog(null, "Please enter a short description of the task");
        if (taskDesc.length() > 50) {
            JOptionPane.showMessageDialog(null, "Please enter a task description of less than 50 characters.", "ERROR",3);
            taskDesc = JOptionPane.showInputDialog(null, "Please enter a short description of the task");
            return false;
        }
        JOptionPane.showMessageDialog(null, "Task successfully captured.");
        return true;
    }

Answer 1

下面是使用while循环执行相同操作的另一种方法

java prettyprint-override">boolean flag = false; 
while(!flag){
  flag = checkTaskDescription();
}

主要的想法是你总是需要检查你从上一次通话中返回了什么。如果checkTaskDescription（）返回false，则再次调用，否则退出循环

Answer 2

如果你这样做会容易得多

public String getTaskDescription() {
    String taskDescription = "";
    do {
        taskDescription = JOptionPane.showInputDialog(null, "Please enter a short description of the task");
    }
    while (taskDescription.length() > 50);

    return taskDescription;
}

当你设计你的方法时，试着让它们尽可能独立。现在，如果输入无效，则返回一个标志并依靠调用方方法再次调用它。

Answer 3

您需要使用方法调用的结果。

boolean ready = false;

do{

    ready = checkTaskDescription();

} while ( ! ready );

将进入do块，将ready分配给checkTaskDescription（）的返回值，然后检查它是否为false。如果为false，它将继续循环，直到为true。

您的check TaskDescription也已损坏。如果您第一次失败，它将再次显示对话框，然后始终返回false。

public boolean checkTaskDescription() {
        taskDesc = JOptionPane.showInputDialog(null, "Please enter a short description of the task");
        if (taskDesc.length() > 50) {
            JOptionPane.showMessageDialog(null, "Please enter a task description of less than 50 characters.");
            return false;
        }
        JOptionPane.showMessageDialog(null, "Task successfully captured.");
        return true;
    }

我更新了该方法，以便在输入失败时“工作”，它将显示错误消息，然后循环并再次显示对话框。这就是为什么您应该验证对话框中的数据，而不是在循环中打开对话框。