Wywołanie obiektu w obiekcie.. z zewnątrz - problem Opcje

[japolak]

Witam.. mam taki kod:

fragment pliku strona.php:

[PHP] pobierz, plaintext 
 
 
class strona {
public $tytul;
public $zawartosc;
 
public function __set($nazwa,$wartosc)
{
	$this->$nazwa=$wartosc;
}
 
public function wzor_obranowania(){?>
		 echo$this->tytul;
	echo $this->zawartosc;
 
}
 
public function wyswietl_dodaj_karte()
{
 
.... costam.... 
}
 
}
 
[PHP] pobierz, plaintext 

nastepny plik wywolujacy obiekty z klasy (fragment):

[PHP] pobierz, plaintext 
 
$wyswietl= new strona();
 
$wyswietl->wyswietl();
echo"<br>";
$wyswietl->tytul="Rejestracja nowego klienta";
$wyswietl->zawartosc=wyswietl_dodaj_karte();
$wyswietl->wzor_obranowania();
 
 
 
$wyswietl->wyswietl_stopke();
 
 
[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

[thek]

Podstawy obiektówki. Zauważ, że funkcja jaka wywala błąd jest wewnątrz definicji klasy strona, przez co musi być wywoływana poprzez obiekt tejże klasy. A skoro tak jest to wywołuj ją prawidłowo, czyli

[PHP] pobierz, plaintext 
$wyswietl->wyswietl_dodaj_karte();
[PHP] pobierz, plaintext 

Bo chyba nie chodzi Ci o wariacje w stylu:

[PHP] pobierz, plaintext 
$wyswietl->zawartosc = $wyswietl->wyswietl_dodaj_karte();
[PHP] pobierz, plaintext 

EDIT: oczywiście to są moje domysły bo kompletnie nie wiem co metoda wyswietl_dodaj_kartę() robi. Dla mnie jest zapewne widokiem jakimś, ale to tylko moje przypuszczenia...

Ten post edytował thek 5.11.2009, 22:56:33

[japolak]

Podstawy obiektówki. Zauważ, że funkcja jaka wywala błąd jest wewnątrz definicji klasy strona, przez co musi być wywoływana poprzez obiekt tejże klasy. A skoro tak jest to wywołuj ją prawidłowo, czyli

[PHP] pobierz, plaintext 
$wyswietl->wyswietl_dodaj_karte();
[PHP] pobierz, plaintext 

Bo chyba nie chodzi Ci o wariacje w stylu:

[PHP] pobierz, plaintext 
$wyswietl->zawartosc = $wyswietl->wyswietl_dodaj_karte();
[PHP] pobierz, plaintext 

EDIT: oczywiście to są moje domysły bo kompletnie nie wiem co metoda wyswietl_dodaj_kartę() robi. Dla mnie jest zapewne widokiem jakimś, ale to tylko moje przypuszczenia...

codzi mi konkretnie o to ze w metodzie wzor_obranowania znajduje sie tabela
a ja chce jeszcze w wybrane komórki tabeli umiescic inne obiekty.. inne metody..

1.
$wyswietl->zawartosc = $wyswietl->wyswietl_dodaj_karte();

to sprawia iż obiekt wyswietl_dodaj_karte wyswietla mi sie calkowicie nad obiektem wzor obramowania.. a chce aby znajdowal sie wewnatrz tabeli ktora obiekt wzor oprogramowania tworzy..

[thek]

Twoim problemem jest to, że wszystko masz w jednej klasie i nie widzisz co jest czym. Gubisz się między modelem, kontrolerem i widokiem. Widok to wzor_obranowania() i jego nawet nie tykaj. To co masz zrobić z zawartością definiujesz wewnątrz metody wyswietl_dodaj_karte() i wywołujesz ją z odpowiednimi parametrami lub używasz $this jeśli tylko zapisujesz do zmiennej $zawartość poprzez $wyswietl->wyswietl_dodaj_karte($parametry), ale broń boże nie wyświetlasz tego co robi! Od tego masz metodę widoku wzor_obranowania() bo to ona wyświetla całość tabelki. Podziel sobie kod na logiczniejsze działy jeśli chcesz mieć całość w jednej i zrób bloki kodu, które funkcjonalnie będą równe modelowi, kontrolerowi oraz widokowi/-om. Teraz masz bowiem coś, co jest jednocześnie metodą modelu (kontrolera? ki czort wie co robisz tak naprawdę wewnątrz więc nie mam pojęcia gdzie to wcisnąć), ale jednocześnie ma być widokiem bo wyświetla dane. Tak się nie robi i dlatego masz problem z wciśnięciem tego. Możesz dopisać do zmiennej zawartość efekt tej metody lub nadpisać ją, ale nie tak jak Ty to robisz tylko jak Ci pokazałem w drugim przypadku. Po drodze jednak pewnie zamiast nadpisywać Ci dane - dopisuje je, przez co kod się wewnątrz wydłuża.

[japolak]

hmm
no dałeś mi teraz do myslenia ..

widać nie dokońca rozumiem reguł obiektowości..

moze zdradze troche wiksza część mojej jednej klasy..>

teoretycznie podzielilem sobie klasy .. na te ktore wyswietlaja jakis teoretycznie statyczny kod.. oraz te ktore pobieraja dane z bazy..

zerknij i oceń.. czy to jest wszystko do kitu..

chcialem we wzorze_obramowania . umieszczac sobie w zaleznosci co dana strona ma zawierac odpowiednie "funkcje / obiekty"

moze wiec inaczej zapytam .. jak zrobic aby nie trzeba bylo pisac po sto razy do każdej funkcji w klasie tabelek tylko raz zrobic wzór i wszystko w niego pakować

[PHP] pobierz, plaintext 
 
 
 
class recepcja extends strona
 
{
 
	public $naglowek;
	public $obiekt;
 
 
protected $przyciski = array ('Recepcja'=>'panel_recepcji.php',
								'Rejestracja karty'=>'dodaj_karte.php',
								'Karnety'=>'pokaz_karnety.php',
								'Formularz dzienny'=>'formularz_dzienny.php',
								'Szukaj'=>'form_szukaj.php',
								'Trenerzy'=>'dyz_trenerow.php',
								'Wyloguj'=>'wyloguj.php' );
 
 
public function wyswietl (){
	$this->doctype();
	echo "<html>\n<head>\n";
	$this->wyswietl_metaznaczki();
	$this->wyswietl_tytul();
	$this->wyswietl_style();
 
	echo"</head>\n<body>\n";
	$this->wyswietl_naglowek();
 
	$this->wyswietl_menu($this->przyciski);
$sprawdzenie=new logowania();
	$sprawdzenie->sprawdz_prawid_uzyt();
	echo $this->spawdzenie;
	echo $this->zawartosc;
 
 
 
}
 
 
 
 
 
 
protected function wyswietl_menu($przyciski){
echo "<table width = '100%' rules='none' class='all' frame='above' border='1' bgcolor = 'white' cellpadding = '1' cellspacing = '1' >\n";
    echo " <tr >\n";
 
    //obliczenie wielkości przycisku
    $szerokosc = 100/count($przyciski);
 
    foreach ($przyciski as $nazwa=>$url)
    {
     $this->wyswietl_przycisk($szerokosc, $nazwa, $url,
                             !$this->czy_to_aktualny_url($url));
    }
    echo " </tr>\n";
    echo "</table>\n";
}
 
protected function czy_to_aktualny_url($url){
	if(strpos($_SERVER['PHP_SELF'], $url)==false)
    {
      return false;
    }
    else
    {
      return true;
    }
}
 
protected function wyswietl_przycisk($szerokosc, $nazwa, $url, $active = true){
    if($active)
    {
      echo "<td width = '".htmlentities($szerokosc)."%'align='center' >
 
            <a href = '".htmlentities($url)."' class='nav' ><span class='menu'>$nazwa</span></a></td>";
    }
    else
    {
     echo "<td width = '".htmlentities($szerokosc)."%'  align='center' style='border-bottom-style: double;'>
 
           <span class='menu' >$nazwa</span></td>";
    }
}
public function wzor_obranowania(){?>
		<table width=100% border =0 cellpadding=0 cellspacing=0><tr><td  valign="top" align="left"><img src = "r1.gif" /></td>
		<td ><?php echo$this->naglowek;?></td><td valign="top" align="right"><img src = "r2.gif" /></td></tr>
	<tr><td></td><td>
	<?php 
	echo$this->obiekt;
 
	?>
 
	</td><td ></td></tr>
	<tr  valign="bottom" align="left" ><td><img src = "r4.gif" /></td><td></td><td valign="bottom" align="right"><img src = "r3.gif" /></td></tr>
	</table><?php 
}
 
public function wyswietl_dodaj_karte()
{
	$sprawdz=new prawid_dane();
	$sprawdz->sprawdz_dodaj_karte_onkeyup();	
	$sprawdz->sprawdz_dodaj_karte_onsubmit();
 
 
 
 
	?>
 
 
 
 
 
<center><h3>Formularz rejestracji nowej karty klienckiej w systemie</h3></center>
 
 
<form
	method="post" action="tworz_karte.php"
	name="dodaj_karte"
	id="basicform"
	onSubmit="return sprawdz_form_dodaj_karte(this);" >
 
 
<fieldset>
	<label for="nr_karty">Nr karty:</label>
	<?php 
	$nr=new dane();
	$nr->nastepny_numer_karty();
 
	?>
	</fieldset>
 
 
 
 
<fieldset>
	<label for="imie">Imie:</label>
	<input
		type="text"
		name="imie"
		id="imie"
		onkeyup="sprawdz_imie(this);" />
	<span class="hint"> Imie może zawierać tylko znaki polskiego alfabetu, musi być bez spacji oraz składać sie od 3 do 20 liter</span>
</fieldset>
 
<fieldset>
	<label for="nazwisko">Nazwisko:</label>
	<input
		type="text"
		name="nazwisko"
		id="nazwisko"
		onkeyup="sprawdz_nazwisko(this);" />
	<span class="hint">Nazwisko musi zaczynać sie i konczyć znakiem polskiego alfabetu, może składac sie z kilku członów oddzielonych "-" lub " " oraz składać sie od 3 do 40 liter</span>
</fieldset>
.......
 
 
......
 
<input type="submit" name="submit" value="Zapisz" />
 
</form>
 
 
<?php
 
}
 
 
 
 
 
function wyswietl_formularz_dzienny()
{
	?>
 
 
	<?php
	$pokaz=new dane();
	$godzina=date(G);
	$id_dnia=date(w);
....
 
bla bla bla
......
 
  					}
}
 
function wyswietl_historia_dzienny_form(){
	$sprawdz= new prawid_dane();
	$sprawdz->sprawdz_hist();
 
	?>Wyświetl formularz z dnia:
	<form  
	onSubmit="return sprawdz_hist1(this);"
	action="formularz_dzienny.php" method="get"> 
	<input name="dzien" type="text"  maxlength="2" size="2" /> - <input name="miesiac" type="text" maxlength="2" size="2"  /> - <input name="rok" type="text" maxlength="4" size="4" /> 
<input type="submit" name='submit' value='wyslij'/>
</form>
	<?php 
 
 
}
 
 
 
 
 
 
 
 
function wyswietl_szukaj_karte()
{
	?>
	<br />
	<table width=100% border =0 cellpadding=0 cellspacing=0><tr><td  valign="top" align="left"><img src = "r1.gif" /></td><td >
	<center><h3>Wyszukiwarka</h3></center>
	</td><td valign="top" align="right"><img src = "r2.gif" /></td></tr>
	<tr><td></td><td>
	<center><form method="post" action='szukaj.php'>
	<table>
	<tr>
	<td><input type="text" name="wyrazenie"></input> </td>
	<td>	
	<select name="metoda_szukania">
		<option value="id_karty">Nr. karty</option>
		<option value="imie">Imie</option>
		<option value="nazwisko">Nazwisko</option>
		<option value="adres">Adres(ulica)</option>
		<option value="miasto">Miasto</option>
		<option value="kod_pocztowy">Kod pocztowy</option>
		<option value="telefon">Telefon</option>
 
	</select>
 
	</td>
	<td><input type="submit" value="szukaj" name="szukaj" /></td>
	</tr>
 
	</table>
	</form></center>
	</td><td ></td></tr>
	<tr  valign="bottom" align="left" ><td><img src = "r4.gif" /></td><td></td><td valign="bottom" align="right"><img src = "r3.gif" /></td></tr>
	</table>
 
 
 
 
	<?php
 
}
 
function sprawdzanie_nr_karnetu(){
?>
 
 
 
<script type="text/javascript"><!--
 
 
 
var minlength = '3'; // minimalna ilość znaków
var delay = '1000';// czas do aktywacji sprawdzania. w milisekundach (1 sec = 1000 milisec.)
var divid = 'sprawdz'; // ID of the DIV where response will be shown
 
// czekaj az użytkownik przestanie pisać
 
function czekaj(){
if(window.mytimeout) window.clearTimeout(window.mytimeout);
window.mytimeout = window.setTimeout("sprawdz2()", delay);
return true;
}
 
//sprawdz dostepność przegladarki
 
function sprawdz2() {
 
if (window.XMLHttpRequest) {
http = new XMLHttpRequest();
} else if (window.ActiveXObject) {
http = new ActiveXObject("Microsoft.XMLHTTP");
}
 
handle = document.getElementById('nazwisko');
var url = 'ajax2.php?';
if(handle.value.length > minlength) {
 
// czas dla IE
 
fetch_unix_timestamp = function()
{
return parseInt(new Date().getTime().toString().substring(0, 10))
}
 
var timestamp = fetch_unix_timestamp();
 
var fullurl = url + 'do=sprawdz_nazwisko&nazwisko=' + encodeURIComponent(handle.value) + '&timestamp=' + timestamp;
 
http.open("GET", fullurl, true);
http.send(null);
http.onreadystatechange = zmiana_stanu_nazwiska;
}else{
document.getElementById(divid).innerHTML = '';
}
}
 
// sprawdź nazwisko
 
function zmiana_stanu_nazwiska() {
if (http.readyState == 4) {
 
var html = http.responseText;
 
document.getElementById(divid).innerHTML = html;
}
}
// -->
</script>
 
Szukaj nr. karnetu:
<table width = "300"><tr><td>
<div id="sprawdzanie">
 
<form>
Wpisz nazwisko:
 
 
<input id="nazwisko"  onkeyup="czekaj()" name="nazwisko" size="20" type="text" />
</form></div>
<br/>
<div id="sprawdz"></div>
 
</td></tr></table>
<?php
 
}
 
 
[PHP] pobierz, plaintext 

[/php]

[thek]

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 
class Strona {
private $head;
public $title;
private $body;
 
public function setHead($dane) {
$this->head = $dane;
}
 
public function setBody($dane) {
 $this->body = $dane;
 }
 
public function display() {
echo '<doctype ble...>
<html>
<head>'.$this->head.
'<title>'.$this->title.'</title>
<head>
<body>'.$this->body.'</body>
</html>';
}
}
[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 
$wyswietl = new Strona();
$wyswietl -> setHead( $sekcjaHead );
$wyswietl ->title = "Rejestracja nowego klienta";
$wyswietl -> setBody( $co_w_body );
$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 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 
public function setBody($dane, $dodanie = FALSE) {
if($dodanie)
    $this -> body .= $dane;
else
    $this -> body = $dane;
}
[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

[japolak]

ok..

Wielkie dzięki za udzielenie wskazówek.
niezmiernie mi sie przydadzą .. teraz troche inaczej na to wszystko patrze;) ...

wiesz ten kod co ci przedstawiłem.. to część mojej pracy inz.

Program mam juz gotowy.. akceptacje promotora tez mam.. ale chcialem sobie kod poukładać jakoś ładniej, zanim oddam do końcowej analizy.. bo jak sam zauważyłeś teraz jest zmiksowany bałagan..

tak sie zastanawiam. skorzystać z jakiejś literatury??

myślalem poczytać troche : Php object - oriented solutions. .ale nie wiem czy to dobra pozycja aby zaczynac??
bo to co aktualnie mam wykonane bylo zalążkiem .. z książki : PHP i MySQL. Tworzenie stron WWW. Vademecum profesjonalisty

moze ty znasz jakaś pozycje wartą uwagi??

[thek]

Szczerze mówiąc, to do PHP nie czytałem żadnych książek drukowanych Moja wiedza wynika głównie ze znajomości inżynierii oprogramowania, C++, który jest bardzo podobny, późniejszego wertowania manuala, googlania oraz masy czasu poświęconego na siedzenie w kodzie. Dobry projekt + chęci = 70% sukcesu. Stąd nie mogę polecić konkretnego wydawnictwa. Zwyczajnie szukam rozwiązań na bieżąco gdy mi problem wykoczy. A raczej wyskakują rzadko bo trochę już połamałem zębów na różnych problemach. Podstawą jest i tak dobry algorytm. Nietóre mają już gotowe implementacje, ale większość musisz sam napisać i przemyśleć. Z czasem zaczniesz widzieć rozwiązania dopiero co przedstawionego Ci problemu od razu. To już kwestia doświadczenia i otrzaskania z problemami. Najwyższe stadium "zboczenia programistycznego" to widzieć algorytmy w zdarzeniach codziennych i próbować je zapisać