Vérifier si mon MVC semble correcte dans sa base et voir le controleur

**dancom5** · 28/08/2014, 04h58

Bonjour la communauté.

Je souhaite qu'on regarde si je respecte la base du MVC et voir si mon contrôleur est correcte.

Ma stucture MVC de base, le contrôleur est l'index.php en racine.
Je ne suis pas certain pour le contrôleur.
Ce sont les objet extanciés que j'ai mis là.

Je ne suis pas un expert alors je voudrais qu'on en tienne compte SVP. Merci. Je me casse la tête
depuis très longtemps, j'ai besoin de l'aide sur mon problème.

Structure de base:

Code :

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

1
2
3
4
5
6
7
8
        index.php
        /MODEL/
            connexion.php
            model.php
        /VIEW/
            layout.php
            template.php
            style.css

index.php contient qui serait mon contrôleur:

Code :

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

1
2
3
4
5
6
7
8
9
        <?php
            require_once './model/model.php';
 
            $oArtiste = new Artiste($cxn);
            $ListerArtistes = $oArtiste->lister();
            $artiste_id  = (empty($_GET['artiste']))?'1':(int)$_GET['artiste'];
            $AfficherArtistes = $oArtiste->afficher($artiste_id);
 
            require './view/template.php';

connexion.php partie model:

Code :

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

1
2
3
4
5
6
7
8
        <?php
            $dnsh = "mysql:host=localhost;dbname=ptg";
            $user = "root";
            $pass = "";
 
            $cxn = new PDO($dnsh, $user, $pass);
            $cxn ->setAttribute(PDO::ATTR_PERSISTENT,true); //a le même effet qu'un singleton, sans les inconvénients
            $cxn ->setAttribute(PDO::ATTR_ERRMODE,PDO::ERRMODE_EXCEPTION);

model.php partie model:

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
        <?php
 
            include './model/connexion.php';
 
            class Artiste{
                private $_conn;
                public function __construct($cxn){
                    $this->_conn = $cxn;
                }
 
                public function lister(){
                    $stmt = $this->_conn->prepare('SELECT * FROM artistes');
                    $stmt->execute();  
                    return $stmt->fetchAll(PDO::FETCH_OBJ);
                }    
 
                public function afficher($id=null){
                    $stmt = $this->_conn->prepare('SELECT * FROM artistes WHERE id = :id');
                    $stmt->bindParam(':id', $id, PDO::PARAM_INT);
                    $stmt->execute();  
                    return $stmt->fetchAll(PDO::FETCH_OBJ);
                }    
            }

layout.php partie view:

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
        <!DOCTYPE html>
        <html>
        <head>
            <meta charset="utf-8" />
            <!--[if lt IE 9]><script src="http://html5shiv.googlecode.com/svn/trunk/html5.js"></script><![endif]-->
            <title></title>
            <meta name="keywords" content="" />
            <meta name="description" content="" />
            <link href="./view/style.css" rel="stylesheet">
        </head>

        <body>

        <?php echo $content; ?>

        </body>
        </html>

template.php partie view:

Ici, je compte faire des sous-couches mais je me demande si c'est pertinent.

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
        <?php ob_start() ?>

            <div class="wrapper">

                <header class="header">
                </header><!-- .header-->

                        <div class="middle">
                            <div class="minheight">

                                <div class="container">
                                    <main class="content">
                                            <?php
                                            foreach($AfficherArtistes as $a_artiste) {
                                                        echo $a_artiste->id ." ". $a_artiste->prenom ." ". $a_artiste->nom;
                                             }
                                            ?>
                                    </main><!-- .content -->
                                </div><!-- .container-->

                                <aside class="left-sidebar">
                                    <main class="content">
                                            <?php
                                            foreach($ListerArtistes as $l_artiste) {
                                                         echo "<a href='http://localhost/index.php?artiste=" . $l_artiste->id . "'>".$l_artiste->prenom ." ". $l_artiste->nom . "</a></br>";
                                            }
                                            ?>
                                    </main><!-- .content -->
                                </aside><!-- .left-sidebar -->

                            </div><!-- .minheight-->
                        </div><!-- .middle-->

                <footer class="footer">
                </footer><!-- .footer -->

            </div><!-- .wrapper -->

        <?php $content = ob_get_clean(); ?>
        <?php include './view/layout.php'; ?>

**Nightfall** · 29/08/2014, 01h10

Ta classe Artiste est trop dépendante de l'utilisation de la BDD. Elle ne devrait pas utiliser directement des fonctions comme execute() ou fetchall() mais passer juste le sql à une fonction qui serait dans connexion.php et cette fonction lui renverrait les données (c'est-à-dire le résultat de fetchall()).

Et la fonction lister() devrait être statique : pas besoin d'instancier un artiste pour récupérer une liste d'artistes. Mais bien sûr en l'état, le fait qu'elle utilise une variable de classe (_conn) t'en empêche.

Sinon dans une appli il y a de multiples contrôleurs. Le fichier index.php que l'on met à la racine est plutôt le contrôleur frontal, c'est-à-dire le point d'entrée unique de l'appli. Ton fichier index.php ressemble plus à un contrôleur parmi d'autres qui serait dans un répertoire CONTROLLEURS.

Ensuite un contrôleur n'est pas simplement un fichier contenant directement du code qui s'exécute mais une classe contenant une méthode par action :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
class ArtisteControlleur
{
  public function lister()
  {
  //...
  }
 
  public function afficher($id)
  {
  //...
  }
}

**dancom5** · 29/08/2014, 05h55

Envoyé par Nightfall

Ta classe Artiste est trop dépendante de l'utilisation de la BDD. Elle ne devrait pas utiliser directement des fonctions comme execute() ou fetchall() mais passer juste le sql à une fonction qui serait dans connexion.php et cette fonction lui renverrait les données (c'est-à-dire le résultat de fetchall()).

Et la fonction lister() devrait être statique : pas besoin d'instancier un artiste pour récupérer une liste d'artistes. Mais bien sûr en l'état, le fait qu'elle utilise une variable de classe (_conn) t'en empêche.

Merci. Dans les grandes lignes, c'est assez bon comme arrangement.

Je ne suis pas certain de comprendre les deux paragraphe, je pourrais avoir un exemple qui me permettrait de mieux visualiser? Surtout le premier paragraphe.

**Nightfall** · 29/08/2014, 20h09

Par exemple : http://bpesquet.developpez.com/tutor...re-mvc/#LV-B-1

Dans le dernier code source de ce paragraphe (Modele.php).

Pour ma 2ème remarque : on voit que la variable de classe $bdd devient une variable statique et peut donc être récupérée via la méthode statique getBdd(), qui se charge d'instancier $bdd si ce n'est pas déjà fait.

Pour ma 1ère remarque :

Code :

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

1
2
3
public function lister(){
    return self::getConnection()->executerRequete('SELECT * FROM artistes');
}

avec executerRequete() qui renvoie le résultat de fetchall(). En récupérant un tableau, tu récupères quelque chose de totalement indépendant de la manière de récupérer les données.

Tu peux aussi t'inspirer de ceci : https://github.com/jpfuentes2/php-activerecord

Vérifier si mon MVC semble correcte dans sa base et voir le controleur

Langage PHP

Discussions similaires

Partager

Partager