Discussion :

[krfa1]

Bonjour,

Je commence à me mettre à la POO, et j'aurais deux question:

Dans l'exemple ci-dessou, est-ce que c'est une bonne manière de profiter de la POO, ou est-ce que c'est inutile?

Deuxièmement, toujours pour le même code, j'ai une erreur dans mon mysql_fetch_array, il me dit que la requete SQL n'est pas bonne, mais je l'ai testée das mysql, et tout est ok, est-ce que je fais qqch de faux? Si oui, comment?

Merci d'avance pour votre aide

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
 
class base_de_donnee
	{
	var $hote = "localhost";
	var $utilisateur = "root";
	var $mot_de_passe = "";
	var $base = "ma_base";
 
	//fonction qui s'exécute à la construction de l'objet
    function __construct()
		{ 
		echo $this->utilisateur;
        mysql_connect($this->hote, $this->utilisateur, $this->mot_de_passe) or die(mysql_error());
		mysql_select_db($this->base) or die(mysql_error());
		}
 
	function execution_requete($requete)
		{
		mysql_query($requete);
		}
 
	//fonction qui s'exécute à la destruction de l'objet
    function __destruct()
		{ 
        mysql_close();
		echo 'test';
		} 
	}
$connection = new base_de_donnee;
$renvoi = $connection->execution_requete("SELECT * FROM tbl_menu");
while($test = mysql_fetch_array($renvoi))
	{
	echo $test['id_menu'];
	}

[Nesmontou]

Salut, il faut que tu modifies ta fonction execution_requete pour qu'elle retourne le résultat de mysql_query :

Code :

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

1
2
3
function execution_requete($requete) {
   return mysql_query($requete);
}

Bon développement

[Yoteco]

Dans l'exemple ci-dessou, est-ce que c'est une bonne manière de profiter de la POO, ou est-ce que c'est inutile?

Alors honnêtement c'est pas très utile ce que tu fais là. Car par exemple ta fonction requête() ne fais rien d'autre que la même chose que la fonction mysql_query(). Ensuite c'est beacoup mieux de passer les valeur en paramètres au constructeur que de faire des variable que tu assignes l'une après l'autres, c'est mieux pour la réutilisabilité. Si tu as un serveur PHP5 essaye de t'habituer à utiliser les mots clé public, private et protected c'est plus propre et mieux protégé .

J'ai déjà essayer plusieurs fois de faire des classe pour gérer les requêtes MySQL mais je me suis rendus compte que sa ne sert pas à grand chose et finalement je me suis tourné vers PDO qui gère ça parfaitement

Voici un exemple de ta classe un peu améliorer:

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
class base_de_donnee
    {
    private $hote;
    private $utilisateur;
    private $mot_de_passe;
    private $base;
 
    //fonction qui s'exécute à la construction de l'objet
    public function __construct($argHost, $argUser, $argPassword, $argDb)
        { 
            $this->hote = $argHost;
            $this->utilisateur = $argUser;
            $this->mot_de_passe = $argPassword;
            $this->base = $argDb;
 
            $this->connect()
        }
         private function connect()
         {
            mysql_connect($this->hote, $this->utilisateur, $this->mot_de_passe) or die(mysql_error());
        mysql_select_db($this->base) or die(mysql_error());
         }
 
    public function execution_requete($requete)
        {
        mysql_query($requete);
        }
 
    //fonction qui s'exécute à la destruction de l'objet
    public function __destruct()
        { 
        mysql_close();
        echo 'test';
        } 
    }
$connection = new base_de_donnee('localhost', 'root', '', 'ma_base');
$renvoi = $connection->execution_requete("SELECT * FROM tbl_menu");
while($test = mysql_fetch_array($renvoi))
    {
    echo $test['id_menu'];
    }

[krfa1]

Merci beaucoup pour votre aide aux deux, ca marche maintenant, maintenant il reste plus qu'a ce que je comprenne quand est-ce que je dois utilisé la POO, et après, c'est super.

Bonne soirée.

Fab