[PHP] Początki programowania obiektowego. Opcje

[BuldogPro]

zaczynam programować obiektowo stworzyłem sobie na początek połączenie. jeśli możecie powiedzcie czy w dobrą stronę zmierzam z programowaniem obiektowym. Tak wygląda mój kod

[PHP] pobierz, plaintext 
<?php 
 class baza {
 	var $dblink;
	var $host = "localhost";
	var $user = "root";
	var $pw;
	var $db;
function __construct() {
		global $host,$user,$pw,$db;
 
		$this->host = $host;
		$this->user = $user;
		$this->pw = $pw;
		$this->db = $db;
 }
 	public function connect() {
		$this->dblink = @mysql_connect($this->host,$this->user,$this->pw) or die("Error!! You can not connect!");
		@mysql_select_db($this->db,$this->dblink) or die("Error! You can not select datebase!");
	}
 
	}
?>
[PHP] pobierz, plaintext 

Ten post edytował BuldogPro 23.01.2010, 22:04:20

[thek]

Coś takiego jak global nie powinno być nawet w użyciu. Konstruktor może mieć parametry, więc ich używaj do przekazania klasie czegoś. Gdzie masz specyfikator dostępu danych (private, public)? Skąd var wytrzasnąłeś? Dlaczego wygłuszasz błędy poprzez @ zamiast je obsłużyć? Jeśli już robisz construct to czemu zapominasz o destruct? Jeśli już tworze obiekt klasy baza to chcę mieć od razu po jego utworzeniu możliwość działań bez konieczności łączenia, więc wszelkie wariacje z connect() nie powinny mieć miejsca. Inna sprawa, że kompletnie olewasz choćby port połączenia. Nie zawsze jest to domyślny 3306. Chcesz to robić tylko "pod siebie" czy może ma to być bardziej skalowalne? Ja bym nieco więcej informacji zamieszczał. Choćby jedno pole o nazwie $error, które zwracałoby ostatni komunikat błędu z bazy danych. Po prostu byłoby to bardziej rozbudowane (IMG:style_emoticons/default/smile.gif)