![]() |
![]() |
![]()
Post
#1
|
|
Grupa: Zarejestrowani Postów: 14 Pomógł: 0 Dołączył: 14.10.2019 Skąd: Białystok Ostrzeżenie: (0%) ![]() ![]() |
Cześć,
Zaczynam naukę OOP i w napisałem skrypt który umożliwia rejestracje/logowanie/zwrot/wypożyczenie książek. Mam świadomość istnienia SOLID oraz PSR - czytałem o PSR 1 / 2 i staram się przestrzegać. Nie znam MVC ani testów np. PHPUnit - to będą kolejne kroki w nauce. Czy taki skrypt można określić obiektowym, czy jest to jakaś hybryda? Zanim pójdę dalej jw chciałbym nauczyć się dobrych nawyków i w prawidłowy sposób posługiwać się OOP. Skrypt nie wszedł na forum, umieściłem na githubie Link do GitHub Moje dodatkowe pytania: 1. Tworzenie obiektu Database w konstruktorze innych klas, czy jest to prawidłowe? 2. Metoda checkCredentials klasy log - czy nie narusza zasady pojedynczej odpowiedzialności? Czy nie powinna np. wyszukiwać użytkownika,a następnie inna metoda powinna weryfikować dane? Dzięki za wszystkie uwagi i poświęcony czas (IMG:style_emoticons/default/smile.gif) |
|
|
![]() |
![]()
Post
#2
|
|
Grupa: Moderatorzy Postów: 36 559 Pomógł: 6315 Dołączył: 27.12.2004 ![]() |
Widze nadal kodu nie testujesz (IMG:style_emoticons/default/wink.gif)
FrontController.php if ($response->getStatusCode() == 301) { header('location:$file.php'); } Po pierwsze zmienne w pojedynczych ciapkach nie sa parsowane, po drugie zmienna $file w ogole tam nie istnieje Naucz sie uzywac === zamiast ==. Zaoszczedzi ci to kiedys sporo czasu na szukaniiu glupich bledow return Yaml::parseFile('C:\xampp\htdocs\LibraryResponse\Core\db.yaml'); Gdy prosilismy bys wywalil konfig z gita, naprawde nie chodzilo nam bys go zastapil sciezka do pliku, do ktorego nie mamy dostepu (IMG:style_emoticons/default/wink.gif) public static function getDB(): object Czemu to zwraca object? Przeciez wiesz, ze to zwraca PDO a nie byle jaki objekt Taki kod
Nie ma wiekszego sensu. Po mergujesz zawsze puste $params z jakas tablica? Przeciez wynikiem bedzie zawsze ta inna tablica. Co tu chcesz tak naprawde zrobic to zmergowac get i post czyli Skracaj zbedne bloki
Piszesz poprostu
Cytat Napotkałem się jeszcze na sformułowanie, że Response powinien zostać zwrócony z View za pomocą ResponseFactory, a nie z Controllera. To będzie następny etap ? No raczej nie |
|
|
![]() ![]() |
![]() |
Aktualny czas: 15.10.2025 - 18:08 |