Как убрать лишний join в query builder?

Question

[snake2]

Есть обычный запрос к бд с помощью query builder laravel
Но также при получении выборки может использоваться условие

$user = USER::query
    ->where(active, '=', 1)
    ->join('roles', 'roles.user_id', '=', 'user_id')
    ->where(function ($query) {
        if($roleId) {
            $query->where('role', '=', $roleId)
        }
    })
    ->get()

Можно ли как-то красиво избавиться от join если $roleId = null
Пока в голову лезет только такое решение

$user = USER::query
    ->where(active, '=', 1)
    ->join('roles', function($join) {
         if($roleId) {
            $join->on('roles.user_id', '=', user.id)
        }
    })
    ->where(function ($query) {
        if($roleId) {
            $query->where('role', '=', $roleId)
        }
    })
    ->get()

Answer 1

[vism]

$userQuery = USER::query()
    ->where(active, '=', 1);
if($roleId){
    $userQuery->join('roles', 'roles.user_id', '=', 'user_id')
    ->where('role', '=', $roleId);
}
$user = $userQuery->get()

Comments

[jazzus]

В принципе, жил автор как-то без Ларавел и еще проживет.

[vism]

jazzus, именно :)))
зато теперь он знаешь как конструировать запрос динамически :)

Да и зачем ему отношения, может он заядлый холостяк.

[snake2]

vism, Смешно, когда на самом деле ваше решение ужасное. А всего надо было использовать when

[vism]

snake2, Я аж засмеялся)))
На самом деле when это ужасное решение, а мое более удобное. Ну хотя да, кто я такой, у меня даже нет примеров кода:)

Ну а в целом тебе бы заняться отношениями, а не этими решениями:)

[snake2]

vism, Это только ваше мнение, что ваше решение удобное, мне же кажется что лучше использовать встроенные средства laravel, чем использовать вот эту самодеятельность.
А по поводу моих отношений вы можете не беспокоиться.

Answer 2

[jazzus]

С помощью отношений