[PHP][OOP] Klasa do łączenia się z bazą danych - czy dobrze jest skonstruowana? Opcje

[Dominator]

Witajcie
Dawno nic w php nie robiłem, więc tak dla przypomnienia zrobiłem sobie klasę, która łączy się z bazą danych. Mam takie pytanie - czy ja dobrze to zrobiłem? Co ewentualnie proponujecie zmienić?
Ogólnie to jestem w trakcie przesiadki na OOP, więc nie krzyczcie (IMG:style_emoticons/default/biggrin.gif)

[PHP] pobierz, plaintext 
<?php
 
Class UserInterface
{
	public function SetDBData($_server, $_port, $_username, $_password, $_database);
	{
		$this->server   = $_server;
		$this->port     = $_port;
		$this->user     = $_username;
		$this->password = $_password;
		$this->database = $_database;
	}
	public function DBConnect()
	{
		Try
		{
			$pdo = new PDO('mysql:host='.$this->server.';dbname='.$this->database.';port='.$this->port, $this->username, $this->password );
		}
		catch(PDOException $e)
		{
			return "Error! Code ->".$e;
		}
	}
}
 
$_UI = new UserInterface();
$this->SetDBData("","","","");
$this->DBConnect();
 
?>
[PHP] pobierz, plaintext 

[Dominator]

Okej, pomajstrowałem i wygląda to tak:

[PHP] pobierz, plaintext 
<?php
 
Class DB
{
	public function Set($server, $port, $username, $password, $database)
	{
		$this->server   = $server;
		$this->port     = $port;
		$this->user     = $username;
		$this->password = $password;
		$this->database = $database;
	}
	public function Connect()
	{
		try
		{
			$pdo = new PDO('mysql:host='.$this->server.';dbname='.$this->database.';port='.$this->port, $this->username, $this->password );
			$pdo->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
			if($pdo)
			{
				return "connected";
			}
		}
		catch(PDOException $e)
		{
			return "Error! Code ->".$e->getMessage();
		}
	}
}
 
$db = new DB();
$db->Set("","","","","");
echo $db->Connect();
 
 
 
?>
[PHP] pobierz, plaintext 

Tylko dostaję błąd: Notice: Undefined property: DB::$username in ... (17 linia)
Dlaczego?

EDIT: Poradziłem sobie, łączy się z bazą (IMG:style_emoticons/default/biggrin.gif)

Tylko pytanie: czy dobrze to jest napisane? co zmienić?

Ten post edytował Dominator 8.05.2013, 20:11:49