如何将Laravel登录逻辑从Controller迁移至Service类并调用

1. 完善UserService类 #

调整App\Services\UserService.php代码，让服务专注处理登录验证的业务逻辑（遵循单一职责原则，不直接处理HTTP跳转）：

<?php

namespace App\Services;

use Illuminate\Support\Facades\Auth;

class UserService
{
    /**
     * 验证用户登录凭证
     * @param array $credentials 登录凭证（邮箱、密码）
     * @return bool 验证结果
     */
    public function attemptLogin(array $credentials): bool
    {
        return Auth::attempt($credentials);
    }
}

2. 修改AuthController调用服务 #

保留控制器的请求验证和HTTP跳转逻辑，将登录验证的业务逻辑委托给UserService：

<?php

namespace App\Http\Controllers;

use Illuminate\Http\Request;
use App\Services\UserService;

class AuthController extends Controller
{
    protected $userService;

    // 推荐用依赖注入，便于后续测试和维护
    public function __construct(UserService $userService)
    {
        $this->userService = $userService;
    }

    public function postLogin(Request $request)
    {
        $validated = $request->validate([
            'email' => 'required',
            'password' => 'required',
        ]);

        // 调用服务类的登录验证方法
        if ($this->userService->attemptLogin($validated)) {
            return redirect()->intended('dashboard')
                ->withSuccess('登录成功');
        }

        return redirect("login")
            ->withSuccess('账号或密码错误，请重新输入');
    }
}

关键说明 #

• 服务层专注业务逻辑处理（此处为验证登录凭证），不直接处理HTTP响应（如跳转），让控制器负责HTTP层面操作，符合MVC职责划分。
• 使用依赖注入实例化UserService，而非直接new UserService()，更贴合Laravel设计思想，也方便后续单元测试。
• 若执意要在服务类中处理跳转也可实现，但会导致服务层与HTTP层耦合，降低代码复用性，不推荐。

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