...

View Full Version : validating email address not working



bhakti_thakkar
05-15-2008, 03:23 PM
hi all,
the following function is failing to give me the right output. there is a '\' in the email address but still the function returns true.
can any one please help me to rectify it




function valid_email($email) {
// First, we check that there's one @ symbol, and that the lengths are right
if (!ereg("^[^@]{1,64}@[^@]{1,255}$", $email)) {
// Email invalid because wrong number of characters in one section, or wrong number of @ symbols.
return false;
}
// Split it into sections to make life easier
$email_array = explode("@", $email);

$local_array = explode(".", $email_array[0]);


for ($i = 0; $i < sizeof($local_array); $i++) {

if (!ereg("^(([A-Za-z0-9!#$%&#038;'*+/=?^_`{|}~-][A-Za-z0-9!#$%&#038;'*+/=?^_`{|}~\.-]{0,63})|(\"[^(\\|\")]{0,62}\"))$", $local_array[$i])) {
return false;
}
}
if (!ereg("^\[?[0-9\.]+\]?$", $email_array[1])) { // Check if domain is IP. If not, it should be valid domain name
$domain_array = explode(".", $email_array[1]);
if (sizeof($domain_array) < 2) {
return false; // Not enough parts to domain
}
for ($i = 0; $i < sizeof($domain_array); $i++) {
if (!ereg("^(([A-Za-z0-9][A-Za-z0-9-]{0,61}[A-Za-z0-9])|([A-Za-z0-9]+))$", $domain_array[$i])) {
return false;
}
}
}
return true;
}

print (valid_email('abc@yahoo.co.in'));
print ("<BR><BR>");
print (valid_email('abc\@del2.vsnl.net.in'));






thanks in advance

chaosprime
05-15-2008, 04:09 PM
It's the backslash before the period in your big regexp. In that context it's being interpreted as a real backslash rather than escaping the period. (And it's impossible for a period to occur at that point anyway, since you're exploding on it.)

runnerjp
05-15-2008, 04:11 PM
what about like this

function valid_email ( $str )
{
return ( ! preg_match ( "/^([a-z0-9\+_\-]+)(\.[a-z0-9\+_\-]+)*@([a-z0-9\-]+\.)+[a-z]{2,6}$/ix", $str ) ) ? FALSE : TRUE;
}

bhakti_thakkar
05-15-2008, 05:50 PM
hi runnerjp,

your function works perfect. :thumbsup:
but let me try up few more test case to make it full proof.

Thanks for your reply



EZ Archive Ads Plugin for vBulletin Copyright 2006 Computer Help Forum