Forum PHP.pl

Dopiero zacząłem przygodę z obiektowym PHP. Uczyłem się z książki 'Zaawansowane tworzenie stron www PHP5 Szybki start'.

Udało mi się wykombinować taki kod:

http://www.wklej.com.pl/show.php?what=20080812201428

Ma to służyć jako miniCMS, to są klasy do zarządzania podstronami.

Chciałbym się zapytać czy to jest wogóle obiektowo, bo mi się wydaje że to są zwykłe funkcje pogrupowane w klasy. 

Nie wiem czy rozumiem ideę obiektowego programowania.

Niestety jest to tylko kod strukturalny zamknięty w klasach i funkcjach.

No trochę lipa. Utwórz np klasę podstrona i w niej metody, źle to IMO podzieliłeś. I stwórz klasę baza a nie polacz

Zmieniłem to w jedną klasę podstrony. 

Nie widzę innej możliwości rozwiązania tego. Według mnie jest wszystko dobrze. Jedna klasa pobiera dane, druga je wyświetla. O to chyba w tym chodzi, nie?

Tak jak ktoś wyżej napisał, myślisz funkcjonalnie. Obiekt to nie zbiór funkcji. Obiekt reprezentuje jakąś konkretną "rzecz".

Pozdrawiam!

Nie, niestety nie chodzi o to. Przynajmniej nie wprost. Obiekty są po to, aby reprezentowały jakieś rzeczy. W przypadku strony www, może to być obiekt Page (zawsze używaj nazw angielskich - to bardzo ważne), który posiada właściwości i operacje. Nie jest jednak reprezentacją dokumentu HTML, tylko danych, które ta stronę tworzą (wpisy w bazie danych lub w pliku tekstowym). Poczytaj więcej, w bardziej specjalistycznych książkach (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

Pozdrawiam.

W sumie nie wiem czy rozpoczęcie nauki programowania obiektowego od PHP to najlepsze rozwiązanie. Ja OOP nauczyłem się na C++ i Tobie też to polecam, chociaż na początku może być trudno, ale nie ma nic za darmo (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg) Poza tym C++ to kanon, który co prawda bywa czasem trochę upierdliwy, bo wszystko musi się zgadzać i jest mniej elastyczne niż nowsze języki, ale to uczy porządnego programowania.
Bazując na swoim przykładzie, polecam naukę podstaw C++, potem wskaźników i wtedy programowanie obiektowe w C++.

Jeszcze odnośnie Twojego kodu, to zrobiłeś zbiór funkcji, jak poprzednicy pisali; klasy charakteryzują się tym, że obiekty z nich utworzone posiadają 1) zmienne i 2) metody (funkcje) i jedno i drugie jest w równym stopniu ważne, więc tworzenie klas tylko ze zmiennymi albo tylko z funkcjami niewiele nam daje, dopiero połączenie obu pozwala wykorzystać masę funkcjonalności.

Tworzysz klasę i później możesz stworzyć wiele obiektów, które są niezależne od siebie, mają inne dane (zmienne), ale korzystają z tych samych metod.

Trywialny przykład:

[PHP] pobierz, plaintext 
<?php
class Prostokat
{
	var $szerokosc;
	var $wysokosc;
 
	public function pole()
	{
		return $this->szerokosc * $this->wysokosc;
	}
}
 
$p1 = new Prostokat;
$p1->szerokosc = 20;
$p1->wysokosc = 10;
echo $p1->pole(); //200
 
$p2 = new Prostokat;
$p2->szerokosc = 65;
$p2->wysokosc = 2;
echo $p2->pole(); // 130
?>
[PHP] pobierz, plaintext 

Coś bardziej zbliżonego do Twojego kodu - obiekt 'Tabela':

[PHP] pobierz, plaintext 
<?php
class Tabela
{
	var $dane;
 
	public function __construct()
	{
		$this->dane = array();
	}
	public function addRow($text)
	{
		$this->dane[] = $text;
	}
	public function display()
	{
		echo '<table>';
		foreach($this->dane as $row)
		{
			echo '<tr><td>'.$row.'</td></tr>';
		}
		echo '</table>';
	}
}
 
$produkty = new Tabela;
$produkty->addRow('margaryna');
$produkty->addRow('masło');
$produkty->addRow('chleb');
$produkty->display();
?>
[PHP] pobierz, plaintext 

