Как сделать связь между 3 таблицами?

Question

[Andrey Markov]

Добрый день, как можно организовать связь между 3 таблицами.

У меня есть 3 таблицы, машины, каталог компонентов машины, и каталог повреждений.

Ни как не могу связать все эти 3 таблицы вместе, что бы удобно осуществлять выборку по ним.
На пример auto->getComponents->getDamagers();

Попытался связать всё это через отдельную таблицу, car_has_damage, где назначил 3 поля, car_id, component_id, damage_id.

Но столкнулся с проблеммой

foreach ($car->components as $component) {
    echo $component->name;

    foreach ($component->getDamages($car->id) as $damages) {
        echo $damage->name;
    }
}

Но вылетает с ошибкой, так как не может получить

$damage->name;  "Trying to get property of non-object"

При этом если ответ вывести через var_dump возвращаться куча NULL значений.

//Car model

public function getComponents(){
  return CarComponent::find()->all();
}

// Component model

public function getDamages($id)
{
    return $this->hasMany(Damage::className(),['id' => 'damage_id'])->viaTable('car_has_damage', ['component_id' => 'id'], function($query)use($id) { $query->onCondition(['car_id' => $id]); });
}

Answer 1

[Дмитрий]

Добрый вечер.
Для начала начните с того, что проверяйте, что передаёте для foreach().
Что-то подобное:
print_r($car->components);
Это не правильно
$component->getDamages($car->id)
Если Вы уж передаёте параметр, то сначала передайте, получите результат, а потом уже передавайте это в foreahc()

$component = $component->getDamages($car->id)
foreach($component as $damages){
 ****************
}

Если Вы обращаетесь к связи так
$component->getDamages($car->id)
то Вы получаете объект ActiveQuery, но Вы не получите данные.
Вот пример. У меня есть две модели, которые связаны через связь getCity()

public function getCity()
    {
        return $this->hasOne(Cities::className(), ['id' => 'city_id']);
    }

Вот, что я сделал, посмотрите результат и сравните.

$test = new RoutesCities(['city_id' => 1]);
echo "<pre>";
print_r($test->getCity());
print_r($test->city);

Результат 1

yii\db\ActiveQuery Object
(
[sql] =>
[on] =>
[joinWith] =>
[select] =>
[selectOption] =>
[distinct] =>
[from] =>
[groupBy] =>
[join] =>
[having] =>
[union] =>
[params] => Array
(
)

[_events:yii\base\Component:private] => Array
(
)

[_behaviors:yii\base\Component:private] => Array
(
)

[where] =>
[limit] =>
[offset] =>
[orderBy] =>
[indexBy] =>
[emulateExecution] =>
[modelClass] => app\modules\routes\models\Cities
[with] =>
[asArray] =>
[multiple] =>
[primaryModel] => app\modules\routes\models\RoutesCities Object
(
[from_rout] =>
[to_rout] =>
[tariff] =>
[arr_route] => Array
(
)

[_attributes:yii\db\BaseActiveRecord:private] => Array
(
[city_id] => 1
)

[_oldAttributes:yii\db\BaseActiveRecord:private] =>
[_related:yii\db\BaseActiveRecord:private] => Array
(
)

[_errors:yii\base\Model:private] =>
[_validators:yii\base\Model:private] =>
[_scenario:yii\base\Model:private] => default
[_events:yii\base\Component:private] => Array
(
)

[_behaviors:yii\base\Component:private] => Array
(
)

)

[link] => Array
(
[id] => city_id
)

[via] =>
[inverseOf] =>
)

Результат 2

app\modules\routes\models\Cities Object
(
[_attributes:yii\db\BaseActiveRecord:private] => Array
(
[id] => 1
[title] => Иркутск
[created_at] => 1506275820
[updated_at] => 1506692111
[status] => 1
[address] =>
[phone] =>
[time_work] =>
)

[_oldAttributes:yii\db\BaseActiveRecord:private] => Array
(
[id] => 1
[title] => Иркутск
[created_at] => 1506275820
[updated_at] => 1506692111
[status] => 1
[address] =>
[phone] =>
[time_work] =>
)

[_related:yii\db\BaseActiveRecord:private] => Array
(
)

[_errors:yii\base\Model:private] =>
[_validators:yii\base\Model:private] =>
[_scenario:yii\base\Model:private] => default
[_events:yii\base\Component:private] => Array
(
[beforeInsert] => Array
(
[0] => Array
(
[0] => Array
(
[0] => yii\behaviors\TimestampBehavior Object
(
[createdAtAttribute] => created_at
[updatedAtAttribute] => updated_at
[value] =>
[attributes] => Array
(
[beforeInsert] => Array
(
[0] => created_at
[1] => updated_at
)

[beforeUpdate] => updated_at
)

[skipUpdateOnClean] => 1
[owner] => app\modules\routes\models\Cities Object
*RECURSION*
)

[1] => evaluateAttributes
)

[1] =>
)

)

[beforeUpdate] => Array
(
[0] => Array
(
[0] => Array
(
[0] => yii\behaviors\TimestampBehavior Object
(
[createdAtAttribute] => created_at
[updatedAtAttribute] => updated_at
[value] =>
[attributes] => Array
(
[beforeInsert] => Array
(
[0] => created_at
[1] => updated_at
)

[beforeUpdate] => updated_at
)

[skipUpdateOnClean] => 1
[owner] => app\modules\routes\models\Cities Object
*RECURSION*
)

[1] => evaluateAttributes
)

[1] =>
)

)

)

[_behaviors:yii\base\Component:private] => Array
(
[0] => yii\behaviors\TimestampBehavior Object
(
[createdAtAttribute] => created_at
[updatedAtAttribute] => updated_at
[value] =>
[attributes] => Array
(
[beforeInsert] => Array
(
[0] => created_at
[1] => updated_at
)

[beforeUpdate] => updated_at
)

[skipUpdateOnClean] => 1
[owner] => app\modules\routes\models\Cities Object
*RECURSION*
)

)

)

