如何减少 laravel 中策略文件中的代码？

Question

我有两个文件 userPolicy.php 和 userObserver.php ，我在这两个文件中编写了一些逻辑，这意味着如果用户有任何活动订阅，则用户不会被删除到此为止，它工作正常。

userPolicy.php

  public function delete(User $user, $item)
    {
       $canceled_subscription=new userObserver();
       return (!$canceled_subscription->deleting($item)) && ($user->hasAdminRole());    
    }

userObserver.php

 public function deleting(Plan $item){
//based on $has_subscriptions it should allow to delete or call the userPolicy.php delete function
        $has_subscriptions=$item->subscriptions()->where('status','!=','canceled')->exists();
        return $has_subscriptions;
    }

现在我想要的是我想重构 userPolicy.php 中的代码，这意味着有没有办法减少 userPolicy.php 文件中的代码（主要是 $canceled_subscriptions），有没有机会减少代码来自 userPolicy.php 请帮我重构代码

Answer 1

我更喜欢通过分离条件来保持代码简洁，这样更容易快速理解。您可以在用户观察者中使用静态函数，如下例所示。

// UserPolicy
public function delete(User $user, $item) 
{
    if (! $user->hasAdminRole()) return false;

    return UserObserver::deleting();
}

// UserObserver
public static function deleting(Plan $item)
{
    return $item->subscriptions()->where('status','!=','canceled')->exists();
}