que fallo veis, validar mail

[MiguelFT]

Código:

if($email == '') {
		die("Espacio en Blanco");
	}
		
if ((!ereg("^[_a-zA-Z0-9-]+(\.[_a-zA-Z0-9-]+)*@[a-zA-Z0-9-]+(\.[a-zA-Z0-9-]+)*$", $email)) or (!getmxrr($emailhost1, $mxhostarr))){
    header("location: fafasdfasdfaslo.html");
}

El espacio en blanco me funciona, lo otro no :S se os ocurre por qué!

[bioxido]

No te podria ayudar diciendote que esta mal, pero te recomiendo una otra funcion para validar emails, aca te la dejo.

Código PHP:

  function comprobar_email($email){ 
    $mail_correcto = 0; 
    //compruebo unas cosas primeras 
    if ((strlen($email) >= 6) && (substr_count($email,"@") == 1) && (substr($email,0,1) != "@") && (substr($email,strlen($email)-1,1) != "@")){ 
       if ((!strstr($email,"'")) && (!strstr($email,"\"")) && (!strstr($email,"\\")) && (!strstr($email,"\$")) && (!strstr($email," "))) { 
          //miro si tiene caracter . 
          if (substr_count($email,".")>= 1){ 
             //obtengo la terminacion del dominio 
             $term_dom = substr(strrchr ($email, '.'),1); 
             //compruebo que la terminación del dominio sea correcta 
             if (strlen($term_dom)>1 && strlen($term_dom)<5 && (!strstr($term_dom,"@")) ){ 
                //compruebo que lo de antes del dominio sea correcto 
                $antes_dom = substr($email,0,strlen($email) - strlen($term_dom) - 1); 
                $caracter_ult = substr($antes_dom,strlen($antes_dom)-1,1); 
                if ($caracter_ult != "@" && $caracter_ult != "."){ 
                   $mail_correcto = 1; 
                } 
             } 
          } 
       } 
    } 
    if ($mail_correcto) 
       return 1;        
    else 
       return 0;    
}  
   

[MiguelFT]

La he probado y tampoco me funcionaba, seria ponerlo tal cual?:

Código:

if($email == '') {
		die("Espacio en Blanco");
	}
function comprobar_email($email){ 
    $mail_correcto = 0; 
    //compruebo unas cosas primeras 
    if ((strlen($email) >= 6) && (substr_count($email,"@") == 1) && (substr($email,0,1) != "@") && (substr($email,strlen($email)-1,1) != "@")){ 
       if ((!strstr($email,"'")) && (!strstr($email,"\"")) && (!strstr($email,"\\")) && (!strstr($email,"\$")) && (!strstr($email," "))) { 
          //miro si tiene caracter . 
          if (substr_count($email,".")>= 1){ 
             //obtengo la terminacion del dominio 
             $term_dom = substr(strrchr ($email, '.'),1); 
             //compruebo que la terminación del dominio sea correcta 
             if (strlen($term_dom)>1 && strlen($term_dom)<5 && (!strstr($term_dom,"@")) ){ 
                //compruebo que lo de antes del dominio sea correcto 
                $antes_dom = substr($email,0,strlen($email) - strlen($term_dom) - 1); 
                $caracter_ult = substr($antes_dom,strlen($antes_dom)-1,1); 
                if ($caracter_ult != "@" && $caracter_ult != "."){ 
                   $mail_correcto = 1; 
                } 
             } 
          } 
       } 
    } 
    if ($mail_correcto) 
       return 1;        
    else 
       return 0;    
}

[MiguelFT]

Ahora tengo esto:

Código:

        $email = clean($_POST['email']);
if($email == '') {
		die("Espacio en Blanco");
	}
		