Comments

[Andrey Markov]

Большое спасибо, вы очень внятно разъяснили суть моего вопроса и указали мне на мои ошибки, я за это благодарен.

Но, всё же так получается что в модели Components, такой параметр как car_id просто отсутствует, и вроде как не хотелось бы его отдельно объявлять так как он там не нужен, не ужели нет метода, получить не объект ActiveQuery а просто данные обратившись напрямую к методу и переда туда данные ?

$component = $component->getDamages($car->id)

[Дмитрий]

AndreyMarkov, Вам дали ссылку на вебинар Елисеева, смотрите, там всё доходчиво объясняется. Ничего страшного нет в том, что бы объявить недостающий параметр.
Хотя трудно сказать, что и как надо делать не видя структуры базы и моделей. Возможно Вам надо использовать не with, а joinWith. В параметрах можно создать псевдонимы таблиц и получить нужные данные.
p.s. смотрите вебинар с 1:46 минуты, там как раз про жадную загрузку есть.

[Дмитрий]

AndreyMarkov,

а просто данные обратившись напрямую к методу и переда туда данные ?

Можно попробовать так

$component = $component->getDamages()->andWhere(['id' => $car->id])

[Andrey Markov]

Добрый день, спасибо за помощь, я выспался и понял, что очень сильно ошибся и не понял главного, что в принципе при обращении напрямую к методу возвращается ActiveQuery.. и мне стоит просто добавить all()...

То есть я могу получить нужные мне данные как

$component = $component->getDamages()->andWhere(['id' => $car->id])->all()

так например

$component = $component->getDamages($car->id)->all();

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

[Дмитрий]

Andrey Markov, а сколько полезного Вы почерпнёте из вебинара, ВЫ себе не представляете)))

p.s. если помог - отмечайте решением) прошу не для собственного удовольствия, а для следующих вопрошающих)

[Andrey Markov]

С данным вебинаром знаком, он действительно хороший, но далек от той цели что я себе поставил, Связать 2 модели через третью не так сложно, а вот связать 3 таблицы через для меня оказалось тяжелее.

Хотя благодаря вашей помощи, я получил более менее рабочий прототип.

Хотя что интересно, единственное не могу понять. Почему если он не находит связь в таблице car_has_damage он отправлять такой запрос.


SELECT * FROM `damage` WHERE 0=1

При переборе компонентов автомобиля если в таблице car_has_damage существует связь типа_компонента с типом_повреждения запрос проходит верно

SELECT * FROM `car_has_damage` WHERE (`component_id`=1) AND (`car_id`=2)
SELECT * FROM `damage` WHERE `id` IN ('3', '2')

а вот во втором случане

SELECT * FROM `car_has_damage` WHERE (`component_id`=2) AND (`car_id`=2)
SELECT * FROM `damage` WHERE 0=1

Конечно я понимаю это связанно с тем что во втором варианте первый запрос не находит связей и приходит пустой, но мне не понятно зачем HasMany пытается найти связь если первый запрос приходит пустым.

Конечно можно сделать всё проще через
Join
public function getDamages(){
return Damage::find()->rightJoin('car_has_damage','car_has_damage.damage_id = damage.id')->andWhere(['car_has_damage.component_id' => $this->id]);
}

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

[Дмитрий]

Andrey Markov, в поисковой модели должен быть такой код

if (!$this->validate()) {
            // uncomment the following line if you do not want to return any records when validation fails
            // $query->where('0=1');
            return $dataProvider;
        }

[Дмитрий]

Andrey Markov, ещё раз пересмотрите ту часть вебинара, которая касается жадной загрузки, а лучше полностью просмотреть.
Связать таблицы можно и три и четыре, главное в голове всё сложить. Не просто это, но можно.
Когда Вы в поисковой модели получаете данные через find(), можно подключать кучу таблиц через псевдонимы.
Примерно так

$query = Routes::find()
            ->with(['city', 'routesCities', 'price'])
            ->joinWith([
                'city' => function(ActiveQuery $query){
                    $query->from(['rc1' => Cities::tableName()]);
                }
            ])
            ->joinWith([
                'routesCities' => function(ActiveQuery $query){
                    $query->from(['rc2' => RoutesCities::tableName()]);
                }
            ])
            ->groupBy('id');

