Problem z Include Opcje

[Fragin]

Witam.
Czy ten oto kod będzie bezpieczny? Tzn. że nikt np. nie będzie mógł podglądać plików?

Kod

<?php
    if ( $_GET['strona'] == "" ){
        include("news.php");
    }

    if ( $_GET['strona'] == "kontakt" ){
        include("kontakt.php");
    }
    else {
        include("error.php");
    }

?>

Bo troszkę go zmieniłem (w oryginale był krótszy), ale teraz jest problem bo jeśli wpisze samą nazwę strony (coś.pl) to zawartość news.php pojawia się dwa razy... A jeśli po strona= będzie np. omg to skrypt zamiast error.php pokazuje news.php. Można coś z tym zrobić? Szczególnie, że dość wolno to chodzi.
Dziękuje za pomoc

Ten post edytował Fragin 2.02.2010, 23:06:29

[Fifi209]

A ja robię to tak:

[PHP] pobierz, plaintext 
	if ($_GET['action']) {
		if (file_exists('actions/'.$_GET['action'].'.php')) {
			include_once('actions/'.$_GET['action'].'.php');
		}else{
			echo 'Not Found';
		}
	}else{
		//include_once('actions/news.php');
	}
[PHP] pobierz, plaintext 

Zaraz ktoś powinien czepić się zabezpieczeń, ale wszystko lata przez .htaccess więc dziwne znaki i tak nie przejdą.

[XianN]

Zaraz ktoś powinien czepić się zabezpieczeń, ale wszystko lata przez .htaccess więc dziwne znaki i tak nie przejdą.

Mozesz pokazac ten kawalek .htaccess ?

[Fifi209]

Mozesz pokazac ten kawalek .htaccess ?

Kod

RewriteRule ^([a-z]+)/$ index.php?action=$1 [L]

Proszę. (IMG:style_emoticons/default/smile.gif)