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]

1. Metody, które i tak zawsze pobierają coś z bazy danych, nie wyobrażam tam sobie nagle czegoś niezwiązanego z bazą danych. Wtedy trzeba by to rozdzielić. No tak, a jeżeli będę dziedziczył (wg mojej teorii), to muszę sprawdzić czy już nie istnieje połączenie dlatego Singleton.

2. Dalej w związku z tym jest tworzenie połączenia itd. ale właśnie samo getInstance to zwykły Singleton.

[PHP] pobierz, plaintext 
/**
     * Returns an instance of this class
     * (this class uses the singleton pattern)
     *
     * @return Doctrine_Manager
     */
    public static function getInstance()
    {
        if ( ! isset(self::$_instance)) {
            self::$_instance = new self();
        }
        return self::$_instance;
    }
[PHP] pobierz, plaintext 

3. Za każdym razem musisz się martwić, żeby przekazać to połączenie do obiektu. Tworząc odpowiednio skonkretyzowany model, wiem, że będzie on korzystał z bazy danych i w konstruktorze, tworzę sobie to połączenie od razu. Tworząc obiekt nie muszę pamiętać, aby przekazać tam połączenie z bazą.