Можно ли исключить find для WithCount?

Question

[jazzus]

Проверяю количество через withcount. Сделал в модели

public function hasModel($model)
  {
    if ($this->withCount($model)->find($this->id)->{$model.'_count'}>0) return true;
    return false;
  }

И чувствую, что можно обойтись без find($this->id). Мы ведь уже обращаемся к найденому объекту. Но без find идет ошибка.

Можно сделать проще с коунтом:

public function hasModel($model)
    {
      if ($this->$model()->count()>0) return true;
      return false;
    }

но насколько я знаю из документации withcount -это более оптимизированный способ подсчитать количество. Или в данном случае разницы нет?

Answer 1

[Александр Ковальчук]

Надо вызвать не модель, а функцию отношения

public function hasModel($relation)
{
  return $this->$relation()->count()>0;
}
public function comments()
{
    return $this->hasMany('App\Comment');
}
...
$this->hasModel('comments')

но вообще лучше использовать has

public function comments()
{
    return $this->hasMany('App\Comment');
}
...
$this->has('comments')

Comments

[jazzus]

Так у меня функция и вызывается в примерах. Функции связанных моделей. По-поводу has - я так и не понял, как его в шаблоне использовать. Если только

if($list->has('products')->get()->count()>0){
...
}

хочется типа

if($list->has('products')){
...}

[Александр Ковальчук]

jazzus, has проверяет есть ли связанная сущность, по сути has делает все то что тебе и надо
она так и работает

if($list->has('products')){
...}

[jazzus]

Что-то я совсем запутался) has возвращает Builder, а мне нужен true/false для проверки

[jazzus]

либо может в самой модели делать

public function hasModel($model)
  {
    if ($this->has($model)) return true;
    return false;
  }

сейчас попробую

[Александр Ковальчук]

jazzus, ок давай по порядку я с примером пост комментарии
Когда использовать has? Если тебе нужно выбрать только посты с комментариями ты пишешь
$posts = $model->has('comments')->get()
в переменную posts попадут только посты с комментариями

если тебе нужно проверить есть ли в посте комментарии надо использовать count

$hasComments = $post->comments()->count()//Количество комментов в посте 0 это false

[jazzus]

Александр Ковальчук, да-да. Я тоже самое выше описал) И мне не нравится такой вариант.

[jazzus]

Можно было бы в контроллере withcount делать и в шаблоне уже products_count>0 проверять, но хочется человекообразным языком написать типа продуктов нет поэтому показываем нет продуктов

[Александр Ковальчук]

jazzus, Ок все для вас, есть еще метод exists
+ сори за этот пример оно не так работает, я протупил вменсто has так работает exists

//Не правильно
if($list->has('products')){
...}
//Правильно
if($list->products()->exists()){
...}

[jazzus]

В принципе у меня ошибок то и нет никаких. И первый и второй вариант в моем посте работают. Я просто консультируюсь можно ли find исключить и есть ли преимущество у withcount в данном случае над count

[Александр Ковальчук]

jazzus, withCount сделает + 1 запрос к бд, а сount на каждую модель отдельный запрос, но только в том случае если ты будешь юзать withCount при выборке, а не на каждую отдельно

$posts = $model->withCount('comments')->get();
...
//В моделе
public function hasModel($model)
  {
    if ($this->{$model.'_count'}>0) return true;
    return false;
  }
//Ну и конечно if тут бесполезный можно просто

public function hasModel($model)
  {
    return $this->{$model.'_count'}>0;
  }

[jazzus]

Александр Ковальчук, Мне не очень хочется получать коллекцию постов, чтобы их посчитать. В данном случае посты нужны, но я также делаю универсальный запрос для проверки и тех моделей, которых получать вообще не нужно. Чтобы hasmodel проверял без цифр есть модель или нет. А почему if бесполезный? Разве функция не может вернуть 0 и это в дальнейшем будет возвращать true?

[Александр Ковальчук]

jazzus, об if когда ты используешь оператор сравнения он всегда возвратит bool поэтому if бесполезный
с универсальным запросом нюанс в том что для каждого такого запроса будет отдельный запрос в бд (это не страшно если моделей не много) но в таком случае лучше использовать

public function hasModel($model)
  {
    //1 запрос в бд
    return $this->$model()->exists();
  }

public function hasModel($model)
  {
    //тоже 1 запрос в бд
    return $this->$model()->count()>0;
  }

public function hasModel($model)
  {
    //2 запроса в бд при чем эта модель у тебя уже есть
    return $this->withCount($model)->find($this->id)->{$model.'_count'}>0;
  }

наверное это надо было написать с самого начала, просто только сейчас полностью понял что именно ты спрашиваешь, ах утро

[jazzus]

Александр Ковальчук, насчет if понял. А по-поводу ->exists() оно проверяет наличие записей а данном случае? Что касается count

return $this->$model()->count()>0;

здесь оно не будет дополнительные запросы к бд делать т.к. будет также как withcount?

[Александр Ковальчук]

jazzus, как exists так и count будут делать 1 запрос в бд, если хочешь без них то надо сначала вытащить все вместе, а потом проверять в коллекции для этого есть with но он вытащит всю сущность, а не только есть такое или нет как exists или количество как count

[jazzus]

exists то, что нужно! Я и не знал, что так можно) В итоге я использовал:

public function hasModel($model)
  {
    return $this->$model()->exists();
  }