Discussion :

[laurentSc]

Bonjour,

j'ai créé une classe bdd qui hérite de PDO pour créer une connection. Mon souci est qu'il dit ne pas trouver la classe et je comprends pas.

index.php :

Code php :

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

1
2
3
4
5
6
7
8
9
10
<?php
//autoloading
include_once 'Autoloader.php';
 
use _classes\Bdd\Bdd;
 
$Bd = new Bdd('localhost','support_eis', 'root', '');
 
var_dump($Bdd);
?>

autoloader.php :

Code php :

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
<?php 
 
class Autoloader {
static function register() {
    define('DIR_ROOT', __DIR__.DIRECTORY_SEPARATOR);
 
    $autoloader = function($full_class_name) {
        // on prépare le terrain : on remplace le séparteur d'espace de nom par le séparateur de répertoires du système
        $name = str_replace('\\', DIRECTORY_SEPARATOR, $full_class_name);
        // on construit le chemin complet du fichier à inclure :a
        // il faut que l'autoloader soit toujours à la racine du site
        $path = DIR_ROOT.$name.'.php';
 
        // on vérifie que le fichier existe et on l'inclut
        // sinon on passe la main à une autre autoloader (return false)
        if (is_file($path)) {
            include $path;
            return true;
        } else {
            return false;
        }
    };
    spl_autoload_register($autoloader);
    }
}
 
Autoloader::register();

Bdd.php :

Code php :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
<?php
 
class Bdd extends PDO {
 
    public $Bdd;
 
function __construct($Bddhost, $Bddname, $Bdduser, $Bddpswd) {
 
$this->Bdd = new PDO('mysql:host=' . $this->Bddhost.';
Bddname=' . $this->Bddname ,// virgule ici et pas point-virgule
$this->Bdduser, 
$this->Bddpswd);
    }
}

Message d'erreur :

Fatal error: Uncaught Error: Class '_classes\Bdd\Bdd' not found in C:\wamp64\www\eis_oo2\index.php on line 7

Pourquoi ?

[laurentSc]

J'ai corrigé une erreur dans Bdd.php causée par un "remplacer partout", mais l'erreur reste .

Bdd.php corrigé :

Code php :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
<?php
 
class Bdd extends PDO {
 
    public $Bdd;
 
function __construct($Bddhost, $Bddname, $Bdduser, $Bddpswd) {
 
$this->Bdd = new PDO('mysql:host=' . $this->Bddhost.';dbname=' . $this->Bddname ,// virgule ici et pas point-virgule
$this->Bdduser, 
$this->Bddpswd);
    }
}

dbname et pas Bddname à la ligne 9

[rawsrc]

Salut,

il manque le namespcae Bdd; dans la définition de ta classe Bdd, puis il faut que tu enregistres la classe dans un dossier Bdd
Je voudrais savoir : tu ne comprends pas ce que j'écris dans mon blog ou tu ne le mémorises pas ? Parce que si c'est incompréhensible, c'est que tu ne dois pas être le seul à galérer et je devrais je pense réécrire certaines parties.

[laurentSc]

Merci de me répondre.
En fait, mon fichier Bdd.php est rangé dans le folder _classes\Bdd (tous les sous-répertoires contenant des fichiers définissant des classes sont dans le folder _classes). Donc faut-il mettre namespace Bdd ou namespace _classes\Bdd ?
Mais de toute façon, dans les 2 cas, il trouve pas la classe PDO.

Code php :

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

1
2
3
namespace Bdd;
class Bdd extends PDO {
...etc

message d'erreur :

Fatal error: Class 'Bdd\PDO' not found in C:\wamp64\www\eis_oo2\_classes\Bdd\Bdd.php on line 3

et pour répondre à ta question : on va dire que comme c'est un problème de POO mais pas de MVC, j'ai pensé que je n'y trouverais pas la réponse (Cela dit, si tu ne m'avais pas répondu, je pensais regarder comment tu avais fait). C'est vrai que quand je l'ai lu, j'ai trouvé ça très difficile à comprendre, mais depuis que je me suis plongé dans le bouquin de Bersini (mi août), je n'ai plus regardé ton blog (ni ton code).

[rawsrc]

