[SOLUCIONADO] Problemas con sistema de registro de usuarios

[iDiegoEscobar]

Hola nuevamente.

Hoy estaba terminando mi sistema de registro de usuarios y al probarlo me di cuenta que aunque el sistema muestra mensajes de invalidez (Campos vacios, constraseña no coinciden, etc) este aún así registra a los usuarios. Por ejemplo tengo una condición que si los campos están vacíos no se procede con el registro y lo probé pero cuando envió el formulario de registro con los campos vacíos me manda el mensaje de que los campos no pueden estar vacíos pero a la vez me manda el mensaje de que el usuario se registro correctamente. Este en mi código a ver si alguien puede decirme que está mal.

Código PHP:

              // Comprobar si el formulario de Registro no se envía vacío // 
            if(empty($name) && empty($lastname) && empty($nick) && empty($sex) && empty($pass) && empty($repass) && empty($mail) 
            && empty($remail) && empty($birth) && empty($country)) 
                 
                { 
                    echo "<p>OOPS! Por favor rellena todos los campos.</p>"; 
                    echo "<a href=\"javascript:history.back(1)\"><div id=\"Button-Space\"><div id=\"Button\">Regresar</div></div></a>"; 
                } 
 
            // Comprobar si el campo 'Nombre' no está vacío // 
            elseif(empty($name)) 
 
                { 
                    echo "<p>Haz olvidado ingresar tu <b>Nombre</b>!</p>"; 
                    echo "<a href=\"javascript:history.back(1)\"><div id=\"Button-Space\"><div id=\"Button\">Regresar</div></div></a>"; 
                } 
 
            // Comprobar si el campo 'Apellido' no está vacío // 
            elseif(empty($lastname)) 
 
                { 
                    echo "<p>Haz olvidado ingresar tu <b>Apellido</b>!</p>"; 
                    echo "<a href=\"javascript:history.back(1)\"><div id=\"Button-Space\"><div id=\"Button\">Regresar</div></div></a>"; 
                } 
 
            // Comprobar si el campo 'Nick' no está vacío // 
            elseif(empty($nick)) 
 
                { 
                    echo "<p>Haz olvidado ingresar tu <b>Apodo</b>!</p>"; 
                    echo "<a href=\"javascript:history.back(1)\"><div id=\"Button-Space\"><div id=\"Button\">Regresar</div></div></a>"; 
                } 
                 
            // Comprobar si el campo 'Nick' es permitido // 
            elseif($nick == 'Admin') 
 
                { 
                    echo "<p>El <b>Apodo</b> que ingresaste no está permitido!</p>"; 
                    echo "<a href=\"javascript:history.back(1)\"><div id=\"Button-Space\"><div id=\"Button\">Regresar</div></div></a>"; 
                } 
                 
            // Comprobar si el campo 'Sexo' no está vacío // 
            elseif(empty($sex)) 
 
                { 
                    echo "<p>Haz olvidado seleccionar tu <b>Sexo</b>!</p>"; 
                    echo "<a href=\"javascript:history.back(1)\"><div id=\"Button-Space\"><div id=\"Button\">Regresar</div></div></a>"; 
                } 
 
            // Comprobar si el campo 'Contraseña' no está vacío // 
            elseif(empty($pass)) 
 
                { 
                    echo "<p>Haz olvidado ingresar tu <b>Contrase&ntilde;a</b>!</p>"; 
                    echo "<a href=\"javascript:history.back(1)\"><div id=\"Button-Space\"><div id=\"Button\">Regresar</div></div></a>"; 
                } 
 
            // Comprobar si el campo 'Repite Tu Contraseña' no está vacío // 
            elseif(empty($repass)) 
                 
                { 
                    echo "<p>Haz olvidado repetir tu <b>Contrase&ntilde;a</b>!</p>"; 
                    echo "<a href=\"javascript:history.back(1)\"><div id=\"Button-Space\"><div id=\"Button\">Regresar</div></div></a>"; 
                } 
  
            // Comprobar si el campo 'Contraseña' & 'Repite Tu Contraseña' coinciden // 
            elseif($pass != $repass) 
 
                { 
                    echo "<p>Las <b>Contrase&ntilde;as</b> no coinciden!</p>"; 
                    echo "<a href=\"javascript:history.back(1)\"><div id=\"Button-Space\"><div id=\"Button\">Regresar</div></div></a>"; 
                } 
 
            // Comprobar si el campo 'Correo Electrónico' no está vacío // 
            elseif(empty($mail)) 
 
                { 
                    echo "<p>Haz olvidado ingresar tu <b>Correo electr&oacute;nico</b>!</p>"; 
                    echo "<a href=\"javascript:history.back(1)\"><div id=\"Button-Space\"><div id=\"Button\">Regresar</div></div></a>"; 
                } 
 
            // Comprobar si el campo 'Repite Tu Correo Electrónico' no está vacío // 
            elseif(empty($remail)) 
 
                { 
                    echo "<p>Haz olvidado repetir tu <b>Correo electr&oacute;nico</b>!</p>"; 
                    echo "<a href=\"javascript:history.back(1)\"><div id=\"Button-Space\"><div id=\"Button\">Regresar</div></div></a>"; 
                } 
 
            // Comprobar si el campo 'Correo Electrónico' & 'Repite Tu Correo Electrónico' coinciden // 
            elseif($mail != $remail) 
 
                {  
                    echo "<p>Los <b>Correos Electr&oacute;nicos</b> no coinciden!</p>"; 
                    echo "<a href=\"javascript:history.back(1)\"><div id=\"Button-Space\"><div id=\"Button\">Regresar</div></div></a>"; 
                } 
                 
            // Comprobar si el campo 'Fecha de Nacimiento' no está vacío // 
            elseif(empty($birth)) 
 
                { 
                    echo "<p>Haz olvidado ingresar tu <b>Fecha de Nacimiento</b>!</p>"; 
                    echo "<a href=\"javascript:history.back(1)\"><div id=\"Button-Space\"><div id=\"Button\">Regresar</div></div></a>"; 
                } 
 
            // Comprobar si el campo 'País' no está vacío // 
            elseif(empty($country)) 
 
                { 
                    echo "<p>Haz olvidado seleccionar tu <b>Pa&iacute;s</b>!</p>"; 
                    echo "<a href=\"javascript:history.back(1)\"><div id=\"Button-Space\"><div id=\"Button\">Regresar</div></div></a>"; 
                } 
 
            // Comprobar si el campo 'Apodo' no tiene espacios en blanco // 
            elseif(preg_match_all('<\s>', $nick)) 
 
                { 
                    echo "<p>El <b>Apodo</b> no puede tener espacios en blanco!</p>"; 
                    echo "<a href=\"javascript:history.back(1)\"><div id=\"Button-Space\"><div id=\"Button\">Regresar</div></div></a>"; 
                } 
 
            // Comprobar si el 'Apodo' ingresado está disponible // 
            $value_nick = mysql_query("SELECT * FROM users WHERE nck_us='".$nick."'"); 
            $counter_nick = mysql_num_rows($value_nick); 
            if($counter_nick >= 1) 
 
                { 
                    echo "<p>El Apodo <b>".$nick."</b> ya se encuentra registrado.<br />Intenta con otro Apodo.</p>"; 
                    echo "<a href=\"javascript:history.back(1)\"><div id=\"Button-Space\"><div id=\"Button\">Regresar</div></div></a>"; 
                } 
                 
            // Comprobar si el 'Correo Electr�nico' ingresado está disponible // 
            $value_mail = mysql_query("SELECT * FROM users WHERE eml_us='".$mail."'"); 
            $counter_mail = mysql_num_rows($value_mail); 
            if($counter_mail >= 1) 
 
                { 
                    echo "<p>El Correo Electr&oacute;nico <b>".$mail."</b> ya se encuentra registrado. 
                    <br />Intenta con otro Correo Electr&oacute;nico.</p>"; 
                    echo "<a href=\"javascript:history.back(1)\"><div id=\"Button-Space\"><div id=\"Button\">Regresar</div></div></a>"; 
                } 
 
            // Encriptando la contraseña con MD5 // 
            else { 
            $pass = md5($pass); 
 
            // Registrando al Usuario // 
            $insert = mysql_query("INSERT INTO users(nme_us,lnm_us,nck_us,sex_us,pss_us,eml_us,brt_us,cnt_us,reg_us)VALUES 
            ('".$name."','".$lastname."','".$nick."','".$sex."','".$pass."','".$mail."','".$birth."','".$country."',NOW())"); 
            echo "<p>Te Haz Registrado Correctamente. Ya Puedes Iniciar Sesión <b>".$nick."</b>.</p>"; 
 
            }  
   

[PIRRUMAN]

Es una cuestion de logica , veras si validas los posibles errores pero terminando las validaciones
tienes independiente la validacion para ingresar o no al sistema, debes unirlas o inicializar una variable a cero y sobre cada error sumarle o agregarle el valor 1, sobre esta variable hacer la condicionante de ingreso

[loncho_rojas]

claro que te va a registrar porque en realidad estas poniendo todo en una sola linea jerarquica de IF ELSEIF...

deberia ser algo como

Código PHP:

if(condicion1){
 
   if(condicion2){
 
      if(condicion2){   
        LOGUEAR
      }else{
      mensaje de error 3
      } 
 
   }else{
   mensaje de error 2
  }
 
}else{
mensaje de error 1
}
 

[iDiegoEscobar]

Iniciado por PIRRUMAN

Es una cuestion de logica , veras si validas los posibles errores pero terminando las validaciones
tienes independiente la validacion para ingresar o no al sistema, debes unirlas o inicializar una variable a cero y sobre cada error sumarle o agregarle el valor 1, sobre esta variable hacer la condicionante de ingreso

¿Cómo podría unirlas? Soy novato en el lenguaje D:

[iDiegoEscobar]

Iniciado por loncho_rojas

claro que te va a registrar porque en realidad estas poniendo todo en una sola linea jerarquica de IF ELSEIF...

deberia ser algo como

Código PHP:

if(condicion1){
 
   if(condicion2){
 
      if(condicion2){   
        LOGUEAR
      }else{
      mensaje de error 3
      } 
 
   }else{
   mensaje de error 2
  }
 
}else{
mensaje de error 1
}
 

Muchas gracias por tu ayuda! ;D