Oczywiście to też przykład trywialny, jedynie obudowanie zwykłego arraya, ale niewiele zmieniając można już poprawić trochę jakość zwyczajnej tablicy, np. walidować dane wejściowe, czyli w funkcji addRow sprawdzać czy $text jest taki jak nam odpowiada, czy nie jest za długi etc. etc.

@qBK w PHP5 nie ma var, jest private, protected, public (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)

Pomęczyłem się pół dnia i trochę już łapię. Zrobiłem to tak aby jak najwięcej kodu móc wykorzystać w przyszłości.

Klasa:

http://www.wklej.com.pl/show.php?what=20080813115015

Wykorzystanie klasy:

http://www.wklej.com.pl/show.php?what=20080813115116

Wg tego zrobie potem zarządzanie newsami, menu itd, wszystko tą samą klasą.

Chciałem jeszcze przenieść zapytanie do klasy jakoś, ale nie wiem zabardzo jak i czy warto.

Jak już się uczysz, to nabieraj od podstaw sensownych przyzwyczajeń - rozdzielaj kod php od html w tym przypadku, w bardziej ogólnym zainteresuj się wzorcem MVC.

Zarządzanie wszystkim tą samą klasą?
Co ma wspólnego menu z newsami? Na zdrowy "chłopski" rozum są to dwa różne obiekty, np do jednego są komentarze, do drugiego ni hu hu, jedno i drugie jest inne. Zapoznaj się z dziedziczeniem, interfejsami (możesz sobie zrobić ogólną klasę (lub interface), faktycznie, ale potem ją implementować, czy rozszerzać do swoich potrzeb nie zawierając w nich zupełnie niepotrzebnych metod czy własności).

Ten post edytował Shili 13.08.2008, 15:50:54

