[HTML][PHP]sprawdzenie, czy wypełniono pola formularza - problem Opcje

[hyhyhy]

Mam taki problem: mam stronę z dość obszernym formularzem (generator quizu) no i chciałbym uniemożliwić pozostawienie pustych pół. Mam dwa kody: jeden działający i jeden nie. Ten drugi dotyczy większej liczby elemtntów, ale wydaje mi się taki sam, czy ktoś mógłby mi podpowiedzieć, co jest nie tak? Dodam, że problem jest taki, że IF się nie wykonuje (tzn zachowuje się tak, jakby $wypelniono zawsze mialo true, nawet kiedy pola są kompletnie puste)

kod, który działa:

[PHP] pobierz, plaintext 
$wszystko_wpisane = true; //-------
for ($x=0;$x<10;$x++)
{
trim($tekst[$x]);
$tekst[$x] = mysql_real_escape_string(htmlspecialchars($tekst[$x]));
if (empty($tekst[$x]) == true) {$wszystko_wpisane = false;}  
}
 
if ($wszystko_wpisane == false) {echo 'Blad! uzupelnij wszystkie pola!';} 
else {   //jesli uzupelniono, dalsze instrukcje
[PHP] pobierz, plaintext 

No i teraz na tej podstawie zrobiłem kod, z którym mam problem:

[PHP] pobierz, plaintext 
function sprawdzanie($liczba_obiektow, $obiekt)
 {
 for ($q=0;$q<$liczba_obiektow;$q++)
 {
 if (empty($obiekt[$q]) == true) {$wypelniono = false;}
 }
return $wypelniono;
 }
 
$wypelniono = true;
sprawdzanie($liczba_pyt, $pytanie[$q]);
sprawdzanie($liczba_odp, $odpowiedz[$q]);
sprawdzanie($liczba_wynik, $wynik[$q]);
 
if ($wypelniono == true) 
{ dalsze instrukcje - wykonują się zawsze, nigdy nie wchodzi else}
else { echo 'Blad, uzupelnij wszystkie pola!';}
[PHP] pobierz, plaintext 

Czy ktoś mógłby mnie naprowadzić? Bo nie widzę różnicy między nimi dwoma, a tylko drugi nie działa...

[Zagiewa]

Mogę się mylić ale pętla for zwraca false albo true i zapisuje ją do zmennej $wypelniono po czym zwracasz ją za pomocą return. Zaraz pod pętlą niepotrzebnie zmieniasz wartość zmiennej $wypelniono na true. Bo wtedy zawsze wartość nadana przez pętlę będzie zmieniona na true. Więc usuń to: $wypelniono = true; i powinno działać ew. zmień kod na taki:

[PHP] pobierz, plaintext 
function sprawdzanie($liczba_obiektow, $obiekt)
      {
      for ($q=0;$q<$liczba_obiektow;$q++)
      {
      if (empty($obiekt[$q])) {
$wypelniono = false;
}else {
$wypelniono = true;
}
      }
      return $wypelniono;
      }
 
      sprawdzanie($liczba_pyt, $pytanie[$q]);
      sprawdzanie($liczba_odp, $odpowiedz[$q]);
      sprawdzanie($liczba_wynik, $wynik[$q]);
 
      if ($wypelniono == true)
      { dalsze instrukcje - wykonują się zawsze, nigdy nie wchodzi else}
      else { echo 'Blad, uzupelnij wszystkie pola!';}
[PHP] pobierz, plaintext 

Ten post edytował Zagiewa 27.02.2011, 13:19:23

[hyhyhy]

I nic, niestety jak usuwam $wypelniono = true to mowi ze nie wypelniono, jak zostawiam to mimo wszystko mowi mi ze pola sa wypelnione.

@Zagiewa:
Próbowałem, i wtedy zawsze jest "błąd, nie wypełniłeś wszystkich pól"

Ten post edytował hyhyhy 27.02.2011, 13:31:50

[Zagiewa]

Spróbuj wykorzystać ten kod który przerobiłem, on od razu wypełnia zmienną wartością true albo false w zależności od tego co zwróci warunek

[Valdi_B]

Wynik zwracany przez funkcję sprawdzanie musisz odczytać w programie głównym.
Do tej pory ten wynik "leci w próżnię".
Może tak:

$wypelniono &&= sprawdzanie(...);

czyli "kumulatywne zerowanie" zmiennej $wypelniono?

Druga możliwość: w funkcji dodaj:
globals $wypelniono;
bo tak jak jest to zmienna $wypelniono jest w funkcji zmienną lokalną.

Ten post edytował Valdi_B 27.02.2011, 16:34:10

[hyhyhy]

Dalej nie działa, nie mam pojęcia, co to może być, wygląda to tak:

[HTML] pobierz, plaintext 
PYTANIA
Array ( [0] => q [1] => q [2] => q [3] => q [4] => q ) 
ODPOWIEDZI
Array ( [0] => q [1] => q [2] => q [3] => q [4] => q [5] => q [6] => q [7] => q [8] => q [9] => q [10] => q [11] => q [12] => q [13] => q [14] => q ) 
WYNIK
Array ( [0] => f [1] => fb [2] => fbf ) 
 
 
nie wypelniono wszystkich pol!
[HTML] pobierz, plaintext 

Ten post edytował hyhyhy 28.02.2011, 19:39:45

[sadistic_son]

Do sprawdzania czy coś jest RUE czy FALSE używa się operatora === sprawdzającego typ danych.

[Valdi_B]

Jeszcze jedno spostrzeżenie co do Twojego kodu:
Zaraz po deklaracji funkcji sprawdzanie() masz jej wywołanie:

[PHP] pobierz, plaintext 
sprawdzanie($liczba_pyt, $pytanie[$q]);
[PHP] pobierz, plaintext 

Zwróć uwagę, że zmienna $q jest to zmienna lokalna w powyższej funkcji, a tu - już nie jest określona.

Poza tym: Skoro w funkcji przeszukujesz tablicę obiektów, to parametrem do tej funkcji powinieneś podać całą tablicę, a nie jej któryś obiekt.
Spróbuj taki kawałek kodu:

[PHP] pobierz, plaintext 
function sprawdzanie($liczba_obiektow, $obiekt) {
  globals $wypelniono;
  for ($q=0; $q<$liczba_obiektow; $q++) {
    if (empty($obiekt[$q])) {
      $wypelniono = false;
    }
  }
}
 
$wypelniono = true;
sprawdzanie($liczba_pyt, $pytanie);
sprawdzanie($liczba_odp, $odpowiedz);
sprawdzanie($liczba_wynik, $wynik);
 
if ($wypelniono) {
  <obsługa przypadku "OK">
} else {
  echo 'Blad, uzupelnij wszystkie pola!';
}
[PHP] pobierz, plaintext 

Edit:
Zakodowałem "podobny" formularz, z kilkoma polami:
<input type="text" name="tt[]" />
i co widzę w $_POST:
1. Dostaję tablicę $tt.
2. Pola które w formularzu zostawiłem puste są w tej tablicy, ale z pustą treścią.

Wniosek:
W pętli nie korzystaj z empty() (bo element tablicy istnieje).
Sprawdzaj długość stringu w tym elemencie - strlen($obiekt[$q]);

Ten post edytował Valdi_B 2.03.2011, 17:05:12

[thek]

Ja nie widzę nigdzie przypisania tego co zwraca funkcja sprawdzająca. Zresztą lepiej by ciut było nieco inaczej funkcję sprawdzającą zrobić.

[PHP] pobierz, plaintext 
function sprawdzanie( $tablica ) {
    if( is_array( $tablica ) && !empty( $tablica ) ) { //sprawdzamy czy to tablica i czy ma choć jeden element
        foreach( $tablica AS $element ) {
            $element = trim($element);  // na wszelki wypadek usuwamy z elementu znaki puste
            if( empty( $element ) ) { return false; } //jeśli trafimy na element pusty to od razu możemy zakończyć funkcję z wynikiem negatywnym
        }
        return true;
    } else {
        return false;
    }
}
[PHP] pobierz, plaintext 

Teraz musimy jedynie konkretne tablice sprawdzać.

[PHP] pobierz, plaintext 
if( !sprawdzanie( $pytania ) ) { echo 'Nie uzupełnione pytania'; }
if( !sprawdzanie( $odpowiedzi ) ) { echo 'Nie uzupełnione odpowiedzi'; }
if( !sprawdzanie( $wyniki ) ) { echo 'Nie uzupełnione wyniki'; }
[PHP] pobierz, plaintext 

Oczywiście funkcja sprawdzająca będzie wyglądać inaczej jeśli masz inaczej zaprojektowaną strukturę danych, ale to już Twój problem bysobie odpowiednią napisać

[Prezi2907]

Do sprawdzania czy coś jest RUE czy FALSE używa się operatora === sprawdzającego typ danych.

dla sprostowania

$a === $b
sprawdza czy identyczne
Prawda jeśli $a jest równe $b i są tego samego typu. (tylko PHP4)

Mogę się nie znać no ale...

[Valdi_B]

Wczorajszy post pisałem z myślą o kontroli odpowiedzi tekstowych (do pola trzeba coś wpisać).

Dopiero teraz skojarzyłem, że w formularzu masz chyba także pytania jednokrotnego wyboru (wybrać jeden z przycisków "radio").
W takich przypadkach kontrolować trzeba z grubsza tak jak miałeś na początku - czy jest dany element w tablicy.

Co do pól opcji - chyba nie ma jak kontrolować "czy wypełniono", bo oba warianty (zaznaczono / nie zaznaczono) to jest jakaś odpowiedź.

Załączam chodzący przykład z dwoma wariantami kontroli - pól tekstowych i grup pól opcji. Porównaj konwencję nadawania "name" polom, czy masz tak samo u siebie.

[PHP] pobierz, plaintext 
<html>
<head>
  <meta http-equiv="content-type" content="text/html; charset=utf-8" />
  <title>Pytania</title>
</head>
 
<body>
<?php
// Sprawdź grupy przycisków "radio"
function spr_rad($n, $tbl) {
  // Nie wybrano nic w grupie "i" -> elem. "i" nie ma w tabl.
  for ($i = 1; $i <= $n; $i++) {
    if (!isset($tbl[$i]))
      return false;  // brak odp. na pyt. "i"
  }
	return true;  // OK
} // spr_rad
 
// sprawdź pola tekstowe (mogą mieć pustą treść)
function spr_txt($tbl) {
  foreach ($tbl as $tt) {
    if (strlen($tt) == 0)
      return false;  // brak odp.
  }
  return true;  // OK
} // spr_txt
 
// Jak wywołano stronę
$rqm = $_SERVER['REQUEST_METHOD'];
 
if ($rqm == 'POST') {
  // czytaj dane z formularza
  $wyb = $_POST['wyb'];
  $pt = $_POST['pt'];
 
  // echo '<pre>'; print_r($_POST); echo '</pre>';
 
  $wyn = true;  // wynik sprawdzenie
  // sprawdź grupy pól "radio" (w form. są 2)
  if (!spr_rad(2, $wyb)) {
    $wyn = false;
    $tt[] = 'Wypełnij wszystkie pola jednokrotnego wyboru';
  }
  // sprawdź pola tekstowe
  if (!spr_txt($pt)) {
    $wyn = false;
    $tt[] = 'Wypełnij wszystkie pola tekstowe<br />';
  }
  if ($wyn) {
    echo 'Kontrola - OK<br />';
  } else {
    echo 'Błędy:<br />' . implode('<br />', $tt);
  }
}
?>
  <h2>Kwestionariusz</h2>
  <form action="<?php echo $_SERVER['PHP_SELF']; ?>"
    method="post" name="frm1">
  <p>Wybierz jedną odpowiedź</p>
  <table border="1" cellspacing="2" cellpadding="2">
    <tr>
      <td>Pytanie 1<br />
        <input type="radio" name="wyb[1]" value="1" /> Odp. pierwsza<br />
        <input type="radio" name="wyb[1]" value="2" /> Odp. druga<br />
        <input type="radio" name="wyb[1]" value="3" /> Odp. trzecia
      </td>
      <td>Pytanie 2<br />
        <input type="radio" name="wyb[2]" value="1" /> Odp. pierwsza<br />
        <input type="radio" name="wyb[2]" value="2" /> Odp. druga<br />
        <input type="radio" name="wyb[2]" value="3" /> Odp. trzecia
      </td>
    </tr>
  </table>
  <p>Pytania tekstowe</p>
  <table border="1" cellspacing="2" cellpadding="2">
    <tr><td>Pyt. pierwsze</td><td><input type="text" name="pt[]" /></td></tr>
    <tr><td>Pyt. drugie</td><td><input type="text" name="pt[]" /></td></tr>
    <tr><td>Pyt. trzecie</td><td><input type="text" name="pt[]" /></td></tr>
    <tr><td>Pyt. czwarte</td><td><input type="text" name="pt[]" /></td></tr>
  </table>
  <p><input type="submit" value="Zapisz" /></p>
  </form>
</body>
</html>
[PHP] pobierz, plaintext 

Warianty kontroli są 2 (a nie 3), bo nie rozumiem o co chodzi z kontrolą samych pytań. Pytania wstawia do formularza Twój skrypt, a użytkownik nic w nich nie zmieni, ma na nie tylko odpowiedzieć.
Dlatego uważam, że kontrolować można tylko odpowiedzi.

Jeszcze jedno: W swoim skrypcie dodaj print_r($_POST).
Jak widzisz jakie dane dostajesz, to łatwiej napisać program ich kontroli.