Discussion :

[FreelancerAnju]

Bonjour, je suis en train de retravailler mon MVC pur l'arrivé de php 7

Donc voici la gestion du controller, action, et la l'inclusion des class et methode.

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
$params = explode('/', $_GET['controller']);  
if ($params[0] == NULL) {
    $params[0] = 'Home';
    $controller = $params[0];
} else $controller = $params[0];
 
if ($params[1] == NULL) {
    $params[1] = 'index';
    $Action = $params[1];
} else $controller = $params[0];
 
if(file_exists('../application/controllers/'.$controller.'.php')){
   require '../application/controllers/'.$controller.'.php';
   $controller = new $controller;  
    if (method_exists($controller, $Action))  
    {  
        $controller->$Action();  
    }else{
        echo "La classe ou la methode n'existe pas !";
    }
}else{
    echo "Page introuvable";
}

Merci de m'aider a optimiser mon code.

[rawsrc]

Salut,

cette partie de code

Code :

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

1
2
3
if(file_exists('../application/controllers/'.$controller.'.php')){
   require '../application/controllers/'.$controller.'.php';
   $controller = new $controller;

devrait être remplacée par un système d'autoloading et de ton côté tu devrais avoir quelque chose dans ce genre

Code :

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

1
2
3
4
5
6
<?php
 
use namespace\Class;
 
$controller = new Class();
// reste du code

[FreelancerAnju]

Merci j'aurai parié que vous allez parler de namesapce.

Mais je ne comprends comment les utiliser.

Pour ce qui est de autoload j'utilise déjà !

Donc le voici :

Code :

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

1
2
3
4
 
function __autoload($class_name) {
    include ROOT . '/application/controllers/' . $class_name . '.php';
}

[rawsrc]

Allez un peu de lecture parce que ton autoloader me semble vachement léger : ici

[FreelancerAnju]

Et pour namespace comment on les utilise ?

[Gaulouis]

et de ton côté tu devrais avoir quelque chose dans ce genre

Code :

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

1
2
3
4
5
6
<?php
 
use namespace\Class;
 
$controller = new Class();
// reste du code

Et pour namespace comment on les utilise ?

A mon avis, FreelancerAnju souhaiterais avoir un factory de controller. Et obtenir :

Code :

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

1
2
$controller_class_name = 'AppName::ModuleName::'.$controller;
$controller = Controller::Factory($controller_class_name);