重构lingling代码

中关村村草

重构lingling代码

在添加getRecentGames和getTvChannels接口过程中，发现很多自己都看不过去的代码，试举两例：

1，判断传入的日期是否是合法形式：xxxx-xx-xx xxxx
编码时，用最土的方式，挨个判断字符是否合法。

1. function isDateValid($date){
   
2.     $date = trim($date);
   
3.     $arr = explode(' ', $date);
   
4.     if(empty($arr)){
   
5.         return false;
   
6.     }
   
7. 
   
8.     $datePart = $arr[0];
   
9.     $elements = explode('-', $datePart);
   
10.     if(count($elements) != 3){
    
11.         return false;
    
12.     }
    
13.     if(! checkDate($elements[1], $elements[2], $elements[0])){
    
14.         return false;
    
15.     }
    
16.     $timePart = $arr[1];
    
17.     $elements = explode(':', $timePart);
    
18.     if(count($elements) != 2 && count($elements) != 3){
    
19.         return false;
    
20.     }
    
21.     foreach($elements as $ele){
    
22.         if(! is_numeric($ele)){
    
23.             return false;
    
24.         }
    
25.     }
    
26.     return true;
    
27. }

复制代码

用正则表达式修改后：

1. public function isDateValid($date){
   
2.     $pattern = '/^ *[0-9]{2,4}(-[0-9]{2}){1,2} [0-9]{2}(:[0-9]{2}){1,2} *$/';
   
3.     $ret = preg_match($pattern, $date, $matches);
   
4.     return $ret == 0 ? false : true;            
   
5. }

复制代码

重构过程中，又复习了下正则表达式，平常极少用到{}，这次痛快的用了N次。

  

2，类似的业务，调用接口的层数也应类似，比如：

添加足球比赛信息：addFootballTvb() –> finalAddTvb()

添加篮球比赛信息：finalAddTvb()

出现这样的调用层次，是因为后一个功能上层得到的数据，正好符合finalAddTvb()的要求，所以将原来private的接口public出来，看起来好像省事了，但在后来阅读代码过程中，总感觉非常别扭。后来改为：

添加篮球比赛信息：addBasketballTvb() –> finalAddTvb()

代码读起来顺畅多了。重构的目的并不为解决功能问题，而是为了提高代码可读性，进而提高可维护性。

蓝猫淘气啦啦

谢谢分享