Witam!
Mam pewien skrypt i w nim kawałek kodu:

[PHP] pobierz, plaintext 
<?php
	if(empty($_GET['page']) or $_GET['page']=='newsy') { include('newsy.php'); }
	elseif($_GET['page']=="rejestracja") { include("rejestracja.php"); }
	elseif($_GET['page']=="logowanie") { include("panel-uzytkownika.php"); }
	elseif($_GET['page']=="admin") { include("panel-admina.php"); }
	elseif($_GET['page']=="wyloguj") { include("wyloguj.php"); }
	elseif($_GET['page']=="login") { include("newsy.php"); }
	elseif($_GET['page']=="komentarz") { include("komentarz.php"); }
	elseif($_GET['page']=="lista") { include("lista.php"); }
	elseif($_GET['page']=="profil") { include("profil.php"); }
	elseif($_GET['page']=="mecze") { include("mecze.php"); }
	elseif($_GET['page']=="galeria") { include("galeria.php");; }
	elseif($_GET['page']=="druzyny") { include("druzyny.php"); }
	elseif($_GET['page']=="kadra") { include("kadra.php"); }
	elseif($_GET['page']=="kontakt") { include("kontakt.php"); }
	elseif($_GET['page']=="moj-profil") { include("moj-profil.php"); }
	elseif($_GET['page']=="newmail") { include("newmail.php"); }
	elseif($_GET['page']=="mymail") { include("mymail.php"); }
	elseif($_GET['page']=="sentmail") { include("sentmail.php"); }
	elseif($_GET['page']=="przypomnij") { include("przypomnij-haslo.php"); }
?>
[PHP] pobierz, plaintext 

według mnie nie wygląda to zbyt ładnie. Jest jakiś sposób aby to uprościć i aby to wyglądało w miarę profesjonalnie?
Pozdrawiam

może switch lub dynacznie ale wtedy musisz troche zabezpieczyć skrypt

edit: @down : racja napisałem głupotę, nie pomyślałem.

@up. Sprawdź sam jak to zadziała

[PHP] pobierz, plaintext 
$list = array('rejestracja.php' => 'rejestracja', 'logowanie.php' => 'logowanie');//uzupełnij sobie sam ;)
$get = $_GET['page'];
if( ($filename = array_search($get, $list))  != false )
{
   include $filename;
}
else
{
  //error 404, chyba masz bana
}
[PHP] pobierz, plaintext 

Walnąłem głupotę więc się poprawiam

[PHP] pobierz, plaintext 
<?php
 
    $list = array('rejestracja' => 'rejestracja.php', 'logowanie' => 'panel-uzytkownika.php');
    $get = $_GET['page'];
 
    switch($get) {
    	case array_key_exists($get, $list):
    	include($list[$get]);
    	break;
    	default:
    	include('error-page.php');
    	break;
   }
?>
[PHP] pobierz, plaintext 

Bardzo dużo elementów tablicy posiada taki sam klucz i nazwę pliku (tyle, że z rozszerzeniem .php).
Możesz zrobić tablicę, w której przechowywać będzies tylko te strony, które posiadają różne pliki w stosunku do nazwy w adrsie URL.
I teraz tak: sprawdzać, czy istnieje w tablicy wyjątek ('logowanie' => 'panel-uzytkownika.php') i jeśli tak, ładować te pliki, a jeśli nie ma w tablicy, to ma załadować plik o tej samej nazwie co request , tylko dodać .php na końcu.
Oczywiście ze względów bezpieczeństwa proponuję sprawdzić, czy nie ma nigdzie w przekazanej wartości żadnych niebezpiecznych ciągów, oraz czy plik istnieje (np. w określonej na sztywno lokalizacji).
Jeżeli nie będzie pliku na serwerze, wtedy include jakiejś strony z błędem.

@mieszkos

tzn. walisz większą głupotę ?

@up

czemu ?

[PHP] pobierz, plaintext 
$modules = array(
       'default' => 'default.php',
       'reg' => 'rejestracja.php',
       'graph' => 'graph/graph.php'
);
 
$id = !empty($_GET['id']) ? $_GET['id'] : 'default';
 
 
if(isset($modules[$id])) {
 
      require_once $modules[$id];
}
else {
 
      require_once $modules['default'];
}
[PHP] pobierz, plaintext 

Dziękuję bardzo za pomoc o to mi chodziło
Można zamknąć