Hah, coś mi od początku to var nie pasowało do PHP-a, ale wszystko działało więc się nie zorientowałem (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg) Przeglądałem pewnie jakiś stary kod i mi się zakorzeniło w głowie. A modyfikatorów nie używałem bo kod private var $x krzaczył mi się i nie wiedziałem o co chodzi (IMG:http://forum.php.pl/style_emoticons/default/biggrin.gif) Dzięki za zwrócenie uwagi.

edit
@marcinpruciak:
Jeśli zrobiłeś już klasę Baza, to dodaj na przykład w niej metodę 'query' albo 'execute'

[PHP] pobierz, plaintext 
<?php
class Baza
{
...
	  public function query($q)
	  {
		return mysqli_query($this->polacz, $q);
	  }
}
?>
[PHP] pobierz, plaintext 

i zamiast wywoływać
$r=mysqli_query($baza->polacz, $q);
możesz pisać
$r=$Baza->query($q)

Jest trochę krócej i później powiedzmy przechodząc z MySQL na PostgreSQL nie będziesz musiał każdego mysqli_query zamieniać na pg_query.

Ten post edytował qBK 13.08.2008, 19:57:45

Nie private var $abc, a private $abc (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)

Trochę pomyślałem, pokombinowałem i mam. Nie jest to MVC, bo gubię się przy oddzielnych trzech klasach, ale jest oddzielony widok od logiki.

[PHP] pobierz, plaintext 
<?php
class Publikuj
{
 public $pub;
 public $new_pub;
 public $id;
 
 public function pobierz_id_pub($id, $pub)
 {
     $this->id = $id;
     $this->pub = $pub;
 }
 
 public function pokaz_pub()
 {
     if($this->pub==Y) {
         echo 'id.'&pub=N&#092;">TAK';
     } else {
         echo 'id.'&pub=Y&#092;">NIE';
     }
 }
 
 public function oblicz()
 {
     if($_GET['pub']==N) {
         $this->new_pub='N';
     } else {
         $this->new_pub='Y';
     }
 }
 
 public function zastosuj()
 {
     $baza=new Baza;
     if($_GET['pub']) {
         $q=&#092;"UPDATE pages SET publikuj='$this->new_pub' WHERE id=\".$_GET['id'].\"\";
         $r=$baza->query($q);
     }
 } 
 
 public function przekierowanie()
 {
         header(&#092;"Locatio: ./pages.php\");
 }
}
?>
[PHP] pobierz, plaintext 

I użycie tej klasy w innej klasie: (muszę to jeszcze ulepszyć)

[PHP] pobierz, plaintext 
<?php
class Page
{
 public $tablica;
 public $r;
 
 public function query()
 {
     $baza= new Baza;
     $q=&#092;"SELECT * FROM `pages` ORDER BY `id`\";
     $this->r = $baza->query($q);
 }
 
 public function tworz_tabele()
 {     
     $pub= new Publikuj;
  
           while($this->tablica = mysqli_fetch_array($this->r)) {
             echo '
                     '.$this->tablica['id'].'
                     Zaznacz
                     '; $pub->pobierz_id_pub($this->tablica['id'], $this->tablica['publikuj']); $pub->pokaz_pub(); $pub->oblicz(); $pub->zastosuj(); 
                 echo '
                     '.$this->tablica['tytul'].'
                     Edytuj
                     Usun
                  ';
         }
         echo ''; 
         $pub->przekierowanie();
 }
}
?>
[PHP] pobierz, plaintext 

I co o tym myślicie? Nieźle jak na pierwszy kontakt z obiektowością?

Źle. Powinieneś mieć logikę wywołującą widok, a nie widok wywołujący logikę.

Czyli najpierw myślisz co trzeba zrobić (np. na podstawie informacji z $_GET)
Potem wykonujesz te swoje działania ($pub->oblicz(), $pub->zastosuj(), $pub->pobierz_id_pub(), a wyniki zapisuj sobie na boku w jakichś zmiennych)
Na końcu, gdy już jesteś pewien co(i czy w ogóle) chcesz wyświetlić, to wstawiasz sobie te zmienne do HTML'a

Abstrahując od twojego problemu, jeżeli chodzi o rozdzielenie logiki to poczytaj o MVC sam niedawno się z tym męczyłem i wydawało mi się to wręcz bez sensowne ale później po przemyśleniu i napisaniu pierwszego skryptu MVC zrozumiałem o co biega tak z biegu podam prosty przykład

[PHP] pobierz, plaintext 
<?php
class model
{
public function show_news($name)
{
// metoda modelu pobiera dane ze źródła danych i zwraca je do kontrolera
$news=file_get_contents("plik/".$name.".php");
return $news;
}
}
class view
{
public function show_news($news)
{
// metoda widoku formatuje dane pobrane z modelu i wyświetla w przeglądarce
echo "To jest widok newsa:<br/>zawiera html<br/>";
echo "<p>".$news."</p>"
}
}
class kontroler
{
public function __construct()
{
$model = new model;
$widok= new view;
}
public function doPage()
{
// szkielet kontrolera wywoluje odpowiedni widok z odpowiednimi danymi pobranymi z
 modelu, zarządza żądaniami użytkownika
if (isset($_GET['id']))
{
$widok->show_news($model->show_news($_GET['id']));
}
else
{
$widok->show_news($model->show_news(0));
}
}
$strona=new kontroler;
?>
<html>
<head>
......
</head>
<body>
<p>......tutaj stala czesc strony ona zawsze pozostanie taka sama (IMG:http://forum.php.pl/style_emoticons/default/biggrin.gif) </p>
<div class="tresc">
<?php
$strona->doPage();
?>
</div>
</body>
</html>
[PHP] pobierz, plaintext 

sens powyższego jest żaden, natomiast moim zdaniem dobrze obrazuje idee MVC. Zauważ że dzięki temu, jeżeli np. będziesz chciał dodać obsługę bazy mysql, to zostanie ci do zmiany tylko model, a jeżeli będziesz chciał dać możliwość obejrzenia newsa w pdf to wystarczy dopisać odpowiedni widok.

Ten post edytował f1xer 22.08.2008, 14:21:34

Tylko jak poradzić sobie z pętlą? Wszystkie przykłady co widziałem nie miały takiej pętli. To jedyny sposób jaki znam. 

Bo ważniejsze zmienne nie pobieram z geta tylko z tablicy. I te wszystkie funckje też muszą być w pętli. 

jeżeli masz jusz dane w tablicy to tą tablicę przekazujesz do widoku a w widoku wyciągasz wszystkie elementy tablicy i dekorujesz html później wyrzucasz do przeglądarki. Generalnie to jest tak w uproszczeniu

Kontroler - przyjmuje zapytanie i podejmuje decyzje co zrobić, jeżeli potrzebne są jakieś dane ze źródła danych (plik,baza,SOAP etc.) to prosi Model o przygotowanie takich danych.
Model - pobiera dane ze źródła danych i zwraca kontrolerowi (w postaći np. tablicy)
Kontroler dostaje dane i wywołuje odpowiedni widok
Widok - dostaje "surowe" dane i przygotowywuje je do prezentacji (np. dostał tablicę to robi na niek foreach i dla każdego wiersza dodaje tagi html) po czym wyrzuca do przeglądarki.
odnośnie twojego mini CMS (też taki pisze (IMG:http://forum.php.pl/style_emoticons/default/biggrin.gif) ) to nie potrzeba duzo roboty od frontu model ma np. 1 metode pokaz_strone($id) i wniej wczytujesz z bazy treść strony, widok też ma co najmniej jedną metodę pokaz($strona) dostaje treść strony od modelu i np. wyrzuca to pomiędzy tagami <p></p> a kontroler np. sprawdza czy w $_GET jest podane id jeżeli tak to wywołuje model z tym id a pozniej widok ze zwróconą przez model wartością, jeżeli id nie ma to wczytuje stronę domyślną.

Ten post edytował f1xer 22.08.2008, 17:32:14

Zedytowałem to tak:

[PHP] pobierz, plaintext 
<?php
class Page
{
 public $tablica;
 public $id;
 public $tytul;
 public $publikuj;
 public $show;
 public $r;
 
 public function query()
 {
 	$baza= new Baza;
 	$q="SELECT * FROM `pages` ORDER BY `id`";
 	$this->r = $baza->query($q);
 }
 
 public function tworz_tablice()
 {
 	$this->tablica = mysqli_fetch_assoc($this->r);
 	return $this->tablica;
 } 
 
 public function tworz_strone()
 {	 
 	$pub= new Publikuj;
 			$this->query();
 			$this->tworz_tablice();
 			$this->id = $this->tablica['id'];
 			$this->tytul = $this->tablica['tytul'];
 			$this->publikuj = $this->tablica['publikuj'];
 			$pub->pobierz_id_pub($this->id, $this->publikuj); 
 			$pub->oblicz(); 
 			$pub->zastosuj();
 			$pub->pokaz_pub();
 			$this->show = $pub->show;
 			$this->widok();
 			$pub->przekierowanie();
 }
 
 public function widok()
 { 
 	echo '<table border="1">
 			<tr>
 				<td width="5%">#</td>
 				<td width="5%">Zaznacz</td>
 				<td width="5%">Publikuj</td>
 				<td width="25%">Tytul</td>
 				<td width="15%">Edytuj</td>
 				<td width="15%">Usun</td>
 			</tr>'; 
 		 
 	while($this->tworz_tablice()) {
 		echo '<tr>
 				<td width="5%">'.$this->id.'</td>
 				<td width="5%">Zaznacz</td>
 				<td width="5%">'.$this->show.'</td>
 				<td width="25%">'.$this->tytul.'</td>
 				<td width="15%"><a href="?id='.$this->id.'">Edytuj</a></td>
 				<td width="15%">Usun</td>
 			</tr>'; 
 	}
 	echo '</table>';
 }
 }
?>
[PHP] pobierz, plaintext 

Chyba o to chodzi?

Mam tylko jeden problem. Metoda tworząca tablicę z wyników z bazy dodaje do tablicy tylko pierwszy wiersz z bazy? Nigdzie nie widzę błędu? Co jest nie tak?

Ten post edytował marcinpruciak 22.08.2008, 23:48:20

Zobacz w manualu opis funkcji mysql_fetch_assoc(), zwróć uwagę na pętle while w przykładzie.

Pozdr

Udało się. Dzięki za sugestię.




Dalej ćwiczę obiektowość, stworzyłem teraz klasę dodawania i edycji stron:
http://wklej.com.pl/show.php?what=20080828234035

Jest to zgodne z obiektowością? Strona działa i ma się dobrze tutaj można zobaczyć jak:

http://195.205.202.32:6893/minicms/index.php?page=pages

Proszę o odpowiedź.

Ten post edytował marcinpruciak 28.08.2008, 22:44:29