such a strict language

[klassicd]

hey everone. would someone mind looking at the code and perhaps telling me what i might be doing wrong?

 <script language="JavaScript" type="text/JavaScript">
function checkForm()
 {
 var is_checked = document.forms["submitscore"].forfeit.checked;
 if (is_checked == false)
   var r1h = parseFloat(document.forms["submitscore"].r1h.value);
   var r2h = parseFloat(document.forms["submitscore"].r2h.value);
   var r1a = parseFloat(document.forms["submitscore"].r1a.value);
   var r2a = parseFloat(document.forms["submitscore"].r2a.value);
   
   var totalHome = parseFloat(r1h + r2h);
   var totalAway = parseFloat(r1a + r2a);
   var totalScore = totalHome + totalAway;
   if (totalScore != 24)
	{
	 alert('The Round 1 and Round 2 scores do not add up to an even 24 rounds.');
	 return false;
	}
   return true;
  }
  return true;
 }
</script>

[beetle]

Well, offhand what I see you're missing is an opening brace after your first if statement. However, you can make your code quite a bit more efficient by storing the form as a reference, a refactoring some of your variables out of existence.

function checkForm()
{
	var f = document.forms["submitscore"];
	if ( !f.forfeit.checked )
	{
		var r1h = parseFloat( f.r1h.value );
		var r2h = parseFloat( f.r2h.value );
		var r1a = parseFloat( f.r1a.value );
		var r2a = parseFloat( f.r2a.value );
		var totalScore = r1h + r2h + r1a + r2a; // already numbers, no need to run parseFloat on total

		if ( totalScore != 24 )
		{
			alert('The Round 1 and Round 2 scores do not add up to an even 24 rounds.');
			return false;
		}
	}
	return true;
}

[klassicd]

Thank you for your help.

[Markdidj]

If you want good error reporting, install NS. The latest Netscape has superb error reporting, and will point you to the exact problem at the line itself (creates a link to problem line). I use it all the time, to check my site and to do ALL error reporting. It also allows the embedding of WMP objects, with all controls. which is cool.....