Design pattern MVC et mon code actuel

Version imprimable

Salut,

**Désolé je ne savais pas ou poster ce message**

Je vais vous montrer la hiérarchie de mon code afin de le confronter au design pattern MVC afin de me donner vos remarques pour voir si mon architecture est "viable".Ça ressemble un peu à du MVC.
Disons que c'est une architecture que j'utilise depuis longtemps et que je trouve très simple (pour moi) donc j'hésite à passer à une conception MVC pure.
Je l'utilise quand je code en spécifique sinon j'utilise Symfony2 donc ça me change on va dire.

Supposons je lance "index.php"

->Dans le fichier Index j'inclue mon template (canevas.php)(juste un include)

-> Dans ce canevas

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
<?php
 
include($_SERVER['DOCUMENT_ROOT'].'/Php/Configuration.php');
 
include($_SERVER['DOCUMENT_ROOT'].'/Php/Gestion_controller.php');
 
 ?>
 
[Mon code html]
 
<?php (file_exists(TEXTE))?include(TEXTE):0;?>
 
[Mon code html]

Dans configuration (une sorte de controleur frontal)

Code:

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
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
 
define("TEXTE",$_SERVER['DOCUMENT_ROOT'].'/Txt/'.basename($_SERVER["PHP_SELF"]));
define("ERROR",'/erreur404.php');
define("SEARCH",'/recherche.php');
define("URL",$_SERVER['REQUEST_URI']);
define("SCRIPT",$_SERVER["PHP_SELF"]);
define("DOMAINE",$_SERVER['HTTP_HOST']);
define("IP",$_SERVER["REMOTE_ADDR"]);
//Chargement des classes
function chargerClasse ($classe)
	{ 
	[Moncode]
	}
spl_autoload_register ('chargerClasse');
session_start();
 
$menuManager = new MenuManager(Database::getInstance());
$produitManager = new ProduitManager(Database::getInstance());
$clientManager = new ClientManager(Database::getInstance());
$panierManager = new PanierManager(Database::getInstance());
 
function urlRedirect($menuManager,$produitManager)
	{
	$aux=0;
	if(SCRIPT==GAMME && isset($_GET['id']) && isset($_GET['gammes']))
		{
		if(!($aux=$menuManager->get((int)$_GET['id']))) 
			{ 
			header("HTTP/1.1 410 Gone"); 
			echo file_get_contents("http://".DOMAINE.ERROR);
			}
		else 
			{
			if(current(explode('?', URL, 2))!=($r_url=Menu::verification_menu($aux)))
				{
				header('HTTP/1.1 301 Moved Permanently', false, 301);
				header('Location: http://'.DOMAINE.$r_url.'');
				}
			}
		}
	else if(SCRIPT==MENU_PRINCIPAL && isset($_GET['id']) && isset($_GET['gammeprincipal']))
		{
		if(!($aux=$menuManager->getPrincipal((int)$_GET['id']))) 
			{ 
			header("HTTP/1.1 410 Gone"); 
			echo file_get_contents("http://".DOMAINE.ERROR);
			}
		else 
			if(current(explode('?', URL, 2))!=($r_url=Menu::verification_menu_principal($aux)))
				{
				header('HTTP/1.1 301 Moved Permanently', false, 301);
				header('Location: http://'.DOMAINE.$r_url.'');
				}
		}
 
	if(!$aux)
		{
		(!isset($cmsManager))?$cmsManager = new CmsManager(Database::getInstance()):0;
		$aux=(pathinfo(SCRIPT, PATHINFO_FILENAME)==pathinfo(ERROR, PATHINFO_FILENAME))?$cmsManager->get(pathinfo(SCRIPT, PATHINFO_FILENAME),CmsManager::DYNAMIQUE):$cmsManager->get(pathinfo(parse_url(URL, PHP_URL_PATH), PATHINFO_FILENAME),CmsManager::DYNAMIQUE);
		}
	if(!$aux)
		{
		header("HTTP/1.1 410 Gone"); 
		echo file_get_contents("http://".DOMAINE.ERROR);	
		}
	return $aux;
	}
 
//Cela sert a rediriger vers la bonne page si url comporte erreur, rediriger vers la bonne.
	$obj=urlRedirect($menuManager,$produitManager);

Dans Gestion_controller

Code:

1
2
3
4
5
6
 
<?php
require_once($_SERVER['DOCUMENT_ROOT'].'/Php/Controller/AjoutProduitController.php');
require_once($_SERVER['DOCUMENT_ROOT'].'/Php/Controller/CreationSuiviCommandeController.php');
require_once($_SERVER['DOCUMENT_ROOT'].'/Php/Controller/ModificationSuiviCommandeController.php');
?>

