système de routage

**laurentSc** · 17/03/2023, 10h09

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

Code php :

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
<?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 php :

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
<?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 :

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 ?

**Pytet** · 17/03/2023, 12h15

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';

**cavo789** · 17/03/2023, 12h32

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.

**laurentSc** · 17/03/2023, 14h54

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...

Envoyé par cavo789

En programmation défensive

jamais vu ce terme...

Pytet ; ça marche !

**cavo789** · 17/03/2023, 18h49

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)

**laurentSc** · 17/03/2023, 21h55

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

Code php :

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

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

système de routage

Langage PHP

Vue hybride

Discussions similaires

Partager

Partager