[JavaScript]Sprawdzanie form Opcje

[SaMi]

Witam,
Chcę sprawdzić czy na wszystkie pytania w ankiecie zostałą udzielona odpowiedz i kombinowałem w ten sposób:

[HTML] pobierz, plaintext 
function checkAll()
{
    var error = false;
 
	for(i=1;i<14;i++)
	{
		var ids="pytanie["+i+"]";
 
		if(document.getElementsByName(ids).value =="" )
		{
		  error = true;
		}
	}
 
	if (error){
	   alert('nie odpowiedziałeś na wszystkie pytania');
	   return false;
	} else {
	   return true;
	}
 
}
[HTML] pobierz, plaintext 

natomiast formularz wygląda tak

[HTML] pobierz, plaintext 
<form  name="pytaniee" method="post" onsubmit="return checkAll()">
  <select name="pytanie[1]" >
    <option value=""></option>
    <option value="1">cos 1</option>
    <option value="2">cos 2</option>
  </select>
   <select name="pytanie[2]" >
    <option value=""></option>
    <option value="1">cos 11</option>
    <option value="2">cos 22</option>
   </select>
  .
  .
  .
[HTML] pobierz, plaintext 

Niestety nie działa mi to.
document.getElementsByName(ids).value zwraca undefined tak jak by nie przyjmowało nazwy dziecka form ze zmiennej, co robię źle w powyższej funkcji ?

[bobo168]

google.pl mnóstwo tego typu skryptów znajdziesz.

[SaMi]

To prawda ale ja chcę wiedzieć co w tym przypadku robię źle.

[erix]

getElementsByName(ids)
Nie ma takiej funkcji.

[bobo168]

getElementsByName(ids)
Nie ma takiej funkcji.

Jak to nie ma ?

[erix]

Jasny gwint, wtopa. :X

[nitek]

spróbuj może zdefiniować nazwy pól nie wskazujące na tablice. coś w rodzaju, pytanie_1, pytanie_2... i zrobić coś takiego

Kod

var error = false;
var ids; //nie defuniuj nowej zmiennej za kazdym razem w petli - zdeifniuj ją raz, potem nadawaj wartość.

    for(i=1;i<14;i++)
    {
        ids="pytanie_"+i;

        // dla testu możesz zrobić w tym miejscu coś takiego (bedziesz wiedzial, co wskazuja zmienne): alert(ids); alert(document.getElementsByName(ids).value);

        if(document.getElementsByName(ids).value =="" )
        {
          error = true;
        }
    }

Ten post edytował nitek 28.11.2008, 08:50:37

[vokiel]

Ale co to za herezje?

document.getElementsByName(ids)

getElementsByName pobiera elementy po nazwie, a tu przekazujecie id, do pobierania po id jest metoda getElementById()
Jak już pobierać po name, to: document.getElementsByName('input'), wtedy dostajesz tablice inputów, sprawdzasz w pętli czy są to inputy typu text, i sprawdzasz ich value.