Javascript validation loop problem

[jrayjr]

Hello, I’m running into a problem when I am trying to validate my form. I have a form that has 10 sections I need to validate with 70 inputs that the user must fill out. I tried doing it all in if statements but it only does half the checks and then breaks out of the function. I’m assuming I have reached and went beyond the number of if statements allowed. So what I am trying to do now is just run it through a loop and have the loop variable increment the input name on the form. Example I have 10 text boxes named TMName1, TMName2, TMName3 etc.. I'm not sure if it's even possilble the way I am trying to do it. This is the code I have.

<script language="JavaScript">
function validate_form ( )
{
for(var x=1; x<10; x++)
{
if ( document.MainForm.TMName(x).value == "" )
{
alert ( "Please fill in the Team Member name." );
document.MainForm.TMName(x).focus ( );
return false;
}
if ( document.MainForm.FDepartment(x).selectedIndex == 0 )
{
alert ( "Please select the department " + document.MainForm.TMName(x).value + " is from." );
document.MainForm.FDepartment(x).focus ( );
return false;
}
if ( document.MainForm.FShift(x).selectedIndex == 0 )
{
alert ( "Please select the shift." );
document.MainForm.FShift(x).focus ( );
return false;
}
if ( document.MainForm.TDepartment(x).selectedIndex == 0 )
{
alert ( "Please select the department  " + document.MainForm.TMName(x).value + "  went to." );
document.MainForm.TDepartment(x).focus ( );
return false;
}
if ( document.MainForm.TShift(x).selectedIndex == 0 )
{
alert ( "Please select the shift" );
document.MainForm.TShift(x).focus ( );
return false;
}
if ( ( document.MainForm.FDepartment(x).selectedIndex == document.MainForm.TDepartment(x).selectedIndex ) && ( document.MainForm.FShift(x).selectedIndex == document.MainForm.TShift(x).selectedIndex ) )
{
alert ( "The From and To Shifts must be different." );
document.MainForm.TShift(x).focus ( );
return false;
}
if ( document.MainForm.STime(x).value == "" ) 
{
alert ( "Please enter the start time." );
document.MainForm.STime(x).focus ( );
return false;
}
}
}
</script>    
 

Thanks,
John

[acoder]

Change, for example, document.MainForm.TMName(x).value to:

document.forms["MainForm"].elements["TMName"+x].value

and likewise for the rest.