PHP : organiser les pages web

Bon, rapidement, voici, ce que j'aurais fait pour améliorer ton code sans en changer foncièrement la logique voulu initialement.

j'aurais toujours laissé un paramètre $_GET['page'] cependant j'aurais vérifié par sécurité que ce dernier ne contient que des caractère de l'alphabet en minuscule ainsi que des chiffres. Cela se fait super simplement avec la fonction preg_match(). En l'occurrence le pattern est le suivant, '/^[a-z0_9]+$/' donc là j'ai au moins un caractère (sinon j'aurais eu * au lieu de +) et ce n'est que des chiffres ou des lettre minuscule. Laisser plus de caractère est un peu risqué, surtout le / et le . avec la méthode que je vais utiliser maintenant donc voilà.

Maintenant je pars du principe que j'ai un répertoire avec l'ensemble de mes partials de page. Mes morceaux de page si tu préfères. Tu vas stocker ce répertoire dans un fichier de configuration par ex ou bien tu peux dire que ce répertoire est qqpart en fonction de là ou du est. ex.

$dir = dirname(__FILE__).'/../pages

La je reprend ce que j'avais expliqué il y a qqjours avec le répertoire include différent du répertoire www. Je crois que c'était à toi que j'expliquais ça sur un problème de .htaccess non ? Bref l'intéret est d'avoir un $dir qqpart.

Maintenant je sais que dans $page provenant de $_GET['page'] j'ai une chaine alphanumérique parfaite pour faire un nom de fichier si je rajoutais un '.php' par exemple. Faisons !

Pour rappel preg_match va renvoyer 1 si il trouve soit vrai. Donc on doit renvoyer une erreur si il renvois 0 si il ne trouve pas.

$dir = dirname(__FILE__).'/../pages;
$page = $_GET['page'];
 
if(!preg_match('/^[a-z0_9]+$/') {
  // hum bizarre ce qu'on m'a envoyé là ... 
  // je retourne une 404
  include $dir.'/error404.php';
  exit; // on vérifie bien qu'on ne va pas plus loin dans l'exécution surtout.
}
 
//on va déduire le nom du fichier à partir de $page
$filename = $page.'.php';

Maintenant j'ai un répetoire et un nom de fichier, bah j'ai plus qu'à regarder si le fichier existe non ? oui  

$dir = dirname(__FILE__).'/../pages;
$page = $_GET['page'];
 
if(!preg_match('/^[a-z0_9]+$/') {
  // hum bizarre ce qu'on m'a envoyé là ... 
  // je retourne une 404
  include $dir.'/error404.php';
  exit; // on vérifie bien qu'on ne va pas plus loin dans l'exécution surtout.
}
 
//on va déduire le nom du fichier à partir de $page
$filename = $page.'.php';
$file = $dir.'/'.$filename;
if(file_exist($file)) {
  include $file;
} else {
  // sinon on demand eune page qui n'existe pas 404.
  include $dir.'/error404.php';
}

Maintenant pour ajouter des pages tu n'as plus qu'à rajouter des fichiers avec un nom correct dans le répertoire qui va bien   Tu n'as plus à toucher à la logique de ton code et rajouter des infos dans des tableaux etc.

Alors il y a plein de limite, mais bon, c'est du code à l'ancienne on dira.