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]

Bonsoir

J'ai deja lu le manuel php. Donc vous dites que les erreurs ne devraient pas s'afficher ? Parce que j'ai déjà essayé et si la fonction ne s'éxécute pas je reçoit correctement ce que contient le die. C'est pas normal ?

[amoiraud]

Bonsoir

J'ai deja lu le manuel php. Donc vous dites que les erreurs ne devraient pas s'afficher ? Parce que j'ai déjà essayé et si la fonction ne s'éxécute pas je reçoit correctement ce que contient le die. C'est pas normal ?

Si c'est normal, avec le @ il ne t'affiche pas l'erreur généré par la fonction mais le contenu du die, pas pratique pour les débugs

[flozza]

C'est bien ce qu'il me semblait! J'ai fait exprès de mettre ces @ avec die, parce que je ne veux pas que les utilisateurs voient l'erreur brute et précise, c'est pas pas propre pour eux je trouve.

[amoiraud]

C'est bien ce qu'il me semblait! J'ai fait exprès de mettre ces @ avec die, parce que je ne veux pas que les utilisateurs voient l'erreur brute et précise, c'est pas pas propre pour eux je trouve.

D'où la deuxième partie du message de stealth35 : "fait des exceptions"

[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 ?

[flozza]

D'où la deuxième partie du message de stealth35 : "fait des exceptions"

Si je fait des exceptions, le but c'est d'afficher l'erreur de la fonction. Mais moi je ne veux pas qu'elle s'affiche. Dans ce cas le @ die est mieux non?

[amoiraud]

Si je fait des exceptions, le but c'est d'afficher l'erreur de la fonction. Mais moi je ne veux pas qu'elle s'affiche. Dans ce cas le @ die est mieux non?

Non tu peut faire une exception qui va à la fois t'envoyer un mail avec le détail précis de l'erreur et afficher un message à l'utilisateur du style "Un problème est survenue, nos administrateurs ont été prévenus, désolé pour la gène, etc..."

[flozza]

Alors effectivement c'est encore mieux. Merci pour cette précision, je vais me faire quelques exceptions du coup ^^

[amoiraud]

N'hésite pas à créer tes propres classes d'exceptions pour simplifier encore plus les choses

[flozza]

Une petite question au passage. Y a t il une différence entre SELF::.. et $this->.. ?

[stealth35]

Une petite question au passage. Y a t il une différence entre SELF::.. et $this->.. ?

self : static
$this-> instance

[flozza]

D'accord!

Bon j'ai regardé pour les exceptions.C'est intéressant comme technique. mais finalement je suis perplexe.. Ca alourdit le code, et en réduit donc les performances, non?

[stealth35]

D'accord!

Bon j'ai regardé pour les exceptions.C'est intéressant comme technique. mais finalement je suis perplexe.. Ca alourdit le code, et en réduit donc les performances, non?

non
le but c'est pas de faire le moins de ligne possible et un code illisible, si tu veux des performances fait autre chose que du PHP, le but est plutot de faire quelque chose de maintenable, lisible et simple