Home/ Questions/Q 726737
In Process

The Archive Base Latest Questions

Editorial Team
  • 0
Asked: 2026-05-14T06:29:10+00:00

My javascript is not working right. It is simple pre-vailidation form and I can

  • 0

My javascript is not working right. It is simple pre-vailidation form and I can not get the script to work. It is supposed to validate each field but I can not get it to validate past the first name. I stripped out all of the other garbage so the code would not be confusing Should be a copy paste to notepad. Little help please 

<?xml version="1.0" encoding="iso-8859-1"?>
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0         Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">
<head>


<script language="JavaScript" type="text/javascript">    
<!--    
function validateForm(theForm) {    
 var name = theForm.firstname.value;
 var name = theForm.lastname.value;     
 var email = theForm.email.value;    
 if (name == "") {    
   alert("Please fill in your First Name.");    
   theForm.firstname.focus();    
   return false;    
 }  
 if (name == "") {    
   alert("Please fill in your Last Name.");    
   theForm.lastname.focus();    
   return false;    
 }

 if (email == "") {    
   alert("Please fill in your email address.");    
   theForm.email.focus();    
   return false;    
 }    
 return true;    
}    
//--> 

</script>


if (!theForm.myCheckbox1.checked {    
 alert("Please check the honor box.");    
 return false;    
}


</head>
<body>
</script>    



<fieldset>
<legend>Fun in the Sun With JavaScript</legend>  
<ul>  


<form action="blah.cgi" method="post"    
     onSubmit="return validateForm(this);">    
First name: <input type="text" name="firstname"> <font color="#FF0000" size="1"     face="Arial, Helvetica, sans-serif"><strong>*</strong></font> <br><br> 
Last name: <input type="text" name="lastname"> <font color="#FF0000" size="1"     face="Arial, Helvetica, sans-serif"><strong>*</strong></font>  <br><br>
Email address: <input type="text" name="email"> <font color="#FF0000" size="1" face="Arial, Helvetica, sans-serif"><strong>*</strong></font>  <br><br>  
Phone Number: <input type="text" name="phone"><br><br>    
<input type="submit" name="submit" value="Submit">    
</form>   



<input type="checkbox" name="myCheckbox" value="someValue"><font color="#FF0000" size="1" face="Arial, Helvetica, sans-serif"><strong>*</strong></font> <P>By checking this Box you are confirming the data is accurate</p>


<p>(* indicates a required field)</p>





</body>
</html>
Share

Leave an answer

You must login to add an answer.

Need An Account,

1 Answer

  1. Editorial Team

    You are declaring 2 variables named name, that might be the problem.

    Change the variable name like the following:

    var firstName = theForm.firstname.value;
var lastName = theForm.lastname.value;

    Don’t forget to update the rest of the code with these variable names

    Edit:

    You might also want to check on those return statements as you are terminating the flow to not reach the next block.

    I also like @psuphish05 suggestion for this part

    • 0