如何精简Laravel框架中Policy策略文件的冗余代码？

现有写法的问题 #

你当前在Policy中手动实例化Observer调用方法的写法不符合职责拆分原则：Observer的deleting方法是设计给模型事件系统自动触发的，不应该作为公共方法被业务代码手动调用，额外实例化也属于冗余逻辑。 #

最优重构方案：业务逻辑下沉到模型 #

将「是否存在有效订阅」的判断逻辑放到Plan模型中作为公共方法，Observer和Policy都直接调用模型方法即可，完全消除Policy对Observer的依赖：

1. 首先在Plan.php模型中新增专属判断方法

// App\Models\Plan.php
public function hasActiveSubscription(): bool
{
    return $this->subscriptions()->where('status', '!=', 'canceled')->exists();
}

2. 简化userPolicy.php的delete方法，直接调用模型方法

// App\Policies\UserPolicy.php
public function delete(User $user, Plan $item): bool
{
    return $user->hasAdminRole() && !$item->hasActiveSubscription();
}

3. 同步修改userObserver.php的deleting方法复用逻辑

// App\Observers\UserObserver.php
public function deleting(Plan $item): bool
{
    return $item->hasActiveSubscription();
}

该方案的优势： #

• 彻底移除Policy中实例化Observer的冗余代码，代码精简度提升50%以上
• 职责拆分清晰：模型管理自身的业务规则判断，Policy负责权限校验，Observer负责事件触发后的逻辑处理
• 有效订阅判断逻辑可全局复用，后续其他业务场景需要用到时不需要重复写SQL
• 调整了判断顺序，先校验用户权限再查数据库，非管理员请求直接返回结果，无需执行SQL，性能更优

兼容现有逻辑的轻量重构方案 #

如果暂时不想调整Observer和模型的现有代码，只精简Policy部分，可以将Observer的判断逻辑改成静态调用：

1. 改造userObserver.php，抽离静态公共方法

// App\Observers\UserObserver.php
public function deleting(Plan $item): bool
{
    return self::hasActiveSubscription($item);
}

public static function hasActiveSubscription(Plan $item): bool
{
    return $item->subscriptions()->where('status', '!=', 'canceled')->exists();
}

2. Policy中直接调用静态方法，无需实例化

// App\Policies\UserPolicy.php
public function delete(User $user, Plan $item): bool
{
    return $user->hasAdminRole() && !\App\Observers\UserObserver::hasActiveSubscription($item);
}

额外优化提示 #

• 原Observer的注释存在逻辑错误，建议修正为* 返回true代表存在有效订阅，禁止删除，避免后续维护踩坑
• 给所有方法加上返回值类型声明，提升代码可维护性
• 给Policy的$item参数加上明确的Plan类型约束，避免传参错误

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