php - 如何减少 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();
}