Exemple d'un "controller"

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
 
<?php
//PANIER CODE PROMOTIONNEL
if(isset($_POST['validation_code_promotion']))
	{
	$validated = Gump::is_valid($_POST,array('code_promotion'=>'required|min_len,2'));
		if($validated === true) 
			{
			$ReductionManager = new ReductionManager(Database::getInstance());
			$_SESSION['reduction']=$ReductionManager->verificationReduction($_POST['code_promotion'],$_SESSION['panier']->totalPanierTTC());
			}
	}
?>

Exemple d'une vue

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
 
<?php include($_SERVER['DOCUMENT_ROOT'].'/headerslider.php'); ?>
 
<div id="arborescence"><?php echo Menu::arbre_menu($Cms->texte()); ?></div>
 
<div class="article">
	<?php echo $Cms->description_1(); ?>
</div>
 
<?php
if(!$Cms->contact())
	echo($Cms->description_2())?"<div class=\"article\">".$Cms->description_2()."</div>":"";
else
	{
	include($_SERVER['DOCUMENT_ROOT'].'/formContact.php');
	echo "<div class=\"article mini_article\">";
		echo $Cms->description_2();
	echo "</div>";		
	}
?>

Citation:
Envoyé par kyride
Code:

1 2 3 4 5 <?php require_once($_SERVER['DOCUMENT_ROOT'].'/Php/Controller/AjoutProduitController.php'); require_once($_SERVER['DOCUMENT_ROOT'].'/Php/Controller/CreationSuiviCommandeController.php'); require_once($_SERVER['DOCUMENT_ROOT'].'/Php/Controller/ModificationSuiviCommandeController.php'); ?>
Pourquoi fais-tu des require sur l'ensemble de tes contrôleurs alors que tu as défini une fonction d'autoload plus haut ?
Sinon l'essentiel c'est que ça marche et que ça fasse ce qu'on lui demande, il y a plein de façons de procéder, surtout sur des petits projets.

Bonne journée à toi

25/03/2015, 12h44
kyride

Salut,

Merci de m'avoir donné une réponse.

Pourquoi fais-tu des require sur l'ensemble de tes contrôleurs alors que tu as défini une fonction d'autoload plus haut ?

Bah c'est juste que (a ce que j'ai compris sur les autoload) mon ChargerClasse, va chercher une classe
Si php voit $test = new Menu;
Et si Menu n'existe pas il va tester ma fonction et aller chercher ma classe si elle existe.

Mais la pour mes "controlleurs" je vois pas vraiment comment faire ma fonction
25/03/2015, 16h40
kyride

En faite je ne vois pas quand la fonction autoload pour mes controllers va être lancé car actuellement je les inclus tous et en fonction de l'url et de l'action de ma page appelante je declenche une action

Par exemple si je suis sur gamme.php j'Inclue tout mes controllers et si dedans je trouve un script qui vaut gamme.php le contrôle sera effectué en lançant les bons modèles par exemple menumanager.
25/03/2015, 16h52
alejandro
Essaye ceci :

index.php
Code:

1 2 3 <?php spl_autoload_register(); $menu = new menu\principal;
tu crée un dossier "menu" à côté de index.php et dedans tu met principal.php

menu/principal.php
Code:

1 2 3 4 5 6 7 <?php namespace menu; class principal { public function __construct() { echo "Hello World"; } }
puis tu appelle index.php et tu regardes ...

après tu peux imaginer ce que tu veux :
Code:

1 2 3 $controller = new controleur\principal; $model = new modele\bdd; $view = new vue\accueil
C'est déjà un bon point de départ.

Bonne journée
25/03/2015, 20h46
kyride

Je vais voir comment je vais appliquer cela à mon projet mais je ne connaissais pas cette utilisation de l'autoload avec les namespace (pour par contre il faut vraiment respecter namespace et nom du dossier)
Je ne savais pas que automatiquement sans préciser de fonction dans l'autoload php faisait le chargement auto.

Franchement merci à toi ;)

J'ai trouvé d'ailleurs un bon truc la dessus
http://www.sitepoint.com/autoloading...sr-0-standard/ ou ça explique bien tout ça.
25/03/2015, 22h42
Tsilefy
Tu fais du PHP, pas de l'assembleur. Un conseil, écris un code lisible, sinon dans 6 mois tu perdras beaucoup de temps à te demander ce que fait par exemple.

Code:

$aux=(pathinfo(SCRIPT, PATHINFO_FILENAME)==pathinfo(ERROR, PATHINFO_FILENAME))?$cmsManager->get(pathinfo(SCRIPT, PATHINFO_FILENAME),CmsManager::DYNAMIQUE):$cmsManager->get(pathinfo(parse_url(URL, PHP_URL_PATH), PATHINFO_FILENAME),CmsManager::DYNAMIQUE);

