Forum PHP.pl

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ć:

[PHP] pobierz, plaintext 
    private function isDateValid($date)
    {
        $isCorrectData = preg_match("|[0-9]{4}-[0-9]{2}-[0-9]{2}|", $date);
        if ($date !== null && !$isCorrectData) {
            $this->_lastErrorCode = "clientSide1";
            $this->_lastErrorDesc = "Data wprowadzona w niewłaściwym formacie. Prawidłowy: YYYY-MM-DD";
 
            return false;
        }
        return $date;
    }
 
    public function getLotto($date = null)
    {
        if(($date = $this->isDateValid($date)) === false) {
            return false;
        }
 
        $data = [];
        $data['type'] = 'getLotto';
        if ($date !== null) {
            $data['type'] .= 'ByDate';
            $data['date'] = $date;
        }
 
        return $this->curl($data);
    }
[PHP] pobierz, plaintext 

- brak autoloadera (composer nie gryzie a mega ułatwia)
-

[PHP] pobierz, plaintext 
        if(isset($response['status']) && $response['status'] == 1){
            $this->_connected = true;
            $this->_token = $response['token'];
            return true;
        }else{
            $this->_lastErrorCode = $response['error']['code'];
            $this->_lastErrorDesc = $response['error']['description'];
            return false;
        }
[PHP] pobierz, plaintext 

Skoro robisz return true to else już nie potrzebne Poza tym odwracając warunek kod będzie czytelniejszy. :

[PHP] pobierz, plaintext 
if(empty($response['status'])) {
            $this->_lastErrorCode = $response['error']['code'];
            $this->_lastErrorDesc = $response['error']['description'];
            return false;
        }
 
        $this->_connected = true;
        $this->_token = $response['token'];
        return true;
[PHP] pobierz, plaintext 

- 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