Discussion :

[oranocha]

Bonjour,

J'ai décidé de me lancer dans l'OO sur PHP et je bloque un point. Voici le contenu de mon fichier class.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
 
class db {            
 
      function connect() {
               $connect = mysql_connect("localhost", "login", "password");
               mysql_select_db("maDB");
               return;
      }
 
      function disconnect() {
               mysql_close($connect);
      }
 
}
 
class constante_gen {
 
      public $valeurconstante;
 
      function constante($const) {
               $db = new db();
               $db -> connect();
 
               $select = "SELECT constante.nom AS nom, constante_c.contenu AS contenu FROM constante, constante_c, langue
	               WHERE constante_c.constante = constante.id
	               AND constante_c.langue = langue.id
	               AND langue.id = '1'
                 AND constante.nom = '".$const."'";
               $query = mysql_query($select) or exit(mysql_error() . "<br/>$select");
               $result = mysql_fetch_array($query);
               $valeurconstante = $result['contenu'];
 
               return;
 
               $db -> disconnect();
      }
 
}

Et voici le contenu de mon fichier test.php :

Code :

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

1
2
3
4
5
6
 
include("include/class.php");
 
$test = new constante_gen();
$test -> constante("NOM");
echo $test -> valeurconstante;

Très simple comme exemple : il va juste rechercher une entrée dans la DB portant le nom "NOM" et doit afficher le contenu. Résultat quand je lance le fichier test.php sur mon serveur, une page blanche, sans erreur mais blanche tout de même :-)

Si je modifie quelque peu ma classe constante_gen pour ne pas exécuter la requête SQL mais l'afficher, tout est OK mais il m'indique que la Query est vide hors si j'effectue exactement la même Query directement sur le serveur SQL, j'obtiens bien un résultat.

Ma classe db semble bien fonctionner car si je tape un password incorrect, mon fichier test.php affiche une erreur disant qu'il est impossible de se connecter.

Donc en gros, pourquoi cette requête ne m'affiche rien ?

Merci :-)

[oranocha]

OK non c'est bon ...

Mon erreur vient de mon appel dans test.php

Au lieu de faire :

Code :

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

1
2
3
4
5
6
 
include("include/class.php");
 
$test = new constante_gen();
$test -> constante("NOM");
echo $test -> valeurconstante;

Il fallait faire :

Code :

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

1
2
3
4
5
 
include("include/class.php");
 
$test = new constante_gen();
echo $test -> constante("NOM");

Et ça marche :-)

[daajack]

Dans ce cas tu utilises ta classe comme si c'était une fonction. C'est inutile de mettre des classes sauf bien sûr si c'est pour le fun de pouvoir dire que l'on fait de la POO.

Dans ton cas une variante plus sympa, sans pour autant être utile serait déjà d'appeller ta fonction __construct() et de balancer ton paramètre dès l'appel par un $test = new constante_gen('NOM'); p.ex. suivi d'un $test->writeMe(). Ce serait dejà plus POO-style. Et encore... Mais pour ça il faudra assigner tes variables à tes objets comme dans ta classe db par un $this->connect = mysql_blabla et un mysql_close($this->connect); Quoiqu'en passant la connection se ferme automatiquement à la fin du script donc si tu as d'autres requête à faire c mieux de la laisser ouverte, et sinon c'est plus simple de la laisser se terminer automatiquement à la fin du script. Même chose dans constante_gen() pour le $valeurconstante = blablabla en $this->valeurconstante...

L'idéal si tu veux faire de la POO, c'est de bien réfléchir à ton app avant de commencer à créer des classes dans tous les sens, en découpant ton environnement en différents objets, composé de propriétés et de méthodes que tu déterminera le plus tôt possible. Mais ça ne sert à rien de créer une classe pour un truc que tu peux faire avec une fonction. Ou à la rigeur tu fais des appels statiques.

Sur ce bon courage et bonne continuation, j'espère ne pas t'avoir brusqué, mais c'est pour ton bien.... ou pour le bien des logiciels et des développeurs en général