switch case正确的使用姿势——在代码之外的思考

switch case正确的使用姿势——在代码之外的思考

一个简单的switch case语句，假设meg有三个case，A、B、和C：

1. 第1种写法
这种写法也在我接手的项目里出现。

String result = "";
switch (meg) {
    case A:
        //do something case A
        result = "A";
        break;
    case B:
        //do something case B
        result = "B";
        break;
    default:
        //do something case C
        result = "C";
}

看起来好像没啥问题？
2.第2种写法

String result = "";
switch (meg) {
    case A:
        //do something case A
        result = "A";
        break;
    case B:
        //do something case B
        result = "B";
        break;
    case C:
        //do something case C
        result = "C";
        break;
    default:
        Log.e(TAG, "Didn't match any case!");
}

第二种写法将default用于异常数据的检查，而不是第一种作为一个case使用。对比一下，我们能够发现第二种写法的质量更高：

• 第二种更容易让阅读者明白这个switch有3个case，分别为A、B和C，而第一种则容易让读者混淆，他还要去看下msg定义了哪几种case。

• 第二种对非正常数据进行了处理，输出了日志，为debug提供了线索；而第一种则没有对非正常case进行处理，也就是埋下了一个安全隐患。试想一下某天要是这里出了问题，要查出来估计也要走点弯路。

代码之外的思考

除了会写代码，写出能跑的代码，是否能够写出健壮稳定的代码？这也是我经常思考的问题。