Laravel模型层数据验证方案的合理性咨询

首先，你想统一验证逻辑、避免多处重复代码的思路完全正确，这是减少冗余、保证逻辑一致性的好做法，没必要接受控制器里的代码重复。不过当前的实现有可以优化的地方：

1. 用Laravel原生的ValidationException替代手动拼接错误信息 #

你现在手动循环拼接错误字符串抛出普通Exception的做法完全没必要，Laravel已经提供了专门的ValidationException，能自动处理错误格式，不管是HTTP请求还是GraphQL场景都能适配：

use Illuminate\Validation\ValidationException;

// 验证失败时直接抛出
if ($validator->fails()) {
    throw ValidationException::withMessages($validator->messages()->toArray());
}

这个异常抛出后，Laravel会自动处理成适合当前请求类型的响应格式（比如HTTP请求会返回422状态码和结构化错误数组，GraphQL也能捕获到结构化的错误信息），不用你手动拼接字符串。

2. 抽离验证规则，进一步减少重复 #

可以把验证规则抽离到User模型的静态方法里，或者单独的请求类（不过因为要在多个场景复用，静态方法更灵活）：

// 在User模型中添加
public static function createRules(): array
{
    return [
        'email' => 'email|max:255|unique:users',
        'password' => 'required|min:6|max:20|confirmed',
        'password_confirmation' => 'required|same:password',
    ];
}

之后不管哪里需要验证规则，直接调用User::createRules()即可，不用重复写规则数组。

3. 建议引入服务层（Service Class）承载业务逻辑 #

你提到把业务逻辑放在模型上层的第二层类，其实这就是Laravel中常用的服务层模式，比直接在模型里写业务逻辑更符合单一职责原则：

• 模型只负责数据持久化、关联定义等ORM相关操作
• 服务层负责封装业务逻辑（包括验证、密码加密、用户创建等）

示例代码：

// app/Services/UserService.php
namespace App\Services;

use App\Models\User;
use Illuminate\Validation\ValidationException;
use Illuminate\Support\Facades\Validator;

class UserService
{
    public function create(array $data): User
    {
        // 统一验证
        $validator = Validator::make($data, User::createRules());
        
        if ($validator->fails()) {
            throw ValidationException::withMessages($validator->messages()->toArray());
        }
        
        // 处理业务逻辑（比如密码加密）
        $data['password'] = bcrypt($data['password']);
        
        // 调用模型创建用户
        return User::create($data);
    }
}

之后在控制器和GraphQL mutation中只需要注入服务类并调用方法：

// UserController中
public function store(Request $request, UserService $service)
{
    $user = $service->create($request->all());
    // 返回响应
}

// GraphQL mutation中
public function resolve($root, array $args, $context)
{
    $service = app(UserService::class);
    $user = $service->create($args);
    // 返回结果
}

这样所有业务逻辑和验证都统一在服务层，完全没有重复代码，架构也更清晰。

总结 #

你的核心想法（统一验证避免重复）是正确的，不需要回到控制器重复写验证。只需要用Laravel原生的ValidationException替代手动拼接错误，再引入服务层封装业务逻辑，就可以避免重复造轮子，同时让代码结构更合理。

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