Witaj Gościu! ( Zaloguj | Rejestruj )

Forum PHP.pl

> Wywołanie obiektu w obiekcie.. z zewnątrz - problem
japolak
post
Post #1





Grupa: Zarejestrowani
Postów: 106
Pomógł: 0
Dołączył: 11.03.2007
Skąd: Łódzkie

Ostrzeżenie: (0%)
-----

Witam.. mam taki kod:

fragment pliku strona.php:
[PHP] pobierz, plaintext 
  1.  
  2.  
  3. class strona {
  4. public $tytul;
  5. public $zawartosc;
  6.  
  7. public function __set($nazwa,$wartosc)
  8. {
  9. 	$this->$nazwa=$wartosc;
  10. }
  11.  
  12. public function wzor_obranowania(){?>
  13. 		 echo$this->tytul;
  14. 	echo $this->zawartosc;
  15.  
  16. }
  17.  
  18. public function wyswietl_dodaj_karte()
  19. {
  20.  
  21. .... costam.... 
  22. }
  23.  
  24. }
  25.  
[PHP] pobierz, plaintext



nastepny plik wywolujacy obiekty z klasy (fragment):

[PHP] pobierz, plaintext 
  1.  
  2. $wyswietl= new strona();
  3.  
  4. $wyswietl->wyswietl();
  5. echo"<br>";
  6. $wyswietl->tytul="Rejestracja nowego klienta";
  7. $wyswietl->zawartosc=wyswietl_dodaj_karte();
  8. $wyswietl->wzor_obranowania();
  9.  
  10.  
  11.  
  12. $wyswietl->wyswietl_stopke();
  13.  
  14.  
[PHP] pobierz, plaintext



i wyskakuje mi error:
Fatal error: Call to undefined function wyswietl_dodaj_karte() in C:\xampp\htdocs\praca_inz\test.php on line 7

co jest nie tak?? jak zrobic aby funkcja wzor_obranowania byla elastyczna i mogla w sobie wyswietlac tez inne fukcje..

Ten post edytował japolak 5.11.2009, 22:30:54
Go to the top of the page
+Quote Post
 
 Start new topic
Odpowiedzi
thek
post
Post #2





Grupa: Moderatorzy
Postów: 4 362
Pomógł: 714
Dołączył: 12.02.2009
Skąd: Jak się położę tak leżę :D
Jak dla mnie to za bardzo chcesz upychać wszystko do wszystkiego. Zauważ, że większość Twoich metod uparcie chce coś wyświetlać, choć wcale nie jest to konieczne. Niech będzie sobie co chcesz w podziale, ale ustawiaj to osobnymi metodami. Osobna do doctype, osobna do meta, osobna do dodawania css i js, osobna do body. Dopiero w metodzie wyświetlającej całość strony składaj to do kupy. Poza tym uważam, że pisanie wszystkiego w jednej klasie wprowadzi chaos do twojego projektu w pewnym momencie będziesz miał tyle metod, że pogubisz się w ich wywołaniach. Zauważ, że przykładowo zdefiniujesz sobie metodę display() i będzie miała ona gotowy szablon strony. wystarczy, że w jej zmienne "wstrzykniesz" kawałki kodu i wszystko zagra. Jak go podzielisz? Twój wybór. Ale tylko ona jedna ma wpływać na całość. Nie zaś tak, że dostaje wszystko na gotowe. Weź sobie przykładowo sytuację, że obiekt ma do zmiany head i body. To zrób obiektowi funkcje je ustawiające, ale oczekuj określonego formatu i typu, byś mógł je do zmiennej wpisać. Gdy już masz wszystko ustawione, wywołaj metodę display() a wyświetli ci całość. Tymczasem Ty upychasz metodę wyświetlającą mikroskopijną część wyniku po innej metodzie, która też to robi, by ta druga robiła to samo w trzeciej. To nie ma sensu. Metodami ustawiaj tylko zmienne wewnątrz klasy, a tylko jedna powinna wyświetlać. Da ci to coś w stylu:
[PHP] pobierz, plaintext 
  1. class Strona {
  2. private $head;
  3. public $title;
  4. private $body;
  5.  
  6. public function setHead($dane) {
  7. $this->head = $dane;
  8. }
  9.  
  10. public function setBody($dane) {
  11.  $this->body = $dane;
  12.  }
  13.  
  14. public function display() {
  15. echo '<doctype ble...>
  16. <html>
  17. <head>'.$this->head.
  18. '<title>'.$this->title.'</title>
  19. <head>
  20. <body>'.$this->body.'</body>
  21. </html>';
  22. }
  23. }
