Как правильно организовать связь между таблицами?

Question

[Иван Петров]

Здравствуйте. Запарился с связями таблицы в Yii 1. Никак не могу правильно получить данные.

У меня есть 2 таблицы:
1 таблица складов, есть id, name, city_id

И собственно с помощью поля city_id должен получить название города из таблицы с городами.

Связь организовываю в модели City

'stockInCity' => array(self::HAS_MANY, 'Stock', 'city_id'),

После вызываю в компоненте

$a = City::model()->with('stockInCity')->findByPk(1250);
        echo "<pre>"; print_r($a); echo "</pre>";

На выходе получаю


где city_id в моем случаем это Москва.

Нужно, как я понимаю, обратиться наоборот к складам, а в stockInCity, чтобы был массив города. Поменять местами получается. Пытался в модели Stock сделать связь

'stockInCity' => array(self::HAS_MANY, 'City', 'id'),

но на выходе получается пустой объект.

Определенно, что-то неправильно делаю, но не могу понять. В чем может быть проблема? Спасибо

Comments

[RidgeA]

Склад может быть в нескольких городах? Зачем массив?

[Иван Петров]

RidgeA: В одном городе может быть несколько складов

[RidgeA]

Если имея AR склада надо получить AR города через relations, то связь надо и прописывать в AR склада.
Если склад находится в одном городе, то смысла делать HAS_MANY нет.
В AR склада что-то вроде

'city' => [self::HAS_ONE, 'City', ['id'=>'city_id']]

Могу ошибиться с массивом ['id'=>'city_id'], возможно надо наоборот, не помню.

Если связь указывать в виде одного значения, а не массива, то yii берет из текущей AR primary key, если для построения relation надо указывать иное поле, то это надо явно указывать.

Answer 1

[Boris Köln]

Если из Stock надо получить City, то в Stock надо указать

'city' => array(self::HAS_ONE, 'City', ['id' => 'city_id']),

P.S. Правильнее называть не "stockInCity", а "stocks"

Comments

[Иван Петров]

А если к примеру, в 1 городе 2 склада?

[Boris Köln]

Склады города - $city->stocks
Город склада - $stock->city

[Иван Петров]

Boris Korobkov: Что - то не то получается. Пройдусь чуть вперед. )) Сама схема сейчас выглядит так:


Например, захожу на детальную, у которого id = 1. В промежуточной таблице поле item_id.


Где видно, что товар на двух складах, а вот собственно таблица складов:


Получается должны получить в итоге 2 склада и 2 города.
Сейчас вытягиваю данные по id склада, т.е. она выводит только то что в первой строке. Не могу понять, получается с таблицей складов еще нужна связь и с таблицей товаров? Или как правильнее все-таки, чтобы вывести оба склада и оба города? Спасибо

[Boris Köln]

1. Вопрос был "как у stock получить city". Я ответил. Что не так? Должно работать.

2. "Как из item получить stocks" - это уже совсем другой вопрос.
В item определить stocks с MANY_MANY. А далее из stock получить city - это предыдущий вопрос.

Подробнее см. www.yiiframework.com/doc/guide/1.1/ru/database.arr

[Иван Петров]

Boris Korobkov:
1) Все отлично
2) Связь есть

'itemMany' => array(self::MANY_MANY, 'Stock',  tbl_item_stock(item_id, stock_id)'),

Просто, когда мы получаем в stock делаем связь с city. То в компоненте нужно, чтобы в findByPk я передавал id открытого товара. Поэтому сейчас не совсем правильно получается. То что передоется в findByPk она принимает как поле id таблицы Stock. И поэтому выводит только один город....

[Boris Köln]

Правильнее назвать stocks, а не itemMany.

findByPk как именно вызывается?

[Иван Петров]

Boris Korobkov:

$a = Stock::model()->with('Stocks')->findByPk(1);
        echo "<pre>"; print_r($a); echo "</pre>";

        foreach ($a->Stocks as $stock) {
            $cities[] = $stock->name;
        }

На данный момент так, после полученные города по циклу. Сейчас там 1, т.е. id товара, который открыт. А в таблице 1 находится в двух городах...

[Boris Köln]

Stock::model()->with('Stocks')->findByPk(1);

Во-первых, у Stock нет связи stocks. Она есть только у Item и City.
Во-вторых, если надо найти ТОВАР=1, то и указывать надо модель ТОВАРА Item::model()...

[Иван Петров]

Во-вторых, если надо найти ТОВАР=1, то и указывать надо модель ТОВАРА Item::model()...

В этом и есть кажется загвоздка? Если указывать Item, то как получать склады и города )

[Boris Köln]

$item = Item::model()->findByPk(1);
foreach($item->stocks as $stock) {
   $city = $stock->city;
}

P.S. with нужен только для оптимизации, когда выбирается много item, чтобы stocks выбирались одним запросом, а не в цикле по каждому item. Для одного item не нужно указывать with

[Иван Петров]

Boris Korobkov:

Получается так, если указываем модель item, и уже с помощью полученных city_id нужно получить название городов)

[Иван Петров]

Решил проблему таким образом)))

$itemId = Yii::app()->request->getParam('id');

            $a = Item::model()->with('Stocks')->findByPk($itemId);

        foreach ($a->Stocks as $stock) {
            $city_id = City::model()->findByPk($stock->city_id);
            $cities[] = $city_id->name;
        }

[Boris Köln]

1. Называйте переменные правильно! Не $a, а $item. Не $city_id, а $city
2. City::model()->findByPk($stock->city_id) лишний. Вместо него $stock->city (как для этого описать relation - выше уже писал)

[Иван Петров]

Boris Korobkov: Делал так, но без City::model()->findByPk($stock->city_id) он ведь никак не может получить запись из таблицы с городами

[Boris Köln]

Может. Как это сделать - см. мой самый первый комментарий.

[Иван Петров]

Boris Korobkov:

И ошибка: Array to string conversion

[Boris Köln]

На какой именно строчке ошибка?

Укажите relations всех моделей.

[Иван Петров]

Boris Korobkov: $city[0]->name так сработало

[Boris Köln]

Значит, у вас HAS_MANY. Прочитайте еще раз мой самый первый комментарий - надо HAS_ONE.
Укажите relations всех моделей.

[Иван Петров]

Массив получал в таком виде

[Иван Петров]

Boris Korobkov: Да, HAS_MANY, потому что может быть и 2 и 3 города

[Boris Köln]

Не может!!! При структуре

1 таблица складов, есть id, name, city_id

у склада только ОДИН город!

[Иван Петров]

Boris Korobkov: У склада да, а в 1 городе может быть несколько городов. Например, как щас у меня))



Этот товар находится на 2 складах, 1 из складов в Москве другой в Казани.

Или как я понял, вы имеете ввиду вытягивать 2 разных массива для каждого склада отдельно?

[Boris Köln]

Если у склада ОДИН город - значит, в stock->relations должен быть city HAS_ONE.
Если у города МНОГО складов - значит, в city->relations должен быть stocks HAS_MANY.