Как правильно написать relation в модели Yii?

Question

[Иван]

Есть две модели: Product и ProductPhoto.

У модели ProductPhoto есть атрибут sort_order (порядок сортировки). Нужно, чтобы по запросу $product->primaryPhoto была доступна первая фотография (с наименьшим sort_order).
Правильно ли я написал relation в модели Product, или есть способ поизящнее?

public function relations() {
  return array(
    'photos' => array(self::HAS_MANY, 'ProductPhoto', 'id'),
    'primaryPhoto' => array(self::HAS_ONE, 'ProductPhoto', 'product_id', 'order' => 'sort_order', 'limit' => '1'),
  );
}

Answer 1

[Yuri Morozov]

Нужно проверить две вещи:

1. Как оно будет вести себя в случае отображения списка продуктов с картинкой (что-то мне подсказывает, что тут Вас ожидает сюрприз)

2. Не добавляет ли HAS_ONE лимит по умолчанию (что-то мне подсказывает, что да)

Что же касается поизящнее - лично мне всегда было жалко расходовать время БД на такие штуки, поэтому конкретная реализация сильно зависит от.

Comments

[Yuri Morozov]

Уточнение по пункту 1: если Вам действительно нужно будет выводить список с картинкой - такое отношение не сработает, увы.
Ну, точнее, сработает, но через задницу.

[Иван]

1. У меня работает во вьюхе CListView вот так <?=$data->primaryPhoto['url']?>. Вочему это через задницу?
2. Да, с лимитом я переборщил, он тут не нужен и в работающей модели у меня его нет.
А как бы сделали вы? Подскажите, пожалуйста, парочку других способов, пришедших на ум.

[Yuri Morozov]

@0neS 1. а Вы итоговый запрос посмотрите (в дев-панели). Я сам пока не проверял, но интуитивно понятно, что тут либо подзапрос нужен, либо группировка, а Yii без дополнительных телодвижений сам такое делать не будет.

2. Любым доступным способом явно прописываю "обложку". Это может быть флаг в таблице изображения, или cover_id у товара + belongs_to, или явно прописанный путь до обложки, или еще что-то типа blob прямо в товаре. Целостностью при этом занимаются хуки или триггеры, тут уж никуда не деться, зато не приходится каждый раз опрашивать кучу изображений, сортировать и извлекать одно, и так для каждой записи.

Ну, жаба душит просто, из-за какой-то сраной картинки БД дрючить.

[Иван]

@metamorph в моем случае эти фотографии товаров выводятся на странице товара следующем образом: крупно главная и отдельным блоком все остальные в порядке сортировки.
В админ-панели фотографии продукта сортируются с помощью jquery-плагина и основной считается первая.
По-вашему, правильнее будет выставлять в бд 1й фотографии флаг primary=1, а остальным sort_order в порядке сортировки?

[Yuri Morozov]

@0neS В Вашем случае вообще непонятно, зачем использовать отдельное отношение.

Вы же наверняка дергаете photos для извлечения всех фотографий, оттуда и выдерните.

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

БД хотя бы пожалейте, какие все чёрствые прям стали.

[Yuri Morozov]

@0neS А вот когда заказчик попросит вывести список товаров с главной картинкой неподалеку (а он стопудово попросит, знаю я их) - тогда и надо будет что-то придумывать.

Но, скажем, имея проставленный флаг primary Вы сможете выдернуть весь список с картинками максимум в два запроса (один на товары, второй на изображения, дальше Yii сам разберется). Можно и в один (left join), но не факт, что это эффективнее будет.

[Иван]

@metamorph заказчик я сам и о списке товаров с главной картинкой я как раз Вам забыл сказать, для этого и понадобилось.

[Yuri Morozov]

@0neS Короче, я скажу так.

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

В Yii, конечно, дофига магии, но она не всегда хорошо работает. Нужно следить, контролировать и при необходимости бить по рукам.

[Иван]

@metamorph спасибо за диалог. Сделаю с флагом.