système de routage

Version imprimable

système de routage

Bonjour,
j'ai le système de routage suivant :
routing.php :

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
<?php declare(strict_types=1);
 
$routes = [
    'vos-temoignages' =>'/controller/Vos-temoignages',
    'politique-de-confidentialite' =>'/controller/Politique-de-confidentialite',
    'nous-contacter' =>'/controller/Nous-contacter',
    'mentions-legales' =>'/controller/Mentions-legales',
    'home' =>'/controller/Accueil',
    'tarifs' =>'/controller/Tarifs',
    'les-nids-et-les-insectes' =>'/controller/Les-nids-et-les-insectes',
    'conseils' =>'/controller/Conseils',
    'action-form-contact' =>'/controller/Action-form-contact',
    'etre-rappele' =>'/controller/Etre-rappele',
    'action-form-rappel' =>'/controller/Action-form-rappel',
 
];
 
$action = isset($_GET['action'], $routes[$_GET['action']]) ? $_GET['action'] : null;
 
if ($action === null) {
    $ctrl = '/controller/Accueil';
} else {
    $ctrl = $routes[$action];
}
 
$page = $ctrl.".php";

et la page d'entrée du site (index.php) :

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
<?php
 declare(strict_types=1);
   const DIR_ROOT = __DIR__.DIRECTORY_SEPARATOR;
   const APP_NAME_NS = 'PasDePanique';
 
   require DIR_ROOT.'vendor'.DIRECTORY_SEPARATOR.'autoload.php'; //autoload de composer
   require DIR_ROOT.'config'.DIRECTORY_SEPARATOR.'bootstrap.php';//definition of the constants
 
   // ROUTING : en charge d'appeler le bon controleur
   include 'config/routing.php';
   ?>
 
<html>
<?php
    include("controller/head.html");
?>
<body>
<?php
    include("controller/header.html");
    include("controller/menu.html");
?>
    <main class="container">
        <section>
            <?php include $page; ?>
        </section>
    </main>
 
    <?php include("controller/footer.html");?>
<script type="text/javascript" src="../main.js"></script>
</body>
</html>

Donc chaque page sera en fait incluse dans index.php (ligne 24 de index.php). Simplement, c'est pas la bonne façon d'inclure un fichier car :

Citation:

Warning: include(/controller/Accueil.php): Failed to open stream: No such file or directory in C:\projets\arnaud\index.php on line 24

Quelle serait la bonne syntaxe SVP ?

17/03/2023, 13h15
Pytet

Bonjour,

Essaye d'enlever le slash initial pour utiliser des chemins relatifs dans la déclaration de tes routes et pour la route par défaut (comme pour les autres include) : 'home' =>'controller/Accueil' et $ctrl = 'controller/Accueil';
17/03/2023, 13h32
cavo789

Tu fais une erreur par omission : tu fais un include de $page sans même avoir vérifié que le fichier existe.

En programmation défensive, c'est une faute et cela mène à des bugs ultérieurs.
17/03/2023, 15h54
laurentSc

Citation:

Envoyé par cavo789

Tu fais une erreur par omission : tu fais un include de $page sans même avoir vérifié que le fichier existe.

Etant le seul programmeur, les fichiers indiqués dans routing.php existent forcément...

Citation:

Envoyé par cavo789

En programmation défensive

jamais vu ce terme...

:merci: Pytet ; ça marche !
17/03/2023, 19h49
cavo789

Ouch... Que tu sois seul ou que vous travaillez à plusieurs cela n'a rien à voir.

Quand tu fais un include il est sain de vérifier que le fichier existe avant. Ton projet va évoluer et peut-être qu'un jour tel fichier disparaîtra pour x raisons.

Si tu ne fais pas cette vérification, c'est très bof et un code peu propre.

(et si tu avais écrit le test, tu n'aurais pas poster ici car immédiatement, tu aurais compris la source du problème)

J'ai mis du temps à mettre au point le test, mais ça y est :

Code:

1
2
3
4
5
6
 
if (($action === null)||(!(file_exists(DIR_ROOT.$routes[$action].".php")))) {
    $ctrl = 'controller/Accueil';
} else {
    $ctrl = $routes[$action];
}

17/03/2023, 23h03
cavo789

Tu avais une variable $page en ligne 26 dans ton tout premier post.

Il suffit de vérifier si file_exists($page) après cette ligne 26.

Si pas trouvé, tu fais un throw() pour stopper ton code avec une exception qui dit clairement pourquoi càd parce que ton fichier n'existe pas.
17/03/2023, 23h27
laurentSc

Pas tout à fait car au moment de faire le test, $page n'existe pas encore...