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ę ?

[marcinq123]

Prawdę mówiąc, po cichu liczyłem na Twoją odpowiedź Crozin ;]

Z racji tego iż chcę nabrać dobrych nawyków, spróbuje podejść kompleksowo.

4. Musiałeś zdefiniować dla klasy DataBaseConnection metodę prepare bo dlaczego niby miałaby ona istnieć sama z siebie? Obiekt PDO udostępnia prepare(), query(), beginTransaction() i masę innych, nie DataBaseConnection.

No właśnie zdefiniowałem tylko prepare() a np. execute(), bindParam() już nie musiałem bo działało ... i dlatego nie rozumiem czemu.

Pomijam tutaj już fakt, że wybieranie danych z bazy i ich reprezentacja w formie jednego obiektu klasy User jest bardzo złym podejściem. Powinieneś mieć raczej jeden obiekt przeznaczony do reprezentowania użytkownika (User) i drugi do wybierania takowych np. z bazy danych, czyli tzw. "model" (UserRepository).

Cenna uwaga, ale napisałem przykład z pierwszego postu tylko żeby sprawdzić czy w ogóle to działa.

[PHP] pobierz, plaintext 
// Klasa DataBaseConnection w ogóle do kosza.
 
class User
{
    private $pdo;
 
    private $name;
 
    public function __construct(PDO $pdo)
    {
        $this->pdo = $pdo;
    }
 
    public function select($login)
    {
        $stmt = $this->pdo->prepare('SELECT ... ');
        $stmt->bind(..., $login);
        $stmt->execute();
 
        if ($stmt->rowCount() === 0) {
            throw new ...Exception('...');
        }
 
        $data = $stmt->fetch();
 
        $this->name = $data['name'];
    }
 
    public function getName()
    {
        return $this->name;
    }
 
//oraz inne metody
}
[PHP] pobierz, plaintext 

Czyli teraz odwołuje się tak ?:

[PHP] pobierz, plaintext 
$user = new User(new PDO('mysql:dbname=base;host=host;', 'user', 'password'));
$user->select('user');
echo $user->getName();
[PHP] pobierz, plaintext 

Moje pytanie brzmi, dajmy na to miałbym klasy User ... Admin, Auth, etc. ( nie wiem co tam jeszcze wymyśle (IMG:style_emoticons/default/tongue.gif) )
Każdy obiekt generuje ( w razie potrzeby połączenia z bazą) poprzez

[PHP] pobierz, plaintext 
$admin= new Admin(new PDO('mysql:dbname=base;host=host;', 'user', 'password'));
//jakieś wywoływane metody
[PHP] pobierz, plaintext 

A czy takie podejście żeby się nie powtarzać ma sens?

[PHP] pobierz, plaintext 
$connection = new PDO('mysql:dbname=base;host=host;', 'user', 'password'));
$admin= new Admin($connection);
[PHP] pobierz, plaintext 

A do reszty się nie ustosunkuje bo ze wstydu nie wiem nawet co powiedzieć (IMG:style_emoticons/default/oneeyedsmiley02.png)