[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

[Niktoś]

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

Przed sql_injection jestem zabezpieczony, korzystam z pdo.

To używasz mysql czy pdo?

Warto by było rozważyć bloki try-catch.Skorzystać z opcji bindowania parametrów:

sth->bindParam(':value', $opinion, PDO::PARAM_INT);

Jeśli parametr nie będzie typu int kwerenda w bloku try sie nie wykona,spowoduje błąd i nastąpi przejście do bloku catch,gdzie będziesz mógł przekierować na inną stronę lub wyświetlić wiadomość o błędzie.

Ten post edytował Niktoś 31.07.2012, 12:17:33