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进行处理,也就是埋下了一个安全隐患。试想一下某天要是这里出了问题,要查出来估计也要走点弯路。
代码之外的思考
除了会写代码,写出能跑的代码,是否能够写出健壮稳定的代码?这也是我经常思考的问题。