Forum PHP.pl > [JavaScript]Sprawdzanie form

Pomoc - Szukaj - Użytkownicy - Kalendarz

Pełna wersja: [JavaScript]Sprawdzanie form

SaMi

27.11.2008, 23:45:13

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

27.11.2008, 23:52:08

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

SaMi

28.11.2008, 00:00:01

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

erix

28.11.2008, 00:01:16

getElementsByName(ids)
Nie ma takiej funkcji.

bobo168

28.11.2008, 00:11:52

Cytat(erix @ 28.11.2008, 00:01:16 )

getElementsByName(ids)
Nie ma takiej funkcji.

Jak to nie ma ?

erix

28.11.2008, 00:15:08

Jasny gwint, wtopa. :X

nitek

28.11.2008, 08:48:03

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;
        }
    }

vokiel

28.11.2008, 13:35:40

Ale co to za herezje?

Cytat

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.

To jest wersja lo-fi głównej zawartości. Aby zobaczyć pełną wersję z większą zawartością, obrazkami i formatowaniem proszę kliknij tutaj.