Bit of a headache

Question

Hi all,
I have found this bit of javascript to validate a users form
input, but the problem is that it does it in steps - ie it will
check the name and if that isn't present it will stop the form and
display just the name error.
However I want it to display all the errors in one go (so if
the name AND the email is left blank, it will display both errors
at the same time!). I am sure it has to do with where you place the
return false value but after an hour of mucking about I just don't
seem to be able to get it to work.
Here is the script:
function checkForm() {
name = document.getElementById("name").value;
email = document.getElementById("email").value;
comment = document.getElementById("comment").value;
if (name == "") {
hideAllErrors();
document.getElementById("nameError").style.display =
"inline";
document.getElementById("name").select();
document.getElementById("name").focus();
return false;
} else if (email == "") {
hideAllErrors();
document.getElementById("emailError").style.display =
"inline";
document.getElementById("email").select();
document.getElementById("email").focus();
return false;
} else if (comment == "") {
hideAllErrors();
document.getElementById("commentError").style.display =
"inline";
document.getElementById("comment").select();
document.getElementById("comment").focus();
return false;
return true;
function hideAllErrors() {
document.getElementById("nameError").style.display = "none"
document.getElementById("emailError").style.display = "none"
document.getElementById("commentError").style.display =
"none"

Answer 1

I think you need to add a ';' (one of each) in the following
script code as follows below:
Your code as shown earlier:
From here:
document.getElementById("nameError").style.display = "none"
document.getElementById("emailError").style.display = "none"
document.getElementById("commentError").style.display =
"none"
To here:
document.getElementById("nameError").style.display = "none";
document.getElementById("emailError").style.display = "none";
document.getElementById("commentError").style.display =
"none";
I will take a look at Elaine's quick advice. Let us know if
that works with ';' at end of those three codes.
Thanks in advance, DWD