[js] Prosze o pomoc przy dokonczeniu walidacji Opcje

[pawelgdz]

[HTML] pobierz, plaintext 
<script>
function sprawdz() {
 
 
      if (document.forms[0].elements['in001'].value == '') {
         document.forms[0].elements['in001'].style.background="red";
         return false;
       }    
 
      if (document.forms[0].elements['in002'].value == '') {
         document.forms[0].elements['in002'].style.background="red";
         return false;
       }    
 
     // .... i tak do in010
 
   } 
 
 
</script>
[HTML] pobierz, plaintext 

Drodzy koledzy mam około 10 pół do sprawdzenia w formularzu

do sprawdzenia uzywam powyzszej funkcji do ktorej odwołuje sie przy zatwierdezaniu formularza, w naglowku

[HTML] pobierz, plaintext 
<form action=index.php method=post onclick="return sprawdz()">
[HTML] pobierz, plaintext 

Funkcja ta sprawdza mi pierwsze napotkane puste pole i koloruje je na czerwono ale powinna mi zaznaczyc wszystkie pola ktore sa puste Gdy zaznacza pierwsze zwraac wartosc false i skrypt przestaje dzialac

[mike]

return powoduje zwrócenie wartości przez funkcję i zakończenie jej działania.

Pozbądź się return a będzie OK.

A jeszcze lepiej zrób tak:

Kod

<script type="text/javascript">
function sprawdz() {

    var result = true;

    if( document.forms[ 0 ].elements[ 'in001' ].value == '' ) {
        document.forms[ 0 ].elements[ 'in001' ].style.background = "red";
        result &= false;
    }

    if( document.forms[ 0 ].elements[ 'in002' ].value == '' ) {
       document.forms[ 0 ].elements[ 'in002' ].style.background = "red";
        result &= false;
    }

    // ...

    return result;

}

</script>

W tej postaci ten kod nadal jest nadający się do kosza, żeby go poprawić umieść sprawdzanie w pętli. Po co masz klepać w kółko to samo.

[^bmb]

@pawelgdz

do sprawdzenia uzywam powyzszej funkcji do ktorej odwołuje sie przy zatwierdezaniu formularza, w naglowku
HTML
<form action=index.php method=post onclick="return sprawdz()">

Zatwierdzenie formularza = onsubmit a nie jak u ciebie onclick.

[pawelgdz]

Czy moglibyscie odwiedzic adres

www.inter-soft.pl/formularz

Kliknij "Wyslij" zakolorują się niewypelnione pola

Teraz wypelnij pierwsze, trzecie, ostatnie (mowimy o polach zakolorowanych na czerwono)

Co widzimy ? formularz sie zatwierdza ... dlaczego ?

Kod zrodlowy zobaczycie klikajac na mojej stronie Pokaz źródlo .



dlugo czekam na jaką kolwiek podpowiedz ale zadam pytanie bo domyslam sie że ..


Jezeli pierwsze pole nie jest wypelnione zwraca FALSE czyli logiczne 0

Drugie pole jest wypelnione zwraca TRUE czyli logiczne 1

jesli mamy :

0 and 1 and 0 and 1 and 0 and 1


to return zwraca ?

ja doliczylem sie ze zwroci true ?

nie wiem jak mam sobie poradzic

[nospor]

jesli mamy :

0 and 1 and 0 and 1 and 0 and 1


to return zwraca ?

ja doliczylem sie ze zwroci true ?

Ojojojojj..... podstawy matematyki, logiki czy czegos tam sie klaniają. Jesli w wyrazeniu gdzie masz same "and" pojawi sie chocby jeden false, to wynik bedzie false, wiec ty sie tu lepiej nie doliczaj tylko troche podszkol

Problem lezy w twoim skrypcie, bo ty w zaleznosci od niektorych warunkow przypisujesz dla result true. ZLE!!! true dla result przypisac masz raz na poczatku. potem jesli jakis warunek bedzie mowil ze pole jest zle, to dla result masz zwrocic false i tylko false.