Discussion :

[laurentSc]

Bonsoir,

j'effectue une connexion PDO qui semble bonne, mais derrière la méthode prepare est refusée...

Voici mon code :

Code :

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

1
2
3
4
5
	include_once "connect.php"; // ls : on effectue une connexion (_once pour le faire une seule fois)
	if (isset($bdd)) echo "connecté<br/>";
 
		$sql = "SELECT * FROM users";
		$qid = $bdd->prepare($sql);

Comme "connecté" est affiché, pas besoin de montrer connect.php ; donc pourquoi ce message :

Fatal error: Call to a member function prepare() on a non-object in C:\wamp\www\devjupiter\new\authenticate.php on line 22

[Tsilefy]

Est-ce que $bdd est bien une instance de PDO?

que dit var_dump($bdd) ?

[laurentSc]

Je viens de mettre un var_dump ; il apparaît 2 fois ; une fois, bien objet PDO, mais la 2e fois NULL. Or, je fais des include de la fonction qui exécute le prepare ; y a-t-il moyen d'identifier la fonction appelante car il est évident que le problème est là, sauf que je ne sais pas où c'est...

[Tsilefy]

Tu peux poster connect.php ici?

[laurentSc]

connect.php :

Code php :

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

1
2
3
4
5
6
7
8
9
10
11
<?php
header('Content-type: text/html; charset=UTF-8');
 
try{
 
	$bdd=new MyPdo;
}
catch(PDOException $e){
	echo "argggggggggggggg".$e->getMessage();
}
?>

et la classe MyPdo est définie dans 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
25
26
27
28
29
30
31
32
<?php
error_reporting(E_ALL ^ E_DEPRECATED ^ E_NOTICE);
ini_set('display_errors','1'); 
// MyPDO
 // ls : la classe MyPdo hérite de la classe PDO (extends)
class MyPdo extends PDO {
 
	static public $DB_NAME = "su_dev_eis"; // ls : static public, je sais + pourquoi...
 
	static public $HOST = "localhost";
 
	// $USER = $_SERVER["PHP_AUTH_USER"];
 
	//static public $PASS =$_SERVER["PHP_AUTH_PW"];
 
// ls : le constructeur de MyPdo appelle le constructeur de PDO en lui passant ses paramètres	
	function __construct() {
	/* ls : la variable $pdo_options, ça date de ma formation de 2012 */ 
	$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
 
	/* ls : connexion PDO : on instancie la classe, qui dérive de la classe PDO (fournie par PHP5) (les paramètres pour wampserver sont host="localhost",
	user="root" et pwd="") */
	parent::__construct('mysql:host=' . MyPdo::$HOST . ';dbname=' . MyPdo::$DB_NAME, "root","", $pdo_options);
 
	}
 
}
// fin MyPDO
?>

[Tsilefy]

Si MyPdo.php est correctement inclus (ce qui semble être le cas), il n'y a rien d'anormal. Il y a forcément quelque chose qui cloche dans le reste du code alors. Peux-tu montrer authenticate.php (au moins jusqu'à la ligne 22)?

[laurentSc]

Je te remercie d'essayer de m'aider !

authenticate.php :

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
38
39
40
41
42
43
44
<?php
error_reporting(E_ALL ^ E_DEPRECATED ^ E_NOTICE); // ls : affichage des erreurs PHP en filtrant les erreurs de notice et les erreurs "deprecated"
header('Content-type: text/html; charset=UTF-8');	// ls : encodage en utf-8 (encoder qd même le fichier (menu "encodage" de Notepad++))
 
function authentification()
{
	GLOBAL $user;		// ls : var GLOBAL : la fct y accède (lecture et écriture) sans avoir à les passer en paramètre (GLOBAL<>LOCAL)
	GLOBAL $pass;
	GLOBAL $dbase;
	// ls : je mets le user et le pwd qui marchent pour wampserver
	$entered_user = $_SERVER["PHP_AUTH_USER"]="root";
	$entered_pass = $_SERVER["PHP_AUTH_PW"]="";
 
	include_once "connect.php"; // ls : on effectue une connection (_once pour le faire une seule fois)
	if (isset($bdd)) echo "connecté<br/>";
 
	if (isset($entered_user) && isset($entered_pass)) {	// on teste l'existence des var. superglobales 
		/* si existent, connexion à bdd mysql "su_dev_eis" et lecture table "users" */
		$auth=false;
 
		$sql = "SELECT * FROM users";
		echo"<br/>var_dump(bdd) :";var_dump($bdd);
		$qid = $bdd->prepare($sql);
		$qid->execute();	// ls : exécution (PDO) de la requête (préparée, même si ça sert à rien)
		while( $ligne=$qid->fetch(PDO::FETCH_OBJ) ) // ls : ligne par ligne, on retourne un objet dont chq propriété est une colonne de la table
			if (($ligne->user == $entered_user) && ($ligne->password == $entered_pass)) {
	// on set la var $auth et on positionne les var globales $user et $pass
				$auth = true;
				$user=$entered_user;
				$pass=$entered_pass;
			}
 
 
	}
	$auth=1; // ls : forçage de l'authentification
 
	return $auth;
}
$auth=authentification();
/*if ($auth) {
	include "connect.php";
}*/ // ls : sert à rien car déjà fait
 
?>

Les commentaires "ls" (mes initiales), c'est pour faciliter le relecture.

[Tsilefy]

Ton code fonctionne parfaitement chez moi. J'ai un fichier MyPDO.php, un fichier connect.php (qui doit inclure MyPDO.php) et authenticate.php qui inclut connect.php dans la fonction authentication.

- Es-tu sûr que le nom de la base, l'utilisateur et le mot de passe sont corrects?
- Es-tu sûr de ne pas inclure connect.php deux fois dans authenticate.php? Il est déjà inclus dans la fonction authentication()
- Essaie de tout mettre dans un seul fichier, et regarde si ça marche ou pas. Si ça marche, le problème est dans l'inclusion des fichiers.

[laurentSc]

En voulant tout mettre dans un seul fichier, j'ai trouvé la solution : déclarer la variable $bdd comme GLOBAL (dans index.php, le fichier principal qui appelle tout le monde et dans authenticate.php)(et dans index.php, il y avait un appel en trop à la fonction authentification() vu que c'est déjà fait dans authenticate.php)

[Tsilefy]

Content que le problème soit résolu, mais déclarer une variable comme global n'est jamais une bonne solution.

Ton problème est un problème de portée: en incluant connect.php dans la fonction authentification(), tu limites la portée de la variable $bdd à cette fonction uniquement, la variable n'existe pas au-delà. La déclarer globale résout ce problème, mais est un signe de mauvaise conception. Les variables globales (ou statiques, pour les classes) sont très problématiques.

Une meilleure solution serait d'inclure connect.php avant la fonction authentification, et de modifier la signature d'authentification en passant $bdd en paramètre:

Code :

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

1
2
3
4
5
 
include('connect.php');
 
function authentification($bdd) {
 //le reste est inchangé, enlève juste le include_once.

[laurentSc]

Je te remercie d'être intervenu ; j'ai adopté ta proposition...