[php] klasa php + pdo Opcje

[jol.us_]

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

[goartur]

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

[jol.us_]

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

[goartur]

Cos w tym stylu.
Pokaze ci inny przyklad:
Database.php

[PHP] pobierz, plaintext 
class Connection
{
    private static $instance;
    public static function getInstance(){
 
        if(!self::$instance){
            try {
                self::$instance = new PDO("mysql:host=$host;", 'root', 'haslo');
                return self::$instance;
            }catch (Exception $Error){
                echo 'Error has been detected:';
                die('Error has been detected: ' . $Error->getMessage());
            }
        }
    }
}
[PHP] pobierz, plaintext 

server.php

[PHP] pobierz, plaintext 
 
class Server
 
{
 
    private $_db;
 
    public static $instance;
 
    public function __construct(){
 
        $this->_db = Connection::getInstance();
 
    }
 
    public function selectAbleSkills(){
 
        $query = $this->_db->prepare("SELECT * FROM skill_type");
 
        $query->execute();
 
        return $query;
 
    }
 
    public static function getInstance()
 
    {
        if(self::$instance === null)
        {
            self::$instance =  new self();
        }
        return self::$instance;
    }
 
 
 
}
[PHP] pobierz, plaintext 

Ten post edytował goartur 20.02.2016, 23:10:32

[jol.us_]

Teraz już bardziej rozumiem. Dziękuję

[com]

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 (IMG:style_emoticons/default/smile.gif)

Tylko znacznie lepiej jest odseparować html od logiki aplikacji (IMG:style_emoticons/default/smile.gif)

goartur singleton to antywzorzec, jak możesz to nie używaj (IMG:style_emoticons/default/smile.gif)

[goartur]

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 (IMG:style_emoticons/default/smile.gif)

Tylko znacznie lepiej jest odseparować html od logiki aplikacji (IMG:style_emoticons/default/smile.gif)

goartur singleton to antywzorzec, jak możesz to nie używaj (IMG:style_emoticons/default/smile.gif)

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

[com]

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

[goartur]

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

[kapslokk]

com Ci napisał - PSR

[viking]

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 (IMG:style_emoticons/default/wink.gif)