代码重构思考

问题：

functiongetStatusText($status){
$text = '';
if($status=='0'){
$text = 'aaa';
}elseif($status=='1'){
$text = 'asdfa';
}elseif ($status==5){
$text = 'asdfasdf';
}elseif ($status=='8'){
$text = 'egwdf';
}elseif ($status==12){
$text = 'asfw';
}elseif ($status == '10'){
$text = 'rwefv';
}
if($text == 'aaa' || $text == 'asfw' ||$text == 'wreb'){
$text = 'weutiw';
}
if(!$text){
$text = 'iuiqewr';
}
return $text;
}
如果对它进行优化重构，你能写出来的尽可能漂亮的代码是什么。

LOW的思考

首先看到的是由if…elseif…语句组成的判断不同$status而得到不同的$text的一个function。输入是$status;输出是$text;

    于是想到也可以用switch。知道switch/case是松散比较，即类似（==：only比较值）;而不是严格比较（===：比较值&数据类型）;通过观察，把‘0’  和12归为一类。还有发现一个不可能出现的情况（$text=’wreb’），就没写判断这部分的代码。

    其次，true和false的情况。1==true;0==false;补充： -1==true; 非0数字的都是true，0是false。

尝试后还发现一个奇怪的情况：

01~07==true;  08~09==false;  -01~-07==true;  -08~-09==false;

001~007==true;  008~009==false;…

…

Why？

用switch写：

function getStatusText($status){    $text = '';    switch ($status) {           case '1':                  $text ='asdfa';                  break;           case '0':           case 12:                  $text ='weutiw';                  break;           case 5:                  $text ='asdfasdf';                  break;           case '8':                  $text ='egwdf';                  break;           case '10':                  $text ='rwefv';                  break;               default:                  $text ='iuiqewr';                  break;    }    return $text;}

错误

没有想到实际它要会有的输入是什么情况。错误以为入参可以是任何类型，布尔值、数值、字符串、浮点型、null…。以为传什么参数都行，进入function中怎么处理以及结果就是按照原来贴的代码去执行的。以为function内部就是正确的了。所以自己去优化代码的时候，就想着要让结果和原来的代码一样才是对的。没想到它的参数只能是数字(完全没往这方面想！！)，要去排除true和false的情况，因为true==1和false==0。要保证传true不等于传1，传false不能等于传0。

看到这个代码，没有考虑布尔值，没有想到它没有把握数字的类型(数值型or字符串型)。被它牵着鼻子走了。然后只是换一种写法，并没有根据这个代码的意图去优化。而且还忽略了其他数字也是==true的。

看到的东西都是很表层的，想深入挖掘可是想不到，挖的还是表层的…

一点也没想到用要数组。

正确思考

一、原来的代码没有考虑布尔值的情况

二、不能准确把握入参一定是数值型还是字符串型的数字

三、if else的性能不如switch ，switch不如用数组

优化重构方案

functiongetStatusText($status){if(!is_numeric($status)) return false;    //入参只能是数字      $arr = array(             0=>'aaa'             1=>'asdfa',             5=>'asdfasdf',             8=>'egwdf',             12=>'asfw',             10=>'rwefv'      );      $text = $arr[$status-0] ?: 'iuiqew';        //数字字符串转为数字      returnin_array($text,array('aaa','asfw','wreb'))?'weutiw' : $text;}

 

 

is_numeric — 检测变量是否为 数字 或 数字字符串

数组来存所有情况，利用数组下标来取值。

我的总结

1.     先思考代码意图。清楚它要的输入是什么。

2.     不要被现有的代码牵着鼻子走了。以为写好的就是对的了，被固化、限制了。想不开，只看到表层，无法深入。

3.     做好限制，排除不合法的情况

4.     想想数组，学会利用数组。