如何将多控制器方法的重复代码提取为外部函数？abort(404)存疑

针对你要把重复控制器代码抽成外部函数的需求，这里提供两种实用方案，同时说明abort相关的顾虑：

方案一：使用Trait（控制器方法复用首选） #

Trait是Laravel中复用控制器代码的常用方式，适合这种多个控制器方法共享的逻辑：

namespace App\Traits;

use Illuminate\Support\Facades\Auth;
use App\Models\User;

trait RetrievesUser
{
    /**
     * 获取授权访问的用户实例
     * @param string|int|null $id
     * @return User
     */
    public function getAuthorizedUser($id = false)
    {
        if (empty($id)) {
            $id = Auth::user()->id;
        }

        if (Auth::user()->isSuperUser()) {
            $user = User::where('id', $id)->withTrashed()->first();

            if (!$user) {
                abort(404);
            }
        } elseif (Auth::user()->id == $id) {
            $user = User::findOrFail($id);
        } else {
            abort(401);
        }

        return $user;
    }
}

在需要的控制器中引入Trait并调用：

class UserController extends Controller
{
    use RetrievesUser;

    public function index(Request $request, $id = false)
    {
        $user = $this->getAuthorizedUser($id);
        // 执行后续业务逻辑
    }
}

方案二：使用服务类（解耦复杂逻辑） #

如果后续逻辑可能扩展，或者需要在非控制器场景复用，用服务类封装更解耦：

namespace App\Services;

use Illuminate\Support\Facades\Auth;
use App\Models\User;

class UserRetrievalService
{
    /**
     * 获取授权访问的用户实例
     * @param string|int|null $id
     * @return User
     */
    public function getAuthorizedUser($id = false)
    {
        if (empty($id)) {
            $id = Auth::user()->id;
        }

        if (Auth::user()->isSuperUser()) {
            $user = User::where('id', $id)->withTrashed()->first();

            if (!$user) {
                abort(404);
            }
        } elseif (Auth::user()->id == $id) {
            $user = User::findOrFail($id);
        } else {
            abort(401);
        }

        return $user;
    }
}

控制器中通过依赖注入调用：

class UserController extends Controller
{
    public function index(Request $request, UserRetrievalService $service, $id = false)
    {
        $user = $service->getAuthorizedUser($id);
        // 执行后续业务逻辑
    }
}

关于abort(404)的顾虑 #

完全不用担心abort在控制器外部调用的问题：

• abort()本质是抛出HttpException异常，Laravel框架的全局异常处理器会自动捕获这类异常，返回对应的HTTP状态码和响应内容；
• 不管是在Trait、服务类还是其他地方调用abort，处理逻辑和控制器内调用完全一致，不会出现异常无法捕获的情况；
• 另外User::findOrFail($id)本身也是找不到用户时抛出404异常，和你手动写的abort(404)效果一致，框架都会统一处理。

内容的提问来源于stack exchange，提问作者poashoas