Skocz do zawartości
Pecetowicz Forum komputerowe

Biblioteka w PHP (SDK) dla API Fakturomania


Rekomendowane odpowiedzi

  • Ekspert

Nie znam fakturomanii, ale kod to takie 2/10. Gdzie mogę demo przetestować?

Odnośnik do odpowiedzi
Udostępnij na innych stronach
8 minut temu, -n3veR napisał:

Nie znam fakturomanii, ale kod to takie 2/10. Gdzie mogę demo przetestować?

Dema nie posiadam, dlaczego kod 2/10? *Chcę wiedzieć co poprawić*.

Dokumentacje API znajdziesz pod - https://api.fakturomania.pl

Odnośnik do odpowiedzi
Udostępnij na innych stronach
  • Ekspert
Napisano (edytowane)

Brak SOLID, KISS i DRY.
Brak wzorców.
Brak formatowania względem PSR.
Brak enkapsulacji.
Raz obiekty budujesz poprzez kompozycję, a raz przez dopisywanie wartości do publicznych properties (sic!).
Client HTTP powinien być Twoją abstrakcją - wydzielony jako osobny Service.
Custom Exceptions - fajnie, że je tworzysz, ale dlaczego są puste w środku?
Commity niewiele mówią co zostało konkretnie zmienione.

Ogólnie, to mogę jeszcze tak długo 😄 

Edytowane przez -n3veR
  • Lubię to! 1
Odnośnik do odpowiedzi
Udostępnij na innych stronach
  • Ekspert

Ten feedback to powinien być od zespołu w którym pracujesz 😉 po to Cię tam wrzucili jako juniora. No ale cóż... 😄 

  • Lubię to! 1
Odnośnik do odpowiedzi
Udostępnij na innych stronach
  • 2 tygodnie później...

Aktualizacja 0.3.0 -

  • Zastosowanie KISS, DRY
  • Zastosowanie niestandardowej klasy HTTPClient jako warstwy abstrakcyjnej w celu odizolowania Guzzle od logiki aplikacji
  • Dodanie nowego formatowania błędów które umożliwi łatwiejszą lokalizacje błędu
  • Naprawa niezauważonych błędów w 0.2.x

https://github.com/Hyd3r1/Fakturomania

https://packagist.org/packages/khaller/fakturomania-sdk

Edytowane przez Hyd3r1
  • Super 1
Odnośnik do odpowiedzi
Udostępnij na innych stronach

Problem wciąż nierozwiązany? Dodaj swoją odpowiedź

Jeśli chcesz dodać odpowiedź, zaloguj się lub zarejestruj nowe konto. Jedynie zarejestrowani użytkownicy mogą komentować zawartość tej strony.

Zarejestruj nowe konto

Załóż nowe konto. To bardzo proste!

Zarejestruj się

Zaloguj się

Posiadasz już konto? Zaloguj się poniżej.

Zaloguj się
×
×
  • Dodaj nową pozycję...