[php] programowanie obiektowe - gdzie go używać? Opcje

[sweter]

Witam, 

od kilku dni uczę się programowania obiektowego i chyba dobrze mi to idzie

Mam tylko problem, bo nie wiem gdzie stosować takie programowanie, a gdzie lepiej jest pisać "tradycyjnie"

Powiedzmy, że mamy taką klasę służącą do tworzenia formularzy i w niej dwie metody:

[PHP] pobierz, plaintext 
 
class formularze{
 
   public $nazwa;
 
   public $wartosc;
 
   public function text(){
 
        echo "<input type=\"text\" name=\"".$this->name."\" value=\"".$this->wartosc."\">";
 
   }
 
   public function password(){        echo "<input type=\"password\" name=\"".$this->name."\" value=\"".$this->wartosc."\">";
 
   }
 
 
 
}
 
[PHP] pobierz, plaintext 

W zależności od zapotrzebowania wyświetlało by się:

[PHP] pobierz, plaintext 
 
$cos = new formularze;
 
$cos -> nazwa = 'Nazwa_pola';
 
$cos -> wartosc = 'Wartość_pola';
 
$cos -> text();
 
[PHP] pobierz, plaintext 

lub

[PHP] pobierz, plaintext 
 
$cos = new formularze;
 
$cos -> nazwa = 'Nazwa_pola';
 
$cos -> wartosc = 'Wartość_pola';
 
$cos -> password();
 
[PHP] pobierz, plaintext 

Czy dobrze rozumiem? Programowaniem obiektowym zajmują się od niedawna więc proszę o wyrozumiałość

Z góry dziękuję za uwagi i rady

[taktu]

przede wszystkim używaj return w metodach zamiast echo, teraz pomyśl czy nie lepiej zrobić metodę input zamiast password i text?

[PHP] pobierz, plaintext 
//i wywołać w ten sposób
 
echo $cos->input('password');
echo $cos->input('text');
 
[PHP] pobierz, plaintext 

Jeżeli nauczysz się dobrze OOP to już nie będziesz chciał pisać 'tradycyjnie'.

[sweter]

OK, dzięki.
Jeszcze takie mam pytanie: tworzę CMS'a (takiego prościutkiego) i na pewno będą w nim takie elementy jak popup'y z komunikatami oraz menu rozwijalne.
Czy powinienem zrobić te elementy za pomocą obiektów??

[Wicepsik]

A nie lepiej zrobić coś takiego?

[PHP] pobierz, plaintext 
class formularze{
   public function input($type, $name, $value=false){
        return '<input type="'.$type.'" name="'.$name.'" value="'.$value.'">';
   }
}
 
$cos = new formularze;
echo $cos->input('text', 'login', 'podaj login');
echo $cos->input('password', 'haslo');
[PHP] pobierz, plaintext 

Nie znam się dobrze na oop, ale czy ten kod jest lepszy?

Ten post edytował Wicepsik 27.12.2009, 18:28:32

[sweter]

Hej, napisałem coś w OOP
Moglibyście rzucić okiem i powiedzieć czy wszystko jest OK?
Moje klasy są niezwykle skomplikowane
Pierwsza łączy się z bazą danych, a druga liczy ilość rekordów z podanego zapytania:

[PHP] pobierz, plaintext 
class db {
		private $host="localhost";
		private $db = "cms";
		private $db_login = "root";
		private $db_password = "";
 
		public function __construct(){
			if(!$con=mysql_connect($this -> host, $this -> db_login, $this -> db_password)){
				echo "Nie udało się nawiązać połączenia z bazą danych";
			}
			if(!mysql_select_db($this-> db)){
				echo "Nie udało się wybrać bazy danych";
			}
		}
	}
 
class query extends db{
		public function query2($zap){
			$wyn = mysql_query($zap);
			return mysql_num_rows($wyn);
		}
	}
 
$o = new query;
$o -> query2("SELECT * FROM `articles`");
[PHP] pobierz, plaintext 

[darko]

Zamiast niepotrzebnie opakowywać standardowe funkcje mysql - skorzystaj z PDO.

[skowron-line]

Zamiast niepotrzebnie opakowywać standardowe funkcje mysql - skorzystaj z PDO.

Większość osób zaczyna od napisania klasy do obsługi Mysql

[darko]

Większość osób zaczyna od napisania klasy do obsługi Mysql

Dokładnie, sam też tak zaczynałem, jednak z perspektywy czasu oceniam, że niepotrzebnie i sporo mojej ówczesnej pracy poszło na marne. Lepiej "pobawić się" dziedziczeniem, zasięgiem zmiennych, nadpisywaniem metod itd. próbując pisać coś pożytecznego, co pozwoli uniknąć klepania kilka razy tego samego kodu.

