classe de connexion

Version imprimable

classe de connexion

Bonjour,

je voudrais créer une classe de connection à une bdd en PDO.

Voici mon code :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
class Db {
 
    public $db;
 
    /**
     * Connect to the database
     * @param $dbhost
     * @param $dbname
     * @param $dbuser
     * @param $dbpswd
     */
    function __construct($dbhost, $dbname, $dbuser, $dbpswd){
        $this->db = new PDO('mysql:host='.$dbhost.',dbname='.$dbname.',charset=utf8', $dbuser, $dbpswd);
    }
...
}
 
et  
 
$db=new Db('mysql:host='.DATABASE_HOST.',dbname='.DATABASE_NAME,DATABASE_USER, DATABASE_PASSWORD);

et les messages d'erreur :

Citation:

Fatal error: in C:\wamp64\www\mvc_udemy\_classes\Db.php on line 21

Citation:

PDOException: in C:\wamp64\www\mvc_udemy\_classes\Db.php on line 21

28/06/2019, 13h16
Celira

Laurent, tu as bien conscience que si on ne voit pas la ligne 21 (et les lignes autour) de ta classe Db , on a aucune chance de voir l'erreur. :roll:

Et sinon, ta chance construit la chaine de connexion "mysql=...", donc il est inutile de la passer en paramètre :

Code:

$db=new Db(DATABASE_HOST, DATABASE_NAME, DATABASE_USER, DATABASE_PASSWORD);
28/06/2019, 15h21
laurentSc

Bonjour Celira,

j'ai modifié l'appel (de la ligne 20 ci-dessus) : $db=new Db(DATABASE_HOST,DATABASE_NAME,DATABASE_USER, DATABASE_PASSWORD); et la ligne 21 correspond à la ligne 13 ci-dessus.
Les messages d'erreur n'ont pas changé :aie:

Je ne m'étais pas posé de question sur ce code, l'ayant récupéré sur Github : https://github.com/AxelPariss/DB/blob/master/src/DB.php et ayant été écrit par un formateur...

Salut,

On sépare le dsn par un point-virgule, avant "dbname" et avant "charset=utf8" :

:arrow: Sans paramètres dans le constructeur :

Code:

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
38
39
40
41
42
43
44
45
46
47
 
<?php 
define('DATABASE_HOST','localhost');
define('DATABASE_NAME','....');
define('DATABASE_USER','...');
define('DATABASE_PASSWORD','...');
class Db {
 
        private $db;
	private $dbhost;
	private $dbname;
	private $dbuser;
	private $dbpswd;
 
    /**
     * Connect to the database
     * @param $dbhost
     * @param $dbname
     * @param $dbuser
     * @param $dbpswd
     */
    function __construct(){//sans paramètres ici
		$this->dbhost=DATABASE_HOST;
		$this->dbname=DATABASE_NAME;
		$this->dbuser=DATABASE_USER; 
		$this->dbpswd=DATABASE_PASSWORD;
		try{
			$this->db = new PDO('mysql:host='.$this->dbhost.';dbname='.$this->dbname.';charset=utf8'
			     , $this->dbuser
			     , $this->dbpswd
			     ,array(PDO::ATTR_ERRMODE=>PDO::ERRMODE_EXCEPTION)//s'ils y a des erreurs ou avertissement, on les affichent
                        );
		} catch(Exception $e){
			die('Erreur de connexion à la base :'.$e->getMessage());//si pdo n'a pas pu se connecter à la base, il renvoi une erreur
		}
    }
	public function prepare($query){
		return $this->db->prepare($query);
	}
}  
 
$db=new Db();
$res=$db->prepare('select * from NOMTABLE');
$res->execute();
while($d=$res->fetch()){//affichage des données...
	echo $d['NOM_COLONNE'];
}

:arrow: Avec les paramètres dans le constructeur :

Code:

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
38
39
40
41
42
43
44
45
46
 
<?php 
define('DATABASE_HOST','localhost');
define('DATABASE_NAME','...');
define('DATABASE_USER','...');
define('DATABASE_PASSWORD','...');
class Db {
 
        private $db;
	private $dbhost;
	private $dbname;
	private $dbuser;
	private $dbpswd;
 
    /**
     * Connect to the database
     * @param $dbhost
     * @param $dbname
     * @param $dbuser
     * @param $dbpswd
     */
    function __construct($dbhost,$dbname,$dbuser,$dbpswd){
		$this->dbhost=$dbhost;
		$this->dbname=$dbname;
		$this->dbuser=$dbuser; 
		$this->dbpswd=$dbpswd;
		try{
			$this->db = new PDO('mysql:host='.$this->dbhost.';dbname='.$this->dbname.';charset=utf8'
			, $this->dbuser
			, $this->dbpswd
			,array(PDO::ATTR_ERRMODE=>PDO::ERRMODE_EXCEPTION));
		} catch(Exception $e){
			die('Erreur de connexion à la base :'.$e->getMessage());
		}
    }
	public function prepare($query){
		return $this->db->prepare($query);
	}
}  
 
