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

[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 

metode SetDBData zmien na protected i wywoluj ja z konstruktora ktory przyjmue potrzebne parametry. DbConnect tworzy instancje obiektu PDO ale gdzie zapisuujesz ta zmiena? zrob pole do ktorego przypisujesz zmienna typu PDO jako protected i moja ostatnia 3 uwaga to nie lap tak szybko wyjatkow lepiej obsluzyc je jak najwyzej

- nazwa UserInterface niewiele mówi i ma mało wspólnego z DB.
- w DBConnect powinieneś chyba zwracać ten obiekt PDO, aby móc na nim operować.
- jak chcesz poza klasa używać $this? To nie zadziała.
- wywal podkreślenia z nazw zmiennych, to nie czasy PHP4

Sam używam podkreślenia do oznaczenia zmiennych, które przekazuje do metody. To zła praktyka?

http://forum.php.pl/index.php?s=&showt...t&p=1041800 kilka postów o tym dlaczego.

[PHP] pobierz, plaintext 
<?php
 
Class DB
{
	protected 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); 
		}
		catch(PDOException $e)
		{
			return "Error! Code ->".$e;
		}
	}
}
 
$db = new DB();
$db->Set("","","","","");
$db->Connect();
?>
[PHP] pobierz, plaintext 

Dostaję: Fatal error: Call to protected method DB::Set() from context '' in...

Wyjaśni ktoś dlaczego?

Nie możesz się odwołać do metody która jest prywatna lub chroniona. http://pl.wikibooks.org/wiki/PHP

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ą

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

Największy błąd to: połączyłeś się i co z tego skoro utraciłeś obiekt PDO, możesz to rozwiązać poprzez przypisanie do bazy pola protected lub wyrzucić obiekt BD na zewnątrz, na szybko:

[PHP] pobierz, plaintext 
<?php
 
Class DB{
	public function __construct($server, $port, $username, $password, $database)
	{
		$this->Set($server, $port, $username, $password, $database);
		$this->$db=$this->Connect();
	}
	protected function Set($server, $port, $username, $password, $database)
	{
		$this->server = $server;
		$this->port = $port;
		$this->user = $username;
		$this->password = $password;
		$this->database = $database;
	}
	protected function Connect()
	{
		$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);
	}
              protected  $server;
	 protected $port ;
	protected $user;
	protected $password;
	protected $database;
	protected $db;
}
try
{
	$db = new DB("","","","","");
}
catch(PDOException $e)
{
	echo "Error! Code ->".$e->getMessage();
}
 
?>
[PHP] pobierz, plaintext 

Pomijam tutaj takie rzeczy jak PSR bo dopiero się uczysz, chociaż warto poczytać już za wczasu żeby wyrobić sobie nawyk
//edit sorki za formatowanie

Dostałem jakieś błędy, ale już je naprawiłem, dziękuję za pomoc!