重构那些事(一)
转后端学习PHP一个多月了，发现我们的后端代码问题很多,可维护性很差，对于新手入门和开发很不友好，所以想提一下改进这些问题的思路，与大家一起探讨一下，一起看看有哪些问题及改进方法吧
1.使用语法糖(即中括号形式)来创建数组 PHP 5.4之后已经可以使用中括号的形式来创建数组了，我们的线上环境刚好是5.4的，所以完全可以大大方方的使用中括号即如下形式来创建数组 $arr = [1, 2, 3];
不建议使用array(1, 2, 3)的形式来创建数组，原因嘛，更简洁！用array()的形式有点啰嗦,就像OC中原来创建数组是如下形式
NSArray *arr = [[NSArray alloc] initWithObjects:@"1", @"2", nil];变成语法糖之后改用了如下方式
NSArray *arr = @[@"1", @"2", @"3"];是不是清爽了很多,既然在新版本中引入了这种中括号的形式来创建数组，说明PHP是更建议使用这种格式来初始化数组的,何不与时俱进,一起改造成用中括号的形式来初始化数组呢？
那么问题来了，现有的成千上万个array()这种形式创建数组的方式怎么办,两种办法
- 用正则表达式全局替换
这种方式最简单，一劳永逸
- 只在新业务或重构中使用语法糖的形式
可能你会担心这种批量替换会出什么问题(实际上当初我在用正则做iOS中语法糖批量替换时也出了一些问题)，那我们可以在新业务或重构代码时把它给替换掉,这是最安全的方式
2.if else 套嵌过深 有很少以下形式的if else
if (condition) {
if (condition) { } else {
if (condition) { }
}
}嵌套过深，不仅造成代码难以维护，更让新人难以理解其中的逻辑,绝对是大忌,那么，怎么改进呢,黄金道路法则,如下
if (condition1) return;
if (condition2) return;//logic让一些不合理的或不想要if逻辑提前终止! 怎么样，是不是清楚了很多
3.if else 太多，应该用switch! 我们的代码中经常出现以下代码片断
$test = 1;
if ($test == 1) {} else if ($test == 2) {} else if ($test == 3 || $test == 4) {}类似这种方式用switch不是非常简洁明了吗，和一些同学聊过这个话题，其他他们也意识到这个问题了
但觉得将if else改成switch可能会不小心忘了break之类的关键字，担心改出bug出来，或者是业务太忙，没时间重构。
对此我是这么看的，一个优秀的程序员应该要有一些代码洁癖，要有在写业务时顺便或者说之后重构代码的意识,这样不仅会给其他小伙伴留下非常良好的专业的印象，而且也方便之后的维护,套用iOS大拿喵神的话来说:”一个有节操的程序员会在乎自己的代码的不合理性，就像在乎饭碗边上有只死蟑螂那样”,能用是不够的，可维护,可拓展才是最重要的
4.变量应该定义在离他最近被使用的位置
这个问题其实我自己也犯过这样的错误,比如将它定义在第10行，但实际使用这个变量却是在第50行，这样造成的后果就是要熟悉逻辑的同学不得不带着这个变量从第10行追踪到50行，累不累啊！
5.方法的参数过多 先来看一段我们商详的一段比较恶心的代码
private function get_detail($detail, $is_oversea = FALSE, $ship_city = '', $origin_price = 0, $price = 0, $presell = NULL, $is_talent_mall = FALSE, $iid = 0, $mart_mall_presell_info = FALSE, $cid = 0, $need_warrant = TRUE, $last_img_url = '') {...}参数多到令人发指!并且没有注释！而且调用此方法的方法有接近1900行!如果现在去重构，还是有点麻烦，如果当初在方法的参数多于3，4个的时候我们想办法去用一种更合理的方式把它重构掉,就不会是现在这种情况了，所以说人人都要有及时重构的意识!好了，现在既然问题出现了，我们总要解决它，如何解决这种参数过多的问题, 我认为可以用设计模式中的委托模式,大概流程如下
代码大致如下
//get_detail所在类
private get_detail($businessDelegate) { namespace DesignPatterns\More\Dispatcher; // 初始化 dispacher 并委托具体的业务逻辑调用方
$dispatcher = new Dispatcher($businessDelegate); // TeamLead 将编写代码的任务委托给 JuniorDeveloper
echo $dispatcher->dealWithBusiness();
}//Dispatcher.php
class Dispatcher
{
protected $businessDelegate; public function __construct($businessDelegate) {
$this->businessDelegate = $businessDelegate;
} private function dealWithBusiness() {
$this->businessDelegate->dealWithBusiness();
}
}//BusinessDelegate
class BusinessDelegate
{
private function dealWithBusiness() {
//do something
}}(全文完)
下一篇应该是重构那些事(二) 2017年，坚持！