Laravel проверить есть ли запись в избранном, как решить проблему?

Question

[geyurgfuyewgfyuguy]

Пытаюсь реализовать проверку при выводе 50 записей, с проверкой на наличие записи в другой таблице по авторизованному пользователю. Но если пользователь не авторизован, выводит все записи как надо по критериям. Если пользователь авторизован, записи не выводит вообще. Если ставлю leftJoin, выводит все как надо, но только если юзер авторизован, добавляет еще записи которые есть в избранном, но этого не должно быть.

Вот код

DB::table('book')
	->leftJoin('favorite', 'book.id', '=', 'favorite.book_id')
    ->select('book.*', 'favorite.id as favorite_checked')
    ->where([
        ['book.country', '=', $country->id],
        ['book.city', '=', 0],
        ['book.approve', '=', 1],
    ])
    ->where(function($q)
    {
    	if( Auth::check() ) $q->where('favorite.user_id', '=', Auth::user()?->id);
    })
    ->paginate(50);

Нужна исключительно переменная favorite_checked - которая на фронтеде выводит, есть ли в закладках книга или нет.

Comments

[geyurgfuyewgfyuguy]

Если здесь сделать вот так $q->where('favorite.user_id', '=', Auth::user()?->id); поменять с where на orWhere - тогда приплетает записи которые есть в закладках, но не относящиеся к критериям других where, что логично.

Answer 1

[Дмитрий]

Вы left join должны ограничивать при присоединении - а не в where.

$user = Auth::user();
DB::table('book')
    ->select('book.*')
    ->where([
        ['book.country', '=', $country->id],
        ['book.city', '=', 0],
        ['book.approve', '=', 1],
    ])
    ->when($user, function($query, $user)
    {
    	  $query->leftJoin('favorite', function($query) use($user){
                 $query->whereColumn('book.id', 'favorite.radio_id')->where('favorite.user_id', '=', $user->id);
          })->addSelect('favorite.id as favorite_checked');

    })
    ->paginate(50);

P.S. А модели вы почему не используете?

Comments

[geyurgfuyewgfyuguy]

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

[Дмитрий]

geyurgfuyewgfyuguy, надо просто прям щас скомандовать себе "Даешь модели!" и вырезать к чертям такое. Я вас уверяю спасете себе миллиарды нервных клеток

[geyurgfuyewgfyuguy]

Дмитрий, а сложно вообще перейти на них? Я понимаю, что сейчас просто сплошной говнокод, хочется однозначно писать правильный код.

[geyurgfuyewgfyuguy]

И на отладку уходит уйма времени)

[Дмитрий]

geyurgfuyewgfyuguy, нет не сложно. особенно если вы будете соблюдать laravel naming conventions. таблица book - должна быть books, favorite - favorites и так далее. Тогда по факту вам нужно выполнить комманду

php artisan make:model Book

после чего код будет:

Book::select('book.*')
    ->where([
        ['book.country', '=', $country->id],
        ['book.city', '=', 0],
        ['book.approve', '=', 1],
    ])
    ->when($user, function($query, $user)
    {
    	  $query->leftJoin('favorite', function($query) use($user){
                 $query->whereColumn('book.id', 'favorite.radio_id')->where('favorite.user_id', '=', $user->id);
          })->addSelect('favorite.id as favorite_checked');

    })
    ->paginate(50);

Можно сказать перешли на 90%, после прописывания fillables - на 100. После чего такие громоздкие where можно будет запихивать в scope и пользоваться массой других полезностей которые дают модели

[geyurgfuyewgfyuguy]

Дмитрий, выглядит реально круто, спасибо! Обязательно изучу и буду переходить на модели, потому что времени уходит уйма

Answer 2

[jazzus]

В Ларавел это делается очень просто с withExists. Можно не писать простыни джоинов.

Comments

[Дмитрий]

для этого нужно сначала модели описать и отношения.

[jazzus]

Дмитрий, одна команда, чтобы создать модель и три строчки кода на отношение. Секунд 15 без автокомплита. И потом сэкономленные часы/дни/годы на проектирование и поддержание простыней говнокода и куча возможностей фреймворка, связанных с отношениями и моделями.

[Дмитрий]

jazzus, да не вопрос - только это надо сделать, ибо в вопросе простой query builder

[jazzus]

Дмитрий, ну да надо сделать, конечно пока не поздно. И я об этом) Заново изобретать реализованное в фреймворке не лучшая идея.