PDO prosty czat - proszę o ocenę |
PDO prosty czat - proszę o ocenę |
23.04.2013, 17:12:53
Post
#1
|
|
Grupa: Zarejestrowani Postów: 352 Pomógł: 59 Dołączył: 16.01.2013 Ostrzeżenie: (0%) |
Witam!
Proszę szanowne grono o ocenę mojego czatu napisanego w OOP, dopiero się uczę, z chęcią przyjmę wszystkie uwagi (szczególnie krytyczne) dot. kodu a także co mogłoby być lepiej config.php
database.php
czat.php
pozdrawiam! |
|
|
23.04.2013, 17:21:17
Post
#2
|
|
Grupa: Zarejestrowani Postów: 103 Pomógł: 17 Dołączył: 15.12.2012 Ostrzeżenie: (0%) |
jak ma być oop to rozdziel walidacje, na osobną klasę
|
|
|
23.04.2013, 17:51:13
Post
#3
|
|
Grupa: Zarejestrowani Postów: 1 333 Pomógł: 137 Dołączył: 25.03.2008 Skąd: jesteś?? Ostrzeżenie: (0%) |
Chat moim zdaniem nie powinien dziedziczyć z bazy a być oddzielną częścią i posiadać obiekt klasy jako zmienną.
-------------------- Mój blog - o wszystkim i niczym ale zazwyczaj związane z informatyką! ;-)
Githube Usługi spawalnicze i monterskie | Park linowy Lublin i Okunince |
|
|
23.04.2013, 18:06:34
Post
#4
|
|
Grupa: Zarejestrowani Postów: 352 Pomógł: 59 Dołączył: 16.01.2013 Ostrzeżenie: (0%) |
w ktore miejsce najlepiej wrzucic obiekt bazy? jak sie wtedy odwolac do jego metod?
|
|
|
24.04.2013, 06:57:25
Post
#5
|
|
Grupa: Zarejestrowani Postów: 142 Pomógł: 24 Dołączył: 30.03.2009 Skąd: Rokitno Szlacheckie Ostrzeżenie: (0%) |
Bazę (PDO) przekazać jako parametr do konstruktora, a to co masz w klasie database przekopiuj do klasy chat
i w tedy
Ten post edytował hind 24.04.2013, 06:57:50 |
|
|
24.04.2013, 08:18:21
Post
#6
|
|
Grupa: Zarejestrowani Postów: 1 333 Pomógł: 137 Dołączył: 25.03.2008 Skąd: jesteś?? Ostrzeżenie: (0%) |
@hind dobrze mówi, ale by jeszcze pójść o krok dalej powiem Ci jak działją fw.
W fw masz coś takiego jak klasa DI to taki rejestr wszystkich elementów składowych systemu, najłatwiejsza jego odmiana ma aż 2 metody set($name, $obiect) i get($name) i służy do przechowywania w sobie wszystkich obiektów typu baza danych, instancaj widoku, routingu, requestu, configu itd. i to ona jest przekazywana do kontrolerów (dla uproszczenia można uznać że u Ciebei to klasa chat) podczas konstrukcji klasy, a sama klasa chat ma zmienną $di i metody setDi() i getDi() dzięki który możesz odwołać się do obiektów zapisanych w niej. Taki zabieg w przyszłośic ułatwi Ci kod, bo obiektów typu chat, form, controller jest wiele sposobem @hind musiał byś za każdym razem do nich wprowadzać wszystkie serwisty (db, request, routing itd), a tutaj wszystko wprowadzasz do Di, a samo Di przekazujesz zawsze dalej i ew w przyszłości dodajesz tylko do niego kolejne serwisy. ;-) -------------------- Mój blog - o wszystkim i niczym ale zazwyczaj związane z informatyką! ;-)
Githube Usługi spawalnicze i monterskie | Park linowy Lublin i Okunince |
|
|
24.04.2013, 08:40:36
Post
#7
|
|
Grupa: Zarejestrowani Postów: 352 Pomógł: 59 Dołączył: 16.01.2013 Ostrzeżenie: (0%) |
Coś tam działam sobie w wolnych chwilach w codeigniterze i przyznam, że sporo załapałem z obiektówki dzięki temu, ale nie bardzo doszukałem się informacji odnośnie architektury całego przedsięwzięcia.
Dzięki wielkie szanownym Panom za informację, będę walczył dalej. A jako, ze już przeszliśmy na FW, to mam jeszcze małe pytanko, mam kilka podstron, których treść trzymam w bazie (strona oparta o codeigniter). Stworzyłem sobie kontroler, który zajmuje się wyświetlaniem kolejnych podstron, przyjmuje 1 parametr z ID podstrony - wtedy url wyglada tak: xxx.pl/home/page/1 (2,3, itd) Co myślicie o takim rozwiązaniu? Może jest jakieś lepsze? |
|
|
24.04.2013, 08:47:31
Post
#8
|
|
Grupa: Moderatorzy Postów: 36 440 Pomógł: 6290 Dołączył: 27.12.2004 |
Używasz PDO, używasz bindowania, a mimo to escapujesz dane i to na dodatek przy pomocy addslashes.... przecież to nie ma żadnego sensu...
-------------------- "Myśl, myśl, myśl..." - Kubuś Puchatek || "Manual, manual, manual..." - Kubuś Programista "Szukaj, szukaj, szukaj..." - Kubuś Odkrywca || "Debuguj, debuguj, debuguj..." - Kubuś Developer |
|
|
24.04.2013, 09:03:54
Post
#9
|
|
Grupa: Zarejestrowani Postów: 352 Pomógł: 59 Dołączył: 16.01.2013 Ostrzeżenie: (0%) |
Doczytałem trochę, faktycznie, wstyd
Ale chyba trzeba się troche powstydzić za swoje dokonania, żeby się nauczyć |
|
|
Wersja Lo-Fi | Aktualny czas: 29.03.2024 - 03:23 |