Discussion :

[laurentSc]

Bonjour,

j'ai ce message d'erreur :

Fatal error: Cannot redeclare class MyPdo in C:\wamp\www\eis\eis\calendar-pdo\includes\class_MyPdo.php on line 34

Ca survient dans ma connexion à la bdd MySQL et pourtant, je prends des précautions pour ne la déclarer qu'une seule fois :

Code php :

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

1
2
3
4
5
6
7
8
9
10
11
12
<?php
include_once (dirname(__DIR__).'\\eis\new\class_MyPdo.php');
header('Content-type: text/html; charset=UTF-8');
//// connexion
try{
				$bdd_planning = new MyPdo("planning_dev");
			}
catch(PDOException $e){
				echo "argggggggggggggg".$e->getMessage();
				return(FALSE);
			}
?>

donc comment faire ?

EDIT (16h53) : il y a bien un moyen de supprimer l'erreur, c'est de commenter l'include du fichier qui fait la connexion, mais du coup la connexion n'est pas faite...et comme j'ai pensé que la connexion pouvait être faite ailleurs, j'ai déclaré en global la variable de connexion (global $bdd;), mais ça ne suffit visiblement pas...

[Invité]

Bonjour,

une piste :

Code :

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

1
2
3
if(empty($bdd)){
	include (dirname(__DIR__).'\\eis\new\class_MyPdo.php');
}

Question : pourquoi créer ce nouvel objet $bdd_planning ?
Il me semble que $bdd devrait suffire, non ?

N.B. question subsidiaire : pourquoi les anti-slashes ?
Ça ne fonctionne pas avec de simples slashes ?

[laurentSc]

Je viens d'essayer la piste :

tout d'abord, il y a 2 variables ($bdd et $bdd_planning) car il y a 2 bdd...

Les 2 variables sont bien NULL , donc du coup, on exécute les 2 includes. Confirmation :

Code php :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
if(empty($bdd)){
	echo "include pour bdd<br/>";
	include_once (dirname(__DIR__).'//eis/new/class_MyPdo.php');
}
 
et
 
if(empty($bdd_planning)){
	echo "include pour bdd_planning<br/>";
	include_once (dirname(__DIR__).'//eis/new/class_MyPdo.php');
}

et elle marche

[rawsrc]

Bonjour,

Si tu t'y mets à la POO, utilise donc les principes de base cela devrait te simplifier la vie : l'autoloading qui te soulage d'aller manuellement piocher tes classes, les espaces de noms qui te permettent de tout ranger proprement et au final le mot-clé use qui te permet de gérer tes dépendances de classes de manière aussi propre.

Au pire, tu as la possibilité de lister les classes déjà chargées par PHP avec la fonction get_declared_classes

[laurentSc]

Mon code est principalement procédural, mais certes, il y a un peu de POO (comme l'utilisation de PDO) ; pour utiliser l'autoload, j'ai fait une petite modif : renommer le fichier où est défini la classe pour qu'il porte le même nom que la classe, et qui soit sous la forme class_nomclasse.php. Et dans le fichier initial, j'ai rajouté ce code :

Code php :

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

1
2
3
4
5
function my_autoloader($class) {
    include 'class_'.$class.'.php';
}
 
spl_autoload_register('my_autoloader');

mais

Fatal error: Class 'class_MyPdo' not found in C:\wamp\www\eis\eis\connect_mysql.php on line 8

[laurentSc]

Bonsoir, je viens de décocher car je voudrais faire marcher l'autoload ; ce code a déjà fonctionné dans un autre projet ; pourquoi pas ici ?

[kalimukti]

Bonsoir, je viens de décocher car je voudrais faire marcher l'autoload ; ce code a déjà fonctionné dans un autre projet ; pourquoi pas ici ?

Avec ton code d'autoload, ton fichier où ta classe PDO est enregistrée doit être class_class_MyPdo.php. Est-ce le cas ? (je ne sais pas pourquoi mais je doute que ce soit le cas ;p)

[laurentSc]

Tu as raison de douter et j'ai mis du temps à comprendre comment tu faisais cette déduction. Ayant fini par comprendre, j'ai corrigé mon code :

Code php :

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

1
2
3
4
5
function my_autoloader($class) {
	var_dump($class);
    //include 'class_'.$class.'.php';
	include $class.'.php';
}

mais même message d'erreur et 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
13
14
<?php
/*if(empty($bdd)){
	include_once (dirname(__DIR__).'\\eis\new\class_MyPdo.php');
}*/
header('Content-type: text/html; charset=UTF-8');
//// connexion
try{
				$bdd = new class_MyPdo("su_dev_eis");
  }
catch(PDOException $e){
				echo "argggggggggggggg".$e->getMessage();
				return(FALSE);
			}		
?>

[kalimukti]

Tu as raison de douter et j'ai mis du temps à comprendre comment tu faisais cette déduction. Ayant fini par comprendre, j'ai corrigé mon code :

Code php :

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

1
2
3
4
5
function my_autoloader($class) {
	var_dump($class);
    //include 'class_'.$class.'.php';
	include $class.'.php';
}

Ca serait plutôt l'inverse:

Code php :

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

1
2
3
function my_autoloader($class) {
    include 'class_'.$class.'.php';
}

Code :

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

$bdd = new MyPdo("su_dev_eis");

Ainsi, l'autoload va chercher la classe MyPdo (qui doit être déclarée avec exactement la même casse dans ton code, pour qu'il la trouve)
dans le fichier class_MyPdo.php'

