[MySQL][PHP]łączenie się z bazą z klasy Opcje

[dżozef]

w tej chwili, żeby połączyć się z bazą wewnątrz klasy używam:

[PHP] pobierz, plaintext 
    public function fetchAll() {
        try {
            $conn = new PDO('mysql:host='.DB_HOST.';dbname='.DB_NAME, DB_USER, DB_PASS);
            $today = date("Y-m-d");
            $query = $conn->prepare('SELECT[...]
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
    public function insertDomain($domain) {
        try {
            $conn = new PDO('mysql:host='.DB_HOST.';dbname='.DB_NAME, DB_USER, DB_PASS);
            $query = $conn->prepare('INSERT INTO `foo` (`Status`)
                                        VALUES (:status);');
            $query->execute(array(
[PHP] pobierz, plaintext 

i tak w każdej metodzie. coś mi mówi, że nie jest to idealne rozwiązanie, bo za każdym razem tworzone jest nowe połączenie (racja?), w jaki sposób mogę to nieco ulepszyć?

[Beniooo]

Ja zrobiłbym tak:
w jakimś pliku config czy coś dajesz

[PHP] pobierz, plaintext 
$conn = new PDO('mysql:host='.DB_HOST.';dbname='.DB_NAME, DB_USER, DB_PASS);
[PHP] pobierz, plaintext 

Następnie dajesz w pliku z klasą:

[PHP] pobierz, plaintext 
public $conn;
public function __construct($connect){
		$this->conn=$connect;
	}
[PHP] pobierz, plaintext 

I następnie gdy chcesz skorzystać z połączenia w funkcji dajesz:

[PHP] pobierz, plaintext 
$this->conn->prepare('SELECT[...]
[PHP] pobierz, plaintext 

Ten post edytował Beniooo 8.06.2013, 17:26:37

[SmokAnalog]

Takie rozwiązanie jest złe, masz rację. Możesz stworzyć metodę getConnection(), która będzie:

• Tworzyła obiekt PDO, jeśli jeszcze nie został utworzony
• Zwracała obiekt PDO, jeśli już został utworzony

Taki schemat działania nazywa się singleton. Możesz ten obiekt przechowywać w zmiennej statycznej Twojej klasy. I wtedy za każdym razem zamiast na przykład:

[PHP] pobierz, plaintext 
$query = $conn->prepare(...);
[PHP] pobierz, plaintext 

Możesz wywoływać:

[PHP] pobierz, plaintext 
$query = self::getConnection()->prepare(...);
[PHP] pobierz, plaintext 

[dżozef]

@SmokAnalog:
obawiam się, że (jeszcze) nie potrafię zrozumieć tej koncepcji, mógłbyś nieco rozwinąć, lub podać link do przykładu?

[SmokAnalog]

Poczytaj o wzorcu singleton: http://phpedia.pl/wiki/Singleton

[dżozef]

ok, poskładałem to do kupy i mam:

[PHP] pobierz, plaintext 
<?php
// ustawienia bazy danych
define ("DB_HOST", "localhost");
define ("DB_USER", "root");
define ("DB_PASS", "mysql");
define ("DB_NAME", "db");
 
// nie zmieniac nic ponizej
require_once 'classes/db.php';
$db=DB::getInstance(array("host" => DB_HOST, "dbname" => DB_NAME, "username" => DB_USER, "password" => DB_PASS));
 
?>
[PHP] pobierz, plaintext 

następnie classes/db.php:

[PHP] pobierz, plaintext 
<?php
class DB {
    private $db_info = array("host" => "localhost", "dbname" => "myDB", "username" => "myDB_user", "password" => "myDB_password");
    private $dbh;
 
    public static $instance = NULL;
 
    private function __construct(array $db_info = null) {
        if(isset($db_info)) {
            foreach($db_info as $key_name => $key_value) {
                if(!in_array($key_name, array("host", "db_name", "username", "password") || empty($key_value))) {
                    throw new Exception("Invalid key passed!");
                }
                $this->db_info = $db_info;
            }
        }
        $this->dbh = new PDO("mysql:host={$this->db_info['host']};dbname={$this->db_info['db_name']}", $this->db_info['username'], $this->db_info['password'], array(PDO::MYSQL_ATTR_INIT_COMMAND => "SET NAMES utf8"));
        $this->dbh->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
        $this->dbh->setAttribute(PDO::ATTR_EMULATE_PREPARES, true);
        $this->dbh->setAttribute(PDO::ATTR_DEFAULT_FETCH_MODE, PDO::FETCH_ASSOC);
    }
 
    public static function getInstance() {
        if(!isset(self::$instance)) {
            self::$instance = new DB();
        }
        return self::$instance;
    }
}
?>
[PHP] pobierz, plaintext 

jakieś uwagi?

Ten post edytował dżozef 8.06.2013, 19:18:17

[ber32]

Witam. Spróbuj tak span

Po co te atrybuty

[PHP] pobierz, plaintext 
 $this->dbh->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
$this->dbh->setAttribute(PDO::ATTR_EMULATE_PREPARES, true);
$this->dbh->setAttribute(PDO::ATTR_DEFAULT_FETCH_MODE, PDO::FETCH_ASSOC);
[PHP] pobierz, plaintext 

Ten post edytował ber32 8.06.2013, 21:50:49

Powód edycji: [Daiquiri]: Bez pozycjonowania proszę

[dżozef]

Witam. Spróbuj tak klik

przedstawiasz mi zupełnie inne rozwiązanie ani trochę tego nie usprawiedliwiając, dlaczego (i czy) ten sposób jest lepszy?

Ten post edytował Daiquiri 8.06.2013, 20:57:45

Powód edycji: [Daiquiri]:

[ber32]

Choćby ta część jest lepsza i prawidłowa od tej przedstawionej wyżej



Ten post edytował ber32 8.06.2013, 21:51:11

[dżozef]

lepsza/prawidłowa w jaki sposób?
poza tym twój przykład to nie jest singleton

[ber32]

w jaki sposób mogę to nieco ulepszyć?

Rób co chcesz

[dżozef]

zrobię przede wszystkim poprawnie, natomiast jeśli nie chcesz pomagać a tylko spamujesz linkiem do własnej strony to następnym razem sugeruję tego nie robić

[ber32]

Następnym razem już tobie nie będę pomagał. Spamowanie to nie pomaganie. Ciekawe że ty linków nie wstawiasz pomagając. (IMG:style_emoticons/default/thumbsdownsmileyanim.gif)
Dzieciak z ciebie. Medorator się znalazł, a może Administrator ?

Ten post edytował ber32 8.06.2013, 19:41:29

[Daiquiri]

Ber32, padło pytanie o uzasadnienie, zatem w czym widzisz problem?

[ber32]

W niczym. Ja też zadałem pytanie po co te atrybuty i nie otrzymałem odpowiedzi. Problem leży gdzie indziej. Pomaganie nie wymaga wyjaśnianie niczego.

[dżozef]

- te atrybuty nie zmieniają absolutnie istoty pytania
- zamiast rzeczowo wyjaśnić dlaczego twoje rozwiązanie jest lepsze spamujesz linkiem bez żadnego wyjaśnienia
- obrażasz się po moim pytaniu
- atak 'ad hominem' jak już skończyły się argumenty

[ber32]

Ok