Walidacja formularza Opcje

[imie_nazwisko]

Witam,
Zrobilem walidacje formularza, która ma sprawdzac czy uzytkownik wprowadzil wartosci we wszystkie pola formularza.
Walidacja dziala jesli wszytskie elementy tablicy row mają "Yes".

W przypadku gdy w row[1]="No" chcialbym aby zostały sprawdzone pozostale dwa pola plec oraz kraj. Niestety tak się nie dzieje, formularz przechodzi gdy wszytskie pola sa puste

[HTML] pobierz, plaintext 
<script LANGUAGE= "JavaScript" type= "text/javascript">
function validateForm(rejestracja){
 
    var tekst='';
    if (rejestracja.plec.value==""){tekst=tekst+"- plec\n";}
    if (rejestracja.zawod.value==""){tekst=tekst+"- zawód\n";}
    if (rejestracja.kraj.value =="" ){tekst=tekst+"- kraj\n";}
    if (tekst!="") {alert ("Błąd #882: \nNie uzupełniłeś następujących pól: \n\n"+tekst);
        return false;
    } else {
		 return true;
    }
}
 
</SCRIPT>
[HTML] pobierz, plaintext 

[PHP] pobierz, plaintext 
<form class="form-horizontal" method="post" onsubmit="return validateForm(this);">
<fieldset>
<legend>Close concession #<?php echo $nr; ?></legend><br>
 
<?php
	if ($row[0]=='Yes') {
	echo '<div class="form-group">';
	echo '	<label  for="inputPeriod" class="col-lg-5 control-label">Podaj pleć: </label>';
	echo '	<div class="col-lg-2">';
	echo '		<div class="input-group date">';
	echo '		<input maxlength="10" type="text" class="form-control" name="plec" placeholder=""  ><span class="input-group-addon"><i class="glyphicon glyphicon-calendar"></i></span>';
	echo '		</div>';
	echo '	</div>';
	echo '</div>';
	}
 
	if ($row[1]=='Yes') {
	echo '<div class="form-group">';
	echo '	<label  for="inputPeriod" class="col-lg-5 control-label">Podaj zawód: </label>';
	echo '	<div class="col-lg-2">';
	echo '		<div class="input-group date">';
	echo '		<input maxlength="10" type="text" class="form-control" name="zawod" placeholder=""  ><span class="input-group-addon"><i class="glyphicon glyphicon-calendar"></i></span>';
	echo '		</div>';
	echo '	</div>';
	echo '</div>';
	}
 
	if ($row[2]=='Yes') {
	echo '<div class="form-group">';
	echo '	<label  for="inputPeriod" class="col-lg-5 control-label">Podaj kraj: </label>';
	echo '	<div class="col-lg-2">';
	echo '		<div class="input-group date">';
	echo '		<input maxlength="10" type="text" class="form-control" name="kraj" placeholder=""  ><span class="input-group-addon"><i class="glyphicon glyphicon-calendar"></i></span>';
	echo '		</div>';
	echo '	</div>';
	echo '</div>';
	}
[PHP] pobierz, plaintext 

[trueblue]

Z Twojego kodu nigdzie nie wynika co tak naprawdę (i gdzie ustalasz) zawierają elementy tablicy $row.

[imie_nazwisko]

Do tablicy tab trafia wynik zapytania MySQL.
Nie wstawilem tego fragmentu kodu poniewaz uwazam ze jest nieistotny

[trueblue]

Wrzuć jednak przed linią 6:

[PHP] pobierz, plaintext 
var_dump($row);
[PHP] pobierz, plaintext 

i pokaż jaki będzie tego wynik.

[imie_nazwisko]

