[OOP-php] [ocena] moja pierwsza klasa w OOP, klasa zliczająca ilość userów on-line Opcje

[Athlan]

Klasa zlicza ile jest userów on-line. Opiera się na MySQL, w tabeli są przypisywane kolejno numery IP oraz czas wygaśnięcia sesji (time() + 180). Jak sesja kolesia się skończy, rekord zostaje usunięty i liczna online spada...

http://dyzio.nowiny.pl/bin/sources/online.class.txt

Jest to moja pierwsza klasa w php więc się nie śmaic (IMG:http://forum.php.pl/style_emoticons/default/dry.gif)

Jak macie jakieś uwagi/sugestie (a takie będą bo noobie w OOP - 1 klasa (IMG:http://forum.php.pl/style_emoticons/default/blink.gif) ) to pisać (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

oceniać (IMG:http://forum.php.pl/style_emoticons/default/cool.gif)

Ten post edytował Athlan 19.07.2006, 08:41:46

[Ludvik]

Pierwsze, co mi się rzuciło w oczy, to komentarze. PHPDoc jest najlepszy rozwiązaniem, opanowanie go nie powinno stanowić większego problemu. Poza tym opisujesz typy zwracane przez php nazwami typów SQL. Bardzo chaotycznie...

Poza tym duuuużo zapytań. Zamiast sprawdzać jednym zapytaniem, a potem wywoływać UPDATE, możesz w MySQL posłużyć się REPLACE. Jest to rozszerzenie standardu, ale jeżeli trzymasz się tylko MySQL, to nie zaszkodzi z niego skorzystać.

W funkcji getIp() posłużyłeś się skrótowym zapisem wyrażenia warunkowego, który u Ciebie wcale niczego nie skraca i jest nieczytelny.

Do metod publicznych też warto wyrobić sobie dobry nawyk dopisywania modyfikatora public.

Z błędami radzisz sobie raczej tak, jak robi się to przy programowaniu strukturalnym. Wyjątki są lepszym narzędziem.

Trochę jeszcze do nauki masz, ale nie jest najgorzej.