请问如何优化这段代码

问答 / 2 / 18 / 创建于 2年前

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

调用这个函数的是controller，这里需要用到User和EmailVerifyLog两个对象。
这里加了if判断对象是为了防止执行方法会报错。
但是这样每取一个对象就要判断一次，那就可能会有无数个if，我自己想到的方案有：
1、两个对象同时取同时判断，就少了一个if了，但是如果email不存在，可能会多执行一次查询
2、在控制器获取对象并判断，把对象传进来，这个函数就负责执行函数就行了
请问大家有没有其他方案？

Yoann

见习助教 4 声望

暂无个人描述~

0 人点赞

你将学到如 RESTFul 设计风格、PostMan 的使用、OAuth 流程，JWT 概念及使用和 API 开发相关的进阶知识。

以构建论坛项目 LaraBBS 为线索，展开对 Laravel 框架的全面学习。应用程序架构思路贴近 Laravel 框架的设计哲学。

讨论数量: 18

JaguarJack

Laravel 9.x 译者 883 声望 / 开发 @ 家里蹲开发公司

public function resetPassword($email,$verifyCode,$newPassword)
{
    $user = User::findByEmail($email)->first();

    if (! $user) {
     return false;
   }

   $codeData = EmailVerifyLog::findByEmail($email)->get()->last();

   if (! $codeData) {
      return false;
    }

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

    return false;
}

尽量提前 return

2年前评论

24K大白羊

现在还蛮喜欢类似Go的编程风格

MZ0x01

这种可读和维护性上应该是比较好的。不过根据函数名的含义，楼主的定义的函数返回true|false 是不够的，可能还需要额外的错误码/错误信息表示具体的失败原因给调用方。

WindyLater

@ModStart 这里应该只是举个例子，具体就是自定义的异常信息了

寞小陌

这个风格的代码看着比较舒服

seth-shi

830 声望

   public function resetPassword($email, $verifyCode, $newPassword)
    {
        // 1 换一种写法
        // $user = User::findByEmail($email)->first();
        $user = User::findByEmail($email)->firstOrFail();
        // 2 千万别 get 在 last, 那样子等于从数据库获取所有可能一千条， 然后在 PHP 从数组的 1000 个取一个
        // $codeData = EmailVerifyLog::findByEmail($email)->get()->last();
        $codeData = EmailVerifyLog::findByEmail($email)->latest()->first();

        if (!$codeData->verifyCode($verifyCode) || !$codeData->notExpired()) {
            return false;
        }

        // update 应该返回成功或者失败，建议上事务
        return $user->updatePassword($newPassword) && $codeData->updateToExpired();
    }

2年前评论

浅步调

课程读者 6 声望

自定义validator，查询用户是否存在，或者查询邮件里验证码是否和入参一致其他地方应该也能用到

2年前评论

cccyzloong

课程读者 181 声望

$user = User::findByEmail($email)->first(); 这个应该可以在验证器里面验证

$codeData = EmailVerifyLog::findByEmail($email)->get()->last(); 放到中间件

2年前评论

Aroad

课程读者 116 声望

first可以优化为，嫌麻烦可以再封装一下

firstOr(function(){
    throw new Exception();
})

下面那个获取log直接倒序first就好了

$codeData = EmailVerifyLog::findByEmail($email)->orderBy('id')->firstOr(function(){
    throw new Exception();
});

2年前评论

小李世界

Laravel 8.x 译者 2.0k 声望 / Doge 先锋 @ dogeow.com

提前 return 掉

2年前评论

Alone88

116 声望

快速入门《Laravel 9 中文文档》

2年前评论

JaguarJack

Laravel 9.x 译者 883 声望 / 开发 @ 家里蹲开发公司

public function resetPassword($email,$verifyCode,$newPassword)
{
    $user = User::findByEmail($email)->first();

    if (! $user) {
     return false;
   }

   $codeData = EmailVerifyLog::findByEmail($email)->get()->last();

   if (! $codeData) {
      return false;
    }

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

    return false;
}

