[PHP] Walidacja forlmularza, Walidacja formularza Opcje

[pavel14]

[PHP] pobierz, plaintext 
<?php
if (isset ($_GET['zlokalizuj'])){
		if (!isset ($_GET['opis'])) {echo "tak nie można";}
		if ( isset ($ulica) == "wybierz" ) {echo "tak też nie można";}}
 
		else 
		{
 
		  while ($result = mysql_fetch_assoc($pobrane_adresy)) 
		{
		$adres = $result['opis'];
 
		echo "<h4>Oczekiwana pomoc:</h4>$_GET[opis]<br/>";
		echo "<h4>Adres zamieszkania</h4>$_GET[ulica] &nbsp&nbsp $_GET[numer]<br/></br>";
		echo "Odpowiedni ośrodek to:<br/>$adres<br/>";
		} 
}
?>
[PHP] pobierz, plaintext 

Może mi ktoś podpowiedzieć, co robię tu nie tak? Wciskam 'submit'. Przy pierwszym warunku działa ok. Przy drugim lipa. Czy jest spełniony, czy nie wynik jest ten sam. Wyświetla za każdym tak jakby był spełniony. Instrukcja 'while' się nie wykonuje. Wartość 'wybierz' jest wartością domyślną z pola 'select'.

Ten post edytował pavel14 25.08.2007, 19:45:08

[tommy4]

isset($ulica) zwraca wartość typu boolean, a ty porównujesz ją do stringu.

[PHP] pobierz, plaintext 
<?php
if(isset($ulica) && $ulica == "wybierz") // if(isset($_GET['ulica']... ?
?>
[PHP] pobierz, plaintext 

i to się chyba nadaje na przedszkole;p

Ten post edytował tommy4 24.08.2007, 20:17:28

[pavel14]

Wiem. Ale gdzieś robię błąd i nie wiem gdzie. To co napisałem to próby. Już wszystkiego próbuje, ale wynik jest ten sam. Nie wiem czemu. Myślę, że coś popierdzieliłem z instrukcją.

[batman]

A nie powinno być tak:

[PHP] pobierz, plaintext 
<?php
if ( isset ($ulica) && $ulica == "wybierz" ) {echo "tak też nie można";}}
?>
[PHP] pobierz, plaintext 

W Twoim zapisie porównywana była wartość wyrażenia isset($ulica) ze stringiem "wybierz".

[pavel14]

[PHP] pobierz, plaintext 
<?php
if (isset ($_GET['zlokalizuj'])){
		if (!isset ($_GET['opis'])) {echo "tak nie można";}
		if (isset ($_GET['ulica'])) {echo "tak też nie można";}
 
		else 
		{
 
		  while ($result = mysql_fetch_assoc($pobrane_adresy)) 
		{
		$adres = $result['opis'];
 
		echo "<h4>Oczekiwana pomoc:</h4>$_GET[opis]<br/>";
		echo "<h4>Adres zamieszkania</h4>$_GET[ulica] &nbsp&nbsp $_GET[numer]<br/></br>";
		echo "Odpowiedni ośrodek to:<br/>$adres<br/>";
		} 
 }
}
?>
[PHP] pobierz, plaintext 

Taka jest wersja pierwotna. Ale chodzi o to, że 'ulica' to pole <select name=\"ulica\">. Wartościś domyślną tego pola jest 'wybierz'. Resztę wartości (nazwy ulic) pobieram z bazy. I jeśli zastosuję (isset ($_GET['ulica']) to warunek zawsze będzie spełniony, ponieważ pole domyslnie nie jest puste. A mnie zależy na tym by nie uwzględniał tej wartości.

Ten post edytował pavel14 24.08.2007, 21:38:45

[batman]

W takim razie warunek będzie miał taką postać.

[PHP] pobierz, plaintext 
<?php
if ( !isset($_GET['ulica']) || $_GET['ulica'] == "wybierz" ) {echo "tak też nie można";}}
?>
[PHP] pobierz, plaintext 

Ten post edytował batman 25.08.2007, 11:28:48

[pavel14]

To też próbowałem. Za każdym razem resetuje tą wartość z pola "select", którą ustawiłem, na domyślną "wybierz". Mógłbyś coś na to poradzić? Byłbym wdzięczny. Jak z pola "radio" pamięta zaznaczoną wartośc, tak z "selecta" nie. Nie wiem jak na to zaradzić.

[mike]

Już nie mogę patrzeć na ten żenujący wątek i na jego tytuł.
Przenoszę na Przedszkole (pisz tam jeszcze przez jakiś czas proszę) i proszę o zmianę tytułu wątku.
Tytuł "Co robię źle?" akurat jest mało trafny a raczej głupi i bezużyteczny.
Zasady formułowania poprawnych tytułów wątków znajdziesz w Regulaminie.