Les sauts de ligne et les accolades ne coûtent rien et améliorent la lisibilité.

Ta configuration fait beaucoup trop de chose. Ne mélange pas dans un seul fichier le code inactif (fonctions, classes) et le code actif (la partie procédurale).

J'imagine que Database::getInstance() retourne un singleton? Est-ce qu'il y une raison précise pour préférer un singleton, plutôt qu'un objet ordinaire.

Code:

$menuManager->getPrincipal((int)$_GET['id'])

N'est pas recommandé pour plusieurs raisons:

- si tu veux contraindre le paramètre à être un entier (ce qui est le cas ici), il vaut mieux mettre la contrainte dans la méthode elle-même. Ça garantit qu'elle est appliquée à chaque fois alors que dans ton cas si jamais tu oublies de caster le paramètre, tu risques d'avoir des résultats imprévisibles.
Donc fais plutôt
Code:

1 2 3 4 5 6 public function getPrincipal($id) { $id = (int) $id; //reste de code }
- Mais caster n'est pas le bon choix ici. Imagine que $_GET['ID'] ait comme valeur "31test". Le résultat sera $id = 31. Je ne pense pas que ça soit ce que tu cherches. Il vaut mieux vérifier si $_GET['id'] est bien un entier, et le rejeter si ce n'est pas le cas. Plus tu élimines les ambiguités, moins tu auras de comportements inattendus (= bugs).
Code:

1 2 3 4 5 6 7 8 9 public function getPrincipal($id) { if(!is_numeric($id)) { throw new Exception("$id n'est pas un entier!"); } //reste du code }

Salut,

Merci de m'avoir répondu également,
Pour info je suis le seul développeur dans ma société donc c'est très important d'avoir un avis extérieur sur mon travail car bon ici tant que ça marche c'est bon.
Citation:
Tu fais du PHP, pas de l'assembleur. Un conseil, écris un code lisible, sinon dans 6 mois tu perdras beaucoup de temps à te demander ce que fait par exemple.
Code:

1 2 $aux=(pathinfo(SCRIPT, PATHINFO_FILENAME)==pathinfo(ERROR, PATHINFO_FILENAME))?$cmsManager->get(pathinfo(SCRIPT, PATHINFO_FILENAME),CmsManager::DYNAMIQUE):$cmsManager->get(pathinfo(parse_url(URL, PHP_URL_PATH), PATHINFO_FILENAME),CmsManager::DYNAMIQUE);
Tu as parfaitement raison.
C'est mon défaut, une fois que j'ai trouvé mon schéma sur le coup je me dis "bon je synthétise" et j'avoue qu'après je me dis "A l'époque j'étais plus intelligent que maintenant" Ahah!

Citation:

Ta configuration fait beaucoup trop de chose. Ne mélange pas dans un seul fichier le code inactif (fonctions, classes) et le code actif (la partie procédurale).

En faite les fonctions écrire en procédural (url_redirect etc) vont être intégrer à un classe configuration abstraite avec des methodes statiques comme l'autoload etc. fin que je centralise toute ma partie "config" dans un même fichier. (Et puis comme ça tout mon travail sera sous forme de classe.

Citation:

J'imagine que Database::getInstance() retourne un singleton? Est-ce qu'il y une raison précise pour préférer un singleton, plutôt qu'un objet ordinaire.

Parfaitement j'utilise le design pattern singleton. Mais mon couplage est plutot faible (argument du constructeur) donc je peux changer facilement ma classe.
Je sais que les avis sur le singleton sont divergeant mais pour mon utilisation c'est parfait. Définie une seul fois, persistant etc.
J'utilise d'autre chose que je ne montre pas comme une classe MyPDO surchargeant PDO et me permettant d'avoir le nombre de requete de la page, la durée, etc. C'est mon petit "bloc" réutilisable et si j'ai plusieurs BDD j'utilise un multiton.

Et pour le reste j'ai appris le principe de l'objet avec le bouquin que j'utilise toujours (si besoin)
class Menu (affichage des données) et Class MenuManager (requete BDD le CRUD)
Au debut je ne faisais pas comme ça (déjà j'utilisais __set et _get, je n'avais pas de manager)
On m'a dit alors que c'était nul donc j'ai changé mais maintenant je me rend compte qu'il n'y a pas vraiment de "bonne" ou "mauvaise" solution.

C'est surtout question de ressenti.

Apparemment rien ne vous choque dans mon architecture globale donc je suis assez content, et puis bon maintenant j'essaye d'utiliser Symfony2. mais pour mes petits projets ou si j'ai envie de Challenge j'utilise cette architecture.