no me sale validar email y password a la vez

[rogertm]

hola una vez mas (y no sera la ultima), tengo un form en el que entre otras cosas inserto email y password de los usuarios, tengo que validar ambos campo, por separado todo bien, pero cuando tengo que armar el muñeco entero nada, aqui les pongo lo que tengo:

Código PHP:

  <?php
function checkMail($usuarioEmail){             
    if(!ereg("^([a-zA-Z0-9._]+)@([a-zA-Z0-9.-]+).([a-zA-Z]{2,4})$",$usuarioEmail)){
        return FALSE;
    }else{
        return TRUE;
    } 
}
$usuarioEmail = $_POST['usuarioEmail'];

// comprobamos si las contraseñas son iguales
if($usuarioPassword !== $usuarioConfirmPassword){
    echo "Las contrase&ntilde;as no coinsiden";
}
elseif(checkMail(!$usuarioMail)){
    echo "La direcci&oacute;n de correo no es v&aacute;lida";
}else{ // aqui hago el INSERT
?>   

ya he probado 20mil y una forma, pero nada, donde es que estoy metiendo el PIE???
en este caso, me comprueba bien si los passwords coinsiden o no, pero no me valida el correo

NOTA: la funcion de validacion del email, lo tome de las FAQ de PHP

saludos y gracias de antemano

[eddwinpaz]

Hola Bueno el codigo esta parcialmente mal estructurado. aqui esta el codigo arreglado y con un Debuger integrado. Asi que apenas lo pegues pruebalo para que cambies el Debug por codigo SQL o la sentencia que le vayas a aplicar. saludos espero que te sirva ! exitos !!!

Este codigo te debe devolver la siguiente pantalla

[TRUE]-> Las claves son iguales!
[TRUE]-> El mail es valido

Otro Tip en la funcion validar_email(); En el Else deberia ir un return de el email yo como estaba debugeando el Codigo y haciendo pruebas puse el TRUE pero hay debe ir el email osea la variable $email;

Codigo

Código PHP:

  <?php
function validar_email($email){             
    if(!ereg("^([a-zA-Z0-9._]+)@([a-zA-Z0-9.-]+).([a-zA-Z]{2,4})$",$email)){
      echo "ERROR-> El email No es valido!<BR>";
      } 
       else {
       
        echo "[TRUE]-> El mail es valido<BR>";
        return;
    } 
}

$password_repetir = "clave";
$password = "clave";

$mail = "[email protected]";

// comprobamos si las contraseñas son iguales
if($password == $password_repetir){
    echo "[TRUE]-> Las claves son iguales!<BR>";
    
   }
   
 else { 
    
      die("ERROR:-> Las claves no coinciden!<BR>");
      }
      
validar_email($mail);

 

?>   

[okram]

Código PHP:

  elseif(checkMail(!$usuarioMail)){ 
   

no seria...

Código PHP:

  elseif(!checkMail($usuarioMail)){ 
   

??

Un saludo,

[eddwinpaz]

mmm.. tambien pero el royo es que tiene otros errorsillos decidi ayudar un poquillo mas

[rogertm]

hola, probare como me dicen, pero para colmo ahora se me ropio la PC , en cuanto todo este bien, le meto mano al codigo y les comento

gracias a ambos

[rogertm]

hola otra vez, pensaba que si ponia todo dentro de un mismo if me saldria, y he estado intentando eso, pero sigue sin salirme, aunque creo que debe ser lago asi:

Código PHP:

  <?php
$usuarioPassword = md5($_POST['usuarioPassword']);
$usuarioConfirmPassword = md5($_POST['usuarioConfirmPassword']);
$usuarioEmail = $_POST['usuarioEmail'];

function checkMailPass($usuarioEmail, $usuarioPassword){            
    if(!ereg("^([a-zA-Z0-9._]+)@([a-zA-Z0-9.-]+).([a-zA-Z]{2,4})$",$usuarioEmail)){
        $error = "La direccion de correo no es valida";
    }
    elseif($usuarioPassword !== $usuarioConfirmPassword){
        $error = "Las contrase&ntilde;as no coinsiden";
    }
}
if(!checkMailPass($usuarioEmail, $usuarioPassword)){
    echo $error;
}else{ // hacemos el insert de los datos }
?>   

sigo metiendo los pies, pero, donde? porque ni siquiera hace la insercion de datos

[okram]

No puedes hacer esto:

Código PHP:

  function checkMailPass($usuarioEmail, $usuarioPassword){            
    if(!ereg("^([a-zA-Z0-9._]+)@([a-zA-Z0-9.-]+).([a-zA-Z]{2,4})$",$usuarioEmail)){
        $error = "La direccion de correo no es valida";
    }
    elseif($usuarioPassword !== $usuarioConfirmPassword){
        $error = "Las contraseñas no coinsiden";
    }
} 
   

Porque lo que devuelves en cualquier caso no es false, por lo que siempre termina ejecutándose lo que encierras dentro de else... prueba asi:

Código PHP:

  <?php
 
function checkMail($usuarioEmail){             
    return ereg("^([a-zA-Z0-9._]+)@([a-zA-Z0-9.-]+)\.([a-zA-Z]{2,4})$",$usuarioEmail);
}
 
$usuarioEmail = $_POST['usuarioEmail'];
 
// comprobamos si las contraseñas son iguales
if($usuarioPassword !== $usuarioConfirmPassword){
    echo "Las contrase&ntilde;as no coinsiden";
} elseif(!checkMail($usuarioEmail)){
    echo "La direcci&oacute;n de correo \"$usuarioEmail\" no es v&aacute;lida";
} else { // aqui hago el INSERT   

Un saludo,

EDITO: Lo olvidé... Este es mi mensaje número 1000! Demoré bastante ( desde 2005? ), pero puedo decir con orgullo que contribuí con 0.05% del total de mensajes en el foro (Cluster se mandó como 40.000, llegaré? ) Ahi voy por la segunda!

[rogertm]

no si cuano yo lo digo, cada vez que te veo aparecer okram, veo la solucion, jajaja, gracias man, ahora todo funciona bien, y gracias eddwinpaz tambien.

saludos y suerte:

PD y/o OFF TOPIC: okram, felicidades por lo de los 1000 mensajes, te sigo te sigo, de lo que hablamos ya me puse manos a la obra:
http://www.forosdelweb.com/f5/quiero...erador-536448/

RE: saludetesssssssssssSs