提高你的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; 
    } 
}