вы неправильно решаете вашу задачу и не читаете доки - это ваша самая грубая ошибка
Можно оптимизировать код и запросы двумя способами
Первый:
для геттеров получать информацию из коллекции, а не из базы
public function getUsersActiveCountAttribute()
{
return $this->Users->where('status', '=', 'active')->count();
}
Второй:
Брать данные из базы предварительно их загрузив
public function UsersActive()
{
return $this->hasMany(User::class)->where('status', '=', 'active');
}
В контроллере не забудьте эти данные подгрузить
Для первого
$goups = Group::with('Users')->get();
Для второго
$goups = Group::withCount(['UsersActive'])->get();
Когда ты получаешь вторым способом то для каждой группы у тебя будет доступно свойство
$group->users_active_count
Советую вам сначала вдумчиво и несколько раз почитать доку