[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]

przecież napisałem Tobie....
Twój problem to

[PHP] pobierz, plaintext 
if(isset($_GET['nazwa']) && isset($_GET['dataR']) && isset($_GET['dataZ']) && isset($_GET['numerZ']))
[PHP] pobierz, plaintext 

za każdym razem to ten warunek u Ciebie decyduje o tym, co robić dalej. Po za tym, zobacz jak wygląda URL po wysłaniu pustego formularza. jeśli sa w nim wypisane zmienne bez wartości to się nie dziw, ze spełnia sie powyższy warunek bo zmienne GET istnieją (łał)
Kolejnym błędem, jest sam ten skrypt. Nie chciałbym natrafić na "coś takiego", tym bardziej podawać w takim czymś swoje dane.

nie wiem tez dalczego napisałeś, że masz problem z walidajcą formularza, skoro Ty nic tak naprawdę nie walidujesz.
Kompletnie nic nie sprawdzasz, po za tym czy istnieją zmienne GET.

Napisałbym,żebyś przewertował manuala lub przeanalizować gotowe skrypty walidacji ale zrób do końca ten skrypt po swojemu a z czasem sam zobaczysz co to za potwór i może zmotywujesz się do większego wysiłku ;d

edit:

własnie... jeszcze jedno. Sprawdz - bo ja nie mam juz czasu a sam nie pamietam bo dawno dawno nie używałem GET, czy zmienne tworzą się w URL mimo tego, że sa puste. Jeśli tak to jest twój problem. Zamiast warunków sprawdzających czy te zmienne istnieją, sprawdz lepiej czy te zmienne posiadaja odpowiednie dane i po kłopocie. Tak powinieneś zrobić, skoro piszesz, że jest to "walidacja" ;p
po zakończeniu dodania do bazy, wstaw header("location: jakis_plik.php");
Mimo wszystko radziłbym Tobie zmienić GET na POST.

Ten post edytował gitbejbe 19.06.2013, 14:00:08