 [PHP/OOP] Moje pierwsze klasy, Co robię źle? :) |
| [PHP/OOP] Moje pierwsze klasy, Co robię źle? :) |
 9.03.2008, 20:02:43
Post
#1
|
|
 Grupa: Zarejestrowani Postów: 654 Pomógł: 17 Dołączył: 19.03.2006 Skąd: z kosmosu ;) Ostrzeżenie: (0%) 
 |
Witam, poniżej zamieszczam moje pierwsze klasy. Chcę zabrać się za artykuł o frameworkach z wortalu PHP.pl (właśnie go drukuje) i chce wiedzieć co robię źle w programowaniu obiektowym
 SimpleRSSElement: http://manual.babciastefa.boo.pl/download/...erss_class.phps http://manual.babciastefa.boo.pl/module,vi...impleRSSElement - Manual SimpleRSSBuilder: http://manual.babciastefa.boo.pl/download/...lder_class.phps http://manual.babciastefa.boo.pl/module,vi...impleRSSBuilder - Manual Kodu nie jestem w stanie zamieścić na forum (za długi - ponad 20 kb) Wiem że jest brak poprawnego kodowania w klasach (później to poprawie  )Piszcie wszystkie uwagi co do kodu... Dziękuję, Babcia@Stefa Ten post edytował Babcia@Stefa 9.03.2008, 20:06:58 -------------------- Środowisko testowe (desktop) - Gedit, lighttpd, sftp, rsync, xfce4-terminal, chromium, firefox4 | System: Gentoo ~x86
O'Neill - serwer WWW @ lighttpd, links, nano, rsyncd, sftpd | System: Debian  |
 |
 
|
 |
|
11.03.2008, 22:25:48
Post
#2
|
|
 Grupa: Zarejestrowani Postów: 102 Pomógł: 12 Dołączył: 27.01.2007 Skąd: north Poziom: 158 Tytuł: Miszcz Ostrzeżenie: (0%)
|
SimpleRSSElement - nawet dobra klasa, trzyma poziom,
Co do rssBuildera... nie lepiej użyć simplexml? Jakiś poziom abstrakcji by był. Tak to wygląda średnio. Jest dobrze. Nie widzę żadnych rażących błędów. -------------------- Blog | plugin system by carbolymer
Residence: #php.pl @ IRCNet "Pralki powstały po to, aby kobiety też mogły programować" |
|
|
|
|
12.03.2008, 07:04:31
Post
#3
|
|
|
Grupa: Zarejestrowani Postów: 654 Pomógł: 17 Dołączył: 19.03.2006 Skąd: z kosmosu ;) Ostrzeżenie: (0%)
|
To SimpleXML teæ tworzy dokumenty XML?
Ja nie wiedziaøem o tym... Dziékujé, Babcia@Stefa -------------------- Środowisko testowe (desktop) - Gedit, lighttpd, sftp, rsync, xfce4-terminal, chromium, firefox4 | System: Gentoo ~x86
O'Neill - serwer WWW @ lighttpd, links, nano, rsyncd, sftpd | System: Debian |
|
|
|
|
12.03.2008, 10:37:28
Post
#4
|
|
|
Grupa: Zarejestrowani Postów: 206 Pomógł: 21 Dołączył: 1.09.2006 Skąd: Edinburgh Ostrzeżenie: (0%)
|
Pierwsza klasa wyglada ok. Drobna uwaga kosmetyczna - kilka (chyba 5) razy powtarza Ci sie
Mozna by jakos wyeliminowac te powtorzenia. Jesli chodzi o Builder, to juz zostalo powiedziane o SimpleXML. Acha - no i bledy jezykowe w wyjatkach ;] Pzdr |
|
|
|
|
12.03.2008, 14:05:56
Post
#5
|
|
|
Grupa: Zarejestrowani Postów: 654 Pomógł: 17 Dołączył: 19.03.2006 Skąd: z kosmosu ;) Ostrzeżenie: (0%)
|
Rozumiem, a więc wrzuce zmienną chronioną:
protected $XML; Do konstruktora: $this->XML = @simplexml_load_string($content); I wszędzie wrzuce: $xml = $this->XML // żeby nie przebudowywać całego kodu I będzie dobrze ;] Pozdrawiam, Babcia@Stefa -------------------- Środowisko testowe (desktop) - Gedit, lighttpd, sftp, rsync, xfce4-terminal, chromium, firefox4 | System: Gentoo ~x86
O'Neill - serwer WWW @ lighttpd, links, nano, rsyncd, sftpd | System: Debian |
|
|
|
 |
|
Wersja Lo-Fi | Aktualny czas: 14.08.2025 - 11:36 |
