代码重构思考

问题：

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.     想想数组，学会利用数组。