查找 Java 枚举的最佳实践

Question

我们有一个 REST API，客户端可以在其中提供表示在 Java 枚举中定义在服务器上的值的参数。

所以我们可以提供描述性错误，我们将这个 lookup 方法添加到每个 Enum。好像我们只是在复制代码（不好）。有更好的做法吗？

public enum MyEnum {
    A, B, C, D;

    public static MyEnum lookup(String id) {
        try {
            return MyEnum.valueOf(id);
        } catch (IllegalArgumentException e) {
            throw new RuntimeException("Invalid value for my enum blah blah: " + id);
        }
    }
}

更新：valueOf(..) 提供的默认错误消息为No enum const class a.b.c.MyEnum.BadValue。我想从 API 中提供一个更具描述性的错误。

Answer 1

也许你可以实现通用静态lookup 方法。

像这样

public class LookupUtil {
   public static <E extends Enum<E>> E lookup(Class<E> e, String id) {   
      try {          
         E result = Enum.valueOf(e, id);
      } catch (IllegalArgumentException e) {
         // log error or something here

         throw new RuntimeException(
           "Invalid value for enum " + e.getSimpleName() + ": " + id);
      }

      return result;
   }
}

那你就可以了

public enum MyEnum {
   static public MyEnum lookup(String id) {
       return LookupUtil.lookup(MyEnum.class, id);
   }
}

或显式调用实用程序类查找方法。

Answer 2

看起来你在这里有一个不好的做法，但不是你想的。

捕获IllegalArgumentException 以重新抛出另一个带有更清晰消息的RuntimeException 可能看起来是个好主意，但事实并非如此。因为这意味着您关心异常中的消息。

如果您关心异常中的消息，那么这意味着您的用户以某种方式看到了您的异常。这很糟糕。

如果您想向您的用户提供明确的错误消息，您应该在解析用户输入时检查枚举值的有效性，如果用户输入不正确，则在响应中发送相应的错误消息。

类似：

// This code uses pure fantasy, you are warned!
class MyApi
{
    // Return the 24-hour from a 12-hour and AM/PM

    void getHour24(Request request, Response response)
    {
        // validate user input
        int nTime12 = 1;
        try
        {
            nTime12 = Integer.parseInt(request.getParam("hour12"));
            if( nTime12 <= 0 || nTime12 > 12 )
            {
                throw new NumberFormatException();
            }
        }
        catch( NumberFormatException e )
        {
            response.setCode(400); // Bad request
            response.setContent("time12 must be an integer between 1 and 12");
            return;
        }

        AMPM pm = null;
        try
        {
            pm = AMPM.lookup(request.getParam("pm"));
        }
        catch( IllegalArgumentException e )
        {
            response.setCode(400); // Bad request
            response.setContent("pm must be one of " + AMPM.values());
            return;
        }

        response.setCode(200);
        switch( pm )
        {
            case AM:
                response.setContent(nTime12);
                break;
            case PM:
                response.setContent(nTime12 + 12);
                break;
        }
        return;
    }
}

Answer 3

当涉及到 Rest/Json 等时，我们所有的枚举都是这样的。 它的优点是错误是人类可读的，并且还为您提供了可接受的值列表。我们正在使用自定义方法 MyEnum.fromString 而不是 MyEnum.valueOf，希望对您有所帮助。

public enum MyEnum {

    A, B, C, D;

    private static final Map<String, MyEnum> NAME_MAP = Stream.of(values())
            .collect(Collectors.toMap(MyEnum::toString, Function.identity()));

    public static MyEnum fromString(final String name) {
        MyEnum myEnum = NAME_MAP.get(name);
        if (null == myEnum) {
            throw new IllegalArgumentException(String.format("'%s' has no corresponding value. Accepted values: %s", name, Arrays.asList(values())));
        }
        return myEnum;
    }
}

例如，如果你打电话

MyEnum value = MyEnum.fromString("X");

您将收到带有以下消息的 IllegalArgumentException：

'X' 没有对应的值。接受的值：[A、B、C、D]

您可以将 IllegalArgumentException 更改为自定义。

Answer 4

如果您希望查找不区分大小写，您可以遍历值使其更友好一点：

 public enum MyEnum {
   A, B, C, D;

      public static MyEnum lookup(final String id) {
        for(MyEnum enumValue: values()){
           if(enumValue.name().equalsIgnoreCase(id)){
              return enumValue;
           }
        }  
        throw new RuntimeException("Invalid value for my enum: " + id);
       }
}

Answer 5

Guava 还提供了这样的功能，如果找不到枚举，它将返回 Optional。

Enums.getIfPresent(MyEnum.class, id).toJavaUtil()
            .orElseThrow(()-> new RuntimeException("Invalid enum blah blah blah.....")))

Answer 6

为什么我们要写那 5 行代码？

public class EnumTest {
public enum MyEnum {
    A, B, C, D;
}

@Test
public void test() throws Exception {
    MyEnum.valueOf("A"); //gives you A
    //this throws ILlegalargument without having to do any lookup
    MyEnum.valueOf("RADD"); 
}
}

Answer 7

IllegalArgumentException 中的错误消息已经足够描述了。

您的方法从一个特定的例外中产生了一个通用例外，并简单地改写了相同的消息。开发人员更喜欢特定的异常类型，并且可以适当地处理这种情况，而不是尝试处理 RuntimeException。

如果目的是让消息对用户更友好，那么对枚举值的引用无论如何都与它们无关。让 UI 代码决定应该向用户显示什么，UI 开发人员最好使用 IllegalArgumentException。

Answer 8

更新：正如 GreenTurtle 正确评论的那样，以下是错误的

---

我只想写

boolean result = Arrays.asList(FooEnum.values()).contains("Foo");

这可能不如捕捉运行时异常的性能，但可以使代码更简洁。捕获这样的异常总是一个坏主意，因为它很容易被误诊。 当检索比较值本身导致 IllegalArgumentException 时会发生什么？然后，这将被视为枚举数的不匹配值。

Answer 9

Apache Commons Lang 3 包含类 EnumUtils。如果您没有在项目中使用 Apache Commons，那么您做错了。你在重新发明轮子！

我们可以使用十几种很酷的方法而不会引发异常。例如：

获取类的枚举，如果找不到则返回 null。

此方法与 Enum.valueOf 的不同之处在于它不会抛出 无效枚举名称的异常并执行不区分大小写 名称匹配。

EnumUtils.getEnumIgnoreCase(SeasonEnum.class, season);

Answer 10

您可以使用静态查找映射来避免异常并返回 null，然后随意抛出：

public enum Mammal {
    COW,
    MOUSE,
    OPOSSUM;

    private static Map<String, Mammal> lookup = 
            Arrays.stream(values())
                  .collect(Collectors.toMap(Enum::name, Function.identity()));

    public static Mammal getByName(String name) {
        return lookup.get(name);
    }
}