Laravel路由声明时在if语句中调用控制器方法失效问题

嘿，我懂你想实现的需求——当用户是管理员时，访问/leave路由就调用TimeController@show方法；如果不是管理员，就返回“not found”。你之前的写法之所以没生效，是因为在闭包里直接写'uses' => 'TimeController@show'根本不是Laravel识别路由控制器的正确方式，闭包本身是用来处理请求逻辑的，不是用来定义路由目标的。

给你两种靠谱的实现方式：

方式一：在闭包中直接调用控制器方法 #

你可以在闭包里判断权限后，直接实例化控制器并调用对应的方法，这样就能正确执行控制器的逻辑了：

Route::get('/leave', function() {
    // 先确保用户已登录，避免未登录时Auth::user()报错
    if(Auth::check() && Auth::user()->admin) {
        // 实例化控制器并调用show方法
        return app(\App\Http\Controllers\TimeController::class)->show();
    } else {
        // 返回404，用abort更符合Laravel的错误处理规范
        abort(404, 'not found');
    }
})->name('admin-time');

方式二：使用路由中间件（推荐） #

其实这才是Laravel中处理权限验证的最佳实践，把权限逻辑抽离成中间件，不仅路由代码更简洁，还能在其他路由上复用这个权限验证逻辑：

1. 首先创建一个管理员验证中间件：

php artisan make:middleware CheckAdmin

2. 打开生成的app/Http/Middleware/CheckAdmin.php，修改handle方法：

public function handle(Request $request, Closure $next)
{
    // 验证用户已登录且是管理员
    if (!Auth::check() || !Auth::user()->admin) {
        abort(404, 'not found');
    }

    return $next($request);
}

3. 在app/Http/Kernel.php的$routeMiddleware数组中注册这个中间件：

protected $routeMiddleware = [
    // 其他中间件...
    'admin' => \App\Http\Middleware\CheckAdmin::class,
];

4. 最后在路由中使用这个中间件：

Route::get('/leave', \App\Http\Controllers\TimeController::class.'@show')
    ->middleware('admin')
    ->name('admin-time');

这样一来，只有管理员能访问这个路由，非管理员访问会直接返回404，路由代码也更清晰易维护。

内容的提问来源于stack exchange，提问作者Nil the babe