ensuite, autre difficultée potentielle dans cette ligne:

Code :

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

include 'class_'.$class.'.php';

dans quel fichier va-t-il chercher tes classes ? tel que c'est dans ton code, ici, il va les chercher à la racine de ton site...

moi je travaille plut$ot avec un code du style:

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
 
     set_include_path(PATH_SEPARATOR.dirname(dirname(__FILE__)));
    //echo(get_include_path());
    spl_autoload_register(function ($classe) {
        $possibilities = array(
            "DATA/".$classe . '.class.php',
            "UTILS/".$classe . '.class.php',
            "IHM/".$classe . '.class.php',
            "XML/".$classe . '.class.php'
        );
        $i=0;
        foreach ($possibilities as $file) {
            if (file_exists(dirname(dirname(__FILE__)).'/'.$file)) {
                require_once($file);
                return true;
            }
            $i++;
        }
        return false;
});

où ce code est à la racine de mon dossier "library" où je mets mes librairies de code....
(ensuite j'appelle l'autoload dans les fichiers où je me sers de mes classes)

[laurentSc]

et zut, une longue réponse perdue

je crois avoir suivi toutes tes indications mais

Fatal error: Class 'MyPdo' not found in C:\wamp\www\eis\eis\connect_mysql.php on line 8

Etat actuel du code :

index.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
error_reporting(E_ALL ^ (E_NOTICE|E_WARNING|E_DEPRECATED));
header('Content-type: text/html; charset=UTF-8');
session_start();
function my_autoloader($class) {
    include 'new/class_'.$class.'.php';
}
 
spl_autoload_register('my_autoloader'); 
 
 
include ((__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
13
14
<?php
/*if(empty($bdd)){
	include_once (dirname(__DIR__).'\\eis\new\class_MyPdo.php');
}*/
header('Content-type: text/html; charset=UTF-8');
//// connexion
try{
				$bdd = new MyPdo("su_dev_eis");
  }
catch(PDOException $e){
				echo "argggggggggggggg".$e->getMessage();
				return(FALSE);
			}		
?>

new/class_MyPdo.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
<?php
error_reporting(E_ALL ^ E_DEPRECATED ^ E_NOTICE);
ini_set('display_errors','1'); 
 
// MyPDO
class MyPdo extends PDO {
 
	protected $DB_NAME;
	protected $HOST = "localhost";
	protected $USER ="root";
	protected $PASS ="";
 
 
 
	function __construct($db_name) {	
 
 
	$this->DB_NAME = $db_name;
	parent::__construct('mysql:host=' . $this->HOST . ';dbname=' . $this->DB_NAME,$this->USER,$this->PASS);
 
	}
 
}
?>

[papajoker]

tu pourrais déjà utiliser un chemin absolu dans ta fonction my_autoloader() et simplement utiliser un écho pour contrôler ce que php ne trouve pas.

Code :

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

1
2
3
4
function my_autoloader($class) {
    echo "chargement du fichier: ".__DIR__.'/eis/new/class_'.$class.'.php';
    include __DIR__.'/eis/new/class_'.$class.'.php';
}

Fatal error: Class 'MyPdo' not found in C:\wamp\www\eis\eis\connect_mysql.php on line 8

2 fois /eis/eis/ ? du coup je ne suis même pas sur ou se trouve la racine et ton fichier index.php et donc du : "__DIR__.'/eis/new/"

Vu dans ton premier post :

include_once (dirname(__DIR__).'\\eis\new\class_MyPdo.php');

dirname(__DIR__) est voulu ? cela retourne le répertoire parent et non celui du fichier !

[kalimukti]

Euh, chez moi ton code fonctionne, si je remplace :

Code :

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

include ((__DIR__).'\\connect_mysql.php');

par :

Code :

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

include ('connect_mysql.php');

à aucun moment je n'ai le message d'erreur dont tu parles...

[laurentSc]

En fait, ça régresse, car même sans essayer l'autoloader, plus moyen de se connecter à la bdd : http://www.developpez.net/forums/d15...r/#post8406526. Donc déjà résoudre ce problème avant de faire marcher l'autoloader...

[laurentSc]

Ce soir, tout fonctionne, donc voici le code qui marche :
autoloader dans index.php (fichier principal) :

Code php :

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

1
2
3
4
5
6
7
function my_autoloader($class) {
	    //echo "chargement du fichier (auto_loader) : ".__DIR__.'\new\class_'.$class.'.php';
    include __DIR__.'\new\class_'.$class.'.php';
   // include 'class_'.$class.'.php';
}
 
spl_autoload_register('my_autoloader');

et bien sûr, commenter l'inclusion de la classe dans les 2 fichiers de connexion. Par exemple :

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
if(empty($bdd_planning)){
	//include_once (dirname(__DIR__).'\\eis\new\class_MyPdo.php');
}
header('Content-type: text/html; charset=UTF-8');
//// connexion
try{
				$bdd_planning = new MyPdo("planning_dev");
			}
catch(PDOException $e){
				echo "argggggggggggggg".$e->getMessage();
				return(FALSE);
			}
?>

[laurentSc]

J'avais de la chance hier soir car ce matin, ça ne marche plus et rien changé au code de l'autoloader mais pour que la classe soit trouvée, il faut que je décommente l'inclusion...

[laurentSc]

Comme le titre ne convient plus du tout et que pas moyen de le modifier, je clos cette discussion.