内联函数

未匹配的标注

重构动机

当某个函数内部代码同函数名称一样清晰易读,可选择去掉这个函数,直接使用其中的代码。非必要的调用总是可以避免的。

重构前

<?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;
    }
}

本文章首发在 LearnKu.com 网站上。

上一篇 下一篇
贡献者:2
讨论数量: 0
发起讨论 只看当前版本


暂无话题~