Forum PHP.pl

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 

W którym miejscu pojawia się problem?

[HTML] pobierz, plaintext 
<input type="submit" class="button" value="Wyślj" />
[HTML] pobierz, plaintext 

Przycisk submit ma być Twoim warunkiem wywołania skryptu walidacji. Tak więc dodaj do tego submit name="cos tam" i w pierwszym warunku zamiast:

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

wstaw

[PHP] pobierz, plaintext 
if(isset($_GET['nazwa przycisku']))
[PHP] pobierz, plaintext 

po za tym sugeruje robić to za pomocą metody POST

Ten post edytował gitbejbe 19.06.2013, 06:17:29

Właśnie problem, polega na tym, że nie mogę zlokalizować w tym kodzie, miejsca błędu.
W tym przypadku , akurat musze wykorzystać metodę _GET.

Dodałem też dodatkową instrukcję if sprawdzającą przycisk i dalej mam ten sam problem, że wysyłam pusty formularz.
Chciałbym właśnie, aby mi przełądowało stronę, na ponowne wprowadzenie danych do formularza.

Nie wiem jak sobie z tym poradzić :/

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

Fakt, masz rację, trochę źle sprecyzowałem swój problem.
W takim razie zabieram się do roboty (IMG:style_emoticons/default/smile.gif)

A powiedz mi, można jakoś sensownie
zrobić sprawdzanie pól w formularzu, ale tak, że nie wszystkie pola muszą być wypełnione aby formularz został wysłany poprawnie ?
Jeśli tak, to czy mógłbyś, oświecić mnię trochę swoją wiedzą ?

Pozdrawiam
Marian

Możesz sprawdzać poprawność danych jeśli określone pole nie jest puste. Tj. jeśli jest to dajmy na to nr telefonu, to czy wpisano tylko cyfry, czy wpisano ich odpowiednią ilość, itp. No i jeśli nie chcesz dodawać pustych wartości to ustal, które pola minimalnie muszą być wypełnione.

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