Forum PHP.pl

Witam wszystkich

Wreszcie zabrałem się za PHP OOP i bibliotekę PDO i mam pewien problem.
Chodzi o to, że napisałem sobie dwie klasy i w nich odpowiednie metody. Mam pewien problem z racji tego, że nie wiem jak dostać się do zmiennej $pdo z poprzedniej metody która odpowiada za połączenie z bazą. Zresztą poniżej pokaże jak to napisałem. Dodatkowo chciałbym prosić o mały komentarz na temat tego czy dobrze rozumiem OOP i PDO

Z góry dziękuje:

[PHP] pobierz, plaintext 
<?php
class base
{
	private $host;
	private $dbname;
	private $user;
	private $password;
	protected function connection()
	{
		$host='localhost';
		$dbname='oop';
		$user='root';
		$password='Nanysek93';
		try
		{
			$pdo = new PDO('mysql:host='.$this->host.';dbname='.$this->dbname.'', ''.$this->user.'', ''.$this->password.'');
			$pdo->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
			echo 'Polaczenie zostalo nawiazane!';
		}
		catch(PDOException $ex)
		{
			echo 'Polaczenie nie zostalo nawiazane: '.$ex->getMessage();
		}
	}
	protected function answer()
	{
		try
		{
			$stmt=$pdo->query('SELECT * FROM pdo');
			while($row=$stmt->fetch())
			{
				echo ''.$row['id'].': '.$row['nazwa'].'<br />';
			}
			$stmt->closeCursor();
		}
		catch(PDOException $ex)
		{
			echo 'Dane nie zostaly pobrane: '.$ex->getMessage();
		}
	}
}
class action extends base
{
	public function callConnection()
	{
		return $this->connection();
	}
	public function callAnswer()
	{
		return $this->answer();
	}
}
$action = new action();
$action->callConnection();
$action->callAnswer();
?>
[PHP] pobierz, plaintext 

Problem widnieje w linii 29 ponieważ nie mam dostępu do zmiennej $pdo z poprzedniej metody i nie bardzo wiem jak się do niej dostać.

Akurat jak stworzyć konstruktor to wiem

To zastosuj to w praktyce i będzie działać.

Doprowadziłem do czegoś takiego, lecz dalej źle.

[PHP] pobierz, plaintext 
<?php
class base
{
	private $host=null;
	private $dbname=null;
	private $user=null;
	private $password=null;
	public function __construct($host, $dbname, $user, $password)
	{
		$this->host=$host;
		$this->dbname=$dbname;
		$this->user=$user;
		$this->password=$password;
	}
	protected function connection()
	{
		try
		{
			$this->pdo=new PDO('mysql:host='.$this->host.';dbname='.$this->dbname.'', ''.$this->user.'', ''.$this->password.'');
			$this->pdo->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
			echo 'Polaczenie zostalo nawiazane!';
		}
		catch(PDOException $ex)
		{
			echo 'Polaczenie nie zostalo nawiazane: '.$ex->getMessage();
		}
	}
	protected function answer()
	{
		try
		{
			$this->pdo->query('SELECT * FROM pdo');
			while($row=$stmt->fetch())
			{
				echo ''.$row['id'].': '.$row['nazwa'].'<br />';
			}
			$stmt->closeCursor();
		}
		catch(PDOException $ex)
		{
			echo 'Dane nie zostaly pobrane: '.$ex->getMessage();
		}
	}
}
class action extends base
{
	public function __construct()
	{
		parent::__construct('localhost', 'oop', 'root', 'Nanysek93');
	}
	public function callConnection()
	{
		return $this->connection();
	}
	public function callAnswer()
	{
		return $this->answer();
	}
}
$action = new action();
$action->callConnection();
$action->callAnswer();
?>
[PHP] pobierz, plaintext 

W sumie można powiedzieć, że trochę z powietrza:
ale to tylko notice.

Dzięki wielkie za pomoc
Masz racje nospor

Kod wygląda teraz tak:

[PHP] pobierz, plaintext 
<?php
class base
{
	private $host=null;
	private $dbname=null;
	private $user=null;
	private $password=null;
	public function __construct($host, $dbname, $user, $password)
	{
		$this->host=$host;
		$this->dbname=$dbname;
		$this->user=$user;
		$this->password=$password;
	}
	protected function connection()
	{
		try
		{
			$this->pdo=new PDO('mysql:host='.$this->host.';dbname='.$this->dbname.'', ''.$this->user.'', ''.$this->password.'');
			$this->pdo->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
			echo 'Polaczenie zostalo nawiazane!';
		}
		catch(PDOException $ex)
		{
			echo 'Polaczenie nie zostalo nawiazane: '.$ex->getMessage();
		}
	}
	protected function answer()
	{
		try
		{
			$stmt=$this->pdo->query('SELECT * FROM pdo');
			while($row=$stmt->fetch())
			{
				echo ''.$row['id'].': '.$row['nazwa'].'<br />';
			}
			$stmt->closeCursor();
		}
		catch(PDOException $ex)
		{
			echo 'Dane nie zostaly pobrane: '.$ex->getMessage();
		}
	}
}
class action extends base
{
	public function callConnection()
	{
		return $this->connection();
	}
	public function callAnswer()
	{
		return $this->answer();
	}
}
$action = new action('localhost', 'oop', 'root', 'Nanysek93');
$action->callConnection();
$action->callAnswer();
?>
[PHP] pobierz, plaintext 

Powiedzcie co jeszcze do poprawy. Działać już działa jak trzeba, ale może da się coś jeszcze poprawić, żeby lepiej spełniało swoją funkcjonalność?

Jeśli funkcjonalnością tej klasy jest pobrać wszystkie rekordy z tabeli pdo to ją spełnia.

Dzięki wielkie wszystkim za pomoc
Myślę, że temat wyczerpany. Przynajmniej wiem na co zwracać uwagę.
Zabieram się za dalsze pisanie