но вобще лучше сделать в запросе orderBy()

Не так не подойдет.

Я их сделал. Просто не хочу два шаблона цеплять. Хочется все как-то в одном файле сделать

а в чому різниця між $sortProductsDesc і $sortProductsAsc?

я так розумію виключно в сортуванні і одночасно показується лише один з них

в варианте сортировки

сортируй в контроллере

Вот сервис
  public function getAllProduct($name = null, $type = null)
   {
       $query = $this->product;

       if(!is_null($name) && !is_null($type)){
           $query = $query->orderBy($name, $type);
       }

       return $query->get();
   }

Вот контроллер

public function sort(Request $request)
   {
       if($request->desc){
           $sortProductsDesc = $this->serviceProduct->getAllProduct('price','DESC');
           return view('catalog.index', compact('sortProductsDesc'));
       }

       if($request->asc){
           $sortProductsAsc = $this->serviceProduct->getAllProduct('price','ASC');
           return view('catalog.index', compact('sortProductsAsc'));
       }

       return $this->serviceProduct->getAllProduct();
   }

а чому тоді не залишити одну змінну і в залежності від того чи є $request->desc зробити виборку одну з правильним сортуванням

А теперь хочу все это реализовать в одном файле. Т.е чтобы не плодить сейчас три разных подкладки

можу дати рекомендація як з таким кодом зробити але це поганий код

зачем это вообще нужно?)

это сервис класс

он не делает ничего полезного

Почему? Вроде сортирует в зависимости от условия

Как бы ты сделал?

смотри какой красивый ход)))
@foreach (Request::is('desc')?$sortProductsDesc:Request::is('asc')?$sortProductsAsc:$categoryProducts as $product)
@endforeach

Спасибо. Хороший ход)