Wykorzystanie atrybutu innej klasy, Próba połączenia z bazą MySQL Opcje

[Kmaciek]

Cześć wszystkim,

Zaczynam zabawę z obiektowym PHP i nadziałem się na pierwszy problem, z którym nie mogę sobie poradzić (google też nie pomogło ani przeszukanie forum). Otóż na mojej stronie mam załadowaną główną klasę, która łączy się z bazą danych:

[PHP] pobierz, plaintext 
class Retail_Core
{
	public $Template, $Auth, $Database;
	function __construct()
	{
		try
		{
			$this->Database = new PDO('mysql:host=localhost;dbname=retail;charset=utf8', 'root', '');
			$this->Database->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
			include(APP_PATH . "model/template.php");
			$this->Template = new Template();
			include(APP_PATH . "model/auth.php");
			$this->Auth = new Auth();
			session_start();
		}
		catch(PDOException $exception)
		{
			echo 'Blad polaczenia z baza danych';
   		exit();
		}
	}
}
[PHP] pobierz, plaintext 

Następnie tworzony jest obiekt

[PHP] pobierz, plaintext 
$Retail = new Retail_Core();
[PHP] pobierz, plaintext 

a zaraz po nim mam zaincludowaną drugą klasę, która ma za zadanie wyciągnięcie informacji z bazy danych:

[PHP] pobierz, plaintext 
class EducationTiles
{
	function getTileTitle($round, $modal)
	{
		try
		{
			$stmt = $this->Retail->Database->prepare('SELECT title FROM education WHERE name=:name AND modal=:modal');
			$stmt->bindValue(':name', $round, PDO::PARAM_INT);
			$stmt->bindValue(':modal', $modal, PDO::PARAM_INT);
			$stmt->execute();
			return $stmt->fetch(PDO::FETCH_ASSOC);
			$stmt->closeCursor();
			unset($stmt);
		}
		catch(PDOException $exception)
		{
			echo 'Blad polaczenia z baza danych';
   		exit();
		}
	}
}
[PHP] pobierz, plaintext 

Niestety w linii gdzie przygotowywane jest zapytanie (linia nr 7) dostaję błędy:
Notice: Undefined property: EducationTiles::$Retail
Notice: Trying to get property of non-object
Fatal error: Call to a member function prepare() on null

Będę wdzięczny za podpowiedź jak to poprawnie wykonać by śmigało

[Wazniak96]

Nie zachowujesz w klasach zasady chermetyzacji danych(i pewnie innych z resztą tez ale się nie przyglądalem). Poczytaj o pradygmatach programowania obiektowego oraz o setterach i getterach. To tak z podstawowych pojęć.

[viking]

$this->retail odnosi się do nulla bo nigdzie nie ma definicji ani wskazania czym to ma być. Poza tym za dużo tego wszystkiego. Jak już to coś w stylu:

[PHP] pobierz, plaintext 
class Retail_Core {
protected $database;
protected $template;
 
public function setTemplate(TemplateInterface $template) {
    $this->template = $template;
}
[PHP] pobierz, plaintext 

Ale powinieneś dążyć do minimalizmu. Zobacz sobie przykłady aplikacji zendowych https://github.com/zendframework/zf3-web, https://github.com/zf3buch/zendframework-ce...e/chapter_18_01 np jak wygląda fabryka kontrolera https://github.com/zf3buch/zendframework-ce...llerFactory.php Do kontrolera wstrzykiwane są tylko minimalne zależności.

[Kmaciek]

Dzięki za rady, poczytałem trochę o hermetyzacji, getterach, setterach i ogólnie o wzorcu projektowym jakim jest fabryka. Starałem się wyciągnąć patrząc na kod aplikacji zendowskich, ale niestety na tą chwilę dla mnie to czarna magia

Jeżeli dobrze zrozumiałem to powinno się dążyć do tego, żeby atrybuty w klasach nie były publiczne, a jedynie żeby istniała możliwość ich ustawiania czy pobierania poprzez wspomniane gettery i settery. Natomiast co do samego kontrolera, powinien on sterować tym co ma się zadziać, a nie tak jak w moim przypadku nawiązywać już połączenie z bazą danych. Jeżeli moje rozumowanie jest błędne będę wdzięczny za naprowadzenie.

Idąc tym tropem zmodyfikowałem moje klasy tak, że wyglądają one następująco:

1. Sam kontroler tworzy niezbędne klasy, a połączenie z bazą danych zostało przeniesione do nowej klasy. Dodana została metoda do pobierania połączenia z bazą.

[PHP] pobierz, plaintext 
class Retail_Core
{
  protected $Template, $Auth, $Database;
	function __construct()
	{
    $this->setClass('Template');
    $this->setClass('Auth');
    $this->setClass('Database');
	}
  private function setClass($class)
  {
    $this->$class = new $class();
  }
  public function getDatabase()
  {
    return $this->Database->getDatabase();
  }
}
[PHP] pobierz, plaintext 

2. Wspomniana klasa "Database" wygląda na tą chwilę następująco:

[PHP] pobierz, plaintext 
class Database
{
	protected $Connect;
	function __construct()
	{
		try
		{
			$this->Connect = new PDO('mysql:host=localhost;dbname=retail;charset=utf8', 'root', '');
			$this->Connect->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
		}
		catch(PDOException $exception)
		{
			echo 'Blad polaczenia z baza danych';
			exit();
		}
	}
	public function getDatabase()
	{
		return $this->Connect;
	}
}
[PHP] pobierz, plaintext 

3. Wszystko zostało poprzedzone autoloaderem:

[PHP] pobierz, plaintext 
function __autoload($className)
{
  require(APP_PATH.'model/'.$className.'.php');
}
[PHP] pobierz, plaintext 

Przyznam się, że trochę się w tym jeszcze gubię i dalej nie wiem w jaki sposób wykorzystać w innej klasie wspomniane połączenie z bazą danych. Oczywiście siedzę dalej nad lekturą w postaci internetu, ale wszelki wskazówki będą mile widziane