Forum PHP.pl

Witam,

Mam pytanie. Jak wcielić zapytania PDO do własnej klasy. Jak to jest realizowane.
Czy takie używanie jak poniżej jest poprawne, czy realizuje się to inaczej?

[PHP] pobierz, plaintext 
<?php
class Strona {
	public $url;
 
	public function PobierzMetaTagi() {		
		try {
			$db = new PDO('mysql:host='.DB_HOST.';dbname='.DB.';charset=utf8', DB_USER, DB_PASS);
			$db->exec("set names utf8");
 
			$wynik = $db->prepare("SELECT title, description, keywords FROM ".DB_PREFIX."strony WHERE url=:url LIMIT 1");
			$wynik -> execute(array(
				':url'=>$this->url
			));
			$rekord = $wynik->fetch(PDO::FETCH_ASSOC);
 
			print '<title>'.$rekord['title'].'</title>';
			print '<meta name="description" content="'.$rekord['description'].'" />';
			print '<meta name="keywords" content="'.$rekord['keywords'].'" />';
 
			$db = null;
 
		} catch (PDOException $e) {
			throw new Exception('Could not connect to database');
		}	
	}
}
?>
[PHP] pobierz, plaintext 

Moje wątpliwości głównie są takie, że przy takim użyciu w każdej funkcji muszę łączyć się z baza. Mało to wygodne.
Może lepiej nawiązać połączenie w ciele strony i przekazać do klasy ma zasadzie "$strona->db = $db"?
A może jeszcze inaczej?

Ten post edytował jol.us_ 20.02.2016, 21:24:05

tworzysz klase database w ktorej laczysz sie z baza i zwracasz polaczenie,
w construct klasy w ktorej chcesz uzywac owego polaczenia robisz cos w stylu

$this->_db = Database::Connect();

nastepnie wywolujesz ja w sposob
$this->_db->prepare
$this->_db->execute()

itp

A takie rozwiązanie jest ok?

klasa

[PHP] pobierz, plaintext 
class Strona {
	private $url;
	private $db;
 
	public function __construct($url, $db) {
		$this->_url = $url;
		$this->_db = $db;
	}
 
	public function PobierzMetaTagi() {		
 
		$wynik = $this->_db ->prepare("SELECT title, description, keywords, robots FROM ".DB_PREFIX."strony WHERE url=:url LIMIT 1");
		$wynik -> execute(array(
			':url'=>$this->_url
		));
		$rekord = $wynik->fetch(PDO::FETCH_ASSOC);
 
		print '<title>'.$rekord['title'].'</title>';
		print '<meta name="description" content="'.$rekord['description'].'" />';
		print '<meta name="keywords" content="'.$rekord['keywords'].'" />';
		print '<meta name="robots" content="'.$rekord['robots'].'" />';
 
	}
}
[PHP] pobierz, plaintext 

wywolanie

[PHP] pobierz, plaintext 
try {
	$db = new PDO('mysql:host='.DB_HOST.';dbname='.DB.';charset=utf8', DB_USER, DB_PASS);
	$db->exec("set names utf8");
} catch (PDOException $e) {
    throw new Exception('Could not connect to database');
}
 
$strona = new Strona('url', $db);
$strona->PobierzMetaTagi();
[PHP] pobierz, plaintext 

Zaczynam dopiero z PHP obiektowo i trochę kombinuje po omacku.
Pełno jest tutków w sieci na temat programowania obiektowego, ale takiego z uzyciem PDO to nie znalazlem.

Ten post edytował jol.us_ 20.02.2016, 21:55:43

Teraz już bardziej rozumiem. Dziękuję

jol.us_ tak twoje jest ok, a tego wyżej nie słychaj tyle że nie _db itp bo masz $db, zresztą underscore nie jest już zalecane

Tylko znacznie lepiej jest odseparować html od logiki aplikacji

goartur singleton to antywzorzec, jak możesz to nie używaj

Czemu nie jest zalecane?
I o co chodzi z singleton?

o psr słyszałeś?
No to co pokazałeś to przecież wzorzec singleton

Nie wiem jak to sie nazywa dlatego, kiedys sie tego nauczylem i uzywalem,
chodzi mi bardziej dlaczego nie uczywac _ w prywatnych zmiennych

com Ci napisał - PSR

Service Manager przez wielu też jest uważany za antywzorzec a jednak jest obecny we wielu frameworkach. Skoro singleton się sprawdza to znaczy że jest poprawny