Как можно сократить код?

Question

[Александр Корегин]

if ($request->building_id == null) {
            $posts = Post::with('buildings')
                ->published()
                ->latest('published_at')
                ->whereCategoryId($request->posts_type)
                ->offset(($request->page - 1) * 6)
                ->take(6)
                ->get(); 
        } else {
            $posts = Post::published()
                ->latest('published_at')
                ->whereCategoryId($request->posts_type)
                ->whereHas('buildings', function ($query) use ($request) {
                    $query->whereId($request->building_id);
                })
                ->offset(($request->page - 1) * 6)
                ->take(6)
                ->get(); 
        };

Можно ли как-нибудь сократить данный код

Comments

[Александр Аксентьев]

А почему вы думаете что если у вас будет whereHas('buildings'), то загружать with('buildings') уже не надо?

[Александр Аксентьев]

->offset(($request->page - 1) * 6)
->take(6)
->get()

https://laravel.com/docs/7.x/pagination

[Дмитрий]

Ну 5 строк повторяется, добавить условие на две не хватающих строчки

Answer 1

[Anton October]

Вот так, как вариант, можно сократить.

$query = Post::whereCategoryId($request->posts_type);

if (!empty($request->building_id)) {
    $query = $query->with('buildings');
} else {
    $query = $query->whereHas('buildings', function ($query) use ($request) {
            $query->whereId($request->building_id);
        });
}

$posts = $query->published()
    ->latest('published_at')
    ->offset(($request->page - 1) * 6)
    ->take(6)
    ->get();

Answer 2

[jazzus]

Для условий в запросах есть when, разбить на страницы можно с помощью paginate, with как писали в комментариях нужен будет везде иначе что-то с логикой.

$buildingId = $request->input('building_id');

Post::published()
    ->whereCategoryId($request->input('posts_type'))
    ->when($buildingId, function ($query, $buildingId) {

      return $query->whereHas('buildings', function ($query) use($buildingId) {
                  $query->whereId($buildingId);
              });
    })
    ->with('buildings')
    ->latest('published_at')
    ->paginate(6);

Comments

[Александр Корегин]

Благодарю за помощь, можете объяснить как происходит запрос when происходит когда $buildingId не пустое ?

[jazzus]

Александр Корегин, будет брать все посты, которые имеют buildings с id == $buildingId

Answer 3

[Александр Аксентьев]

https://laravel.com/docs/7.x/queries#conditional-c...

А еще необязательно писать все в одну цепочку.

$posts = Post::with('buildings');
$posts->where('1', '=', '2');
$posts->where('3', '=', '4');
$posts->orderBy('id', 'desc');