23-11-2009, 04:44
szczerze mówiąc, to ja tą walidację ciut inaczej bym zrobił
empty() sprawdza Ci czy na pewno zmienna jest pusta, a nie czy jej nie ma... No i przede wszystkim - skąd niby serwer ma wiedzieć, że zmienna $nrproduktu zawierać będzie wartość z formularza? Może właśnie w tym rzecz?
Kod:
$nrproduktu = $_POST["nrproduktu"];
$nazwaproduktu = $_POST["nazwaproduktu"];
$cena = $_POST["cena"];
if (empty($nrproduktu) || empty($nazwaproduktu) || empty($cena))
{
print "Nie zostały wypełnione wszystkie pola";
}
else
{
$nrproduktu = addslashes($nrproduktu);
$nazwaproduktu = addslashes($nazwaproduktu);
$cena = addslashes($cena);
$query = "insert into produkty values ('".$nrproduktu."', '".$nazwaproduktu."', '".$cena."')";
$result = mysql_query($query);
if ($result)
{
print "Towar <b>".$nazwaproduktu."</b> został dodany do bazy danych.";
}
else
{
print "W bazie istnieje już produkt o tym numerze.";
}
}
empty() sprawdza Ci czy na pewno zmienna jest pusta, a nie czy jej nie ma... No i przede wszystkim - skąd niby serwer ma wiedzieć, że zmienna $nrproduktu zawierać będzie wartość z formularza? Może właśnie w tym rzecz?
"Kod ma być koszerny!"
![[Obrazek: 106.gif]](http://www.siemiwidzi.pl/smilies/usmiechy/emocje/106.gif)