Używanie Exceptionów a typowanie Opcje

[athabus]

Ostatnio ciekawi mnie temat używania Exceptionów - muszę przyznać, że trochę słabo to czuję. Kiedyś Exception stosowałem do rzeczy, które po prostu mogły spowodować niedziałanie aplikacji - np. brak dostępu do pliku czy coś podobnego. Ostatnio jednak widzę, że coraz częściej w nowych bibliotekach Exceptiony stosuje się w inny sposób tj. do wymuszenia typowania metody bez zwracania NULL.

Np taka klasa:

[PHP] pobierz, plaintext 
<?php
 
class MySampleRepository
{
    public function getItemV1(int $id): SomeObject
    {
        //pobieramy item, który może nie istnieć (null)
        $item = $this->fetch($id);
 
        if (!$item) {
            throw new ItemNotFoundException('Item does not exist');
        }
 
        return $item;
    }
 
    public function getItemV2(int $id): ?SomeObject
    {
        //pobieramy item, który może nie istnieć (null)
        $item = $this->fetch($id);
 
        return $item;
    }
}
[PHP] pobierz, plaintext 

Pierwsza metoda (podejście o którym mówię) nigdy nie zwraca nulla, tylko obiekt lub rzuca wyjątek. Druga metoda (tradycyjna) zwraca obiekt lub null.

Które podejście stosujecie? Dlaczego.

Ja po rozmowie z kolegą zacząłem skłaniać się ku pierwszej wersji i brak możliwości zwrócenia oczekiwanego obiektu traktuję jako wyjątek.Podświadomie wydaje mi się to bardziej logiczne i wymusza trochę więcej dyscypliny.

[markonix]

Ja bym jednak wolał aby repozytoria zwracały puste tablice/kolekcje oraz null w przypadku gdy pobieram w zamierzeniu jeden obiekt.

[PHP] pobierz, plaintext 
if (MySampleRepository::getItemV1($id) !== null) {
    //
}
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
if ($object = MySampleRepository::getItemV1($id)) {
    //
}
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
$object = MySampleRepository::getItemV1($id) ? ? Alternative::get($id);
[PHP] pobierz, plaintext 

Generalnie wszystko np. w Laravel typu get(), find(), first(), firstWhere(), config() zwraca null gdy nie znajduje nic i ciężko by mi się było przerzucić na łapanie wyjątków. Najnormalniej w świecie było by więcej kodu i byłby dla mnie mniej czytelny.

edit: Oprócz findFail(), który właśnie zwraca zgodnie z intencją wyjątek ResourceNotFound, idealny do wyświetlania resource/{id} i wyświetlenia po prostu 404ki bez zbędnych warunków.

Temat w każdym razie mnie zaciekawił bo akurat ostatnio często rozmyślam na repository pattern. Znalazłem na Stacku temat: https://stackoverflow.com/questions/175532/...en-it-cant-prod

Ten post edytował markonix 31.12.2018, 17:29:56