如何处理基于枚举的开关中的默认情况？

Question

我有一个枚举，它根据枚举类型评估字符串是否有效。使用开关执行验证。默认操作是这样的： default: return false;.

一位程序员添加了一个新类型并忘记更新 isValid(..)，导致每次评估答案时对 isValid(..) 的调用为 return false。

您认为正确的处理方式是什么？

public enum AnswerType {
  TEXT("string"),
  INT("integer"),
  FLOAT("float");

  final String type;

  AnswerType(final String type) {
    this.type = type;
  }

  /**
   * Checks whether the given answer is valid for this answer type.
   * @param answer The provided answer.
   * @return true if the answer is valid; false otherwise.
   */
  public boolean isValid(final String answer) {

    switch (this) {
      case TEXT:
        return !StringUtils.isEmpty(answer);

      case INT:
        return NumberUtils.isDigits(answer);

      case FLOAT:
        return NumberUtils.isNumber(answer);

      default:
        return false; // Not sure this is best practise.
    }
  }
}

Answer 1

这是一个奇怪的设计（并且您的代码无法按原样达到默认情况）...为什么不将方法放入每个枚举常量中呢？

public static enum AnswerType {
  TEXT("string") {
    public boolean isValid(String answer) { return !StringUtils.isEmpty(answer); }
  },
  INT("integer") {
    public boolean isValid(String answer) { return NumberUtils.isDigits(answer); }
  },
  FLOAT("float") {
    public boolean isValid(String answer) { return NumberUtils.isNumber(answer); }
  };

  private final String type;

  AnswerType(final String type) {
    this.type = type;
  }

  /**
   * Checks whether the given answer is valid for this answer type.
   *
   * @param answer The provided answer.
   *
   * @return true if the answer is valid; false otherwise.
   */
  public abstract boolean isValid(final String answer);
}

或者，如果您使用 Java 8，您可以进一步简化代码：

public static enum AnswerType {
  TEXT("string", s -> !StringUtils.isEmpty(s)),
  INT("integer", NumberUtils::isDigits),
  FLOAT("float", NumberUtils::isNumber);

  private final String type;
  private final Predicate<String> isValid;

  AnswerType(final String type, Predicate<String> isValid) {
    this.type = type;
    this.isValid = isValid;
  }

  /**
   * Checks whether the given answer is valid for this answer type.
   *
   * @param answer The provided answer.
   *
   * @return true if the answer is valid; false otherwise.
   */
  public boolean isValid(final String answer) {
    return isValid.test(answer);
  }
}

Answer 2

如果随着时间的推移需要添加答案类型，为什么还要使用枚举？您可以使用 isValid 方法制作一个 Answer 接口，然后在 StringAnswer、IntegerAnswer 等中实现它。然后当有人想要添加他们不会有的答案类型时修改现有代码。

如何处理基于枚举的开关中的默认情况？

How to deal with default case in an enum based switch?

java

oop

enums

switch-statement