Question

Estoy intentado dar de alta a un usuario en la bbdd vía PHP.
Recogo el email y la password de un formulario y aplico el siguiente código .
Siempre me va al ultimo else y me envía a la página de errorcaptcha.html. y no me muestra lo que he puesto en los var_dump.

   <?php
      session_start();
      if(strtoupper($_REQUEST["captcha"]) == $_SESSION["captcha"]){
         // REMPLAZO EL CAPTCHA USADO POR UN TEXTO LARGO PARA EVITAR QUE SE VUELVA A INTENTAR
         $_SESSION["captcha"] = md5(rand()*time());

         // INSERTA EL CÓDIGO EXITOSO AQUI
$dbhost = "localhost";
$dbuser = "corbachu";
$dbpass = "xxxx";
$conx = mysql_connect($dbhost, $dbuser, $dbpass);

 mysql_select_db("corbachu_percy", $conx)or die ('Error al seleccionar la Base de Datos: '.mysql_error());;

    $email = htmlentities($_POST["email"]); 
    $password = htmlentities($_POST["password"]);   
    var_dump($email,$password);

// Comprobamos si el nombre de usuario o la cuenta de correo ya existían

      $checkemail = mysql_query("SELECT email FROM usuarios WHERE email='$email'");
        $email_exist = mysql_num_rows($checkemail);
        if ($email_exist > 0) {
            echo 'El nombre de usuario o la cuenta de correo están ya en uso';
            formRegistro();
        }else{

        $query = 'INSERT INTO usuarios (email, password) VALUES (\''.$email.'\' ,\''.$password.'\')';

    }   
        ?>

        <SCRIPT LANGUAGE="javascript">
            location.href = "zona.php";
            </SCRIPT>' 

<?      
    mysql_close();
?>  
        // FIN DEL DE CODIGO EXITOSO 
<?        
      }else{
         // REMPLAZO EL CAPTCHA USADO POR UN TEXTO LARGO PARA EVITAR QUE SE VUELVA A INTENTAR
         $_SESSION["captcha"] = md5(rand()*time());
         // INSERTA EL CÓDIGO DE ERROR AQUÍ

         header ("Location: errorcaptcha.html");
      }
?>

Answer 1

Ahora que consigues que entre "donde debe" vamos a ver por qué no hace la inserción. Este es tu código:

$query = 'INSERT INTO usuarios (email, password) VALUES (\''.$email.'\' ,\''.$password.'\')';

Ahí sólo estás construyendo una cadena de texto y la almacenas en la variable $query, no puedes esperar que se ejecute sola =) Necesitas ejecutarla con:

mysql_query($query);

Me da la impresión de que estás haciendo mucho copy&paste sin gran idea de lo que haces ¿me equivoco?

Ahora vienen un par de consejos:

1) Ya te dijo Leonardo en uno de sus comentarios que las funciones que estás utilizando para conectar con MySQL no están recomendadas ya que van a desaparecer y se recomienda sustituirlas por mysqli o PDO.

2) Veo según el código que nos vas compartiendo que estás repitiendo y diseminando los parámetros de configuración y conexión a la BBDD por todos los scripts de tus página de tal manera que el mantenimiento te será muy difícil en un futuro. Deberías aprender más sobre PHP y la posibilidad de crear scripts de configuración o clases para el manejo de las conexiones con la BBDD (y en el futuro plantillas, patrones MVC...) Los ejemplos que aplicas están bien como ejemplos pero para una aplicación en producción te serán difícil de mantener.

Answer 2

Si siempre se va al último else tu error en principio no tiene nada que ver con el título de la pregunta "alta en bbdd php" ya que corresponde al siguiente if:

if(strtoupper($_REQUEST["captcha"]) == $_SESSION["captcha"]){

A ver si algún admin te edita el título.

Respecto a tu problema, parece ser que esa comparación siempre está devolviendo un valor falso, y por eso va al último ese.

Verifica que $_SESSION["captcha"] almacena el valor en mayúsculas, en caso contrario deberás reemplazar la línea por:

if(strtoupper($_REQUEST["captcha"]) == strtoupper($_SESSION["captcha"])){

Este error ya lo tuviste anteriormente y lo solucionaste con esta pregunta (http://entredesarrolladores.com/5633/captcha-muestra-imagen-solo-la-primera-vez)