内联函数

重构改善既有代码的设计—笔记 /

重构动机

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

重构前

<?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 网站上。

版主 1.1k 声望

坚持比努力可怕

贡献者：2

推荐文章：

更多推荐...

PHP 核心特性 - 匿名函数 26 / 10 |

Hyperf 常用助手函数 15 / 4 |

Api接口实战：常用函数集合 58 / 8 |

PHP 7.4 新语法：箭头函数 12 / 7 |

100 个最常用的 PHP 函数 53 / 5 |

每日五个 PHP 函数（2） 10 / 7 |

讨论数量: 0

发起讨论只看当前版本

暂无话题~