[PHP] ocena skryptu na GitHub

pytanie zadane 23 grudnia 2017 w PHP przez `Krzychuu Stary wyjadacz (13,940 p.)
edycja 23 grudnia 2017 przez `Krzychuu

Witam

Czy mógłby ktoś zajrzeć do kodu i powiedzieć czy idę w dobrym kierunku, chodzi mi np czy robię we właściwym momencie wywołanie obiektu itp, najbardziej zależy mi na klasach.

https://github.com/KrzychuuWEB/Log-in

1 odpowiedź

odpowiedź 23 grudnia 2017 przez Comandeer Guru (601,530 p.)

Mam wrażenie, że miałeś kod proceduralny i zamiast na funkcje, podzieliłeś go na klasy. To totalnie nie wygląda jak kod w OOP.

Np. CheckValue nie ma sensu. Posiada tylko jedną metodę, która służy wyłącznie do walidacji loginu. Niemniej ta walidacja jest zrobiona w sposób błędny. Walidator nie ma prawa wiedzieć, skąd dane, które otrzymuje, pochodzą. W tym wypadku walidator sam tworzy połączenie z bazą i wyciąga stamtąd login. Ba, klasa odpowiedzialna za łączenie się z bazą od razu przeprowadza walidację, co tym bardziej pozbawia sensu walidator w obecnym kształcie.

Dodatkowo wszystko jest tight coupled – klasy mają sztywno wpisane zależności. Powinieneś stosować Dependency Injection, zatem nie:

public function __construct()
    {
        $this->LoginDB = new LoginDB();
    }

public function __construct(LoginDB $loginDB)
    {
        $this->loginDB = $loginDB;
    }

1	komentarz 23 grudnia 2017 przez `Krzychuu Stary wyjadacz (13,940 p.) a mógłbyś podesłać jakiś projekt w który jest dobrze napisany w oop?

1	komentarz 23 grudnia 2017 przez HaKIM Szeryf (87,590 p.) a mógłbyś podesłać jakiś projekt w który jest dobrze napisany w oop? Choćby to: https://github.com/norzechowicz/mydrinks

1	komentarz 23 grudnia 2017 przez Comandeer Guru (601,530 p.) No właśnie nie chciałem tego posyłać, a nic innego pod ręką w PHP nie mam… Czemu nie chciałem? Ponieważ to jest totalnie przesadzony przykład. Techniki tam zastosowane tak naprawdę mają sens dopiero w dużej skali.

1	komentarz 23 grudnia 2017 przez HaKIM Szeryf (87,590 p.) Wiem, wiem. Sam się zastanawiałem czy nie daje mu fizyki kwantowej w pierwszej klasie... No ale... :p Jak znajdę coś lepszego to podeślę.