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