项目越做越大,代码越来越乱,维护困难。其实原因有很多,起初为了实现功能,并没有注重代码的结构,尤其是为了完成任务的公司。虽然很多公司一直考虑复用、封装,但是有时想要达到想要的效果也比较难。其实,很多东西我也不懂,但是写完代码之后,偶尔还是会去把代码的结构稍微的修改修改,尽量的少给自己挖坑吧。
举一个简单的例子吧。
接口有大量的方法,调用接口时需要使用 switch 进行判断,大致结构如下:
private function makeXML($xmlName, $xmlNameParam)
{
$requestData = null;
switch ($xmlName) {
// ...
case 'sendOrder':
$requestData = $this->sendOrder($xmlNameParam);
break;
case 'ecfareQuery':
$requestData = $this->ecfareQuery($xmlNameParam);
break;
case 'getInterAV':
$requestData = $this->getInterAV($xmlNameParam);
break;
// ...
}
return $requestData;
}
以上 case 中都是调用对应的方法来拼接接口所需的 XML 信息的。
方法的参数分别传递了拼接接口方法的名称和参数。
这样的判断结构代码感觉不好管理,增加一个方法方法,就需要增加一个 case 调用,而且看的时候也比较乱。当 case 特别多的时候,其实代码看着也是很别扭的。
因为上面提到的问题,因此进行了相关的改进,改进方式如下:
1、先定义一个方法数组,用于保存方法名,定义如下:
protected $arr = [];
public function __construct()
{
// 初始化接口方法
$this->arr = [
'getAV' => 'getAV',
'sendOrder' => 'sendOrder',
'ecfareQuery' => 'ecfareQuery',
'getInterAV' => 'getInterAV',
// ...
];
}
将来该接口下有了新的方法,直接添加到数组里就可以了。这样可以省去大量的 switch case 的判断。
2、修改 switch 的结构,代码如下:
private function makeXML($xmlName, $xmlNameParam)
{
$requestData = null;
$fun = $this->arr[$xmlName];
$requestData = $this->$fun($xmlNameParam);
return $requestData;
}
将 switch case 的结构改成了以数组下标的方式进行调用的方式。这样就把所有的方法统一管理了起来。如果需要有 default 的话,可以判断传递的 $xmlName 是否在数组中,如果不在数组中,则调用其他的方法。这样就完成了 default 的功能。
以上思路其实来自于 MFC 框架对 Windows 消息的处理。
相当于 C 语言中的函数指针数组,或 C# 里的委托。
具体举几个例子来看一下效果吧。
面向过程的函数变量(网上找的)
function come() //定义com函数
{
echo "来了<p>";
}
function go($name = "jack") //定义go函数
{
echo $name."走了<p>";
}
function back($string) //定义back函数
{
echo "又回来了,$string<p>";
}
$func = "come"; //声明一个变量,将变量赋值为“come”
$func(); //使用变量函数来调用函数come()
$func = "go"; //重新给变量赋值
$func("Tom"); //使用变量函数来调用函数go()
$func = "back"; //重新给变量赋值
$func("Lily"); //使用变量函数来调用函数back();
例子2:
对于 PHP 中面向对象的方式如下(自己实现的):
<?php
class test
{
public $arr = [];
public function __construct()
{
$this->arr = array(
'func1'=>'func1',
'func2'=>'func2',
'func3'=>'func3',
'func4'=>'func4',
'func5'=>'func5',
);
}
public function submit($func, $str)
{
$f = $this->arr[$func];
$this->$f($str);
}
static public function func1($str)
{
print 'func1' . ' ' . $str . "\n";
}
static public function func2($str)
{
print 'func2' . ' ' . $str . "\n";
}
public function func3($str)
{
print 'func3' . ' ' . $str . "\n";
}
private function func4($str)
{
print 'func4' . ' ' . $str . "\n";
}
private function func5($str)
{
print 'func5' . ' ' . $str . "\n";
}
}
$t = new test(); // 实例化类
$f = $t->arr['func1'];
test::$f('abc'); // func1 func2 是静态方法
$f = $t->arr['func2'];
test::$f('abc');
$f = $t->arr['func3']; // func3 的调用
$t->$f('abc');
// func4 func5 的调用需要使用 submit 方法进行分发
$t->submit('func4', 'abc');
$t->submit('func5', 'bcd');
以上代码的输出结果如下:
func1 abc
func2 abc
func3 abc
func4 abc
func5 bcd
以上就是通过数组来存储 case 值和 函数名 从而改变代码结构的方法,虽然这种方法并不是特别好的改进方法,但是比起非常大的 switch case 而言要清晰一些,希望可以对各位有所帮助!
由于本人水平有限,难免有误,中肯取代无理,客观代替谩骂,希望可以互相学习,共同提高!