 [PHP] Logowanie, zwrot/wypozyczenie ksiazek OOP - mocno początkujący |
| [PHP] Logowanie, zwrot/wypozyczenie ksiazek OOP - mocno początkujący |
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: Zarejestrowani Postów: 14 Pomógł: 0 Dołączył: 14.10.2019 Skąd: Białystok Ostrzeżenie: (0%)
|
Trochę mnie nie było, jednak musiałem przyswoić dość dużą dawkę wiedzy.
Napisałem praktycznie wszystko od nowa. 1) Oparte jest teraz na MVC - tak, wiem prymitywne MVC - przerabiałem też MVC oparte na wyrażeniach regularnych i preg_match, co w przyszłości tutaj wdrożę jednak w celach naukowych takie prymitywne lepiej mi wszystko ilustrowało. 2) Dodane template engine Twig 3) Usunięte zapytania SQL bazujące na kolejności kolumn w bazie 4) Dodałem obsługę wyjątków (?) czy jest to zrobione poprawnie? Wcześniej była mowa, że wyjątki są duszone w środku (IMG:style_emoticons/default/smile.gif) 5) Było mówione o dziedziczeniu klas - tutaj wszystkie Controllers dziedziczą po głównym kontrolerze Core\Controller - czy jest to akceptowalne? 6)Wiem, że muszę po upraszczać trochę kod bo miejscami if na ifie. 7)Mam problem z Type Hinting w Model User metoda authenticate - zwracam obiekt lub bool, a z tego co czytałem dopiero od php 8 będzie możliwość typowania dwóch różnych form. Wszystko wgrałem pod nowy link, ponieważ mało rzeczy jest wspólnych z poprzednim. Jak zwykle dzięki. Odwiedź moją stronę |
|
|
|
mrpickles [PHP] Logowanie, zwrot/wypozyczenie ksiazek OOP - mocno początkujący 14.10.2019, 18:03:39 
viking Najpierw zastosuj autoloader, potem wyrzuć wszystk... 14.10.2019, 18:38:18 ohm klasa Log jest dość mocno myląca, brak namespace... 14.10.2019, 20:13:08 mrpickles 1. Dodałem autoloadera
2. Nie znam smarty ani inne... 15.10.2019, 20:06:49 athabus Niestety trochę błądzisz, ale moim zdaniem to bard... 15.10.2019, 21:03:22 mrpickles Tak wiem, błądzę muszę poukładać wiedze.
1. zrobi... 17.10.2019, 21:00:47 viking Przykładowo ta nieszczęsna klasa database. W przyp... 18.10.2019, 07:24:15 mrpickles Usunąłem vendor/autoload w klasach - pozostałość b... 18.10.2019, 09:39:59 athabus Ogólnie po zmianach wygląda już znacznie lepiej. P... 18.10.2019, 10:02:48 viking https://martinfowler.com/eaaCatalog/
Dlaczego głów... 18.10.2019, 10:03:49 nospor Njapierw klasa dziedziczy po Database
class Regist... 18.10.2019, 10:21:30 athabus Jeszcze taka ogólna uwaga odnośnie dziedziczenia. ... 18.10.2019, 10:36:44 mrpickles Ustosunkuje się tylko niektórych elementów, bo dos... 18.10.2019, 11:49:49 
nospor Cytat(mrpickles @ 18.10.2019, 11:49:4... 18.10.2019, 12:15:38 athabus To o czym piszesz to jest częsty błąd w tutorialac... 18.10.2019, 12:14:40 mrpickles Zakręciłem się i pracowałem na klonie, a zrobiłem ... 18.10.2019, 13:12:23 nospor Generalnie nie chodzi o odzielenie php od html a o... 18.10.2019, 13:33:05 mrpickles Ok czyli MVC się kłania. Dzięki za wyjaśnienia, ... 18.10.2019, 13:37:07 phpion Wiele już zostało powiedziane ale ja dodam od sieb... 3.11.2019, 18:59:27 nospor Jest zdecydowanie lepiej.
echo View::renderTempl... 20.11.2019, 10:53:34 athabus No prosze a już myślałem, że się poddałeś ;-)
Ko... 21.11.2019, 08:36:44 mrpickles Dzięki za uwagi:)
1) Powalczę z gitem - zamiast n... 21.11.2019, 16:45:44 athabus Tak w telegraficzny skrócie, to to co zaproponował... 21.11.2019, 17:19:04 viking Możesz też zmienić podejście i poczytać o https://... 21.11.2019, 18:53:53 mrpickles Utknąłem, nie mam pomysłu, przeglądałem dokumentac... 5.12.2019, 22:28:48 athabus Najprościej chyba możesz to zrobić przechwytując w... 6.12.2019, 21:45:45 mrpickles Zrobiłem tak, że kontroler zwraca Response, a w Ru... 8.12.2019, 20:51:37 nospor Widze nadal kodu nie testujesz
FrontController.ph... 7.01.2020, 16:34:45 mrpickles Przez ostatni czas przerabiałem Laravela, teraz wr... 29.02.2020, 14:31:25 nospor Zrozum, rzeczy konfiguracyjne nie moga lezej w GIT... 29.02.2020, 17:49:42 mrpickles Dzięki teraz wiem o co chodzi. Tak używam laravela... 1.03.2020, 00:11:46  |
|
Aktualny czas: 14.10.2025 - 19:41 |
