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

一、分析

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

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

public static > T valueof(Class 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 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;

}

}