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];
}

**cavo789** · 17/03/2023, 22h03

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.

**laurentSc** · 17/03/2023, 22h27

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

système de routage

Langage PHP

Discussions similaires

Partager

Partager