项目越做越大,代码越来越乱,维护困难。原因很多吧。起初为了实现功能,并没有注重代码的结构,外包公司嘛。虽然公司的项目负责人一直考虑复用、封装,但是我觉得基本上没有达到想要的效果。因为整个代码中没有用到设计模式,这样结构肯定好不到哪去。虽然很多功能都封装了,但是分层特别的乱,有种为了封装而封装的感觉。其实,很多东西我也不懂,但是写完代码之后,偶尔还是会去把代码的结构稍微的修改修改,尽量的少给自己挖坑吧。
代码结构的问题
接口有大量的方法,调用接口时需要使用 switch 进行判断,大致结构如下:
1 private function makeXML($xmlName, $xmlNameParam) 2 { 3 $requestData = null; 4 5 switch ($xmlName) { 6 // ... 7 case 'sendOrder': 8 $requestData = $this->sendOrder($xmlNameParam); 9 break; 10 case 'ecfareQuery': 11 $requestData = $this->ecfareQuery($xmlNameParam); 12 break; 13 case 'getInterAV': 14 $requestData = $this->getInterAV($xmlNameParam); 15 break; 16 // ... 17 } 18 19 return $requestData; 20 }
以上 case 中都是调用对应的方法来拼接接口所需的 XML 信息的。
方法的参数分别传递了拼接接口方法的名称和参数。
这样的判断结构代码感觉不好管理,增加一个方法方法,就需要增加一个 case 调用,而且看的时候也比较乱。
代码结构的改进
因此进行了相关的改进,改进方式如下:
1、先定义一个方法数组,用于保存方法名,定义如下:
1 protected $arr = []; 2 3 public function __construct() 4 { 5 // 初始化接口方法 6 $this->arr = [ 7 'getAV' => 'getAV', 8 'sendOrder' => 'sendOrder', 9 'ecfareQuery' => 'ecfareQuery', 10 'getInterAV' => 'getInterAV', 11 // ... 12 ]; 13 }
将来该接口下有了新的方法,直接添加到数组里就可以了。这样可以省去大量的 switch case 的判断。
2、修改 switch 的结构,代码如下:
1 private function makeXML($xmlName, $xmlNameParam) 2 { 3 $requestData = null; 4 5 $fun = $this->arr[$xmlName]; 6 7 $requestData = $this->$fun($xmlNameParam); 8 9 return $requestData; 10 }
将 switch case 的结构改成了以数组下标的方式进行调用的方式。这样就把所有的方法统一管理了起来。
以上思路来自于 MFC 框架对 Windows 消息的处理。
相当于 C 语言中的函数指针数组,或 C# 里的委托。
知识补充:
面向过程的函数变量(往上找的)
1 function come() { //定义com函数 2 echo "来了<p>"; 3 } 4 5 function go($name = "jack") { //定义go函数 6 echo $name."走了<p>"; 7 } 8 9 function back($string) //定义back函数 10 { 11 echo "又回来了,$string<p>"; 12 } 13 14 $func = "come"; //声明一个变量,将变量赋值为“come” 15 $func(); //使用变量函数来调用函数come() 16 $func = "go"; //重新给变量赋值 17 $func("Tom"); //使用变量函数来调用函数go() 18 $func = "back"; //重新给变量赋值 19 $func("Lily"); //使用变量函数来调用函数back();
对于 PHP 中面向对象的方式如下(自己实现的):
1 <?php 2 3 class test 4 { 5 public $arr = []; 6 7 public function __construct() 8 { 9 $this->arr = array( 10 'func1'=>'func1', 11 'func2'=>'func2', 12 'func3'=>'func3', 13 'func4'=>'func4', 14 'func5'=>'func5', 15 ); 16 } 17 18 public function submit($func, $str) 19 { 20 $f = $this->arr[$func]; 21 $this->$f($str); 22 } 23 24 static public function func1($str) 25 { 26 print 'func1' . ' ' . $str . "\n"; 27 } 28 29 static public function func2($str) 30 { 31 print 'func2' . ' ' . $str . "\n"; 32 } 33 34 public function func3($str) 35 { 36 print 'func3' . ' ' . $str . "\n"; 37 } 38 39 private function func4($str) 40 { 41 print 'func4' . ' ' . $str . "\n"; 42 } 43 44 private function func5($str) 45 { 46 print 'func5' . ' ' . $str . "\n"; 47 } 48 } 49 50 $t = new test(); // 实例化类 51 52 $f = $t->arr['func1']; 53 test::$f('abc'); // func1 func2 是静态方法 54 $f = $t->arr['func2']; 55 test::$f('abc'); 56 57 $f = $t->arr['func3']; // func3 的调用 58 $t->$f('abc'); 59 60 // func4 func5 的调用需要使用 submit 方法进行分发 61 $t->submit('func4', 'abc'); 62 $t->submit('func5', 'bcd');
以上代码的输出结果如下:
1 func1 abc 2 func2 abc 3 func3 abc 4 func4 abc 5 func5 bcd
欢迎大家关注微信公众号:“码农UP2U”