Discussion :

[PIEPLU]

Bonjour,

Je suis en train de tester la création d'un site basique en MVC pour bien comprendre le schéma.

Cependant, je ne sais pas comment créer le router. Vous pourriez m'aider ?

Voici un ZIP de ce que j'ai commencé : http://www.partage-fichiers.com/upload/k8myn2b5/mvc.zip



Merci de votre aide

[ABCIWEB]

As-tu lu ce tuto ?

Il y a aussi cet exemple de routeur

[laurentSc]

Bien qu'étant loin d'être un spécialiste, j'ai vu un problème dans les requêtes de ton modèle, et même un dans le PHP : dans la requête (city.php ligne 8), il y a un $ dans le nom d'une colonne ($formule) et dans le PHP (même fichier, ligne 6), il faut utiliser la concaténation :

Code :

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

$formule = "(6366*acos(cos(radians(".$latitude."))*cos(radians(`latitude`))*cos(radians(`longitude`) -radians(".$longitude."))+sin(radians(".$latitude."))*sin(radians(`latitude`))))";

D'autre part, même si j'ai peu de connaissance en MVC, je sais que dans le routeur, il faut créer le controller, qui lui-même va créer la vue et le modèle...

[Spartacusply]

Bien qu'étant loin d'être un spécialiste, j'ai vu un problème dans les requêtes de ton modèle, et même un dans le PHP : dans la requête (city.php ligne 8), il y a un $ dans le nom d'une colonne ($formule) et dans le PHP (même fichier, ligne 6), il faut utiliser la concaténation :

Code :

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

$formule = "(6366*acos(cos(radians(".$latitude."))*cos(radians(`latitude`))*cos(radians(`longitude`) -radians(".$longitude."))+sin(radians(".$latitude."))*sin(radians(`latitude`))))";

Non, le "$formule" est tout à fait normal car c'est la variable qu'il créée juste avant. De plus l'utilisation de guillemets doubles dans une chaine de cractère avec des variables est tout à fait valide et autorisé (voir ceci), même s'il est vrai qu'il est recommandé de privilégier l'utilisation de la concaténation, surtout pour une raison de lisibilité.

D'autre part, même si j'ai peu de connaissance en MVC, je sais que dans le routeur, il faut créer le controller, qui lui-même va créer la vue et le modèle...

Ca c'est exact, ton routeur doit rediriger vers un controleur, et non un modèle, et c'est ce contrôleur qui va se charger d'instancier (et non créer) les modèles et les classes nécessaires.

[laurentSc]

Ca c'est exact, ton routeur doit rediriger vers un controleur, et non un modèle, et c'est ce contrôleur qui va se charger d'instancier (et non créer) les modèles et les classes nécessaires.

Oui, juste une erreur de vocabulaire et normal, vu que ça fait des mois que je n'ai pas fait de POO. Sinon, dans le code du constructeur du controller (instancié par le routeur), on aura un truc de ce type :

Code :

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

1
2
3
/*  la vue doit connaître aussi le modèle */
$this->model=new model();
$this->view=new view($this->model);

et pour ne pas avoir à instancier les classes, dans le routeur (pas de moi, mais d'un expert du MVC) :

Code php :

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

1
2
spl_autoload_register('my_autoloader'); 
/* grâce à ça, plus besoin d'include pour les classes : à tt new xyz correspondant à une classe inconnue, cette fonction sera appelée ; si tt fichier de classe est nommé "xyz_class.php", il sera includé si nécessaire */

[PIEPLU]

Je ne suis pas sur d'avoir tout compris dans vos échanges.
J'ai tout de même essayé de le faire.

http://paf.im/TjYaS

Cependant, je ne sais pas comment utiliser un template (dossier views/layout)

Si vous pouvez m'aider.

Merci

[laurentSc]

1) Kaspersky refuse d'ouvrir ton lien ;

2) Même si je suis persuadé que le MVC est ce qu'il y a de mieux (pour coder tout seul, donc sans Framework, Zend, Symphony and co), ça fait des mois que je n'y ai pas touché (pas eu le temps !) ;

3) Si ça peut t'aider, j'ai mis en PJ le zip d'une formation (passionnante) que j'avais suivie (fin 2012) sur le sujet (le contenu du zip est le code du projet MVC résultat de la formation)

[PIEPLU]

Merci pour ton fichier.

Je vous remets tout de même mon zip avec mes fichiers classés en mvc (enfin, j'ai essayé, vous me direz).
J'en suis à cette étape - Je ne comprend pas comment appliquer le template générale (views/layout).

mvc_v2.zip

Merci

[laurentSc]

Quelques remarques d'un non-spécialiste :

- je suis presque sûr qu'il faut créer des classes controller, model et view, ce qui n'est pas le cas chez toi ;

- dans ton routeur, tu includes controllers/accueil.php; or ce fichier n'existe pas.

[Nightfall]

Quelques remarques :

D'après ton index.php, tu sembles ne pas faire la distinction entre un contrôleur et une page. Index.php (appelé contrôleur frontal) reçoit en paramètres le contrôleur et l'action du contrôleur, pas le nom d'une page. C'est ensuite l'action du contrôleur qui va inclure une vue.

Parce que ton fichier news.php pour l'instant il ne fait qu'afficher les news, mais il va falloir qu'il puisse en ajouter, en modifier une, etc... Donc dans news.php il te faut plusieurs actions.

Exemple : tu appelles index.php?controleur=News&action=View. Index.php va alors inclure le fichier controllers/news.php, qui contiendra au minimum :

Code :

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

1
2
3
4
5
6
7
8
9
10
include(dirname(__FILE__).'/../models/news.php');
 
class NewsController
{
    public function ViewAction()
    {
        $news = recuperer_news();
        include(dirname(__FILE__).'/../views/news.php');
    }
}

Et index.php, après avoir inclus ce fichier, va instancier le contrôleur et appeler la méthode View (syntaxe à vérifier) :

Code :

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

1
2
3
4
5
 
$controleur = $_GET['controleur'] . 'Controller';
$controller = new $controleur();
$action = $_GET['action'] . 'Action';
$controller->$action();

Pour le template général, à ton stade je dirais que tu pourrais commencer par faire simplement dans ta vue :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
<?require 'layout/entete.php';?>
<h1>Les news du site</h1>
 
<?php
foreach($news as $n) {
 
	echo '<div class="news">Article : '.$n['titre'].' by '.$n['auteur'].'</p></div>';
 
}
require 'layout/pied.php';
?>

Mais pour pouvoir utiliser un seul fichier comme gabarit tu as besoin de cette technique :
http://bpesquet.developpez.com/tutor...e-mvc/#LII-C-1