提高你的Java代码质量吧：使用valueof前必须进行校验

一、分析

每个枚举都是java.lang.Enum的子类，都可以访问Enum类提供的方法，比如hashCode、name、valueOf等，其中valueOf方法会把一个String类型的名称转变成枚举项，也就是在枚举项中查找字面值和该参数相等的枚举项。

我们来深入分析一下该valueOf方法的源代码：

public static <T extends Enum<T>> T valueof(Class<T> enumType,String name){
    //通过反射，从常量列表中查找
    T result = enmuType.enumConstantDirector().get(name);
    if(result != null)
        return result;
    if(name == null)
        throw new NullPointerException("Name is null");
    //最后排除无效参数异常
    throw new IllegalArgumentException("No enum const" + enumType + "." + name);
} 

valudOf方法通过反射从枚举类的常量声明中查找，若找到就直接返回，若找不到则抛出无效参数异常。valueOf的本意是保护编码的枚举安全性，使其不产生空枚举对象，简化枚举操作，但是却又引入了一个我们无法避免的IllegalArgumentException异常。

二、场景

我们来看如下代码：

public static void main(String[] args){
    //注意summer是小写
    List<String> params = Arrays.asList("Spring","summer");
    for(String name:params){
        //查找字面值与name相同的枚举项
        Season s = Season.valueOf(name);
        if(s != null){
            //有枚举项时
            System.out.println(s);
        }else{
            //没有枚举项
            System.out.println("没有相关枚举项");
        }
    }
} 

summer无法转换Season枚举，根据上面的分析，就会抛出IllegalArgumentException异常，一旦抛出异常，后续的代码就不运行了！！！

这与我们的习惯很不一致，例如我们从一个List中查找一个元素，即使不存在也不会报错，顶多返回indexOf方法返回-1。

三、建议

1.使用try-catch捕捉异常

try{
    Season s = Season.valueOf(name);
    //有该枚举项时处理
    System.out.println("s");
}catch(Exception e){
    System.out.println("无相关枚举项");
} 

2.扩展枚举类

由于Enum类定义的方法都是final类型的，所以不希望被覆写，我们可以通过增加一个contains方法来判断是否包含指定的枚举项，然后在继续转换，代码如下：

enum Season{
    Spring,Summer,Autumn,Winter; 

    //是否包含枚举项
    public static boolean contains(String name){
        //所有的枚举值
        Season[] season = values();
       //遍历查找
       for(Season s : season){
           if(s.name().equals(name)){
               return true;
           }
       } 

       return false;
    }
}