[javascript] Sprawdzanie formularza Opcje

[ddzkm]

Witam!

Mam formularz i skrypt JS, który sprawdza, czy wymagane pola są wypełnione czy nie (w tym drugim przypadku blokuje wysyłanie formularza do serwera). Niestety zupełnie nie wiedzieć dlaczego po wpisaniu dowolnej wartości w polu "imię" skrypt kończy pracę i wysyła formularz, nawet w przypadku gdy pozostałe pola nie są wypełnione, w innym przypadku zaś o dziwo działa poprawnie. Co tu może być nie tak? Dodam, ze próbowałem kombinować z jednym zbiorowym warunkiem ("||") ale efekt też był ten sam...

Skrypt:

[HTML] pobierz, plaintext 
<script TYPE="text/javascript">
<!--  
function empty_check()
    {
    var form_data = document.forms["contact"];
 
    if (form_data.username.value == "")
        {
        alert("Aby wysłać wiadomość musisz wypełnić wszystkie pola formularza!");
        return false;
        }
 
    if (form_data.e-mail.value == "")
        {
        alert("Aby wysłać wiadomość musisz wypełnić wszystkie pola formularza!");
        return false;
        }
 
    if (form_data.subject.value == "")
        {
        alert("Aby wysłać wiadomość musisz wypełnić wszystkie pola formularza!");
        return false;
        }
 
    if (form_data.text.value == "")
        {
        alert("Aby wysłać wiadomość musisz wypełnić wszystkie pola formularza!");
        return false;
        }
 
 
    alert("Aby wysłać wiadomość musisz wypełnić wszystkie pola formularza!");
    return false;
 
    return true;
    }
//-->
</SCRIPT>
[HTML] pobierz, plaintext 

Formularz:

[HTML] pobierz, plaintext 
<FORM NAME="contact" ACTION="send.php" METHOD="post">
 
<TABLE CELLSPACING="5" CELLPADDING="0" STYLE="border: 1px solid #AEAEAE;">
<TR>
<TD ALIGN="left">
Twoje imię, nazwisko lub pseudomin:<FONT COLOR="#FF0000">*</FONT>
</TD>
<TD ALIGN="left">
 
<INPUT TYPE="text" NAME="username" VALUE="">
 
</TD>
</TR>
<TR>
<TD ALIGN="left">
Twój adres e-mail:<FONT COLOR="#FF0000">*</FONT>
</TD>
<TD ALIGN="left">
 
<INPUT TYPE="text" NAME="e-mail" VALUE="">
 
</TD>
</TR>
<TR>
<TD ALIGN="left">
Temat wiadomości:<FONT COLOR="#FF0000">*</FONT>
</TD>
<TD ALIGN="left">
 
<SELECT NAME="subject"><OPTION VALUE="" SELECTED>[ wybierz temat ]</OPTION>
<OPTION VALUE="1">Temat 1</OPTION>
<OPTION VALUE="2">Temat 2</OPTION>
</SELECT>
 
</TD>
</TR>
<TR>
<TD ALIGN="left">
Treść wiadomości:<FONT COLOR="#FF0000">*</FONT>
</TD>
<TD ALIGN="left">
<TEXTAREA NAME="text" COLS="39" ROWS="10"></TEXTAREA>
</TD>
</TR>
<TR>
<TD COLSPAN="2"><HR>
</TD>
</TR>
<TR>
<TD COLSPAN="2" ALIGN="center">
 
<INPUT TYPE="submit" NAME="button" VALUE="Wyślij wiadomość" onclick="return empty_check()">
 
</TD>
</TR>
</TABLE>
</FORM> 
[HTML] pobierz, plaintext 

[glh]

Zrob tak:
1. Zmien nazwe pola (w formularzu, parametr name)e-mail na email,
2. Zmien typ przycisku w formularzu z submit na button,
3.W zdarzeniu onclick tego przycisku wpisz tylko empty_check()
4. I najwazniejsze czyli funkcja empty_check (bo bez urazy, ale ta twoja to straszna kaszana):

[HTML] pobierz, plaintext 
function empty_check()
   {
    pustych=0;
    fd=document.forms['contact'];
 
    if (fd.username.value=="") pustych++;else
    if (fd.email.value=="") pustych++;else
    if (fd.subject.value =="") pustych++;else
    if (fd.text.value=="") pustych++;
 
    if (pustych==0) fd.submit();else
    {
     alert('Wypelnij wszystkie pola!');
    }
   }
[HTML] pobierz, plaintext