Klasa API do gier Totalizatora Sportowego |
Klasa API do gier Totalizatora Sportowego |
22.04.2016, 20:56:28
Post
#1
|
|
Grupa: Zarejestrowani Postów: 260 Pomógł: 26 Dołączył: 26.01.2012 Skąd: Ci to przyszło do głowy? Ostrzeżenie: (0%) |
Cześć, czy moglibyście ocenić jak działa moje autorskie API pobierające wartości losowań różnych gier totalizatora sportowego?
Aktualnie działa pobieranie Lotto, Lotto Plus, oraz EkstraPensja https://github.com/G910/LottoAPI Każda ocena i podpowiedź jest cenna! -------------------- |
|
|
22.04.2016, 21:30:07
Post
#2
|
|
Grupa: Zarejestrowani Postów: 8 068 Pomógł: 1414 Dołączył: 26.10.2005 Ostrzeżenie: (0%) |
Strasznie dużo kodu się duplikuje. Każda metoda praktycznie w 98% ma ten sam kod.
Masz tu jeden z przykładów jak można by napisać:
- brak autoloadera (composer nie gryzie a mega ułatwia) -
Skoro robisz return true to else już nie potrzebne Poza tym odwracając warunek kod będzie czytelniejszy. :
- brak namespace - $_clientSecret - tak się już nie pisze (chodzi o _ dla oznaczenia widoczności) - Skoro zwracasz w odpowiedzi array to dlaczego przy błędzie zwracasz fals? Powinna być pusta tablica. - Wyjątki a raczej ich brak. Zamiast zgadywać czy zadziałało czy nie wystarczy rzucić w pysk Exception - Debug - echo ? seriously ? To tak na dzień dobry |
|
|
Wersja Lo-Fi | Aktualny czas: 23.05.2024 - 17:53 |