[erix]

Nie do końca się zgodzę - jeśli chodzi o wydajność, to - niestety - PDO jest na szarym końcu. Używam, kiedy muszę coś MAŁEGO i szybko napisać.

A opakowanie standardowych funkcji się przydaje, chociażby dla obiektowości. Fakt, jest jeszcze mysqli, ale doradzanie PDO w środowisku, w którym będzie wykorzystywany tylko MySQL, to dla mnie nieporozumienie.

Poszukaj benchmarków na Sieci; nie wszystko złoto, co się świeci. A jeśli chodzi o np. bindowanie zapytań - mysqli. Choć nie zawsze bindowanie zapytań jest szybsze niż wstrzyknięcie całego zapytania; wszystko zależy od konkretnego przypadku.

~sweter - metoda query powinna być w klasie DB, a nie wyniku. Klasa wyniku, to - jak nazwa wskazuje - powinien być wynik + metody pozwalające na przeszukiwanie, wyciąganie danych, itp.

[sweter]

Czyli nie znaleźliście błędów? Uff..
Teraz druga sprawa:
przeczytałem troszkę tutoriali i dowiedziałem się, że można zrobić taką metodę, która będzie się nazywała tak klasa i przy tworzeniu obiektów będzie się od razu wywoływać (tak chyba było w PHP 4). I nie chodzi tu o __construct!
Przykładowo:

[PHP] pobierz, plaintext 
$obiekt = new NazwaKlasy(jakies_wartosci);
[PHP] pobierz, plaintext 

Możecie powiedzieć jak to zrobić w PHP 5, bo nie chce mi się przekopywać tych wszystkich tutoriali

[Mephistofeles]

Może i PDO nie jest mistrzem wydajności, ale do takich zastosowań sprawdza się bardzo dobrze. A czy autor potrzebuje od razu maksymalnej wydajności? Niech się uczy przyszłościowych rozwiązań, a do odpowiedzi co jest lepsze dojdzie kiedyś sam .

Co do twojego pytania, to właśnie do tego służy konstruktor.

Ten post edytował Mephistofeles 29.12.2009, 18:15:22

[darko]

A opakowanie standardowych funkcji się przydaje, chociażby dla obiektowości.

Sztuka dla sztuki. Zresztą jak sam wspomniałeś jest obiektowe mysqli, jest PDO, są różne gotowe rozwiązania, nie bardzo widzę sens tworzenia kolejnych opakowywaczy dla standardowych funkcji mysql. Chyba że ktoś musi mieć koniecznie "po swojemu". A PDO polecam nie ze względów wydajnościowych, a dla świętego spokoju z sql injection.

[sweter]

OK, teraz mam coś takiego:

[PHP] pobierz, plaintext 
class db {
		private $host="localhost";
		private $db = "cms";
		private $db_login = "root";
		private $db_password = "";
 
		public function __construct(){
			if(!$con=mysql_connect($this -> host, $this -> db_login, $this -> db_password)){
				echo "Nie udało się nawiązać połączenia z bazą danych";
			}
			if(!mysql_select_db($this-> db)){
				echo "Nie udało się wybrać bazy danych";
			}
		}
 
		public function query($zap){
			if($wyn = mysql_query($zap)){
				// wykonywanie zapytania
				echo "Znalazlem ".mysql_num_rows($wyn)." rozwiazan.<br />";
				global $wiersz;
				for($q=0; $q<mysql_num_rows($wyn); $q++){
					$wiersz[$q] = mysql_fetch_assoc($wyn);
				}
			}else{
				echo "Nie moge wykonac zapytania. MySQL zwraca blad: <i>".mysql_error()."</i>";
			}
		}
 
		public function sort(){
			// wyświetlanie tablicy z rekordami
			echo "<pre>";
			print_r($wiersz);
			echo "</pre>";
		}
	}
 
$o = new db;
$o -> query("SELECT id, title, date_and_time  FROM `articles`");
$o -> sort();
 
[PHP] pobierz, plaintext 

Jednak wywołanie metody sort() nie wyświetla mi tablicy

[darko]

W klasie db lepiej dodać pole protected $wiersz, w metodzie query wywal global $wiersz (na marginesie global działa na odwrót) i daj $this->wiersz[$q] = mysql_fetch_assoc($wyn); a w metodzie sort odwołać się: print_r($this->wiersz) zamiast print_r($wiersz). Unikaj globali w klasach bo rozwalasz hermetyzację.

Ten post edytował darko 29.12.2009, 19:13:54