[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)

[BuldogPro]

czyli tak już lepiej?

[PHP] pobierz, plaintext 
  class baza {
 protected $pw;
 protected $host;
 protected $db;
protected $user;
protected $dblink;
function __construct($host, $user, $pw, $db) {
  $this->host = $host;
  $this->user = $user;
  $this->pw = $pw;
  $this->db=$db;
		$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!");
 }
public function __destruct()	{
	mysql_close($this->dblink);
	}
	}
[PHP] pobierz, plaintext 

i wywołanie

[PHP] pobierz, plaintext 
<?php $baza= new baza($host, $user, $pw, $db); ?>
[PHP] pobierz, plaintext 

Ten post edytował BuldogPro 23.01.2010, 22:48:39

[Crozin]

Dlaczego wszyscy zaczynają zabawę z OOP od robienia czegoś do "obsługi" połączenia z bazą danych?
Zacznij od przejrzenia co PHP udostępnia właściwie w standardzie - tak, chodzi o PDO - tego używaj do komunikacji z bazą danych.

A dla nauki to sobie zrób coś prostszego (tak, napisanie dobrego sterownika dla bazy danych nie jest łatwym zadaniem) jak np. prosta strona oparta na implementacji MVC (lub którejkolwiek z wariacji tego wzorca) - przy okazji w praniu powinieneś poznać kilka przydatnych wzorców projektowych.

eśli już robisz construct to czemu zapominasz o destruct?

A co to... jak jest konstruktor to musi być destruktor? (IMG:style_emoticons/default/smile.gif)

Ten post edytował Crozin 23.01.2010, 23:06:09