[php]mon bout de code est laid

Marsh Posté le 19-11-2002 à 15:49:08

Code :

$file[0]="/";
if($_GET['page']=='')
{
$file[1]="./";
}
else
{
$file[1]=$_GET['page'];
}
if($_GET['sec']=='')
{
$file[2]="index.php";
}
else
{
$file[2]=$_GET['sec'].".php";
}
$file=$file[1].$file[0].$file[2];
if(is_file($file))
{
include($file);
}

bon voilà, comme vous pouvez le voir ce bout de code récupere
dans l'url les infos concernant page et sec.
vérifie si le fichier existe, et inclu le fichier

mais je trouve un peu degueulasse mon idée sur les tableaux !

comment pourrais-je faire autrement !!

sinon ça fonctionne à la rigueur m'enfin c po bo!!

Marsh Posté le 19-11-2002 à 15:49:08

Marsh Posté le 19-11-2002 à 15:55:05

Tu peux déjà simplifier ton code, en virant les accolades non nécessaires.

Le $file[0] est toujours un slash? Pas nécessaire non plus.

Sinon, pas grand-chose à redire...

Message édité par Fred999 le 19-11-2002 à 15:55:15

Marsh Posté le 19-11-2002 à 17:40:56

Je pense que ça peut se réduire comme ça :

$file = ($_GET['page'] == '' ? "./" : $_GET['page'])."/".($_GET['sec'] == '' ? "index.php" : $_GET['sec'].".php" );

if(is_file($file))
include($file);

On pourrait même peut-être inclure l'affectation dans le is_file (si c'est géré comme en C), mais c'est déjà assez réduit comme ça. Un chouilla moins lisible peut-être par contre, et encore...

Message édité par sielfried le 19-11-2002 à 17:41:04

---------------
StarCraft Professional Gaming Database | [Ze Topic] Starcraft/BroodWar

Sujets relatifs:

Leave a Replay

Make sure you enter the(*)required information where indicate.HTML code is not allowed

Name *

Email *

URL

Message *