Как составить правильно отношение?

Question

[Данил Сысоев]

У меня стоит такая задача. Например: Есть 2 таблицы. Пользователей и например телефонов

id|name|money
1|Кент|0
2|Тнек|100
3|Нект|100

id|user_id|model|buy
1| 1|Nokia|100
1| 1|Xiaom|150

У пользователей 2 и 3 есть деньги на телефон Nokia, но они его еще не купили. Как получить список этих пользователей?

Users::where('money', '>=', 100)->hasMany('Mobiles');
Я видимо совсем в трех соснах запутался. Какое отношение использовать, и какую таблицу делать подчиненной? Или здесь отношениями не обойтись, и придется выбирать их циклом?

Comments

[Илья]

а что значит user_id во второй таблице?

[Данил Сысоев]

Илья, тот кто купил этот телефон

[Илья]

мм, я бы сделал так:
таблица юзеров
id|name|money
таблица телефонов
id|model|buy
таблица покупок
user_id|phone_id

А там уже можно будет сделать модели и отношения у них.

Текущая реализация структуры таблиц довольна странная)

[jazzus]

Илья, а эта типа нормальная. Потом автор решает продавать еще 100 групп товаров и делает еще 200 таблиц и 400 отношений. Я интернет магазины не проектировал, но думаю товары нужно выносить в products, категории в categories, а заказы в orders.

[Данил Сысоев]

3@jazzus, Я привел просто пример, у Васи было 5 яблок, 3 груши отдал Пете. Просто на примере реального проекта это показалось бы запутанным.
Если описать ее в двух словах, то мне нужно получить список пользователей, которые не купили тариф под определенным номером.

[Илья]

jazzus, нормальная, для выполнения задачи. Ясное дело для интернет-магазина не годится, но о нём и речи не шло. С чего Вы взяли, что у автора интернет-магазин?)

[jazzus]

Илья, если фантазировать то смысла корректировать структуру нет т.к. настоящая логика неизвестна. В примере интернет-магазин где продают телефоны

Answer 1

[jazzus]

Тогда так

$users = User::doesntHave('phones')
             ->where('money', '>=', 100)
             ->get();

// в User
public function phones()
{
   return $this->hasMany('App\Models\Phone');
}

Comments

[Данил Сысоев]

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

[Данил Сысоев]

От себя еще добавлю к этому решению, чтобы было полноценное

$users = User::whereDoesntHave('phones', function (Builder $query) {
                   $query->where('model', 'Nokia');
             })->where('money', '>=', 100)
             ->get();

// в User
public function phones()
{
   return $this->hasMany('App\Models\Phone');
}

[jazzus]

Данил Сысоев, если фильтровать нужно то да и для query лучше использовать scope с переменной модели, чтобы код не дублировать. А так doesntHave достаточно.