Как сократить код при связи многие ко многим?

Question

Руслан Абсалямов @rusline18

Laravel

Как сократить код при связи многие ко многим?

Может я конечно непрвильно читал документацию, но связь многие ко многим работает при помощи find(), Но если мне нужно выеси все значение у которого id_user равно авторизованному пользователю. То тут у меня проблема вышла. Возможно ли мне сократить код?

$directions = Direction::where('id_user', Auth::user()->id)->with('executes');
        $execute = [];
        foreach ($directions->get() as $value){
            foreach (Direction::find($value->id)->executes()->get() as $item){
                $execute[] = $item->name;
            }
        }
        dd($execute);

Вопрос задан более трёх лет назад
128 просмотров

4 комментария

Подписаться 1 Простой 4 комментария

Руслан Абсалямов @rusline18 Автор вопроса

aleksejjjj, Немного не понятен вот такой код, Я же должен все перекинуть hasManyThrough в Direction модель верно?
ТО есть примерно так должно быть

class Direction extends Model
{
    protected $table = 'direction';
    protected $fillable = ['id_user', 'name'];

    public function executes()
    {
        return $this->hasManyThrough('App\Execute', 'App\Execute_direction');
    }
}

class Execute_direction extends Model
{
    protected $table = 'execute_direction';
    protected $fillable = ['direction_id', 'execute_id'];
    public $timestamps = false;

    public function execute()
    {
        return $this->hasOne('App\Execute');
    }
}

И мне смутил Вот такой пример Auth::user()->executes В моем случае

$directions = Direction::where('id_user', Auth::user()->id)->executes()->get();
foreach ($directions as $value){
             $execute[] = $value->name;
             foreach (Direction::find($value->id)->executes()->get() as $item){
             $execute[] = $item->name;
          }
     }

Написано более трёх лет назад

Руслан Абсалямов @rusline18 Автор вопроса
aleksejjjj, Но тут промежуточная таблица все равно нужна же как понимаю, потому что в 1 направлений может быть несколько исполнителей, допустим 5 и более. Я просто реализовал ее примерно как в ответе на комментарий было указано
$executors = Executor::whereHas('directions', function($query) { $query->where('id_user', auth()->user->id); })
Написано более трёх лет назад
Руслан Абсалямов @rusline18 Автор вопроса
aleksejjjj,
Вариант 2: - Направление имеет несколько исполнителей, и исполнитель принадлежит нескольким направлениям. Тогда связь многие ко многим и промежуточная таблица нужна.

Да просто утрирована я взял то что сейчас беспокоило.
Но модель Execute_direction не нужна ни в одном из этих случаев

То есть модель полностью убрать в этом случае, если мне нужно потом убирать в случае если пользователь захочет убрать исполнителя
Написано более трёх лет назад
Руслан Абсалямов @rusline18 Автор вопроса

aleksejjjj, Типо просто запрос к бд данных напрямую?

Написано более трёх лет назад

Решения вопроса 1

6 комментариев

Руслан Абсалямов @rusline18 Автор вопроса

AibekAS Вы уверены что так должно быть?
Я использую связь многие ко многим, а получаю данные из таблицы Direction. А мне нужно id этой моделе Direction где id_user является авторизованным и вывести все подходящие значение из таблицы Execute. Вот примерно что я должен получить.
Если не понятно то у одного направление имеется несколько исполнителей, я сейчас в данном примере который Вы описали получаю это направление а не исполнителей. Поэтому я и сделал такой код у себя.

Написано более трёх лет назад
anlamas @anlamas

Руслан Абсалямов, честно, не могу понять что вы хотите вывести. Если вы опишите нормально, то смогу помочь. А так, извините. Просто опишите задачу.

Написано более трёх лет назад
Руслан Абсалямов @rusline18 Автор вопроса
AibekAS, Допустим Направление "тренинг" и в нем 5 исполнителей.
В ларавеле такое примерно
class User extends Model { /** * The roles that belong to the user. */ public function roles() { return $this->belongsToMany('App\Role'); } }

И в контроллере
$roles = App\User::find(1)->roles()->orderBy('name')->get();

В таком примере выведит как раз этих 5 исполнителей из другой модели.
Принцип должен быть ясен.
У меня есть модель направление Direction
id name id_user

Есть модель исполнителей Execute
id name id_user

И есть промежуточная таблица Execute_direction
direction_id execute_id

Мне нужно вывести не по определенному id в моделе Direction, а вместо find($id) сделать условие по id_user который будет равен Auth::users()->id.
Вот примерно такой. То есть мне нужны исполнители которые указаны в промежуточной таблицы
Написано более трёх лет назад
anlamas @anlamas
Руслан Абсалямов,
не знаю зачем вы эту тираду пишите, вместо конкретной задачи.
Что я имею ввиду под задачей.
получаем список исполнителей, которые имеют направления с id_user равным auth()->user->id
$executors = Executor::whereHas('directions', function($query) { $query->where('id_user', auth()->user->id); })
Написано более трёх лет назад
Руслан Абсалямов @rusline18 Автор вопроса

