[js] walidacja radio Opcje

[agataperz]

Witam,

mam taki skrypt

[PHP] pobierz, plaintext 
function checkradio(){
   var radio=false;
   for (var i=0; i<document.formularz.length; i++){
      if (document.formularz.elements[i].type == "radio"){
         var x = document.formularz.elements[document.formularz.elements[i].name].length;
            for (var y=0; y<x; y++){
               if (document.formularz.elements[i+y].checked) break;
            }
            if (y == x){
               return false;
            } else radio=true;
         i = i+x-1;
      } else radio=true;
   }
   return radio;
} 
[PHP] pobierz, plaintext 

oraz

[PHP] pobierz, plaintext 
<form name="formularz" action="finalizacja.php" method="post" onsubmit="return sprawdz(this);return checkradio(this)">
[PHP] pobierz, plaintext 

funkcja sprawdz działa poprawie - sprawdza pola input text
niestety funkcja checkradio nie chce działać. możecie mi powiedzieć co jest źle?

[kamil4u]

@Magic WWW: Kłóciłbym się - szczególnie przy takiej drobnostce jak walidacja formularza.
@agataperz: Robisz dość podstawowy błąd.

Kod

onsubmit="return sprawdz(this);return checkradio(this)"

Funkcja checkradio się nie wykona, dlatego że zostanie zwrócona wartość funkcji sprawdź i ta część kody zostanie zakończona.
Przykład:

Kod

<span onclick="return true; alert(123);">KLIK</span>

Rozwiązaniem problemu jest stworzenie nowej funkcji "sprawdź wszystko" lub zapisanie tego w takiej formie:

Kod

return (sprawdz(this) && checkradio(this))

Lecimy dalej...
Jak w funkcji jako parametr przekazujesz "this": checkradio(this) to w funkcji nie musisz już korzystać z
document.formularz.length, a z przekazanej referencji:

[JAVASCRIPT] pobierz, plaintext 
function checkradio( formularz ){
//pod zmienną formularz masz to samo co wcześniej w document.formularz
// jest to lepsze rozwiązanie, gdyż skrypt nie musi ponownie szukać tego elementu
[JAVASCRIPT] pobierz, plaintext 

W ogóle ta Twoja funkcja mi się nie podoba... Poczytaj o getElementsByTagName, a następnie napisz taki kod(ja przedstawiłem, go w pseudokodzie):

Kod

function sprawdz( formularz ){ //sprawdza czy któryś z inputów został zaznaczony - prawda jeżeli został, fałsz jeżeli nie został
zmienna INPUT = formularz.pobierz_wszystkie_inputy;
PĘTLA po INPUT{
   JEŻELI( INPUT.type == radio i INPUT.checked ) ZWRÓĆ prawdę
}
ZWRÓĆ fałsz
}

Powodzenia