comment ameliorer ce code?

**chrisl0** · 25/02/2009, 21h18

bonjour voila j'ai fais ce petit script pour faire des pseudo-frame
et je voulais savoir si il etait potable :

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
32
33
34
35
 
<html>
<head>
<title>Mon site</title>
</head>
<body>
<a href="index.php?page=livre">Signez le livre d'or</a><br></br>
<a href="index.php?page=contact">Nopus contactez</a><br></br>
<a href="index.php?page=imc">Calculez votre indice de masse corporel</a><br></br>
<?php
// Tableau des pages autorisées
 
$pages = array(
"livre" => "livre.php",
"contact" => "contact.php",
"imc" => "imc.php"
); // fin du tableau
if(!empty($_GET['page'])){
//Récupération du paramètre d'URL
$page = $_GET["page"];
 
// On teste si la page existe et si oui on l'incut
if (isset($pages[$page])) { 
  include($pages[$page]); 
}
// Sinon on inclut une page par défaut
if (empty($_GET['page']) || !isset($_GET['page']))
{
include ('lol.php'); /* Inclusion de la page par défaut si la superglobale est vide */
}
}
 
?>
</body>
</html>

merci d'avance pour votre aide

**N1bus** · 26/02/2009, 02h29

Bonjour,

Ce serait peut être plus lisible avec un switch() :

Code :

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

1
2
3
4
5
6
7
8
9
 
$page = $_GET["page"];
 
switch($page){
  case "livre"   : include("livre.php");   break;
  case "contact" : include("contact.php"); break;
  case "imc"     : include("imc.php");     break;
  default        : include("lol.php"); 
}

comment ameliorer ce code?

Langage PHP

Vue hybride

Discussions similaires

Partager

Partager