Discussion :

[adc15]

Bonsoir,

J'essaye desespérement depuis quelques temps de mettre en place le design pattern Singleton pour pouvoir me connecter à ma base de donnée

Voici d'abord mon code de Singleton :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
<?php
 
 
//Classe de connexion à la BDD - Design Pattern Singleton
class Connection{
 
	private static $instance;
	private $host = 'localhost';
	private $user = 'root';
	private $password = '';
	private $dbName = 'db354423753';
 
	private function __construct() 
	{	
		try
		{
			self::$instance = new PDO('mysql:host=' . $this->$host . ';dbname=' . $this->$db, $this->$user, $this->$password);
 
			self::$instance->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
		}
		catch (Exception $e)
		{
			die('Erreur de connexion: ' . $e->getMessage());
		}
	}
 
	private function __clone() {}
 
	static public function getInstance() 
	{
		if (! (self::$instance instanceof self) ) 
		  self::$instance = new self();
 
		return self::$instance;
	}
 
}

Et je fais ensuite :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

$bdd = Connection::getInstance()->getBdd();

Et paf un beau message d'erreur :

Fatal error: Cannot access empty property in C:\wamp\www\Site\model\Connection.class.php on line 16

Pourtant j'ai regardé pas mal d'autres code de Singleton et je vois vraiment pas ce qui ne vas pas dans le mien !

Merci de votre aide, et bonne soirée !

[stealth35]

$this->val pas $this->$val

[Shikiryu]

En plus de ce que dit stealth (syntaxe des attributs de l'objet), le pattern n'est pas très bon ici.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
 
static public function getInstance() {
	if (! (self::$instance instanceof self) ) 
		self::$instance = new self();
	return self::$instance;
}

Ici, à chaque fois que vous demandez "l'instance", vous regardez si la variable static "$instance" de l'objet Connection est une instance de l'objet Connection sinon vous le construisez.
Or, dans le constructeur de Connection, vous mettez dans $instance un objet PDO...

Donc à chaque fois que vous appelerez getInstance(), il instanciera un nouveau Connection.

Ce serait plus un truc comme ça :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
class Connection{
 
	private static $instance;
	protected $db; // sera un objet PDO
	private $host = 'localhost';
	private $user = 'root';
	private $password = '';
	private $dbName = 'db354423753';
 
	private function __construct(){	
		try{
			$this->db = new PDO('mysql:host=' . $this->host . ';dbname=' . $this->db, $this->user, $this->password);
			$this->db->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
		}catch(PDOException $e){
			die('error : '.$e->getMessage());
		}
	}
 
	private function __clone() {}
 
	static public function getInstance() 
	{
		if (! (self::$instance instanceof self) ) 
		  self::$instance = new self();
 
		return self::$instance;
	}
 
}

[stealth35]

autant faire juste un instanceof PDO

[Shikiryu]

Moui, mais tu perds la possibilité de surcharger avec tes propres méthodes.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

Connection::getInstance()->getAllArticles();

te donnera une erreur car getAllArticles() n'existe pas pour la classe PDO.

Or, rien ne t'empêche avec ma classe de faire :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
...
public function getAllArticles(){
    $sql = "SELECT * FROM ARTICLES WHERE ETAT = 1";
    $stmt = $this->db->prepare($sql);
    $stmt->execute();
    return $stmt->fetchAll();
}
...

Enfin, là, c'est un exemple un peu simple. Bref, ça créé des raccourcis pas trop mal et permet de surcharger un peu tout ça

[stealth35]

t'utilises SingletonFactory et c'est réglé
https://github.com/stealth35/SingletonFactory

[adc15]

Shikiryu, effectivement je vois le problème de instanceof, je vais changer ça.

stealth, j'ai regardé ton Singleton Factory, mais ça m'a paru bien compliqué

Benjamin, si je fais ta méthode qui consiste, si j'ai bien compris, à mettre une variable dans chaque classe qui contiendra l'instance de PDO, je n'ai plus besoin du singleton, c'est ça ?

Mais bon, c'est le choix de adc15.

Moi j'ai choisi ça parce que j'ai vu que c'était une bonne solution quand on se connectait à un serveur/bdd, après si y'en a d'autres plus adaptées je suis preneur...

(et aucun retour de sa part )

J'essayais de faire marcher mon bazar mais je vous avais pas oublié :p