如何封装多角色用户资料更新逻辑？寻求灵活优化方案

你的抽象类思路完全可行，这是策略模式的典型应用场景，能很好地封装不同角色的更新逻辑，同时提升扩展性。下面是具体的优化和落地建议：

1. 完善抽象类定义 #

抽象类要明确核心契约，同时提取公共逻辑（比如用户实例、请求对象的初始化），避免重复代码：

abstract class ProfileUpdater
{
    protected $user;
    protected $request;

    public function __construct(User $user, Request $request)
    {
        $this->user = $user;
        $this->request = $request;
    }

    // 定义验证规则的契约
    abstract protected function getValidationRules(): array;

    // 定义具体更新逻辑的契约
    abstract protected function performUpdate(array $validatedData): void;

    // 统一对外执行入口，封装公共流程
    public function handle()
    {
        // 公共验证步骤
        $validatedData = $this->request->validate($this->getValidationRules());
        
        // 执行角色专属更新逻辑
        $this->performUpdate($validatedData);
    }
}

2. 实现各角色的具体更新类 #

针对医生、诊所分别实现抽象类，专注各自的业务规则：

class DoctorProfileUpdater extends ProfileUpdater
{
    protected function getValidationRules(): array
    {
        return [
            'license_number' => 'required|string|unique:doctors,license_number,' . $this->user->id,
            'specialty' => 'required|string',
            // 其他医生专属字段的验证规则
        ];
    }

    protected function performUpdate(array $validatedData): void
    {
        // 更新医生专属表信息
        $this->user->doctor->update($validatedData);
        // 同步更新用户表的通用字段（如昵称）
        $this->user->update([
            'name' => $validatedData['name'] ?? $this->user->name
        ]);
    }
}

class ClinicProfileUpdater extends ProfileUpdater
{
    protected function getValidationRules(): array
    {
        return [
            'clinic_name' => 'required|string|unique:clinics,name,' . $this->user->id,
            'address' => 'required|string',
            // 其他诊所专属字段的验证规则
        ];
    }

    protected function performUpdate(array $validatedData): void
    {
        $this->user->clinic->update($validatedData);
        $this->user->update([
            'name' => $validatedData['clinic_name'] ?? $this->user->name
        ]);
    }
}

3. 用工厂模式解耦控制器与具体实现 #

控制器里不要直接实例化具体更新类，通过工厂类根据角色匹配对应的更新器，进一步提升灵活性：

class ProfileUpdaterFactory
{
    public static function make(User $user, Request $request): ProfileUpdater
    {
        return match($user->role) {
            'doctor' => new DoctorProfileUpdater($user, $request),
            'clinic' => new ClinicProfileUpdater($user, $request),
            default => throw new InvalidArgumentException('不支持的用户角色')
        };
    }
}

4. 控制器调用逻辑 #

控制器变得简洁，只负责核心流程的串联：

public function update(Request $request, $userId)
{
    $user = User::findOrFail($userId);
    
    $updater = ProfileUpdaterFactory::make($user, $request);
    $updater->handle();

    return response()->json(['message' => '资料更新成功']);
}

方案优势 #

• 封装性：每个角色的验证规则、更新逻辑都封装在独立类中，职责单一，便于维护
• 灵活性：新增角色时，只需新增一个继承ProfileUpdater的类，在工厂里添加一行匹配规则即可，符合开闭原则
• 可测试性：每个更新类可以单独编写单元测试，无需依赖其他角色的逻辑

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