$db=new Db(DATABASE_HOST,DATABASE_NAME,DATABASE_USER,DATABASE_PASSWORD);
$res=$db->prepare('select * from NOMTABLE');
$res->execute();
while($d=$res->fetch()){//affichage des données...
	echo $d['NOM_COLONNE'];
}

Bonjour Toufik83, j'ai mis les points-virgule là où tu dis :
Code:

1 2 3 4 5 6 function __construct($dbhost, $dbname, $dbuser, $dbpswd){ debug($dbuser);debug($dbspwd); $this->db = new PDO('mysql:host='.$this->dbhost.';dbname='.$this->dbname.';charset=utf8' , $this->dbuser , $this->dbpswd); }
mais j'ai un autre message d'erreur maintenant :

Citation:

Fatal error: Uncaught exception 'PDOException' with message 'SQLSTATE[HY000] [1045] Accès refusé pour l'utilisateur: ''@'@localhost' (mot de passe: NON)' in C:\wamp64\www\dvp\_classes\Db.php on line 24

la ligne 24 étant ici la ligne 5 (et comme je suis en local (Wampserver), j'ai mis user=localhost et mot de passe vide...)

30/06/2019, 19h51
Toufik83

Citation:

Accès refusé pour l'utilisateur: ''@'@localhost'

Tu-es sûr que t'as mis le bon nom d'utilisateur ?

Montre moi tout le code de la classe s'il te plait....
30/06/2019, 19h55
laurentSc

Je m'étais trompé dans mon message : host=localhost et user=root...
30/06/2019, 21h59
Toufik83

Je viens de remarquer que tu as un @ en trop dans l'erreur Accès refusé pour l'utilisateur: ''@'@localhost' .

Vérifies dans php.ini si le paramètre sql.safe_mode est activé, si oui désactive le et redémarre le serveur local puis refais le teste de connexion à la base.

J'ai vérifié ; c'était déjà désactivé. Par contre, j'ai quand même redémarré le serveur et le message d'erreur est différent :

Citation:

Parse error: syntax error, unexpected ';' in C:\wamp64\www\dvp\_classes\Db.php on line 23
Code:

1 2 3 4 5 6 7 function __construct($dbhost, $dbname, $dbuser, $dbpswd){ debug($dbuser);debug($dbspwd); $this->db = new PDO('mysql:host=' . $this->dbhost . '; dbname=' . $this->dbname ; $this->dbuser, $this->dbpswd); }
La ligne 23 correspond ici à la ligne 4.

Code:

1
2
3
 
$this->db = new PDO('mysql:host=' . $this->dbhost . ';
        dbname=' . $this->dbname ;

:aie: c'est normal parce que c'est une erreur de syntaxe.

Correction :

Code:

1
2
3
4
5
 
$this->db = new PDO('mysql:host=' . $this->dbhost.';
                         dbname=' . $this->dbname ,// virgule ici et pas point-virgule
                         $this->dbuser, 
                         $this->dbpswd);

Et si l'option sql.safe_mode est déjà en "Off", eh ben là je ne comprends pas pourquoi tu as deux "@" qui s'affichent dans l'erreur !

Si je rajoute un try/catch

Code:

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
class Db {
 
    public $db;
 
    /**
     * Connect to the database
     * @param $dbhost
     * @param $dbname
     * @param $dbuser
     * @param $dbpswd    
     */
 
function __construct($dbhost, $dbname, $dbuser, $dbpswd) {
   try {     
        debug($dbuser);debug($dbspwd);
        $this->db = new PDO('mysql:host=' . $this->dbhost . ';
        dbname=' . $this->dbname ,
        $this->dbuser , 
        $this->dbpswd);
        }
    catch(Exception $e){
            die('Erreur de connexion à la base :'.$e->getMessage());
        }
    }
...
autres méthodes
}

Citation:

Erreur de connexion à la base :SQLSTATE[HY000] [1045] Acc�s refus� pour l'utilisateur: ''@'@localhost' (mot de passe: NON)

et sans try/catch

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
class Db {
 
    public $db;
 
    /**
     * Connect to the database
     * @param $dbhost
     * @param $dbname
     * @param $dbuser
     * @param $dbpswd    
     */
 
function __construct($dbhost, $dbname, $dbuser, $dbpswd) {
   //try {     
        debug($dbuser);debug($dbspwd);
        $this->db = new PDO('mysql:host=' . $this->dbhost . ';
        dbname=' . $this->dbname ,
        $this->dbuser , 
        $this->dbpswd);
   /* }    
    catch(Exception $e){
            die('Erreur de connexion à la base :'.$e->getMessage());
        }*/
    }

Citation:

Fatal error: in C:\wamp64\www\dvp\_classes\Db.php on line 27
PDOException: in C:\wamp64\www\dvp\_classes\Db.php on line 27

La ligne 27 est ici la ligne 19...

Bonjour,

Ecris le code sur UNE SEULE LIGNE...
Code:

1 2 3 ... $this->db = new PDO('mysql:host=' . $this->dbhost . ';dbname=' . $this->dbname; $this->dbuser, $this->dbpswd); ...
Mais NON : il faut toujours que tu fasses ton "kéké"... :roll:

01/07/2019, 12h52
laurentSc

et j'aurais vu quoi ?
Pas de kéké ; si je mets le code sur plusieurs lignes, c'est que si PHP me signale une ligne erronée, je sais plus facilement où se trouve l'erreur...

Mais avec $this->db = new PDO('mysql:host='.$this->dbhost.';dbname='.$this->dbname;this->dbuser,$this->dbpswd);

Citation:

Parse error: syntax error, unexpected '->' (T_OBJECT_OPERATOR) in C:\wamp64\www\dvp\_classes\Db.php on line 24
01/07/2019, 13h21
Invité

1- c'est quand même fou : tu ne sais même pas faire un copier-coller !

Tu as oublié un $ !!!

2- précédemment, je n'ai fait que "pointer" la syntaxe...
Maintenant, concernant les variables :

Code:

$this->db = new PDO('mysql:host='.$dbhost.';dbname='.$dbname; $dbuser, $dbpswd);

Cela dit,... je n'y connais rien à la POO... :aie:
01/07/2019, 14h08
laurentSc
Pour une fois que tu me reproches de pas avoir copié-collé (et non le contraire) !
Cela dit, j'ai remis le $ (et enlevé les $this-> )
Code:

1 2 3 function __construct($dbhost, $DBNAME, $dbuser, $dbpswd) { $this->db = new PDO('mysql:host='.$dbhost.';dbname='.$DBNAME; $dbuser,$dbpswd); }
Citation:

Fatal error: in C:\wamp64\www\dvp\_classes\Db.php on line 24
PDOException: in C:\wamp64\www\dvp\_classes\Db.php on line 24

:aie:
01/07/2019, 14h16
Invité

Citation:

Envoyé par laurentSc

...(et enlevé les $this-> )...

1- As-tu essayé AVEC ou pas ??? *

2- POURQUOI n'as-tu pas TESTE les codes donnés par Toufik83 ??

3- Question subsidiaire : ça fait COMBIEN de jours / de mois / d'années que tu essaye de faire une classe de connexion en POO ???

* Comme je l'ai dit, "je n'y connais rien à la POO...".

1- j'ai essayé avec et sans

2- j'a copié-collé son code, mais même erreur

3- je sais ; un certain nombre

4- ayant fait du copier-coller, je m'attendais à ce que ça marche, mais du coup, j'ai ressorti mes fiches (je m'étais fait un aide-mémoire pdo) et ça marche cette fois :D

Code:

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
class MyPdo extends PDO {
/* on privilégie les var de classe (statiques) aux constantes : ainsi modifiables par programme */
	static public $DB_NAME = DATABASE_NAME;
 
	static public $HOST = DATABASE_HOST;
 
	static public $USER = DATABASE_USER;
 
	static public $PASS = DATABASE_PASSWORD;
 
 
 
// le constructeur de MyPdo appelle le constructeur de PDO en lui passant ses paramètres	
	function __construct() {
	$pdo_options[PDO::ATTR_ERRMODE] = PDO::ERRMODE_EXCEPTION;
	$pdo_options[PDO::ATTR_EMULATE_PREPARES] = false;//important sur les configs récentes
	$pdo_options[PDO::ATTR_DEFAULT_FETCH_MODE] = PDO::FETCH_OBJ;//pour le mode objet
	$pdo_options[PDO::MYSQL_ATTR_INIT_COMMAND] = "SET NAMES utf8";//pour l'utf-8
 
	parent::__construct('mysql:host=' . MyPdo::$HOST . ';dbname=' . MyPdo::$DB_NAME, MyPdo::$USER, MyPdo::$PASS, $pdo_options);
	}
}
 
 
puis $db = new MyPDO();