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ć.

Od góry do dołu źle. Przydałby się konstruktor (przyjmujący jako argumenty dane połącznia). Kolejna sprawa, używasz wewnątrz metod zmiennych lokalnych, a nie właściwości klasy. Więc nie ma prawa działać, nie przekazuje uchwytu do połączenia itd. Musisz się odwoływać poprzez $this -> zmienna. Przykład prawidłowego użycia:

[PHP] pobierz, plaintext 
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();
}
}
[PHP] pobierz, plaintext 

Akurat jak stworzyć konstruktor to wiem

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

Kolega wyżej podał ci rozwiązanie

[PHP] pobierz, plaintext 
[b]$this -> pdo[/b] = new PDO(
[PHP] pobierz, plaintext 

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 

while($row=$stmt->fetch())

A te $stmt to ci sie z powietrza bierze??

Wlacz wyswietlanie bledow i je analizuj a jak nie potrafisz to chociaz je nam przytaczaj

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

Wykorzystujesz obiekt $stmt, który nie istnieje.

[PHP] pobierz, plaintext 
$stmt = $this->pdo->query('SELECT * FROM pdo');
[PHP] pobierz, plaintext 

Ale ten notice informuje cie ze zmiennej nie ma... kurcze, mysl troche... spojrz na swoj pierwszy kod i zobacz skad brales te $stmt. Potrafisz tylko bezmyslnie kopiuowac? No nie na tym polega programowanie

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