Discussion :

[RGuy_]

Bonjour,

Je m'essaie à la POO en php.

J'ai créé une classe qui me permet de gérer les commandes

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
 
class commandes {
	private $dbh;
 
	function __construct() {
		try {
			$dbh = new PDO('mysql:host=xxx.xx.fr;dbname=yy', 'lllooo', 'huhu', array(PDO::ATTR_PERSISTENT => true));
		} catch (PDOException $e) {
			print "Error!: " . $e->getMessage() . "<br/>";
			die();
		}
	}
 
	function cmdes_non_exp() {
 
		$stmt = $dbh->prepare("select id_cmde from commandes where id_etat = ?");
 
		$cpt = 0;
		if ($stmt->execute(array(1))) {
			while ($row = $stmt->fetch()) {
							print "Commande No $row[0]\n";
			}
		}
	}

Ce code provoque l'affichage de l'erreur

Fatal error: Call to a member function prepare() on a non-objec

Pourquoi ? Si j'insère le code du contructeur dans la méthode, la requête sql fonctionne... Il me semble que le database handler devrait être créé par le constructeur puis accessible depuis les différentes méthodes non ? Sinon, il faudrait ouvrir la cnx dans chaque méthode ? Il doit bien y avoir un moyen de rationnaliser tout cela mais lequel ? J'ai beau essayer, je ne trouve pas la solution.

Merci pour votre aide.

RG

[cerede2000]

Normal:

Code :

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

1
2
 
$stmt = $dbh->prepare("select id_cmde from commandes where id_etat = ?");

Je ne voit pas de méthode prepare dans ta class
Et si cela appartient a la class PDO, alors tu doit inclure le fichier php de la class PDO dans le fichier php de ta class

[RGuy_]

Euhh....

Code :

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

$dbh = new PDO(...)

Devrait créer un objet PDO ( classe livrée en std avec php5 ) et l'assigner à $dbh. La méthode prepare est une methode de l'objet PDO. Pourquoi devrais-je le déclarer ?

[koopajah]

Montres le script où tu crées justement l'instance de ta classe pour voir.

Sinon ajoutes un "var_dump($dbh)" avant de faire le prepare pour voir ce que contient ton $dbh justement.

et je dirai, en plus qu'il ne faut pas utiliser $dbh mais $this->dbh mais ca je n'en suis pas sur (mais un test rapide semble le confirmer)

[RGuy_]

le "$this" ne fonctionne pas lors de l'instantiation de la classe..

Code :

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

$this->$dbh = new PDO(...)

Ca me renvoi l'erreur

Fatal error: Cannot access empty property in

le var_dump($dbh) renvoi NULL.
la classe commande est utilisée comme suit

Code :

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

1
2
3
 
			$commandes = new commandes();
			$commandes->cmdes_non_exp();

[cerede2000]

Code :

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

$this->$dbh = new PDO(...)

C'est:

Code :

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

$this->dbh = new PDO(...)

[koopajah]

C'est $this->dbh que j'ai écrit, faut pas mettre le $ pour les membres d'une classe.