A priori, vu le code de l'autoloader, cela devrait être

Code php :

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

1
2
3
4
5
namespace _classes\Bdd;
 
class Bdd
extends \PDO
{

Tu as oublié le \ devant PDO,
c'est, soit tu définis la dépendance explicitement et tu peux t'en passer comme ça :

Code php :

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

1
2
3
4
5
6
7
namespace _classes\Bdd;
 
use PDO;  // dépendance
 
class Bdd
extends PDO
{

soit l'autre méthode indiquée juste avant.

[laurentSc]

Il n'y a que la 2e méthode qui marche car avec la première :

Error: Class '_classes\Bdd\PDO' not found in C:\wamp64\www\eis_oo2\_classes\Bdd\Bdd.php on line 10

et Bdd.php :

Code php :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
<?php
namespace _classes\Bdd;
//use PDO;  // dépendance
class Bdd extends \PDO {
 
    public $Bdd;
 
function __construct($Bddhost, $Bddname, $Bdduser, $Bddpswd) {
 
$this->Bdd = new PDO('mysql:host=' . $Bddhost.';dbname=' . $Bddname ,// virgule ici et pas point-virgule
$Bdduser, 
$Bddpswd);
    }
...etc
}

A noter que dans le bouquin de Bersini, jamais vu l'emploi de l'antislash avant le nom de la classe dont on hérite (par contre, il ne parle que de classes qu'on a soit même définies) et jamais vu non plus l'emploi de use.

[laurentSc]

Pourquoi la méthode d'héritage la plus simple ne marche pas ?

[rawsrc]

ben je n'ai aucun souci chez moi, les deux façons de faire fonctionnent impec.
Je ne comprends pas ce que tu cherches à faire, tu dérives de PDO donc l'instance de ta classe Bdd est de même nature que PDO, pourquoi avoir une variable public $bdd à l'intérieur ?

[laurentSc]

Mon but est simplement de créer une instance PDO connectée à la bdd. J'ai changé le statut de la variable. C'est mieux comme ça ?

Bdd.php :

Code php :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
<?php
namespace _classes\Bdd;
use PDO;  // dépendance
class Bdd extends PDO {
 
private $Bdd;
 
function __construct($Bddhost, $Bddname, $Bdduser, $Bddpswd) {
$this->Bdd = new PDO('mysql:host=' . $Bddhost.';dbname=' . $Bddname ,// virgule ici et pas point-virgule
$Bdduser, 
$Bddpswd);
return ($this->Bdd);
}
...
}

[laurentSc]

J'ai encore modifié le code de la classe Bdd donné au post #9. Comme on ne va pas créer 50 instances PDO, j'ai rendu l'attribut statique :

Bdd.php :

Code php :

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
<?php
namespace _classes\Bdd;
use PDO;  // dépendance
class Bdd extends PDO {
 
        static $Bdd;
 
        public function __construct(){}
 
        public function connect($Bddhost, $Bddname, $Bdduser, $Bddpswd) {
        self::Bdd = new PDO('mysql:host=' . $Bddhost.';dbname=' . $Bddname ,// virgule ici et pas point-virgule
        $Bdduser, 
        $Bddpswd);
        return (self::Bdd);
    }
...
}

ensuite je tente une connexion :

Code php :

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

1
2
3
4
//autoloading
include_once dirname(__DIR__).'\Autoloader.php';
 
include_once dirname(__DIR__)."\connect_mysql.php";

connect_mysql.php :

Code php :

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

1
2
3
4
5
6
7
8
9
10
11
12
<?php
use _classes\Bdd;
//// connexion
try{
				$Bdd = new Bdd();
				$Bdd2=$Bdd->connect('localhost','support_eis', 'root', '');
  }
catch(PDOException $e){
				echo "argggggggggggggg".$e->getMessage();
				return(FALSE);
			}		
?>

mais il ne trouve pas la classe Bdd.

Error: Class '_classes\Bdd' not found in C:\wamp64\www\eis_oo2\connect_mysql.php on line 6

Vois-tu un souci ?

[laurentSc]

Je viens de penser à un souci potentiel, et je voudrais ton avis : le mécanisme de point d'entrée unique n'est pas en place (pas réussi). Or, quand je fais use _classes\Bdd; dans connect_mysql.php, on suppose sûrement être à la racine du site, non ? (et ce n'est pas le cas)

[Loralina]

Bonjour,
use _classes\Bdd; correspond dans le cas présent à l'espace de noms et non la classe.

Deux possibilités :

Code :

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

1
2
use _classes\Bdd\Bdd;
$Bdd = new Bdd();

Ou :

Code :

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

1
2
use _classes\Bdd;
$Bdd = new Bdd\Bdd();

https://www.php.net/manual/fr/langua....importing.php

[laurentSc]

Quand tu dis 2 possibilités, ça veut dire l'une ou l'autre ou bien que les 2 sont possibles ?

En tout cas, elles ont le même effet : l'erreur a disparu mais une autre est signalée :

Parse error: syntax error, unexpected '=' in C:\wamp64\www\eis_oo2\_classes\Bdd\Bdd.php on line 11

Bdd.php :

Code php :

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
<?php
namespace _classes\Bdd;
use PDO;  // dépendance
class Bdd extends PDO {
 
    static $Bdd;
 
        public function __construct(){}
 
        public function connect($Bddhost, $Bddname, $Bdduser, $Bddpswd) {
        self::Bdd = new PDO('mysql:host=' . $Bddhost.';dbname=' . $Bddname ,// virgule ici et pas point-virgule
        $Bdduser, 
        $Bddpswd);
        return (self::Bdd);
    }
..
}

tu vois ce qui se passe ?

[Loralina]

Oui, les deux sont possibles.
Pour l'autre erreur : self::$Bdd.

[rawsrc]

Salut,

Juste pour avoir une connexion à la base tu dérives la classe PDO, ok.

Voici comment faire :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
class Bdd
extends \PDO
{
    private static $pdo;
 
    public function __construct()
    {
        if (self::$pdo === null) {
            self::$pdo = new \PDO('mysql:host=localhost;port=3306;dbname=nom_base_de_donnees;charset=utf8', 'root', 'mot_de_passe');
        }
        return self::$pdo;
    }
}

Ainsi tu sers toujours la même connexion même si tu fais appel plusieurs fois à la classe avec new Bdd().
Je tiens à préciser que tu te sers toujours du même canal de connexion par contre les classes instanciées seront différentes (c'est subtil, mais ça n'a pas tellement d'importance)

[rawsrc]

Je viens de penser à un souci potentiel, et je voudrais ton avis : le mécanisme de point d'entrée unique n'est pas en place (pas réussi). Or, quand je fais use _classes\Bdd; dans connect_mysql.php, on suppose sûrement être à la racine du site, non ? (et ce n'est pas le cas)

Tu n'es pas arrivé à faire une redirection sur Apache ?
Du style tout rediriger vers index.php

[laurentSc]

Je me suis inspiré de ton post #15, mais pour pas tout chambouler :

Code php :

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
namespace _classes\Bdd;
use PDO;  // dépendance
class Bdd extends PDO {
 
    private static $Bdd;
 
        public function __construct(){}
 
        public function connect($Bddhost, $Bddname, $Bdduser, $Bddpswd) {
 
            if (self::$Bdd === null) {
                self::$Bdd = new PDO('mysql:host=' . $Bddhost.';dbname=' . $Bddname ,// virgule ici et pas point-virgule
                $Bdduser, 
                $Bddpswd);
            }
            return (self::$Bdd);
    }
...
}

contrairement à ta pratique, j'ai essayé de tout faire avec Wamp, donc pour la redirection, j'ai mis ton code dans un fichier .htaccess à la racine du site :

Code :

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

1
2
3
4
5
6
7
RewriteEngine on
RewriteCond %{REQUEST_URI} /(index)(\..{3,4})?$ [NC]
RewriteRule ^.* http://dev.mvc.fr [R=301,L]
RewriteCond %{REQUEST_FILENAME} !^/css/.*$
RewriteCond %{REQUEST_FILENAME} !^/js/.*$
RewriteCond %{REQUEST_FILENAME} !^/img/.*$
RewriteRule . /index.php [QSA,L]

Ca redirige bien vers http://dev.mvc.fr mais ça affiche une page débile "ce site web est à vendre" etc.