Laravel Türkiye Discord Kanalı Forumda kod paylaşılırken dikkat edilmesi gerekenler!Birlikte proje geliştirmek ister misiniz?

Servis Katmanlı Yapı Hakkında Soru

myusuf

Merhaba, bir yapı geliştiyorum. 2. yapıda Laravel'in route model binding özelliğini kullanıyorum ama bu durumda servis katmanı biraz gereksiz gibi duruyor. 1. yapıda servis daha bağımsız kalıyor. Sizce hangisi kullanılmalı?

Yapı
Controller:

class ConsulateController extends Controller
{
    public function __construct(private readonly ConsulateService $service) {}

    public function index(?int $countryId = null)
    {
        if (is_null($countryId)) {
            return $this->service->getAll();
        }

        return $this->service->getByCountryId($countryId);
    }

    public function show(int $consulateId)
    {
        return $this->service->getById($consulateId);
    }
}

Servis:

class ConsulateService
{
    public function getAll(): Collection
    {
        return Consulate::with('country')->get();
    }

    public function getByCountryId(int $countryId): Collection
    {
        return Consulate::where('country_id', $countryId)->get();
    }

    public function getById(int $consulateId): ?Consulate
    {
        return Consulate::find($consulateId);
    }
}

Yapı:
Controller:

class ConsulateController extends Controller
{
    public function __construct(private readonly ConsulateService $service) {}

    public function index(?Country $country = null)
    {
        if (is_null($country)) {
            return $this->service->getAll();
        }

        return $this->service->getByCountry($country);
    }

    public function show(Consulate $consulate)
    {
        return $this->service->get($consulate);
    }
}

Servis:

class ConsulateService
{
    public function getAll(): Collection
    {
        return Consulate::with('country')->get();
    }

    public function getByCountry(Country $country): Collection
    {
        return Consulate::where('country_id', $country->id)->get();
    }

    public function get(Consulate $consulate): ?Consulate
    {
        return $consulate->load('country');
    }
}

altayevrim

Düşünceme karşı çıkacak çok fazla kişi olacaktır da, şu sıralar controller katmanında olabildiğince az işlem olması gerektiği konusunda direten insanlar var. Haksız da sayılmazlar, adı üstünde controller, kontrol ediyor; işlemden ziyade yönetme yapması daha mantıklı.

Fakat şahsi düşüncem şu, basit işlemler için ya da ileride geliştirmeyeceğimizi düşündüğümüz şeyler için bu kadar da çok kasmaya gerek yok. Ha tabii arada görüyorum blade içerisinde yarınlar yokmuş gibi php yazanlar var, ona net karşıyım.