提高你的Java代码质量吧：小心switch带来的空值异常

一、分析 

使用枚举定义常量时，会有伴有大量的switch语句判断，目的是为每个枚举解释其行为。 

我们知道，目前的Java的switch语句只能判断byte、short、char、int类型（JDK7已经允许使用string类型），为什么枚举也能跟在switch后面呢？很简单，因为编译的时候，编译器判断出switch后面的参数是枚举类型，然后就会根据枚举的排序值继续匹配。如下： 

 

public static void doSports(Season season){ 
switch(season.ordinal()){ 
    case Season.Spring.ordinal(): 
        …... 
    case Season.Summer.ordinal(): 
        …... 
    } 
} 

 

看明白了吧，switch语句先计算season变量的排序值，然后与枚举常量的每个排序值进行对比。 

二、场景 

看这样一个例子： 

 

public static void doSports(Season season){ 
    switch(season){ 
        case Spring: 
            System.out.printf("春天放风筝"); 
            break; 
        case Summer: 
            System.out.println("夏天游泳"); 
            break; 
        case Autum: 
            System.out.println("秋天捉知了"); 
            break; 
        case Winter: 
            System.out.println("冬天滑冰"); 
            break; 
        default: 
            System.out.println("输入错误"); 
            break; 
    } 
} 

 

这段代码有没有问题？我们先来看看如何被调用的，因为传递的是Season类型，也就是一个实例（枚举的本质就是类）对象，那就当然允许为null了，我们就传递一个null进去看看，代码如下： 

 

public static void main(string[] args){ 
    doSports(null); 
}

 

似乎应该打印，“输入错误！"，因为在switch中没有匹配，输出default代码块。然而，我们看看真实结果： 

Exception in thread "main" java.lang.NullPointerException 

at Client.doSports(Client.java:9) 

at Client.main(Client.java:5) 

怎么会是空指针异常呢？这就与枚举和switch的特性有关了，此问题也在开发中经常发生的。根据上面分析，如果season为null，无法执行ordinal方法，于是报空指针异常了。 

三、建议 

问题清楚了，建议在switch语句枚举前加入参数是否为null。