[PHP] $_GET - kilka pytań Opcje

[d.stp]

Mam kilka pytań odnośnie tablicy $_GET

1. Czy takie coś jest bezpieczne:

[PHP] pobierz, plaintext 
switch($_GET['page']) {
	case 'rejestracja':
		include('./includes/register.php');
	break;
	default:
		include('./includes/default.php');
	break;
}
[PHP] pobierz, plaintext 

2. Robię mini cmsa i mam w pliku index.php takie coś jak wyżej i w miejscu gdzie mam content ładuje pliki z $_GET - to dobre rozwiązanie?

3. Mam na stronie podstronę artykuły i każdy artykuł ma swoje id i po wejściu np. w index.php?page=articles&id=2 ładuję artykuł z bazy danych mysql, a zabezpieczam tak:

[PHP] pobierz, plaintext 
if ($_GET['page'] == 'articles' and is_numeric($_GET['id'])) // wyswietl artykul
[PHP] pobierz, plaintext 

jeszcze sprawdzam w bazie czy artykuł o takim ID istnieje:

[PHP] pobierz, plaintext 
if (mysql_num_rows($id) == 0) // nie istnieje
[PHP] pobierz, plaintext 

takie coś wystarczy?

Przed sql_injection jestem zabezpieczony, korzystam z pdo. Te kody wyżej to tylko przykłady

[Evinek]

1 - jest okej, ale oczywiście można zrobić to lepiej. Ogólnie do miniCMS będzie raczej okej.
2 - Źle. Oddziel logikę od prezentacji. Poczytaj o klasach szablonów. http://forum.php.pl/index.php?showtopic=201664 - tu możesz przejrzeć trochę mojego kodu, może zrozumiesz.
3 - $id = (int)$_GET['id'] - chyba lepiej będzie. Ogólnie idziesz w dobrą stronę z zabezpieczeniami.
4 - Dobre sprawdzanie.

@DOWN:
Teraz widzę jak mało wiem. ;p

Ten post edytował Evinek 31.07.2012, 12:15:38