内联函数
重构动机
当某个函数内部代码同函数名称一样清晰易读,可选择去掉这个函数,直接使用其中的代码。非必要的调用总是可以避免的。
重构前
<?php
namespace app\index\controller;
class Index
{
// 手机号正则
public const REGEX_MOBILE = '/^1[3-9]\d{9}$/'
// 验证手机号码格式
public function isMobileNumber($number):bool
{
return preg_match(self::REGEX_MOBILE, $this->trimMobileNumber($number)) === 1;
}
private function trimMobileNumber($number)
{
return trim($number);
}
}
重构后
<?php
namespace app\index\controller;
class Index
{
// 手机号有19开头的,也方便后续升级
public const REGEX_MOBILE = '/^1[3-9]\d{9}$/';
// 验证手机号码格式
public function isMobileNumber($number):bool
{
return preg_match(self::REGEX_MOBILE, trim($number)) === 1;
}
}