[PHP] pobierz, plaintext
Oczywiście to baaaaaaaaaardzo okrojona wersja, ponieważ head powinno mieć podział na zmienną title (często zmieniana zmienna), tablicę znaczników meta, tablicę dołączanych arkuszy stylów, tablicę dołączanych skryptów itp, wraz z odpowiednimi metodami. Takie podejście mocno upraszcza nawet późniejsze dziedziczenie.
Bo ostatecznie kończy się to poprzez wywołanie
[PHP] pobierz, plaintext 
  1. $wyswietl = new Strona();
  2. $wyswietl -> setHead( $sekcjaHead );
  3. $wyswietl ->title = "Rejestracja nowego klienta";
  4. $wyswietl -> setBody( $co_w_body );
  5. $wyswietl -> display();
[PHP] pobierz, plaintext
Zauważ, że to, co będzie w head i body jest dla mnie obojętne dopóki będzie miało odpowiedni format. Mogę tam wrzucić tablicę, którą mi funkcja display() sama wstawi odpowiednio do szablonu (tak jak zrobiłem z title), albo jako ciąg znakowy jak teraz to mam. Bez mieszania, kombinowania. a jeśli zechcę coś zmienić z wyświetlaniem to po prostu inny display mogę napisać bez modyfikacji czegokolwiek innego. Czyli przechodząc na terminologię MVC -> napisałbym inny widok dla tej klasy (IMG:style_emoticons/default/smile.gif) U Ciebie skończyło by się to poprawianiem kilku metod zapewne.
Co istotniejsze. Dopóki nie wywołam funkcji display, mogę zmieniać nawet setki razy title czy inną część bez wpływu na całość tego co Ci zostanie pokazane. Bo i też jest to cały czas na serwerze tylko. Sekcję body możesz nawet składać. Wystarczy, że przerobisz funkę by zamiast podmiany mogła dodawać dodatkowo. A więc choćby coś w stylu:
[PHP] pobierz, plaintext 
  1. public function setBody($dane, $dodanie = FALSE) {
  2. if($dodanie)
  3.     $this -> body .= $dane;
  4. else
  5.     $this -> body = $dane;
  6. }
[PHP] pobierz, plaintext

EDIT: czemu wspomniałem o trybie dodawania? Bo to świetna baza wyjściowa dla tworzenia modułów. Zrobisz tylko pierwszy w trybie normalnym, a resztę w trybie dodawania i masz budowanie sekcji body z klocków wedle uznania. Najlepiej zaś jeszcze display domyślnie ustalić na jakiś wzorzec i go wypełniać. innymi słowy znacznie bardziej rozbudowany niż mój. Z większą ilością kod html, pomiędzy którymi tylko zmienne wkładamy.

Ten post edytował thek 7.11.2009, 15:26:37
Go to the top of the page
+Quote Post

Posty w temacie
- japolak   Wywołanie obiektu w obiekcie.. z zewnątrz - problem   5.11.2009, 22:27:44
- - thek   Podstawy obiektówki. Zauważ, że funkcja jaka wywal...   5.11.2009, 22:54:13
- - japolak   Cytat(thek @ 5.11.2009, 22:54:13 ) Po...   6.11.2009, 16:19:31
- - thek   Twoim problemem jest to, że wszystko masz w jednej...   6.11.2009, 16:51:51
- - japolak   hmm no dałeś mi teraz do myslenia .. widać nie d...   6.11.2009, 18:50:13
- - thek   Jak dla mnie to za bardzo chcesz upychać wszystko ...   7.11.2009, 15:06:19
- - japolak   ok.. Wielkie dzięki za udzielenie wskazówek. nie...   8.11.2009, 10:40:26
- - thek   Szczerze mówiąc, to do PHP nie czytałem żadnych ks...   8.11.2009, 23:57:44

« Następny starszy · Object-oriented programming · Następny nowszy »
 

Reply to this topicStart new topic
2 Użytkowników czyta ten temat (2 Gości i 0 Anonimowych użytkowników)
0 Zarejestrowanych:

 

Tryb wyświetlania: Przełącz na: Standardowy · Przełącz na: Linearny+ · Drzewo

Śledź ten temat · Wyślij temat na e-mail · Wydrukuj ten temat · Subskrybuj to forum

RSS Aktualny czas: 16.10.2025 - 14:29
Powered By IP.Board © 2025  IPS, Inc.
All changes by PHP.pl Administrators
Hosting zapewnia NQ.pl hosting, trac, svn