Codice..va bene?

[Sora88]

<?
$page=$_GET['pg'];
if (!$page){$url="index.html";}
else {
switch($page){
case 'primo':
$url='primo.html';
break;
case 'secondo':
$url='secondo.htm';
break;
....ecc...
}
}
if (!file_exists($url)){$url='error.htm'}
else {include($url);}
?>

Va bene questo codice?? ( i link sono index.php?pg=**)

[mdsjack]

il miglior modo per saperlo è provarlo, no?

[Sora88]

^_^ Vero...provato tutto ok! ^^

Volevo solo chiedere..magari c'è un modo migliore, qualche correzione! ^^

[aserena]

Originariamente inviato da Sora88
Va bene questo codice?? ( i link sono index.php?pg=**)

Per andar bene va bene, solo che potresti pulirlo un pochettino:

<?
$page=$_GET['pg'];
 if (!$page)
 $url="index.html";
 else {
  switch($page){
  case 'primo':
  $url='primo.html';
  break;
  case 'secondo':
  $url='secondo.htm';
  break;
  ....ecc...
  }
 }
 if (!file_exists($url))
 $url='error.htm';
 else include($url);
?>

Se hai una sola istruzione nell'if non serve che ci metti le parentesi graffe, poi se lo identi diventa ancora + leggibile ciao