[PHP] Poprawne wczytanie $_POST do klasy Opcje

[k8dz]

Mam formularz i po kliknięciu wyślij potrzebuję przekazać jego wartości do klasy, aby móc na nich wykonać operacje. Zastanawiam się jak to poprawnie powinno być wykonane. Czy takie rozwiązanie jest poprawne, czy powinienem to jakoś inaczej zrobić?

[PHP] pobierz, plaintext 
$cena_plyty = 25; //domyślna wartość zmiennej
 
if ( isset($_POST['cena_plyty']) and is_numeric($_POST['cena_plyty']))
{
	$cena_plyty = $_POST['cena_plyty'];
}
 
$wycena = new Wycena;
$wycena->ustaw_cene($cena_plyty);
echo $wycena->wyswietl_wycene();
[PHP] pobierz, plaintext 

[vokiel]

Tak, generalnie jest OK.

Chociaż często stosuje się też inny sposób działania - wyświetlanie błędów w przypadku nieudanej walidacji danych wejściowych (zamiast przepuszczania na domyślnych wartościach). Tym samym jeśli tylko gdzieś trafisz na nieprawidłowe dane to nie próbujesz ich naprawiać, nie podstawiasz danych domyślnych (bo mogą spowodować anomalie), tylko zwracasz użytkownikowi błąd i pokazujesz gdzie zostały wpisane niepoprawne dane.