QueryBuilderで絞り込みをしてからのWithをしてよいでしょうか？

Question

以下のテーブルがあるとします。

lessons(id,title,open)
students(id,name)
options(id,name)
lesson_students(id,lesson_id,student_id)
lesson_options(id,lesson_id,option_id)

とあるstudentsのlessonsをlesson_optionsと一緒に取得したい場合に、以下のQueryBuilderとWithを書きました。
(lessons.openがfalseの場合、lesson_students自体を取得されないようしたい)

$this
->lesson_students
->join('lessons', 'lesson_students.lesson_id', '=', 'lessons.id')
->where('lesson_students.student_id', xxx)
->where('lessons.open', true)
->with([
  'lesson',
  'lesson.options',
])->get();

QueryBuilderとWith（Eloquent）は一緒に記述してもいいものでしょうか？
より良い書き方があれば教えてほしいです。

Answer 1

上級WHERE節 WHERE EXISTS句を使ってはどうですか？
或いはリレーションの存在のクエリを利用してみてください。

//  app/Lesson.php或いはapp/Models/Lesson.php
public function students(){
   return $this->belongsToMany(Student::class);
}

//  app/Student.php或いはapp/Models/Student.php
public function lessons(){
   return $this->hasManyThrough(Lesson::class);
}

// Controller
$this
->lesson_students
->where('student_id', xxx)
->whereExists('lessons', function($query){  
    return $query->where('open', true);
})
->with([
  'lessons',
  'lessons.options',
])->get();

Comments

[love_engineerin]

ありがとうございます！
whereHasで期待した動作になりました。
joinにしたい場合、existにしたい場合で使い分けようと思います。

Answer 2

多対多リレーションではどうでしょうか？

->join('lessons', 'lesson_students.lesson_id', '=', 'lessons.id')

リレーションではすでにstudentを絞り込んでるので、joinを使う目的がよくわかりません。

//  app/Lesson.php或いはapp/Models/Lesson.php
public function students(){
   return $this->belongsToMany(Student::class);
}

//  app/Student.php或いはapp/Models/Student.php
public function lessons(){
   return $this->hasManyThrough(Lesson::class);
}

// Controller
$this
->lesson_students
->where('student_id', xxx)
->with([
  'lessons' => function($query){  
    return $query->where('open', true);
  },
  'lessons.options',
])->get();

Comments

[love_engineerin]

ありがとうございます！
ただこれだとlessons.openがfalseの場合でも、lesson_studentsは取得できてしまいますよね？
lessons.openがfalseの場合、lesson_studentsを取得されないようにするために、joinして先に絞り込んでからwithを使う必要があると考えています。

「lessons.openがfalseの場合、lesson_studentsを取得されないようにする」という説明が抜けておりました。
すみません。

Answer 3

$xxx = '123456789';
$this
->lesson_students
->with([
  'lesson' => function($query) use ($xxx){
     return $query->where('student_id', $xxx);
  },
  'lesson.options',
])->get();

この書き方でどうでしょうか？

Comments

[love_engineerin]

ありがとうございます！
ご回答いただいて、質問に問題があることがわかりました。
修正いたしました。
申し訳ございません。

withの中で絞り込んでもlesson_students自体が絞り込まれないので、joinにする必要があると考えた次第です。