Thread: help with email validation

Hi All,

I've added a line to make sure the email address is not blank...but I can't get the validation part working. if i uncomment the commented lines in red, i get the error: error '8004020e' ...this has to do with validation when i google. am i using that validator the wrong way?

Code:

<script>
function onSubmitForm() {
    var formDOMObj = document.frmSend;

    if (formDOMObj.EqNo.value == "")
	alert("Please enter EQNumber!")
    if (formDOMObj.EqDesc.value == "")
	alert("Please enter EQDescription!")
    if (formDOMObj.EmailAdd.value == "")
	alert("Please enter your email address!")
   // if (emailValidator(formDOMObj.EmailAddress.value,"Please enter a valid email address"))    else
        return true;
    return false;
}

//function emailValidator(){
//	var emailExp = /^[\w\-\.\+]+\@[a-zA-Z0-9\.\-]+\.[a-zA-z0-9]{2,4}$/;
//	if(formDOMObj.EmailAdd.match(emailExp)){
//		return true;
//	}
//else{
//		return false;
//	}
//}
</script>

Thanks peeps!!!!

Thanks Mehere & RR....

RR, I tried the below but it didn't work, kept getting error 8004020e...and it no longer alerted for the other fields. So, the only way to not get the error was to comment out the lines.

Code:

<script>
function onSubmitForm() {
    var formDOMObj = document.frmSend;
//    var emailExp = new RegExp("^[\w\-\.\+]+@[a-zA-Z0-9\.\-]+\.[a-zA-z0-9]{2,4}$");
 
    if (formDOMObj.EqNo.value == ""){
    alert("Please enter EQNumber!");
    return false;
    }
    if (formDOMObj.EqDesc.value == ""){
    alert("Please enter EQDescription!");
    return false;
    }
//    if (!emailExp.test(formDOMObj.EmailAdd.value)){
//    alert("Please enter a valid email address");
//    return false;
//    }
    return true;
}
</script>

So I tried the var line from mehere, and it worked with your if , alert, return... like below:

Code:

<script>
function onSubmitForm() {
    var formDOMObj = document.frmSend;
    var emailExp = /^[\w\-\.\+]+\@[a-zA-Z0-9\.\-]+\.[a-zA-z0-9]{2,4}$/; 
    if (formDOMObj.EqNo.value == ""){
    alert("Please enter EQNumber!");
    return false;
    }
    if (formDOMObj.EqDesc.value == ""){
    alert("Please enter EQDescription!");
    return false;
    }
    if (!emailExp.test(formDOMObj.EmailAdd.value)){
    alert("Please enter a valid email address");
    return false;
    }    return true;
}
</script>

On this, do you mean they wouldn't have to enter the .com, .net, etc? i think i need the whole thing...(.com) because I am using asp mail (cdo).

<edit>I couldn't get it to work using \w. You could try this expression "^[A-Za-z0-9\-.+]+@[A-Za-z0-9.-]+[.]+[A-Za-z]{2,4}$" That will only accept email addresses in the format name@domain.ext You may want to accept emails with name@domain in which case change +[.] to +\. </edit>

Do you mean a JS error or an ASP error?

hmmm...not sure...maybe asp?

error '8004020e'

the line is 222 which is from my cdo code...

this line:

Code:

.Send

That is an error with CDO, normally related to an invalid email address.

When you uncomment the JS code and enter something like 'Rebelle' in the email text box, what happens?

Hi RR,

If i uncomment the line and enter Rebelle...I don't get the error but I also leave the other fields blank so i could get alert but I don't get the alerts.....

it's when i leave the email address field blank that I get the error......but when i leave the other fields blank too, i don't get any alerts....just the error.

ok...i retried and retried with the original line but same thing kept happening.

so I tried changing the " and " to / and / ... like below:

from:

Code:

var emailExp = new RegExp("^[\w\-\.\+]+@[a-zA-Z0-9\.\-]+\.[a-zA-z0-9]{2,4}$");

to:

Code:

var emailExp = new RegExp(/^[\w\-\.\+]+@[a-zA-Z0-9\.\-]+\.[a-zA-z0-9]{2,4}$/);

will this cause any issues doing it like this?