[batman]

Szczerze przyznam, iż nie wiem, gdzie możesz mieć błąd. Jeśli nie działa ten warunek, to dzieje się magia. Może formularz wysyłasz postem a nie getem, może masz włączone register_globals i zmienna ci się nadpisuje, a poza tym znalezienie błędu we fragmencie kodu raczej nie należy do najłatwiejszych. Wklej cały kod tej strony i się zobaczy.

[pavel14]

[HTML] pobierz, plaintext 
<form action="<?php echo $SERVER['PHP_SELF'];?>" method="GET" name="lokalizator">
[HTML] pobierz, plaintext 

[PHP] pobierz, plaintext 
<?php
.........
$wyswietlany_blok .= "<input type=\"radio\" name=\"opis\" value=\"Pomoc\" id=\"1\">Pomoc<br/>";
$wyswietlany_blok .= "<input type=\"radio\" name=\"opis\" value=\"Rodzina\" id=\"2\">Rodzina<br/>";
 
.........
 
 
	$wyswietlany_blok .= "
		<br/>
		<select name=\"ulica\">
		<option value=\"default\">wybierz</option>";
 
	//wyswietla nazwy ulic z tabeli ulice
	while ($rowu = mysql_fetch_assoc($pobrana_lista)) 
		{
		$id = $rowu['id_ulicy'];
		$nazwa_ulicy = $rowu['nazwa_ulicy'];
		$numer_ulicy = $rowu['numer'];
		$wyswietlany_blok .= "<option value=\"$nazwa_ulicy\">$nazwa_ulicy</option>";
		}
		$wyswietlany_blok .= "</select>";
 
		echo $wyswietlany_blok;
		echo "<br/><br/><input type=\"submit\" name=\"zlokalizuj\" value=\"Zlokalizuj\">";
 
 
		if (isset ($_GET['zlokalizuj'])){
		if (!isset ($_GET['opis'])) {echo "tak nie można";}
		if ($_GET['ulica'] == "wybierz" ) {echo "tak też nie można";}
 
		else 
		{
		...................
		} 
}}
?>
[PHP] pobierz, plaintext 

Tak to po krótce wygląda. Pewnie robię głupi błąd. Ale nie wiem gdzie.

Ten post edytował pavel14 25.08.2007, 19:53:04

[batman]

No i wszystko stało się jasne:

[PHP] pobierz, plaintext 
<?php
if ( !isset($_GET['ulica']) || $_GET['ulica'] == "default" ) {echo "tak też nie można";}
?>
[PHP] pobierz, plaintext 

Teraz zadziała.

[pavel14]

Chwała Tobie Batmanie. Znowu ocaliłeś Gotham City .
A możesz mi podpowiedzieć jak przekazać zaznaczone pola, tak by trzeba było wybrać te niezaznaczone po przeładowaniu formularza?

Ps: Wiedziałem, że to będzie błahostka.

Ten post edytował pavel14 25.08.2007, 20:56:55

[batman]

W taki sam sposób jak w przypadku ulicy:

[PHP] pobierz, plaintext 
<?php
if ( !isset($_GET['jakies_pole']) ) {echo "wybierz jakis pole";}
?>
[PHP] pobierz, plaintext 

lub w przypadku list rozwijanych

[PHP] pobierz, plaintext 
<?php
if ( !isset($_GET['jakas_lista']) || $_GET['jakas_lista'] == "default" ) {echo "wybierz cos z jakiejs listy";}
?>
[PHP] pobierz, plaintext 

przy założeniu, ze default jest opcją domyślną (niewybraną).

[pavel14]

Chyba się nie rozumiemy. Chodzi o to, że w momencie kiedy wcisnę "zlokalizuj" to resetuje mi wszystkie zaznaczone i wybrane pola. A chce by w momencie gdy zapomnę o jednym polu to pojawia się odpowiedni komunikat (ale tę sprawę mam już załatwioną) a pozostałe wartości, które wybrałem w dalszym ciągu są zaznaczone, po przeładowaniu formularza. Odhaczam niewypełnioną pozycją, daję "zlokalizuj" i wtedy daje wynik.

Ten post edytował pavel14 26.08.2007, 19:59:29

[batman]

W takim przypadku musisz zrobić co następuje.
Dla pól tekstowych ustawiasz atrybut value na $_GET['nazwa_pola'].
Dla droplisty trzeba stworzyć instrukcję warunkową, a następnie do znacznika <option> dodać zmienną $selected

