[MySQL][PHP] Problem z walidacją formularza Opcje

[--Marian--]

Witam,
mam problem z walidacją formularza.
Stworzyłem formularz który dodaje mi wydarzenia do bazy.
Problem w tym, ze nawet jak nic nie wpiszę do formularza, to mi wysyła go i wyswietla komunikat że dodało wydarzenie do bazy,
czego oczywisscie fizycznie nie wykonało. Jeśli podam wszystkie pola, to wydarzenie normalnie jest dodawane do bazy,
chciałbym też mieć możliwość wypełnienia nie wszystkich pól formularzai żeby dodawane zostało wydarzenie do bazy z mniejszą iloscią danych.
Przesyłam kody:

[formularz]

[HTML] pobierz, plaintext 
 
<form method="GET" action="dodawanieNiestac.php">
<div class="form">
<legend>Dodaj Wydarzenie</legend>
 
<label>
<span>Nazwa Wydarzenia</span>
<input type="text" class="add" name="nazwa" />
</label>
 
<label>
<span>Inna informacja</span>
<input type="text" class="add" name="inna"/>
</label>
 
<label>
<span>Numer Zjazdu</span>
<input type="text" class="add" name="numerZ"/>
</label>
 
<label>
<span>Data Rozpoczęcia</span>
<input type="text" class="add" name="dataR"/>
</label>
 
<label>
<span>Data Zakończenia</span>
<input type="text" class="add" name="dataZ"/>
</label>
 
<label>
<input type="submit" class="button" value="Wyślj" />
</label>
</div>
</form>
 
 
[HTML] pobierz, plaintext 

[Obrabianie formularza]

[PHP] pobierz, plaintext 
 
$nazwa=$_GET['nazwa'];
$inna=$_GET['inna'];
$numerZ=$_GET['numerZ'];
$dataR=$_GET['dataR'];
$dataZ=$_GET['dataZ'];
 
$data = $dataR;
 
 
 
 
	if(isset($_GET['nazwa']) && isset($_GET['dataR']) && isset($_GET['dataZ']) && isset($_GET['numerZ']))
	{
 
	$result = mysqli_query($con,"SELECT * FROM niestacjonarne WHERE wydarzenie='".$nazwa."' AND data='".$data."'");
	$row=mysqli_fetch_array($result,MYSQLI_NUM);
 
			if ($row == 0 ) 
			 {
			 	while ($data <= $dataZ)
				 {
		 	mysqli_query($con,"INSERT INTO niestacjonarne (numer_zjazdu, data, wydarzenie) VALUES ('".$numerZ."', '".$data."', '".$nazwa."')");		
 
			$data = strtotime($data . '+ 1 day');
				 }
			echo "Dodałeś nowe wydarzenie do bazy !";
			echo"<p><a href='logowanie.php'>Powrót</a></p>";
 
			 }
			else
				{
		  		require_once('logowanie.php');	
		  		echo "Istnieje już takie wydarzenie w tym dniu !"; 
           		}
 
}
	elseif(isset($_GET['nazwa']) && isset($_GET['dataR']) && isset($_GET['dataZ']) &&  isset($_GET['inna']) && isset($_GET['numerZ']))
			{
				$result = mysqli_query($con,"SELECT * FROM niestacjonarne WHERE wydarzenie='".$nazwa."' AND data='".$data."'");
	$row=mysqli_fetch_array($result,MYSQLI_NUM);
 
			if ($row == 0 ) 
			 {
			 	while ($data <= $dataZ)
				 {
		 	mysqli_query($con,"INSERT INTO niestacjonarne (numer_zjazdu, data, wydarzenie, dodatek) VALUES ('".$numerZ."', '".$data."', '".$nazwa."', '".$inna."')");		
 
			$data = strtotime($data . '+ 1 day');
				 }
			echo "Dodałeś nowe wydarzenie do bazy !";
			echo"<p><a href='logowanie.php'>Powrót</a></p>";
 
			 }
			else
				{
		  		require_once('logowanie.php');	
		  		echo "Istnieje już takie wydarzenie w tym dniu !"; 
           		}
			}
 
 
	elseif(isset($_GET['nazwa']) && isset($_GET['dataR']) && isset($_GET['dataZ']))
	{
		$result = mysqli_query($con,"SELECT * FROM niestacjonarne WHERE wydarzenie='".$nazwa."' AND data='".$data."'");
	$row=mysqli_fetch_array($result,MYSQLI_NUM);
 
			if ($row == 0 ) 
			 {
			 	while ($data <= $dataZ)
				 {
		 	mysqli_query($con,"INSERT INTO niestacjonarne (data, wydarzenie) VALUES ('".$data."', '".$nazwa."')");		
 
			$data = strtotime($data . '+ 1 day');
				 }
			echo "Dodałeś nowe wydarzenie do bazy !";
			echo"<p><a href='logowanie.php'>Powrót</a></p>";
 
			 }
			else
				{
		  		require_once('logowanie.php');	
		  		echo "Istnieje już takie wydarzenie w tym dniu !"; 
           		}
	}
 
             else
				{
		  		require_once('logowanie.php');	
 
           		}		
 
 
 
 
 
[PHP] pobierz, plaintext 

[gitbejbe]

tak jak napisał kolega wyżej.

To ty ustalasz co ma zawierać pole i które z nich ma być obowiązkowe do wypełnienia. Walidacja to mechanizm składająca się z 3 głównych kroków
- pobranie danych
- sprawdzenie danych
- wysyłka danych - czy jak to innaczej ująć ;p

lecąc od początku
walidacje zawsze zaczynaj od warunku. Najlepszym z nich jest sprawdzenie czy został wciśnięty submit. Najprostrze i najlepsze rozwiązanie. Dopiero w tym warunku pobierasz dane i robisz reszte rzeczy.

Każde pole sprawdzaj pod względem dozwolonych znaków. Email ma być emailem, tel ma być tel a imie czy nazwisko ma nie być puste, nie krótsze niż 3 naki i nie dłuższe od 20 + zawierać tylko litery. Do tego przydadzą Ci się wyrażenia regularne.

Dopiero jak wszystko jest ok, przechodzisz do zapisu do bazy danych i kończysz walidacje.

wskazówka: przed zapissem do bazy danych musisz postawić warunek w którym sprawdzisz czy wszystkie dane z formularza są poprawne. Wszystkie informacje o nieprawidłowym wypełnieniu formularza zapisuj do jednej zmiennej, łatwiej Ci będzie wyświetlić błędy i stworzyć odpowiedni warunek.
np:

stworz zmienną np $error = '';
i teraz każdy błąd dodawaj w ten sposób : $error.= 'tresc bledu<br>';
jeśli chcesz wyświetlić wszystkie błędy dajesz poprostu echo $error;
Też kiedyś byłem początkujący i pamietam, ze to bardzo mi pomogło, bo pisałem mase niepotrzebnych warunków i zmiennych ; )

i przed dodaniem do bazy dajesz warunek if(empty($error)) { //kod zapisu do bazy }

edit:
mały prezent ode mnie. Sprawdzenie maila ;p pewnie i tak byś szukał po googlach
if(preg_match('/^[a-zA-Z0-9.\-_]+\@[a-zA-Z0-9.\-_]+\.[a-z]{2,4}$/D', $email) == false) { $error.='Błędny adres email'; }

Ten post edytował gitbejbe 19.06.2013, 18:11:11