代码优化心得

请问如何优化这段代码

public function resetPassword($email,$verifyCode,$newPassword)
{
    $user = User::findByEmail($email)->first();
    if ($user) {
        $codeData = EmailVerifyLog::findByEmail($email)->get()->last();
        if($codeData){
            if ($codeData->verifyCode($verifyCode) && $codeData->notExpired()) {
                $user->updatePassword($newPassword);
                $codeData->updateToExpired();
                return true;
            }
        }
    }
    return false;
}

上一篇看了大家的方案，蛮有启发的。总结了一下，主要有两种做法：
一、拆分职责，引入过滤器中间件
二、直接修改现有的代码

先说说拆分职责

我们回头看到，前两个产生if的地方都是在检查对象，再往下是验证code。对于一个具体的resetPassword()来说，检查对象相对会是一个比较“泛”的行为；而验证code则是resetPassword()的一个门槛。说白了，就是我压根还没开始真正的reset，就先有两种行为挡住了。所以我们可以尝试把这些行为作为resetPassword()的前置工作，那resetPassword()就改一下psw就行了

使用验证器：表单验证

'email' => 'exists:users,email'

使用中间件：

public function handle(Request $request, Closure $next)
{
    $email = $request->input('email');
    $verifyCode = $request->input('verifyCode');
    $codeData = EmailVerifyLog::findByEmail($email)->latest()->first();
    if ($codeData->verifyCode($verifyCode) && $codeData->notExpired()) {
        $codeData->updateToExpired();
        return $next($request);
    }
    throw new \Exception();
}

接着是直接修改现有的代码

1、使用文档提供的firstOr()、firstOrFail()可以直接让我们省去if：未找到时抛出异常
2、提前return，这个打破了我以往认为的代码行数越少越好的刻板思想，提前return的代码可读性比嵌套if高太多了

以下是我参考大家方案后采用的修改：

public function resetPassword($email,$verifyCode,$newPassword)
{
    if (!($user = $this->getUserInstanceByEmail($email))) {
        return false;
    }

    if(!($codeData = $this->getVerifyLogInstanceLatest($email))){
        return false;
    }

    if ($codeData->verifyCode($verifyCode) && $codeData->notExpired()) {
        $user->updatePassword($newPassword);
        $codeData->updateToExpired();
        return true;
    }

    return false;
}

这里将获取对象的行为进行封装，是受到最近看laravel源码的影响。
我发现源码中会尽可能将一些行为细化到单一的方法中，然后进行逻辑处理的方法只会调用这些单一的行为，而不用理会行为的具体实现。这样做会使得代码可读性、可扩展性大大提高。