[PHP] pobierz, plaintext 
<select name="nazwa_listy">
	<?php
	for($i=0;$i<10;$i++) {
		$selected = '';
		if($opcja == $_GET['nazwa_listy']) {
			$selected = 'selected';
			// nie pamietam czy nie powinno byc $selected = 'selected="selected"';
		}
		echo '<option value="'.$i.'" '.$selected.'>opcja '.$i.'</option>';
	?>
</select>
[PHP] pobierz, plaintext 

Zmienna $opcja jest Twoją wartością dla konkretnego wiersza.

Dla radiobuttona i checkboxa też trzeba użyć instrukcji warunkowej

[PHP] pobierz, plaintext 
<?php
$checked = '';
if($opcja == $_GET['nazwa_listy']) {
	$checked = 'checked="checked"';
}
<input type="checkbox" name="nazwa" value="1" '.$checked.' />
?>
[PHP] pobierz, plaintext 

W ten oto sposób masz zachowane wartości formularza.

Ten post edytował batman 26.08.2007, 22:16:34

[pavel14]

Mam takie o to tabele:
osrodek [id_osrodka, nazwa_osrodka, opis, id_pomocy]
pomoc [id_pomocy, rodzaj_pomocy, opis] ulica_osrodka
[id_osrodka, id_ulicy] ulice [id_ulicy, nazwa_ulicy, numer, parzystosc]
i kod

[PHP] pobierz, plaintext 
<?php
$zapytanie_o_adres2 = "SELECT o.opis FROM osrodek AS o 
LEFT JOIN pomoc p ON (p.id_pomocy=o.id_pomocy) 
LEFT JOIN ulica_osrodka uo ON (uo.id_osrodka=o.id_osrodka) 
LEFT JOIN ulice u ON (u.id_ulicy=uo.id_ulicy) 
WHERE u.nazwa_ulicy= '$_GET[ulica]'AND p.opis = '$_GET[opis]'\";
 
$pobrane_adresy2 = mysql_query($zapytanie_o_adres2) or die (mysql_error());	
	while ($result2 = mysql_fetch_assoc($pobrane_adresy2)) 		
	{
	 $adres2 = $result2['opis'];
	}
?>
[PHP] pobierz, plaintext 

Problem mam taki: Chcę by adres wyświetlał się w zależności od wartości w polu "parzystosc" tabeli "ulice".
Robiłem tak, że do zapytania dawałem jeszcze ".... AND parzystosc = 'cos';" ale nie działa to w przypadku,
gdy ulica jest podzielona na wiele częsci. Może jakiś warunek tu trzasnąć?? Próbowałem to robić przez
porównanie wartości w polu 'parzystosc' przy wybranej 'ulicy' i w zależności wykonywałem odpowiednie zapytanie,
ale coś robiłem nie tak.

Ten post edytował pavel14 6.09.2007, 17:48:04

[batman]

[SQL] pobierz, plaintext 
SELECT 
    u.nazwa_ulicy,
    u.numer,
    o.opis 
FROM osrodek o, 
    ulice u, 
    ulica_osrodka uo 
WHERE o.id_osrodka = uo.id_osrodka AND uo.id_ulicy = u.id_ulicy AND u.parzystkosc = [jakis_warunek]
[SQL] pobierz, plaintext 

Sądzę, że to zapytanie powinno rozwiązać Twój problem. Nie testowałem go, więc gdzieś mógł wkraść się błąd

Ten post edytował batman 6.09.2007, 07:16:02

[pavel14]

Otóż nie. Tego rozwiązania juz próbowałem, a chodzi mi o to, żeby w momencie, gdy wybiorę ulicę to sprawdza jaka jest wartość pola 'parzystosc'
i wtedy mogę wykonać odpowiednie zapytanie. Próbowałem to tak rozwiązać:

[PHP] pobierz, plaintext 
<?php
$zapytanie_o_adres1 = "SELECT o.opis FROM osrodek AS o 
		LEFT JOIN pomoc p ON (p.id_pomocy=o.id_pomocy) 
		LEFT JOIN ulica_osrodka uo ON (uo.id_osrodka=o.id_osrodka) 
		LEFT JOIN ulice u ON (u.id_ulicy=uo.id_ulicy) 
		WHERE u.nazwa_ulicy= '$_GET[ulica]'AND p.opis = '$_GET[opis]' AND u.parzystosc = 'tak'";
$pobrane_adresy1 = mysql_query($zapytanie_o_adres1) or die (mysql_error());
	while ($result1 = mysql_fetch_assoc($pobrane_adresy1)) 
		{
		$adres1 = $result1['opis'];	
		}
 
 
$zapytanie_o_adres2 = "SELECT o.opis FROM osrodek AS o 
		LEFT JOIN pomoc p ON (p.id_pomocy=o.id_pomocy) 
		LEFT JOIN ulica_osrodka uo ON (uo.id_osrodka=o.id_osrodka) 
		LEFT JOIN ulice u ON (u.id_ulicy=uo.id_ulicy) 
		WHERE u.nazwa_ulicy= '$_GET[ulica]'AND p.opis = '$_GET[opis]' AND parzystosc = 'nie'";
$pobrane_adresy2 = mysql_query($zapytanie_o_adres2) or die (mysql_error());
	while ($result2 = mysql_fetch_assoc($pobrane_adresy2)) 
		{
		$adres2 = $result2['opis'];	
		}		
 
$zapytanie_o_p = "SELECT parzystosc FROM ulice WHERE nazwa_ulicy = '$_GET[ulica]'";
$pobrane_p = mysql_query($zapytanie_o_p) or die (mysql_error());
	while ($resultp = mysql_fetch_assoc($pobrane_p)) 
		{
		$parz = $resultp['parzystosc'];	
		}		
		//wysylanie formularza
		if (isset ($_GET['zlokalizuj'])) //jezeli zostal ustawionu przycisk 'zlokalizuj'
 
		{
		if (!isset ($_GET['opis'])) {echo "Wybierz rodzaj pomocy";} 
		elseif ($_GET['ulica'] == "default") {echo "Wybierz ulicę";}
		elseif ($_GET['numer'] == "") {echo "Wpisz numer (max trzy cyfry)";}
 
		switch ($parz){
				 case 'tak':
				 echo "<h4>Oczekiwana pomoc:</h4>$_GET[opis]<br/>";
				 echo "<h4>Adres zamieszkania</h4>$_GET[ulica] &nbsp&nbsp $_GET[numer]<br/></br>";
				 echo "Odpowiedni ośrodek to:<br/>$adres1<br/>";
				 break;
			case 'nie':
			echo "<h4>Oczekiwana pomoc:</h4>$_GET[opis]<br/>";
			echo "<h4>Adres zamieszkania</h4>$_GET[ulica] &nbsp&nbsp $_GET[numer]<br/></br>";
			echo "Odpowiedni ośrodek to:<br/>$adres2<br/>";
			break;
}}
?>
[PHP] pobierz, plaintext 

ale robię to źle bo tak mi nie porówna wartości, z tą którą wybrałem. A tak bym chciał to rozwiązać. Wybieram ulicę - porównuję - odpowiednie zapytanie wykonuję. Tak mi się bynajmniej wydaje.

Ps: Z tym zaznaczeniem

[PHP] pobierz, plaintext 
<?php
......
if($opcja == $_GET['nazwa_listy'])
......
?>
[PHP] pobierz, plaintext 

ta $opcja to jak ma być??

Ten post edytował pavel14 6.09.2007, 18:41:42

[batman]

Szczerze przyznam, że nie wiem o co Ci może chodzić. Może ktoś inny postara się o interpretację lub napiszesz jaśniej o co chodzi.

[pavel14]

Oki postaram się. Mam tabele jak wyżej podałem i kod PHP. Chcę by w momencie wybrania nazwy ulicy sprawdzić jaka jest wartość pola 'parzystosc' dla tej wybranej właśnie ulicy $_GET['ulica']. Bo jeśli bym zrobił tak jak Ty sugerowałeś w powyższym zapytaniu, to musiałbym dać warunek, który sprawdza czy wprowadzony numer do pola 'numer' w formularzu, jest parzysty, czy też nie "if ($_GET['numer'] %2)" i dawałem SWITCH case'nie'; case 'tak'; (bo jak inaczej sprawdzić, które zapytanie ma się wykonać??). Tak próbowałem. Ale np. w momencie, gdzie mam przedział numerów, załóżmy od 1-10 i wszystkie numery bez względu na parzystość są przydzielone do jednego ośrodka, to zawsze będzie się wykonywało zapytanie, które spełnia warunek założmy ... AND u.parzystosc = 'nie'. A chodzi mi o to, że ma sprawdzać wartosc 'parzystosc' dla wybranej ulicy i na tej podstawie wykonywać odpowiednie zapytanie. Czyli mam ulice X i numer założmy 4, a wartość pola 'parzystosc' jest 'null' (dałem taką wartość domyślną w momencie gdy dany zbiór numerów jest przydzielony do jednego ośrodka), numer jest parzysty i wykonałoby się zapytanie spełniające warunek parzystości numeru. Może taki przykład zawartości tabeli:

nazwa_ulicy numer parzystosc
Xxxx 1-20 null
Xxxx 21-29 nie
Xxxx 22-30 tak

Czyli w pierwszej linii numery są wszystkie dla jednego, w drugiej tylko nieparzyste, a w trzeciej parzyste, a ulica jest ta sama. Tu właśnie mam zagadkę jak to uskutecznić by rozdzielał prawidłowo. A od tego zależy włąsnie adres ośrodka.
Ps: Mam nadzieję, że jakoś w miarę zrozumiale napisałem. A może mógłbyś jakiś lepszy pomysł podsunąć? Byłbym wdzięczny naprawdę.

Ten post edytował pavel14 6.09.2007, 21:20:01