Tworzenie połączenia z bazą danych Opcje

[pitu]

Mam pytanie, zaczynam dopiero z OOP. Stworzyłem klasę służącą do połączenia z bazą danych.

Klasa:

[PHP] pobierz, plaintext 
<?php
class Db{
	private $dbh;
 
	public function __construct(){
		$host = 'localhost';
		$dbname = '';
		$user = '';
		$pass = '';
 
		try{
            $this->dbh = new PDO("mysql:host=$host;dbname=$dbname", $user, $pass);
            $this->dbh->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
		}
		catch(PDOException $e){
			echo $e->getMessage();
			exit();
		}
    }
 
	public function __destruct(){
		$this->dbh = null;
    }
}
?>
[PHP] pobierz, plaintext 

W innej klasie mam:

[PHP] pobierz, plaintext 
private $db;
 
	public function __constructor(Db $db){
		$this->db = $db;
	}
public function delArticle($id){
		$del = $this->db->exec('DELETE FROM artykuly WHERE id = '.$this->id);
	}
[PHP] pobierz, plaintext 

W pliku głównym:

[PHP] pobierz, plaintext 
$db = new Db;
$article = new Article($db);
echo $article->delArticle(1);
[PHP] pobierz, plaintext 

Niestety otrzymuję błąd z informacą, że $db nie jest obiektem i nie mogę wywołać na jego rzecz metody exec()

[PHP] pobierz, plaintext 
Call to a member function exec() on a non-object...
[PHP] pobierz, plaintext 

[b4rt3kk]

Czy ta inna klasa dziedziczy po Db? Jeśli tak to wystarczy tyle:

[PHP] pobierz, plaintext 
public function delArticle($id){
		$del = $this->dbh->exec('DELETE FROM artykuly WHERE id = '.$this->id);
	}
[PHP] pobierz, plaintext 

i zmienić private $dbh; na protected?

[pitu]

Ok dzięki, mam jednak pytanie czy dało by się obejść bez dziedziczenia po klasie Db? Chciałem przekazać w parametrze obiekt klasy Db. Jest to tylko nauka OOP i myślę, że przy rozbudowie skryptu lepszym rozwiązaniem byłoby właśnie przekazanie obiektu Db niż ciągłe dziedziczenie.

[b4rt3kk]

Zmienna typu private jest dostępna tylko wewnątrz klasy, tak więc musisz dopisać funkcję która ją zwróci: __get().

[pitu]

Zailustruję jak ja myślę, chcąc przekazać obiekt Db jako parametr w innej klasie. Lecz to nie działa.

W pliku głównym tworzę egzemplarz obiektu Db oraz egzemplarz artykulu:

[PHP] pobierz, plaintext 
$db = new Db;
 
$article = new Article($db);
[PHP] pobierz, plaintext 

W klasie Article mam:

[PHP] pobierz, plaintext 
public function __constructor($db){
		$this->db = $db;
	}
public function delArticle($id){
		$del = $this->db->exec('DELETE FROM artykuly WHERE id = '.$this->id);
	}
[PHP] pobierz, plaintext 

Czyli według mojego rozumowania w konstruktorze klasy Article przekazuję new Db które przypisane jest do właściwości $db. Czyli $db powinno być egzemplarzem klasy Db bez jej dziedziczenia. Niestety PHP wywala, że $db nie jest obiektem.

[b4rt3kk]

Bo klasa nie jest obiektem przecież, tylko klasą. (IMG:style_emoticons/default/smile.gif)

[PHP] pobierz, plaintext 
 
public function __constructor(Db $db){
		$this->db = $db->wartoscDbh();
}
[PHP] pobierz, plaintext 

Dopisz do klasy Db:

[PHP] pobierz, plaintext 
public function wartoscDbh() {
  return $this->dbh;
} 
[PHP] pobierz, plaintext 

I tutaj argument funkcji jest zbędny, ponieważ operujesz na obiekcie, a nie na zmiennej, swoją drogą nigdzie nie przypisujesz wartości. Więc albo dodaj do konstruktora $this->id = costam, albo zmień w zapytaniu na $id.

[PHP] pobierz, plaintext 
public function delArticle($id){
		$del = $this->db->exec('DELETE FROM artykuly WHERE id = '.$this->id);
	}
[PHP] pobierz, plaintext 

Ten post edytował b4rt3kk 4.10.2012, 01:15:14

[CuteOne]

@UP klasa to obiekt pełną gębą (IMG:style_emoticons/default/wink.gif)

@pitu

Wiesz w jakim celu używasz klasy Db a nie bezpośrednio PDO? Bo jeżeli ma ona służyć TYLKO połączeniu z bazą to taki kod jest całkowicie bez sensu i może być zastąpiony zwykłą funkcją...

[PHP] pobierz, plaintext 
function connect(){
 
$host = 'localhost';
$dbname = '';
$user = '';
$pass = '';
 
	try{
            $dbh = new PDO("mysql:host=$host;dbname=$dbname", $user, $pass);
            $dbh->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
	}
	catch(PDOException $e){
		echo $e->getMessage();
		exit();
	}
 
return $dbh;
}
 
 
$db = connect();
 
$art = new Article($db);
[PHP] pobierz, plaintext 

Zobacz sobie do Zenda jak tam zaprojektowano obsługę bazy danych i wyciągnij wnioski

Ten post edytował CuteOne 6.10.2012, 23:50:09

[pamil]

@UP klasa to obiekt pełną gębą (IMG:style_emoticons/default/wink.gif)

Klasa nie jest obiektem, obiekt to instancja klasy. Gdyby klasa była obiektem, każda klasa implementowałaby przymusowego Singletona.

[lord2105]

Proponuję wykorzystać Singletona, zastosować go w klasie DB i wykonywać w ten sposób:

[PHP] pobierz, plaintext 
DB::getInstance()->exec('QUERY');
[PHP] pobierz, plaintext 

[phpion]

W klasie Article mam:

[PHP] pobierz, plaintext 
public function __constructor($db){
		$this->db = $db;
	}
public function delArticle($id){
		$del = $this->db->exec('DELETE FROM artykuly WHERE id = '.$this->id);
	}
[PHP] pobierz, plaintext 

Czyli według mojego rozumowania w konstruktorze klasy Article przekazuję new Db które przypisane jest do właściwości $db. Czyli $db powinno być egzemplarzem klasy Db bez jej dziedziczenia. Niestety PHP wywala, że $db nie jest obiektem.

Nie __constructor tylko __construct. Po tej zmianie powinno śmigać.