Yii2. Как сделать выборку записей из БД по условию?

Question

[Дмитрий Петрик]

Задача такая, нужно выбрать из БД все книги у которых есть определенный тег. В БД у каждой книги есть поле tags в нем теги записаны следующим образом: "1,5,7,12". Т.е. просто цифры перечисленные через запятую. В модели создаю такую функцию:

public static function getBooksByTag($tag)
    {
           return Books::find()->where(in_array($tag, explode(',', 'tags')))->all();
    }

По замыслу вызываю функцию в любом месте, и получаю список нужных книг. На деле такая реализация не работает. Функция всегда возвращает все книги. Как допилить эту функцию до ума? Что я тут делаю не так?

Можно конечно в лоб пойти, через foreach перебрать все книги, у каждой книги перебрать все теги сверяя их с искомым. Просто хочется разобраться с той функцией выше

Upd. Сейчас сделал так. Все работает как надо. Но нельзя ли сделать элегантнее?

public static function getBooksByTag($tag)
{
   $books = Books::find()->all(); //выгребаем все книжки
   $books_id[]=null; //это будет массив с id-шниками книг у которых есть нужные теги
    foreach ($books as $book){ 
        if (in_array($tag, explode(',', $book->tags))){  //проверяем есть ли наш тег в списке тегов книги
            $books_id[]=$book->id; //если тег есть, то добавляем id книги к нашему массиву  
         }
    }
return Books::findAll($books_id); //возвращаем все найденные книги
}

Answer 1

[Егор Мокеев]

Конкретно по задаче

public static function getBooksByTag($tag)
{
    $pattern = "(^{$tag}$)|(^{$tag},)|(,{$tag},)|(,{$tag}$)";
    return Books::find()->where("tags REGEXP '{$pattern}'")->all();
}

А по-хорошему список тегов должен хранится в отдельной таблице и доставаться через связи

Comments

[Дмитрий Петрик]

Мсье знает толк... :)) А если тегов будет больше чем 4?

[Mylistryx]

Мсье не знает толк, а дело говорит! Теги прописать через таблицу связей (pivot table вроде в Yii2) и все проблемы отвалятся сами собой. Извращение - это как раз то, что делаете Вы.

[Mylistryx]

Рано нажал отправить. В итоге получим Таблица1 - таблица с ID тега и его значением, Таблица2 - Таблица с книгами, Таблица3 - таблица связи, в которой хранится ID тегов и ID книг. Это даст возможность легко вешать к книге неограниченное кол-во тегов, выбирать по ним, да и сама структура БД так выглядит правильно.

[Дмитрий Петрик]

Mylistryx: идея Ваша понятна, однако, для меня не приемлема. Таблицы связи нет. id тегов записываются через запятую в столбец tags в таблице книг. Добавление таблицы связей потребует переписи кучи кода написанного другими людьми. Овчинка выделки не стоит

[Егор Мокеев]

Дмитрий Петрик: почитайте на досуге про регулярные выражения, вам явно не помешает...
Код рабочий. А регулярка вычисляет 4 положения тега: одиночный, в начале, середине и в конце списка

[Дмитрий Петрик]

Егор Мокеев: простите мне мое невежество. Все отлично работает!

Answer 2

[primitiveko]

var_dump(explode(',', 'tags')) проверьте

$this->tags мб?

Comments

[Дмитрий Петрик]

Выражение вида in_array($tag, explode(',', 'tags')) даст булевый ответ. По сути получим либо where(true), либо where(false). И при любом варианте на выходе получаются все книги. Похоже, я не правильно условия для выборки задаю.

Answer 3

[Rowdy Ro]

что-то непонятное делаете

in_array возвращает true.

return Books::find()->where(['tags'=>implode(",", $tag)])->all();

Comments

[Дмитрий Петрик]

Ну так я понимаю, что не то делаю :) Но Вы сейчас тоже не то делаете. $tag - это тег по которому я делаю поиск. Т.е. это не массив. Это просто какое-то число. Массив из тегов лежит в базе в столбце tags.

Answer 4

[Mikhail Osher]

Начнем с того, что не самым лучшим образом реализовали книги/тэги.
Вы зашиваете их в книгу, хотя надо было сделать через many-to-many.

books: id, title, ...
tags: id, title, ...
books_tags: book_id, tag_id

Comments

[Дмитрий Петрик]

А чем Ваш вариант лучше моего? Меньше запросов в БД будет? Быстрее работать будет или что? На предмете БД в универе нам тоже рассказывали что Ваш вариант правильный. Но, честно говоря, проектирование БД никогда не было моей сильной стороной, я так и не понял плюсов реализации через таблицу связей...

Answer 5

[Александр Исаев]

Mikhail Osher, а почему человек не самым лучшим образом реализовал? Почему надо many-to-many? Лишняя таблица появляется в базе данных.

Comments

[Дмитрий Петрик]

Тут все ситуативно. Например, в данном случае many-to-many было бы правильным решением. Представьте себе задачу, когда Вам нужно сделать выборку книг по тегам. В моем случае, это нужно перебирать все книги, извлекать массив с тегами и искать в нем нужный тег. Очень много лишних действий.
В реализации many-to-many нужный тег находится в таблице tags и через таблицу books_tags джоином можно сразу выбрать нужные книги.
Мне еще предстоит заняться здесь рефакторингом.