![]() |
![]() |
![]()
Post
#1
|
|
Grupa: Zarejestrowani Postów: 1 385 Pomógł: 48 Dołączył: 23.05.2007 Ostrzeżenie: (0%) ![]() ![]() |
Postanowiłem się w końcu zacząć uczyć programowania obiektowego. (IMG:http://forum.php.pl/style_emoticons/default/guitar.gif) Na początek napisałem skrypt który graweruje formularze. Czy kod jest optymalny? Co w nim jest niepotrzebne a co zmienić?
|
|
|
![]() |
![]()
Post
#2
|
|
Grupa: Developerzy Postów: 823 Pomógł: 12 Dołączył: 18.12.2005 Ostrzeżenie: (0%) ![]() ![]() |
Po 1.
http://forum.php.pl/index.php?s=&showt...st&p=466253 Kolejne nie-OOP, kopiujesz metody jak popadnie. Nie lepiej miec metody, które wyzwalają inną metodę (wpisują typ i zakres danych) np protected $this->_pole_input( paramy ) i zastępują tylko type. Po 2. Po grzyb pisać generator formularzy (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif) ? zawsze znajdzie się jakiś parametr, który odrzuci klasę, np dodatkowe atrybuty. Warto o tym pomyśleć i oprócz name itd dorobić metody addAttribute($sName, $sValue). Po 3. Fajnie by było rozszerzyć takie klasę o validację, pobieranie wartości z POST'u jeżeli są przesłane, a jeżeli nie to z bazy danych (jeżeli to edycja czegoś, po postowaniu dane z bazy zamieniają się na dane z POST'u chwilowo w wyświetleniu formularza). Ogólnie to nie OOP (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif) |
|
|
![]() ![]() |
![]() |
Aktualny czas: 24.08.2025 - 08:36 |