AibekAS, Да такой результат мне нужен, значит whereHas это добавляет к связанной таблицы запрос как понимаю, верно?

Написано более трёх лет назад
anlamas @anlamas
Руслан Абсалямов,
можете сами посмотреть
DB::connection()->enableQueryLog(); // ваши запросы $queries = DB::getQueryLog(); dd($queries)
Написано более трёх лет назад

Пригласить эксперта

Ваш ответ на вопрос

Войдите, чтобы написать ответ

Войти через центр авторизации

Похожие вопросы

Laravel

Простой
Laravel всегда возвращает ответ 200?
- 1 подписчик
- вчера
- 75 просмотров
1

ответ
Laravel

Простой
Почему в Laravel 11 некорректно срабатывает rehash пароля?
- 1 подписчик
- 24 апр.
- 48 просмотров
0

ответов
Laravel

Простой
Почему я получаю это Call to a member function attach() on null?
- 1 подписчик
- 21 апр.
- 74 просмотра
0

ответов
Laravel

Простой
Как в модели Belongsto обратиться по нескольким полям?
- 1 подписчик
- 19 апр.
- 107 просмотров
1

ответ
Laravel

Простой
Как локально работать stevebauman/location для определения геопозиции в Laravel?
- 1 подписчик
- 18 апр.
- 43 просмотра
1

ответ
HTML

+2 ещё

Простой
Как подкрасить 2 блока в разные цвета по всей ширине в дочернем Blade-шаблоне?
- 1 подписчик
- 18 апр.
- 66 просмотров
2

ответа
PHP

+1 ещё

Простой
Переписать грамотнее и с помощью Laravel реально ли и каким лучше образом?
- 1 подписчик
- 13 апр.
- 201 просмотр
3

ответа
Laravel

Простой
Как остановить wire:poll?
- 1 подписчик
- 12 апр.
- 61 просмотр
0

ответов
Vue.js

+2 ещё

Простой
Как побороть проблему CORS в Laravel?
- 1 подписчик
- 10 апр.
- 144 просмотра
0

ответов
Laravel

Простой
В Laravel очереди работают не последовательно в Bus::chain?
- 2 подписчика
- 10 апр.
- 148 просмотров
1

ответ
Показать ещё Загружается…

Бэкенд-разработчик на Laravel (Middle Level)

StreamHunt.tv

от 150 000 до 250 000 ₽

PHP / Laravel разработчик

DIGITAL SECTOR • Краснодар

от 100 000 до 150 000 ₽

Team Lead Laravel

nopreset

от 180 000 ₽

Подключить сервер к сети

27 апр. 2024, в 02:39

1500 руб./за проект

Необходимо сверстать приложение согласно макету Figma используя React

26 апр. 2024, в 22:22

1500 руб./за проект

Написать модуль подключения матрицы Sony к ПЛИС (Verilog)

26 апр. 2024, в 21:30

15000 руб./за проект

aleksejjjj, Немного не понятен вот такой код, Я же должен все перекинуть hasManyThrough в Direction модель верно?
ТО есть примерно так должно быть
class Direction extends Model { protected $table = 'direction'; protected $fillable = ['id_user', 'name']; public function executes() { return $this->hasManyThrough('App\Execute', 'App\Execute_direction'); } }

class Execute_direction extends Model { protected $table = 'execute_direction'; protected $fillable = ['direction_id', 'execute_id']; public $timestamps = false; public function execute() { return $this->hasOne('App\Execute'); } }

И мне смутил Вот такой пример Auth::user()->executes В моем случае
$directions = Direction::where('id_user', Auth::user()->id)->executes()->get(); foreach ($directions as $value){ $execute[] = $value->name; foreach (Direction::find($value->id)->executes()->get() as $item){ $execute[] = $item->name; } }
aleksejjjj, Но тут промежуточная таблица все равно нужна же как понимаю, потому что в 1 направлений может быть несколько исполнителей, допустим 5 и более. Я просто реализовал ее примерно как в ответе на комментарий было указано
$executors = Executor::whereHas('directions', function($query) { $query->where('id_user', auth()->user->id); })
aleksejjjj,
Вариант 2: - Направление имеет несколько исполнителей, и исполнитель принадлежит нескольким направлениям. Тогда связь многие ко многим и промежуточная таблица нужна.

Да просто утрирована я взял то что сейчас беспокоило.
Но модель Execute_direction не нужна ни в одном из этих случаев

То есть модель полностью убрать в этом случае, если мне нужно потом убирать в случае если пользователь захочет убрать исполнителя
aleksejjjj, Типо просто запрос к бд данных напрямую?

Answer 1 · 2018-03-18 09:05:55

Конечно, неправильно. Не знаю, где ты нашел такой пример в документации

// вижу там такой пример
$users = User::with('podcasts')->get();
// твой код сокращается до
$directions = Direction::where('id_user', Auth::user()->id)->with('executes')->get();

Как сократить код при связи многие ко многим?

Войдите, чтобы написать ответ

Минуточку внимания

Войдите на сайт