Laravel控制器如何实现单函数完成权限校验并自动跳转

你最初的写法不生效的核心原因是：PHP中函数内部return的响应不会自动中断外层控制器方法的执行，只有控制器方法直接return的响应，才会被Laravel的HTTP内核捕获并返回给客户端。你没有在控制器方法中return校验函数的返回值，重定向响应会被直接丢弃，程序会继续往下执行返回视图的逻辑。

你不需要拆分两个方法，以下两种方案都可以满足需求：

方案1：零判断直接调用（最贴合你的原始预期） #

修改校验方法，在校验不通过时直接发送重定向响应并终止执行，控制器内只需要单行调用即可，不需要额外写if判断、不需要return方法返回值：

public static function redirectIfUserShouldNotSeeOrder(Order $order)
{
    if ($order->user_id !== auth()->id()) {
        redirect()
            ->route('orders.index')
            ->with('status', 'You are not authorized to view this order')
            ->send();
        exit();
    }
}

public function order(Order $order)
{
    self::redirectIfUserShouldNotSeeOrder($order);
    return view('orders.show', compact('order'));
}

public function payPage(Order $order)
{
    self::redirectIfUserShouldNotSeeOrder($order);
    return view('orders.pay', compact('order'));
}

原理说明：->send()方法会直接将重定向响应发送给客户端，后续的exit()会中断PHP执行流程，完全不会走到返回视图的逻辑。

方案2：可测试性更好的规范写法（推荐） #

不强制终止进程，利用短三元运算符实现单行返回，避免exit导致单元测试中断的问题，更符合Laravel的开发规范：

public static function redirectIfUserShouldNotSeeOrder(Order $order)
{
    if ($order->user_id !== auth()->id()) {
        return redirect()
            ->route('orders.index')
            ->with('status', 'You are not authorized to view this order');
    }
    return null;
}

public function order(Order $order)
{
    return self::redirectIfUserShouldNotSeeOrder($order) ?: view('orders.show', compact('order'));
}

public function payPage(Order $order)
{
    return self::redirectIfUserShouldNotSeeOrder($order) ?: view('orders.pay', compact('order'));
}

原理说明：校验不通过时方法返回重定向响应，会被控制器直接return给HTTP内核；校验通过时方法返回null，短三元运算符会自动执行后面的视图返回逻辑，控制器内仅需一行代码即可完成校验+响应的全流程。

• 你目前拆分两个方法的写法逻辑完全正确，只是代码冗余度较高，上述两种方案都可以在不引入中间件、不拆分多个工具方法的前提下实现需求，足够轻量。
• 如果后续同类权限校验逻辑持续增加，可以考虑使用Laravel内置的Gate/Policy授权功能，是这类场景的官方推荐实现方式。

内容的提问来源于stack exchange，提问作者Cyrus V.