尽量提前 return

2年前评论

24K大白羊

现在还蛮喜欢类似Go的编程风格

MZ0x01

WindyLater

@ModStart 这里应该只是举个例子，具体就是自定义的异常信息了

寞小陌

这个风格的代码看着比较舒服

Mutoulee

课程读者 333 声望 / Developer @ Dobeen.Net

文档：快速入门《Laravel 9 中文文档》

$user = User::findByEmail($email)->firstOrFail();

2年前评论

susa

16 声望


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

判断条件不多的话，感觉没必要过分优化
$user = User::findByEmail($email)->first();只是查询存在的话可以换成 $user = User::findByEmail($email)->exists();和doesntExist();

2年前评论

php_yt

1.1k 声望

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

2年前评论

seth-shi

830 声望

   public function resetPassword($email, $verifyCode, $newPassword)
    {
        // 1 换一种写法
        // $user = User::findByEmail($email)->first();
        $user = User::findByEmail($email)->firstOrFail();
        // 2 千万别 get 在 last, 那样子等于从数据库获取所有可能一千条， 然后在 PHP 从数组的 1000 个取一个
        // $codeData = EmailVerifyLog::findByEmail($email)->get()->last();
        $codeData = EmailVerifyLog::findByEmail($email)->latest()->first();

        if (!$codeData->verifyCode($verifyCode) || !$codeData->notExpired()) {
            return false;
        }

        // update 应该返回成功或者失败，建议上事务
        return $user->updatePassword($newPassword) && $codeData->updateToExpired();
    }

2年前评论

王小大

Laravel 9.x 译者 151 声望

最有必要优化的就是如下语句

EmailVerifyLog::findByEmail($email)->get()->last();

改为

EmailVerifyLog::findByEmail($email)->latest()->first();

2年前评论

DotO

3 声望

public function resetPassword($email, $verifyCode, $newPassword)
{
    // 建個 findLastEmail 的 scope
    if($codeData = EmailVerifyLog::findLastEmail($email)) {
        if (
            $codeData->verifyCode($verifyCode)
            && $codeData->notExpired()
            && $user = User::findFirstEmail($email) // 建個 findFirstEmail 的 scope
        ) {
            $user->updatePassword($newPassword);
            $codeData->updateToExpired();
            return true;
        }
    }

    return false;
}

考量的點在於，會先驗證 email，所以 email 相關的會先處理過關之後才將 user 抓取出來做處理

2年前评论

看上隔壁小花了啦

221 声望

public function resetPassword($email, $verifyCode, $newPassword)
{
    $user = User::findByEmail($email)->first();
    if (! $user) {
        return false;
    }

    $codeData = EmailVerifyLog::findByEmail($email)->get()->last();
    if (! $codeData) {
        return false;
    }

    if (!$codeData->verifyCode($verifyCode)) {
        return false;
    }

    if (!$codeData->notExpired()) {
        return false;
    }

    DB::beginTransaction();
    try {
        $user->updatePassword($newPassword);
        $codeData->updateToExpired();
        DB::commit();
    } catch (\Throwable $e) {
        DB::rollBack();
        return false;
    }

   return true;
}

2年前评论

33qis

23 声望

如果需要返回true false 加个try catch

public function resetPassword($email,$verifyCode,$newPassword)
{
    $user = User::findByEmail($email)->firstOrFail();
    $codeData = EmailVerifyLog::findByEmail($email)->orderByDesc('id')->firstOrFail();
    if ($codeData->verifyCode($verifyCode) && $codeData->notExpired()) {
        $user->updatePassword($newPassword);
        $codeData->updateToExpired();
    }
}

2年前评论

讨论应以学习和精进为目的。请勿发布不友善或者负能量的内容，与人为善，比聪明更重要！

帮助

请问如何优化这段代码

高认可度评论：

社区赞助商

关于 LearnKu

资源推荐

服务提供商

其他信息

请问如何优化这段代码

高认可度评论：

社区赞助商

关于 LearnKu

资源推荐

服务提供商

其他信息

请登录