Discussion :

[flozza]

Bonjour

Je revient à la charge, toujours dans l'apprentissage du POO. décidémment..
Je reçoit cette erreur en essayant de regrouper 2 fonctions touchant mysql dans une classe.

J'ai donc index.php, avec

Code html :

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

1
2
 
<?php include("functions.php"); ?>

functions.php dont les premières lignes sont:

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
 
class bdd {
	private $host="xxx";
	private $user="xxx";
	private $base="xxx";
	private $pwd="xxx";
 
	public function connect() {
		if(!empty($this->host) && !empty($this->user) && !empty($this->base) && !empty($this->pwd)) {
			$db=@mysql_connect($this->host, $this->user, $this->pwd) or die("<span class='error'>Erreur de connexion à la base</span>");
			@mysql_select_db($this->base,$db) or die("<span class='error'>Erreur de sélection de base</span>");
			mysql_set_charset('utf8');
		}	
	}
 
	public function execute($req) {
		$sql = @mysql_query($req) or die("<span class='error'>Erreur d'éxécution !</span><br>[".$req."]<br><u>Réponse:</u>".mysql_error()); 
		return $sql;	
	}	
}
 
$bdd = new bdd();
$bdd->connect();

Et j'obtiens l'erreur

Fatal error: Call to a member function execute() on a non-object in /public_html/configs/functions.php on line 172

Voici la fonction en cause dans functions.php ligne 172:

Code php :

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

1
2
3
4
5
6
7
8
9
10
 
function isHeAdmin($pse) {
	$checkRankSql = "SELECT * FROM comptes WHERE login_user='".$pse."'";
	$checkRank = $bdd->execute($checkRankSql);
	$cr=mysql_fetch_assoc($checkRank);
	if($cr['rank_user']==1) {
		return true;
	}
	else { return false; }
}

Pourtant, l'objet est bien déclarée au début de ce même fichier, à la suite de la classe, et la fonction membre execute() est bien public.. Désolé de vous embeter encore :/

Et si au passage, vous trouvez quelque chose à redire sur tout ça n'hésitez pas mon but est d'apprendre.

Merci

[Bovino]

Pourtant, l'objet est bien déclarée au début de ce même fichier, à la suite de la classe, et la fonction membre execute() est bien public.

Oui, mais $bdd est inconnu dans la fonction isHeAdmin

[flozza]

Ok J'ai donc contourné ce problème en rajoutant global $bdd; dans la fonction isHeAdmin. Mais est-ce vraiment la bonne solution?

[Celira]

Il vaudrait mieux passer la connexion en paramètre à la fonction, ou mettre cette fonction comme méthode de ta classe bdd et utiliser l'objet courant comme connexion.

[stealth35]

vire les @ et fait des exceptions

[flozza]

vire les @ et fait des exceptions

Je veux bien mais pour quelle raison?

[rawsrc]

Bonsoir,

Pour la simple et bonne raison que l'arobase empêche le déclenchement des erreurs/avertissements. Bref ton code se déroule silencieusement mais il t'est impossible de savoir s'il y a eu des erreurs. Aucune utilité, puisque tu sera incapable de savoir si une routine a bien fait ce pour quoi elle a été prévue.
Donc ne jamais utiliser l'arobase, sauf si tu as prévu un autre moyen te permettant d'attraper un déroulement erroné d'une routine.
Un peu de lecture ici

[flozza]

Il vaudrait mieux passer la connexion en paramètre à la fonction, ou mettre cette fonction comme méthode de ta classe bdd et utiliser l'objet courant comme connexion.

Bonsoir

Est-ce vous pouvez m'éclaircir sur cette idée ? Intégrer la fonction isHeAdmin dans l'objet bdd c'est ça ?