problem z Switch Opcje

[tdr]

Oto mój kod:

[PHP] pobierz, plaintext 
<?php
if(isset($_GET['strona'])){
switch($_GET['strona']){
    case 'kontakt':
        $url="includes/kontakt.php";
        $tytul="Kontakt";
        $opis="Zaczerpnij kontaktu o nas";
    break;
 
     case 'onas':
        $url="includes/onas.php";
        $tytul="O NAs";
        $opis="Coś więcej o Nas";
    break;
 
     case 'rejestruj':
        $url="includes/rejestruj.php";
        $tytul="Rejestracja";
        $opis="Zarejestruj się.";
    break;
 
    default:
        $url="includes/glowna.php";
        $tytul="Strona glowna";
        $opis="Opis strony głównej.";
    }
}else{
    $url = 'includes/glowna.php';
    $tytul="Strona glowna";
    $opis="Opis strony głównej.";
}
?>
[PHP] pobierz, plaintext 

CHciałbym stworzyć skrypt który ma posłużyć za index.php, ale już sam nie wiem, czy dobrze robie... co dalej miałbym zrobić aby zaladować poszczególne strony?

Ten post edytował tdr 6.05.2013, 14:45:50

[Crozin]

1. Przede wszystkim: DRY - w bloku DEFAULT i ELSE masz dokładnie to samo nie wiadomo po co.
2.

[PHP] pobierz, plaintext 
$strona = isset($_GET['strona']) ? $_GET['strona'] : '';
 
switch ($storna) {
  ...
}
[PHP] pobierz, plaintext 

3. Kod wydaje się być OK, daj sobie po SWITCHu:

[PHP] pobierz, plaintext 
var_dump($strona, $url, $tytul, $opis);
[PHP] pobierz, plaintext 

I zobacz co zwraca.