Ocena podejścia do Dependency Injection, PDO + OOP + Dependency injection Opcje

[marcinq123]

Witam, uczę się OOP, zacząłem dla testów pisać klasę obsługująca bazę na podstawie PDO ( Trudno to nazwać DIC ). I proszę Was o ocenę czy w dobrą stronę to zmierza czy może podejście mam złe i później sobie skomplikuje życie ?

"Klasa DIC"

[PHP] pobierz, plaintext 
class DataBaseConnection
{
	protected $pdo;
	protected $mysqlHost = 'Host';
	protected $mysqlLogin = 'Login';
	protected $mysqlPassword = 'Haslo do bazy';
	protected $mysqlDatabase = 'Baza';
	public function __construct()
	{
 
				try
	{
		$this->pdo = new PDO('mysql:dbname='.$this->mysqlDatabase.';host='.$this->mysqlHost.';', $this->mysqlLogin, $this->mysqlPassword);
		$this->pdo->setAttribute(PDO::ATTR_DEFAULT_FETCH_MODE, PDO::FETCH_ASSOC);
 
	}
	catch(PDOException $exception) 
	{
		echo $exception->getMessage();
	}
	}
 
	public function prepare($sql)
	{
		return $this->pdo->prepare($sql);
	}
 
}
[PHP] pobierz, plaintext 

Pomijając kwestię iż trzymam dane do połączenia w klasie, czy ma mniej więcej tak to wyglądać ? Ciekawi mnie dlaczego musiałem zdefiniować funkcję prepare ... jak jej nie dopiszę to wywala mi później że nie istnieje ;x

Klasa User:

[PHP] pobierz, plaintext 
class User
{	
	private $login;
	private $query;
	public function __construct($baza)
	{
 
		$this->pdo = $baza;
 
	}
 
	public function selectUser($user)
	{
		$this->login = $user;
 
		$this->query = $this->pdo->prepare('SELECT * FROM users WHERE login =:login');
		$this->query->bindParam(':login', $this->login, PDO::PARAM_STR);
		$this->query->execute();
 
		if($this->query->rowCount() > 0)
		{
 
		$row=$this->query->fetch();
		$this->nazwa = $row['data'];
		}else{echo ' dupa ';}
	}
 
 
 
	public function getUserData()
	{
 
		return $this->nazwa;
 
	}
 
} 
[PHP] pobierz, plaintext 

Kod napisany tylko żeby sprawdzić czy się uruchamia.

A wywołuje metody tak:

[PHP] pobierz, plaintext 
$user = new User(new DataBaseConnection());
 
$user->selectUser('jakisUser');
echo $user->getUserData();
[PHP] pobierz, plaintext 

Ogólnie działa, no chyba że skasuje metode prepare w klasie DataBaseConnection to wtedy już nie ;p Tylko pytanie jak już wcześniej napisałem czy dobrą drogą idę ?

[Damonsson]

Jeżeli to jest po prostu zwykły model, to musisz mieć połączenie z bazą, więc po wymuszać przekazywanie czegoś, bez czego jak nie przekażesz to nie będzie działać? A skoro musisz to przekazać to takie użycie DI, jest wg mnie średnim pomysłem.
Dokładnie, miałbyś je po prostu dziedziczyć, użyć Singleton i tyle. Czasami ludzie próbują na siłę doszukiwać się problemów, z tymi wadami Singleton.

Ten post edytował Damonsson 5.01.2015, 22:23:45