[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

[Athlan]

No i wlasnie dlatego powinna byc statyczna.

dobra poprawiłem...

Albo sciemniasz, albo podales zly link, bo ja nie widze abys cos dostoswal sie do uwag bigZbiga.

nie widzisz?
- poprawiłem deklarację zmiennych na początku (dodalem private)
- z opisów usunąłem nazwy funkcji (fakt... po co to)
- static function encodeIP()
- class ONLINE - nazwa klasy z wielkich liter
- no i teraz dałe static na getIP()

@nospor - możliwe że patrzyles na klasę wcześniej i nie odświeżyłeś strony jak ja zobaczyłeś po raz drugi (cache)... wyczyść sobie cache i daj f5 albo wejdź z np badzIEwia (lol) a potem na mnie krzycz bo mi się smutno zrobiło (IMG:http://forum.php.pl/style_emoticons/default/sad.gif)

pozdro (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

Ten post edytował Athlan 19.07.2006, 10:20:08