Вывести значение модели Laravel?

Question

[Максим]

есть модель Comment

в ней делаю функцию вывода значения like для пользователя. Если он лайнкул то 1, если дизлайк то 0

public function islike(){
        $likes = CommentLike::where('comment_id', $this->id)->where('user_id', Auth::user()->id)->first();
        return $likes->like;
    }

при выводе комментариев

<button class="btn-like">
   <svg><use xlink:href="#like"></use></svg>
   span class="like-count">{{ $comment->likes() }} - {{$comment->islike()}}</span>
</button>
<button class="btn-like dislike">
   <svg><use xlink:href="#like"></use></svg>
   <span class="dislike-count">{{ $comment->dislikes() }} - {{$comment->islike}}</span>
 </button>

хочу вывести значение, но пишет

ErrorException
Undefined offset: 1

Comments

[JhaoDa]

Слышал, поди, что код можно отлаживать? Открою секрет — это делается не через Тостер.

[Максим]

JhaoDa, он возвращает {"like":1}

а нужно чтобы просто число 1

[Дмитрий Кузнецов]

попробуйте вместо return $likes->like; вывести return true/false.

[JhaoDa]

Максим, кто возвращает, где возвращает... ты серьёзно думаешь, что мы тут в угадайку играем?

Хотя да, часто играем, потому что вопрошающие вообще дупля не отрезают в происходящем. Но это не есть хорошо.

[Максим]

Дмитрий Кузнецов, вот так выводит

$likes = CommentLike::where('comment_id', $this->id)->where('user_id', Auth::user()->id)->select('like')->first();
        return $likes['like'];

но вот так нет хочет return $likes->like;

[Владимир Кохан]

В модели напиши

protected $casts = [
        'likes' => 'object',
    ];

Answer 1

[Константин Б.]

За такое вас могут отправить в ад. Научитесь пользоваться отношениями. Нельзя из модели получать данные так как вы их получаете.
ваш код можно привести как минимум к такому виду

class Comment extends Model
{
    public function likes()
    {
        return $this->hasMany(CommentLike::class);
    }
    public function isLikeByUser($user = null)
    {
        if (!$user) {
            return false;
        }
        $users = $this->likes->pluck('user_id');
        return in_array($user->id, $users);
    }
}

В контроллере нужно обязательно запросить лайки сразу все, это оптимизирует количество запросов к бд до 2-х вместо ваших n + 1
$comments = Comment::with('likes')->get();

А уже в шаблоне вы можете просто запросить по пользователю
{{ $comment->isLikeByUser(Auth::user()) ? 1:0 }}

Comments

[NubasLol]

Всегда ли оптимально оптимизировать запросы к бд через with?

Ведь сам with медленный

[JhaoDa]

NubasLol, ну-ка, ну-ка, с чего это он медленный?

[NubasLol]

JhaoDa, он сначала загружает основную коллекцию, потом берет все ее id, делает where in на модели связи, и получает коллекцию моделей.

Потом бегает по всей это коллекции и связывает ее с основной, сопостовля ключи.

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

Но у меня много моментов в проекте, где with оаботает дольше самого запроса на получение, даже сложного

[JhaoDa]

NubasLol, т.е. в случае модели и сотни её детей сделать 100+1 запросов без with будет быстрее, чем 1+1 с with, верно?

[NubasLol]

JhaoDa, думаю такое возможно в некторых ситуациях, но я не тестил. Так то бд работает быстрее php

Как нибудь побалуюсь, посмотрю

[JhaoDa]

NubasLol, т.е. проблему N+1 выдумали недалёкие погромисты?

[NubasLol]

JhaoDa, так разные ситуции есть. Ну поятно, что обычно от нее надо избавляться. Состояние говнки и все такое

[Константин Б.]

NubasLol, ситуации разные и работать с ними нужно по-разному. Я просто дал общий совет. Конечно иногда бывают случаи когда очень много связей и они грузят систему. Но в моем случае такое было один раз всего. Чаще всего бывает что узким местом становятся пара тысяч запросов на странице когда with не используется

[NubasLol]

Константин Б., Ну да, ну я не как ошибку указал, а просто спросил)) Для себя

[Константин Б.]

NubasLol, да и использование отношений без with делает все тоже самое что и с ним, только запросов получается больше