Добавил в модель несуществующие параметры
public $city_id;
И ищу нужные данные

$query->andFilterWhere([
            'r.id' => $this->id,
            'r.number' => $this->number,
            'r.tariff' => $this->tariff,
            'r.created_at' => $this->created_at,
            'r.updated_at' => $this->updated_at,
            'r.status' => $this->status,
            'rc1.id' => $this->city_id,
        ]);

Параметр public $city_id; обязательно добавить в правила валидации.
Вот Вам связанные три таблицы.

[Andrey Markov]

slo_nik, по сути вот это почти решает мою проблему осталось найти возможность передать нужный параметр.

в самой модели ...

public function getDamages()
{
    return $this->hasMany(Damage::className(), ['id' => 'damage_id'])
            ->viaTable('car_has_damage',['component_id' => 'id']);
}

foreach (\app\models\base\Component::find()->with('damages')->all() as $component ) {

    //Выводим компонент
    echo Html::tag('div',$component->name);
    //Выводим все повреждения связанные с этим компонентом
    foreach ($component->damages as $damage) {
        echo Html::tag('span',' ----- ' .$damage->name);
    }
}

Практический работает идеально, за исключением того что выбирает повреждения всех машин.
Это естественно так как пока он работает только с 2 полями в таблица car_has_damage ['component_id','damage_id']

Теперь если прямо в модели пропишу

public function getDamages()
{
    return $this->hasMany(Damage::className(), ['id' => 'damage_id'])
            ->viaTable('car_has_damage',['component_id' => 'id'],
            function($query) { return $query->where(['car_id' => 1]); });
}

То уже на выходе получу модули а внутри модулей все повреждения связанные с первым автомобилем.

SELECT * FROM `car_component`

SELECT * FROM `car_has_damage` WHERE (`car_id`=2) AND (`component_id` IN (1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12, 13, 14, 15, 16, 17, 18, 19, 20, 21, 22, 23, 24, 25, 26, 27, 28, 29, 30, 31, 32, 33, 34, 35, 36, 37, 38, 39, 40, 41))

SELECT * FROM `damage` WHERE `id` IN ('15', '11')

Но пока я не могу найти способ как из контроллера передать ID нужной мне машины в viaTable.

если это сделать так

foreach (\app\models\base\Component::find()->with(['damages' => function($query) { return $query->andWhere(['car_id' => $car->id ]) ])->all() as $component ) {

То у меня получается вот такой ответ :

Unknown column 'car_id' in 'where clause';

Что в принципе естественно так как в таблица damage такое поле просто не существует.

SELECT * FROM `damage` WHERE (`car_id`=1) AND (`id` IN ('15', '11', '13'))

Если же всё сделать через joinWith ... то я получаю только те компоненты у которых есть повреждения связанные с этой машиной.

Прошу прощение, или я очень запутался или во что то не могу очень сильно вникнуть.

[Дмитрий]

Andrey Markov, не делайте так никогда.

foreach (\app\models\base\Component::find()->with('damages')->all() as $component )

Получается, что в цикле Вы обращаетесь к базе данных, это плохо.
Делайте так

// вначале файле, где все use
use app\models\base\Component;
// в нужном месте файла
$model = Component::find()->with('damages')->all();
foreach ($model as $component )

Я вижу, что вебинар Вы так и не посмотрели.
Параметры передаются в поисковых моделях, там же дополнительные условия для выборки из базы делаются.
Смотрите вебинар, потратьте с толком несколько часов.

[Andrey Markov]

Получается, что в цикле Вы обращаетесь к базе данных, это плохо.

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

Я про поисковую модель понял, но хотеться это сделать иначе. В принципе выше проведенный вариант почти меня устраивает, особенно если найду как обратиться к viaTable.

В остальном всё равно спасибо вам большое.

[Дмитрий]

Andrey Markov,

Я это понимаю, просто это быстрый пример для тестирования выборки и не более.

Даже для тестирования так делать не надо. Получается, что при каждой итерации цикла Вы делаете новый запрос в базу.

Я про поисковую модель понял, но хотеться это сделать иначе.

Как именно? Не изобретайте велосипед.

Answer 2

[Максим Федоров]

foreach ($component->getDamages($car->id) as $damages) {
     echo $damage->name;
}

у вас в единственном числе нужно, вы просто как всегда то, что нужно в единственном делаете в множественном, какой-то талант прямо :)

вот так нужно:

foreach ($component->getDamages($car->id) as $damage) {

UPD: Почему ваша IDE вам не подсвечивает ошибки?

UPD: Вижу у вас одна какая-то проблема. Вам крайне рекомендую просмотреть видео по связям в Yii2, очень толковое, я там даже менюшку набросал (в комментариях)
https://www.youtube.com/watch?v=URlo4QjNNao

Comments

[Andrey Markov]

Я опечатался когда писал пример для сайта, на деле же у меня почему то при обрещении к методу $component->damages всё работает но не могу в таком варианте передать ID а вот если делаю вот так $component->getDamages(); вылетает такая фигня так как место класса Damage возвращается NULL, NULL, NULL , app\models\Component и так далее