function comprobar_email($email){ 
    $mail_correcto = 0; 
    //compruebo unas cosas primeras 
    if ((strlen($email) >= 6) && (substr_count($email,"@") == 1) && (substr($email,0,1) != "@") && (substr($email,strlen($email)-1,1) != "@")){ 
       if ((!strstr($email,"'")) && (!strstr($email,"\"")) && (!strstr($email,"\\")) && (!strstr($email,"\$")) && (!strstr($email," "))) { 
          //miro si tiene caracter . 
          if (substr_count($email,".")>= 1){ 
             //obtengo la terminacion del dominio 
             $term_dom = substr(strrchr ($email, '.'),1); 
             //compruebo que la terminación del dominio sea correcta 
             if (strlen($term_dom)>1 && strlen($term_dom)<5 && (!strstr($term_dom,"@")) ){ 
                //compruebo que lo de antes del dominio sea correcto 
                $antes_dom = substr($email,0,strlen($email) - strlen($term_dom) - 1); 
                $caracter_ult = substr($antes_dom,strlen($antes_dom)-1,1); 
                if ($caracter_ult != "@" && $caracter_ult != "."){ 
                   $mail_correcto = 1; 
                } 
             } 
          } 
       } 
    } 
    if ($mail_correcto) 
       return 1;        
    else 
       die("Espacssssssnco");    
}

y pongo una "q" y me lo acepta!

[jackson666]

Me parece que te la estas complicando demasiado....
Pensa, que necesitas saber del e-mail?

1) si tiene arroba y este no se repite
2) si tiene el .com, .net, etc y no se repite
3) si tiene un minimo de caracteres

Nada mas...

[MiguelFT]

y como podria quedar es que he probado muchisimos y ninguno me funciona y efectivamente, tienes razon!

[Osdiwe]

Te paso un enlace a un post muy interesante sobre validaciones de campos con expresiones regulares. Encontrarás como validar un e-mail. Espero te sirva:

http://techtastico.com/post/8-expresiones-regulares-para-php-muy-utiles/

[jackson666]

A ver:

Código PHP:

  
$email = 'este e-mail no sirve';

$largo = strlen($email);

$com_net = substr($email, -4, $largo);

#el resto te las dejo xa q t diviertas


if(strstr($email,"@")){

echo "no tiene arroba!!"

#aca diez por poner algo, dsps cambialo si queres
}else if($largo < 10){

echo "email muy corto!!";

}else if($com_net !== '.com'){

#bla bla bla

}else if($com_net !== '.net'){

#bla bla bla

}  
   

Si no queres tener todo esto junto, metelo en una funcion y llamala para validar...

[Ember]

yo siempre uso esta expresión regular, no sé si te será útil:

var addressPattern = /^[^\s\n]+\@[a-zA-Z0-9][a-zA-Z0-9\-\.]*\.[a-zA-Z]{2,4}$/;

[MiguelFT]

Edito: Estoy leyendo

[MiguelFT]

Lo que no entiendo es por qué al poner:

Código:

if($email == '') {
		die("Espacio en Blanco");
	}

$largo = strlen($email);

$com_net = substr($email, -4, $largo);

//el resto te las dejo xa q t diviertas

if(strstr($email,"@")){

header("location: aaerrro.html");

// aca diez por poner algo, dsps cambialo si queres

}
else if($largo < 10){

header("location: corto.html");

}else if($com_net !== '.com'){

header("location: nocom.html");

}else if($com_net !== '.net'){

header("location: nonet.html");

}

pongo un espacio en blanco (lo dejo en blanco) y FUNCIONA pongo algo sin @, un 7, una x o lo que sea y el otro código NO funciona cuando se ve simple :S

[jackson666]

Código PHP:

  #aca te corregi esto 
if(empty($email)) { 
        die("Espacio en Blanco"); 
    } 
 
$largo = strlen($email); 
 
$com_net = substr($email, -4, $largo); 
 
#aca me habia equivocado yo 
if(! strstr($email,"@")){ 
 
header("location: aaerrro.html"); 
 
// aca diez por poner algo, dsps cambialo si queres 
 
} 
else if($largo < 10){ 
 
header("location: corto.html"); 
 
}else if($com_net !== '.com'){ 
 
header("location: nocom.html"); 
 
}else if($com_net !== '.net'){ 
 
header("location: nonet.html"); 
 
}  
   

[MiguelFT]

Lo del empty funciona perfecto efectivamente, la igual que lo otro que tenia yo pero ni aún corrigiendo eso funciona, pongo "asjk" y me lo admite el cabr*n

Gracias!