Discussion :

[ibasticour]

Bonjour,
voilà, j'ai un petit soucis, je suis entrain de faire un forum et.. peu importe, je dois utiliser mysql.


Donc j'ai les classes suivante:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
<?
class FSection {
}
 
class FForum {
}
 
class FTopic {
}
 
class FPost {
}

Et j'ai une class mysql qui permet de se conencter à la basse de donnée.

Seulement dois-je utiliser les metode pour chercher les donnée dans la base de donnée à partir des class forum ou de la class mysql?

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
<?
class mysql {
    function __construct(){
        mysql_connect('host', 'log', 'mdp');
        mysql_selectdb('db');
    }
 
    function getFPost($id) {
        return $FPost =  mysql_query("select * from 'table' where id=".$id);
    }
}
class FPost {
    function __construct($FPost) {
        $this->titre = $FPost['titre'];
        $this->msg = $FPost['message'];
    }
}
$db = new db();
$id = 1;
$post[$id] = newFPost($db->getFPost($id));

Ou bien plustot quelque chose comme:

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
<?
class FPost {
    function __construct($FPost) {
        $this->titre = $FPost['titre'];
        $this->msg = $FPost['message'];
    }
    function obtainFPost($id) {
        $FPost = mysql_querry ("SELECT * FROM 'table' WHERE id = ". $id);
        $this->titre = $FPost['titre'];
        $this->msg = $FPost['message']
    }
}
 
mysql_connect('host', 'login', 'mdp');
$id = 1;
$fpost[$id] = new FPost($FPOST = array('titre' => '', 'message' => '');
$fpost[$id]->obtainFPost($id)
mysl_close();

Je sais que les deux partie sont incorrecte ici syntaxiquement, mais c'est pour mieux me faire comprendre de ce que je veux dire mysql dans ou hors des classe du forum.


Pourriez vous m'aidez?
Merci d'avance.

[Phelim]

Personnellement, je préfère l'exemple 2.

Il faut eviter de donner des traitements trop nombreux à une meme classe. Ta classe a un role, il ne faut pas qu'elle le depasse.

Son role ecrire une requete SQL et manipuler les données pour les envoyer dans la base.

Son role n'est pas d'etablir la connexion avec la base et de realiser les requetes, pour ça il faut une autre classe.

[Wharenn]

Hello !
L'exemple un me parait pour ma part plus approprié mais avec quelques modifications. En effet, si tu veux vraiment que ton objet Mysql corresponde exactement à son nom (c'est à dire à la manipulation de la base Mysql), tu ne dois pas y intégrer des méthode concernant l'objet FPost. Car imagines si tu as FPost, FForum, Fx, Fy, F1, F2, etc... ta classe Mysql va devenir un vrai bazard.

Il faudrait que ta classe Mysql se limite aux fonctions de base Mysql du style Connect, Query, Update, Close. Pour construire l'objet FPost tu peux ensuite utiliser un truc du style Factory.

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
 
<?
class mysql {
    function __construct(){
        mysql_connect('host', 'log', 'mdp');
        mysql_selectdb('db');
    }
 
    function query($req) {
        return mysql_query($req);
    }
 
    function close() {
        mysql_close();
    }
}
class FPostFactory {
    static function getFPost($id) {
        $mysql = new Mysql(x,y,z);
        $FPost = $mysql->query("select * from 'table' where id=".$id);
 
        return new FPost($FPost);
    }
}
class FPost {
    function __construct($FPost) {
        $this->titre = $FPost['titre'];
        $this->msg = $FPost['message'];
    }
}

Et pour l'utilisation :

Code :

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

1
2
 
$myPost = FPostFactory::getFPost($id);

Et voila ! Et comme ca ta classe Mysql est ulta clean et tu peux construire tes objets FPost de manière super simple grâce à la Factory
Par contre j'ai mis l'instanciation de la classe Mysql dans la Factory, c'est pas très judicieux, vaut mieux que ce soit créé en dehors.

[kaymak]

Suffit d'utiliser une méthode static pour récupérer la connexion courante, genre singleton. Et puis on à rarement besoin d'ouvrir plus d'une connexion à une bdd.
Pour ma part j'éviterais le constructeur qui prend un tableau en entrée, je préfère encore avoir une méthode dédié comme FPost::construireDepuisTuple($tuple).