Korektny zapis triedy pre pracu s DB – PHP – Fórum – Programujte.com

V PHP programujem uz nejaku dobu a rozhodol som sa naucit programovat v tomto jazyku objektovo. Napisal som si velmi jednoduchu triedu na pracu s databazou, no nie som si uplne isty spravnostou vsetkych zapisov. Bol by som velmi rad, keby mi ju skontroloval niekto zbehli v danej problematike a pripadne ju opravil tak, aby bola zapisana korektne. Konkretne ma zaujima nasledovne:

1- zvolil som viditelnost premennych spravne ?
2- zvolil som viditelnost funkcii spravne ?
3- pouzivam spravne pseudopremennu $this ?
4- funkcia query ma dostat ako parameter z vonku SQL kod (ktory sa "dosadi" do premennej $sql). Niesom si isty, ci mam aj tejto premennej $sql definovat nejaku viditelnost, alebo nie, pokial ano, tak aku.

<?php

class Database {

	private $db_host = 'localhost';

	private $db_user = 'root';

	private $db_pass = '';

	private $db_database = 'projekt';	

	private $link;	

	private $db_selected;

	private $result;

	

	public function connect() {

		$this->link = mysql_connect($this->db_host, $this->db_user, $this->db_pass);

		if (!$this->link) {

			die();

		}

		

		$this->db_selected = mysql_select_db($this->db_database, $this->link);

		if (!$this->db_selected) {

			die ();

		}	

	}

	

	public function query($sql) {

		$this->result = mysql_query($sql, $this->link);

		return $this->result;

	}

}

?>



$mysql = new Database();

$mysql->connect();

$mysql->query('SELECT * FROM ...');

-ohledne viditelnosti - bud musim ty promenne neastavovat v constructoru, nebo je musis udelat viditelne (jinak nenastavis jmeno db, a tak beze zmeny kodu tridy)
-viditelnost metod ok
-viditelnost $link - ok
-promenna select_db - ok neni, rekspektive ne v tuto chvili, kdyz v pripade neuspeneho vyberu db ukoncujes script, to nedava smysl. a i kdyby si tu promennou nahradil lokalni ($selec = ...; if ($select) die()) tak stejne by to bylo spatne, protoze v objektovem kodu se pouzivaji vyjimky (exceptions)
-viditelnost se definuje pouze clenskym promennym, tedy $sql je ok

jozosandy: Je to napsane spravne, ale neni to OOP :)

Já myslím, že OOP a výjimky jsou poměrně nezávislé koncepty. Takže pokud se teď chceš učit jen OOP, nechť tam (zatím) zůstane die.

Lamicz: Co na tom není OOP?

Tohle je tak malý kus kódu, že až nemá smysl se o něm nějak víc bavit. Zajímavé to začne být až když to začneš kombinovat s dalším kódem (třídami).

jozosandy
V tomto případě použití ti OOP nic moc nepřináší. Doporučuji nastudovat nějakou literaturu. Mně se docela líbila kniha PHP: Objects, Patterns, and Practice[1], napsal Matt Zandstra. (Dá se sehnat i jako PDF ke stažení.)

hrach
v objektovem kodu se pouzivaji vyjimky (exceptions)
Mohou, nemusí. Nedělal bych z toho nějaké obecné pravidlo. Ale je pravda, že die() je fůj.

[1] http://www.amazon.com/PHP-Objects-Patterns-Practice-Second/dp/1590599098

ano, samozřejmě že se nemusí, ale já sem taky neřekl, že musí, já jsem jen řekl, že se to tak děla :D