[PHP] pobierz, plaintext 
array(13) { [0]=> string(2) "No" [1]=> string(2) "No" [2]=> string(2) "No" [3]=> string(3) "Yes" [4]=> string(3) "Yes" [5]=> string(3) "Yes" [6]=> string(3) "Yes" [7]=> string(4) "2250" [8]=> string(5) "asdad" [9]=> string(0) "" [10]=> string(0) "" [11]=> string(0) "" [12]=> string(0) "" }
[PHP] pobierz, plaintext 

[trueblue]

Czy w takim razie jeśli jeden element ma "No", to pozostałe niezależnie jaką mają wartość i tak mają przechodzić walidację?

[imie_nazwisko]

Powinno być sprawdzane pole tekstowe w formularzu dla które przypisane jest YES w row np.

Gdy row[0]=Yes & row[1]=Yes & row[2]=Yes sprawdzane powinno być każde pole tj. plec, zawod, kraj
Gdy row[0]=No & row[1]=Yes & row[2]=Yes sprawdzane powinno być pole zawod oraz kraj
Gdy row[0]=No & row[1]=No & row[2]=Yes sprawdzane powinno być wyłącznie pole kraj
Gdy row[0]=No & row[1]=Yes & row[2]=No sprawdzane powinno być wyłącznie pole zawód
itd

Napisany przeze mnie kod JS sprawdza wszytskie pola niezaleznie od wartości w row.

Aby rozwiazać problem postanowiłem zmodyfikować kod JS:

[HTML] pobierz, plaintext 
<script LANGUAGE= "JavaScript" type= "text/javascript">
function validateForm(rejestracja){
 
    var tekst='';
    alert ("abc1");
    if (rejestracja.plec.value==""){tekst=tekst+"- plec\n";}
    alert ("abc2");
    if (rejestracja.zawod.value==""){tekst=tekst+"- zawód\n";}
    alert ("abc3");
    if (rejestracja.kraj.value =="" ){tekst=tekst+"- kraj\n";}
    alert ("abc4");
    if (tekst!="") {alert ("Błąd #882: \nNie uzupełniłeś następujących pól: \n\n"+tekst);
        return false;
    } else {
		 return true;
    }
}
 
</SCRIPT>
[HTML] pobierz, plaintext 

Nastepnie sprawdzilem zachowanie skryptu w przypadku gdy wartosc row[0]="Yes", row[1]="No" i row[2]="Yes".
Zgodnie z tym co napisalem wyzej skrypt powinien sprawdzić czy pole "plec" lub "kraj" jest wypelnione, i wyswietlic alert abc1, abc2, abc3, abc4 i jeśli pole plec lub kraj nie jest wypelnione to zwrocić błąd.

Dzieje się inaczej: Wyświetla się wylacznie alert abc1 i abc2. Skrypt prawdopodobnie "krzaczy" przy rejestracja.zawod.value=="" poniewaz pole zawod nie istnieje.

Moje pytanie - czyistnieje jakis warunek sprawdzajacy istnienie "rejestracja.zawod.value" ?

Ten post edytował imie_nazwisko 14.11.2014, 06:41:15

[trueblue]

Czekaj, ale ta walidacja ma pośredni związek z warunkami w skrypcie PHP.
W zależności od Yes lub No, drukujesz na ekranie odpowiednie pola w tym skrypcie, a nie robisz ich walidację na poziomie PHP.
W JS sprawdzasz wszystkie pola, nawet jeśli w PHP wydrukujesz tylko jedno.
Albo w JS powinieneś sprawdzać pola, które istnieją, albo w PHP drukować wszystkie.

P.S. Oprócz tego powinieneś zrobić walidację na poziomie PHP, czyli użyć Yes lub No, nie do uwarunkowania drukowania pola, ale właśnie walidacji (w PHP).

[imie_nazwisko]

w JS powinieneś sprawdzać pola

Jak to zrobić?

[trueblue]

[HTML] pobierz, plaintext 
if(rejestracja.zawod!==undefined){
//
}
[HTML] pobierz, plaintext 

[imie_nazwisko]

Problem rozwiązany. Dziękuje za pomoc.