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

Question

[Руслан Абсалямов]

Может я конечно непрвильно читал документацию, но связь многие ко многим работает при помощи 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);

Comments

[Руслан Абсалямов]

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

[anlamas]

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

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

Comments

[Руслан Абсалямов]

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

[anlamas]

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

[Руслан Абсалямов]

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]

Руслан Абсалямов,
не знаю зачем вы эту тираду пишите, вместо конкретной задачи.
Что я имею ввиду под задачей.
получаем список исполнителей, которые имеют направления с id_user равным auth()->user->id

$executors = Executor::whereHas('directions', function($query) {
    $query->where('id_user', auth()->user->id);
})

[Руслан Абсалямов]

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

[anlamas]

Руслан Абсалямов,
можете сами посмотреть

DB::connection()->enableQueryLog();
// ваши запросы
$queries = DB::getQueryLog();
dd($queries)