[PHP] Wstawianie tekstu z plików, Przerobienie skrypy by działał w register_globals off Opcje

[-Kościej-]

Przez kilka ostatnich lat korzystałem z takiego małego skryptu, który ułatwiał mi bardzo życie. Na górze menu i poniżej treść.
Wszystko było by bardzo fajnie gdyby nie to, że bez włączonego register_globals nie działa poniższy kod.

[PHP] pobierz, plaintext 
	<? 
$rozsz=".php";
if(file_exists("modules/$show$rozsz")){ 
include("modules/$show$rozsz"); 
}
else {
include("modules/index$rozsz"); 
}
?>
[PHP] pobierz, plaintext 

Adres:

[HTML] pobierz, plaintext 
?show=plik
[HTML] pobierz, plaintext 

Po jakimś czasie udało mi się wypłodzić coś takiego

[PHP] pobierz, plaintext 
	<? 
$rozsz=".php"; 
if(file_exists("modules/".$_GET['show']."$rozsz")){ 
include("modules/".$_GET['show']."$rozsz"); 
} 
else{ 
include("modules/index$rozsz"); 
}
 
?>
[PHP] pobierz, plaintext 

Niby wszystko działa, ale teraz rodzi się pytanie do Was, czy jest to dobrze zrobione i czy tak to może działać, czy za tydzień error_log będzie miał kilka GB?

[ADeM]

Tak będzie trochę lepiej:

[PHP] pobierz, plaintext 
if( isset( $_GET['show'] ) && file_exists( "modules/" . $_GET[ 'show' ] . $rozsz ) ) { 
include( 'modules/' . $_GET[ 'show' ] . $rozsz );
[PHP] pobierz, plaintext 

[-Kościej-]

Całość teraz wygląda tak:

[PHP] pobierz, plaintext 
	<? 
$rozsz=".php"; 
if( isset( $_GET['show'] ) && file_exists( "modules/" . $_GET[ 'show' ] . $rozsz ) ) 
{ include( 'modules/' . $_GET[ 'show' ] . $rozsz );
}
else {
include("modules/index$rozsz"); 
}
?>
[PHP] pobierz, plaintext 

Dziękuję bardzo za pomoc