error query was empty

[ecosysonidos]

Pues bien tengo un problema con mi codigo
lo que pretendo hacer
busco en mi bdd el mail de un usuario que se va a registrar, si el email que estan ingresando "existe" entonces no lo deja registrarse y manda el mensaje de "el usuario existe" eso lo hace bien es decir me compara si existe no lo registra de nuevo y si no existe lo registra....

el problema es cuando el usuario ya existe seguido de el mensaje de "el usuario Existe" me manda Error query was empty.


este es mi codigo php

Código PHP:

  <?php
error_reporting ( E_ERROR );
$con = mysql_connect("localhost","root","");
if (!$con)
  {
  die('Could not connect: ' . mysql_error());
  }
 
mysql_select_db("mk3", $con);
$name=$_POST['name'];
$direc=$_POST['direc'];
$tel=$_POST['tel'];
$email=$_POST['email'];
$pass=$_POST['pass'];
$hoy=date("j, n, Y");
 
if(!isset($name))
    echo  "";
else{
    if($name=="")
    {
        echo "debes de insertar el nombre";
    }
        else{
            if($direc=="")
                {
                    echo "debes de insertar la direc";
                }
                else{
                    if($tel=="")
                {
                    echo "debes de insertar el tel";
                }
                    else{
                        if($email=="")
                    {
                        echo "debes de insertar el email";
                    }
                        else{
                            if($pass=="")
                        {
                            echo "debes de insertar el pass";
                        }
                            else{
                        $result = mysql_query("SELECT Email FROM usuarios WHERE Email='$email'");
                        $row=mysql_num_rows($result);
                            }
if($row==1)
 
    echo "el usuario existe";
else
      $insertar="INSERT INTO usuarios (Nombre, Direccion, Telefono, Email, Password, Fecha)
    VALUES
        ('$_POST[name]','$_POST[direc]', '$_POST[tel]', '$_POST[email]', '$_POST[pass]', '$_POST[hoy]')";
 
if (!mysql_query($insertar,$con))
  {
  die('Error: ' .  mysql_error());
  }
echo "Nuevo Contacto Ingresado con Exito";
 
 
    }
}
}
}
}
mysql_close($con);
?>   

les agradeceria si me ayudan a mejorar mi codigo

[quimfv]

Código PHP:

<?php
error_reporting ( E_ERROR );
$con = mysql_connect("localhost","root","");
if (!$con){
  die('Could not connect: ' . mysql_error());
}
mysql_select_db("mk3", $con);
$name=$_POST['name'];
$direc=$_POST['direc'];
$tel=$_POST['tel'];
$email=$_POST['email'];
$pass=$_POST['pass'];
$hoy=date("j, n, Y");
 
if(!isset($name)){
    echo  "";
}else{
 if($name==""){
  echo "debes de insertar el nombre";
 }else{
  if($direc==""){
   echo "debes de insertar la direc";
  }else{
   if($tel==""){
    echo "debes de insertar el tel";
   }else{
    if($email==""){
     echo "debes de insertar el email";
    }else{
     if($pass==""){
      echo "debes de insertar el pass";
    }else{
      $result = mysql_query("SELECT Email FROM usuarios WHERE Email='$email'");
      $row=mysql_num_rows($result);
      if($row==1) {
       echo "el usuario existe";
      }else{
        $insertar="INSERT INTO usuarios (Nombre, Direccion, Telefono, Email, Password, Fecha) ".
                  "VALUES ('$_POST[name]','$_POST[direc]', '$_POST[tel]', '$_POST[email]', '$_POST[pass]', '$_POST[hoy]')";
        if (!mysql_query($insertar,$con)) {
         die('Error: ' .  mysql_error());
        }else{
         echo "Nuevo Contacto Ingresado con Exito";
        }
       }
      }    
     }
    }
   }
  } 
 }
mysql_close($con);
?>
 

Mejorable.... p.e. si defines un indice unico sobre Email, podras lanzar directamente el INSERT si tiene exito es que no habia repetición si falla es que ya habia un usuario con ese email... eso ahorraria una query...

Te he cerrado bien los condicionales... asi creo que te funcionará.

Intenta ordenar tu codigo ayuda mucho a controlar lo que estas haciendo.

[ecosysonidos]

Iniciado por quimfv

Código PHP:

<?php
error_reporting ( E_ERROR );
$con = mysql_connect("localhost","root","");
if (!$con){
  die('Could not connect: ' . mysql_error());
}
mysql_select_db("mk3", $con);
$name=$_POST['name'];
$direc=$_POST['direc'];
$tel=$_POST['tel'];
$email=$_POST['email'];
$pass=$_POST['pass'];
$hoy=date("j, n, Y");
 
if(!isset($name)){
    echo  "";
}else{
 if($name==""){
  echo "debes de insertar el nombre";
 }else{
  if($direc==""){
   echo "debes de insertar la direc";
  }else{
   if($tel==""){
    echo "debes de insertar el tel";
   }else{
    if($email==""){
     echo "debes de insertar el email";
    }else{
     if($pass==""){
      echo "debes de insertar el pass";
    }else{
      $result = mysql_query("SELECT Email FROM usuarios WHERE Email='$email'");
      $row=mysql_num_rows($result);
      if($row==1) {
       echo "el usuario existe";
      }else{
        $insertar="INSERT INTO usuarios (Nombre, Direccion, Telefono, Email, Password, Fecha) ".
                  "VALUES ('$_POST[name]','$_POST[direc]', '$_POST[tel]', '$_POST[email]', '$_POST[pass]', '$_POST[hoy]')";
        if (!mysql_query($insertar,$con)) {
         die('Error: ' .  mysql_error());
        }else{
         echo "Nuevo Contacto Ingresado con Exito";
        }
       }
      }    
     }
    }
   }
  } 
 }
mysql_close($con);
?>
 

Mejorable.... p.e. si defines un indice unico sobre Email, podras lanzar directamente el INSERT si tiene exito es que no habia repetición si falla es que ya habia un usuario con ese email... eso ahorraria una query...

Te he cerrado bien los condicionales... asi creo que te funcionará.

Intenta ordenar tu codigo ayuda mucho a controlar lo que estas haciendo.

si es un desorden el mio pero ya solucione el problema que tenia en el error
die('Error: ' . mysql_error()); me mandaba que mi query estaba vacio ya lo sulicione el codigo funciona a la perfeccion pero gracias por la recomendacion tratare de ordenarlo mas saludos