简化if（做一些操作）然后返回[关闭]

Question

这可能很常见，可能是一个愚蠢的问题，但我真的很想知道其他人如何处理这个问题。

说我有：

private void actionPerformed(ActionEvent evt){
     String text = textfield.getText();
     if(isValid(text)){
         // do something eg:
         list.add(text);
     }
}

private boolean isValid(String text){
    if(text.isEmpty()){
      displayErrorMessage("empty string!");
      return false;
    }
    if(hasInvalidChars(text)){
      displayErrorMessage("Invalid chars");
      return false;
    }
    ....
    return true;
}

isValid(String) 方法感觉怪怪的，我认为一个方法应该只做一件简单的事情，但是 isValid() 肯定违反了它，它显示一个错误消息然后返回一个布尔值。

这样好吗？还是有其他方法可以绕过它？

我能想到的另一个不太优雅的解决方法是实现一个标志。即：

private void actionPerformed(ActionEvent evt){
       String text = textfield.getText();
       verify(text);

       if(!errorIsDisplayed){
          list.add(text);
       }   
}

private boolean verify(String text){
    if(text.isEmpty()){
      displayErrorMessage("empty string!");
      errorIsDisplayed = true;
    }
    if(hasInvalidChars(text)){
      displayErrorMessage("Invalid chars");
      errorIsDisplayed = true;
    }
    ....
    errorIsDisplayed = false;
}

我希望你能清楚地看到我试图解决的问题，我认为我缺少一个简单的解决方案，可能是由于睡眠不足和天气原因，我不知道。其他人将如何实现这一点？

Answer 1

如果你喜欢，
你可以定义一个错误类型的enum，而isValid方法只返回类型。

enum Type{
    VALID("Valid"),
    EMPTY("Empty String"),
    // other types.

    public final String message;
}

对于某个文本字符串，可以定义一个方法getType来获取其对应的Type。

Type getType(String text){...}

那么你的isValid 方法应该只检查这条消息是否有效。

boolean isValid(Type type){
    return type == Type.VALID;
}

而在actionPerformed 中，您可以根据文本字符串的类型采取行动。

actionPerformed(...){
    String text = ...
    Type type = getType(text);
    if (isValid(type)){
        //Actions for valid text
    } else {
        //Actions for invalid text
        System.out.println(type.message);
    }
}

Answer 2

如果你真的想要，将List<String> unacceptableReasons 的参数传递给isValid() 并将错误消息添加到其中。并继续返回布尔值。 actionPerformed()可以承担显示消息的责任。

errorIsDisplayed 字段的解决方案更难理解，弥补了 IMO 可读性较差的代码。