Drukowana wersja tematu

Napisany przez: duch4ever 22.08.2018, 11:14:55

Witam, mam aplikację w symfony która łączy się za api. Żeby wyciąć logikę z kontrolera chcę utworzyć serwis który zajmie się wysyłaniem zapytań do api. Np. wystawienie ofert do serwisu aukcyjnego, i tu pytanie jakie argumenty klasa powinna przyjmować?

[PHP] pobierz, plaintext 
 
class OfferService
{
    private $apiClient;
 
    public function __construct($apiClient)
    {
         ///
    }
 
    public function createOffer($data)
   {
        $apiClient->sendReq('/newOffer' .... , $data)
 
         // dalsza logika, jest jej sporo, stworzenie oferty to kilka zależnych requestów
    }
}
 
[PHP] pobierz, plaintext 

I tu pytanie czy to jest poprawne i zgodne ze sztuką? Czy serwis powinien trzymać klienta api? Może macie jakieś dobre przykłady?

Napisany przez: markuz 22.08.2018, 12:08:59

Zgodnie z DI klasa powinna otrzymać w konstruktorze wszystkie zależności (albo w seterach opcjonalne). Więc tak, jeżeli Twój serwis wymaga apiClient to jak najbardziej przekazanie go w ten sposób jest zgodne ze sztuką.

Napisany przez: duch4ever 22.08.2018, 12:20:21

Dzięki, mam dodatkowe pytanie, jak obsługiwać błędy w takim przypadku?

Np. metoda createOffer powinna zwracać true/false, response z api czy możne exception jeśli coś jest nie tak a true jeśli zapytanie się powiodło?

Napisany przez: markonix 22.08.2018, 12:38:28

Wg mnie Exception jest lepszy bo możesz wyrzucić konkretny problem (jeżeli API go zwraca), natomiast false dużo Ci nie powie.