Este formulario de registro estaria bien ?

Iniciado por nobo, 20 Febrero 2012, 00:49 AM

0 Miembros y 2 Visitantes están viendo este tema.

nobo

Pues eso... estaria bien este codigo ?

Código (php) [Seleccionar]
    if (isset($_POST) && isset($_SERVER['HTTP_REFERER']) && $_SERVER['HTTP_REFERER'] == 'http://www.elzulo***/alta_usuario.php' && !empty($_POST)) {
        $username = $_POST["strNick"];
        $pass = $_POST["strPassword"];
        $password = $_POST["password"];
        $email = $_POST["strEmail"];
        $mail = $_POST["mail"];
        $telefono = $_POST["intNtelf"];
        $telefono2 = $_POST["telefono"];
         
        $checkuser = comprobaruser($_POST["strNick"]);
        $checkmail = comprobaruser($_POST["strEmail"]);
         
              if($pass!=$password) {
                 echo "Las contraseñas no coinciden"; }
                    if($email!=$mail) {
                 echo "El email no coincide"; }
             
                 if($telefono!=$telefono2) {
                 echo "Los telefonos no coinciden";
              }else {
                 if ($checkuser !=0) {
                     echo "El usuario" .$username. "ya existe"; }
                     
                     if ($checkmail !=0){
                         echo "El mail" .$email. "ya existe"; }
             
              }
        }else {
         
        if (isset($_POST["form1"])) {
          $insertSQL = sprintf("INSERT INTO tblusuario (strNombre, strEmail, intActivo, strPassword, strDireccion) VALUES (%s, %s, %s, %s, %s)",
                               GetSQLValueString($_POST['strNombre'], "text"),
                               GetSQLValueString($_POST['strEmail'], "text"),
                               GetSQLValueString($_POST['intActivo'], "int"),
                               GetSQLValueString($_POST['strPassword'], "text"),
                               GetSQLValueString($_POST['strDireccion'], "text"));
         
          mysql_select_db($database_conexionzulo, $conexionzulo);}}

Shell Root

Volvemos a lo mismo, no somos adivinos, tienes que decirnos que es lo que pasa, errores, dudas, etc?
Por eso no duermo, por si tras mi ventana hay un cuervo. Cuelgo de hilos sueltos sabiendo que hay veneno en el aire.

nobo

#2
Realmente todavia no lo he probado jejeje me da miedo... Simplemente saber si los if else estan bien... Es decir que posibles errores habria :P

Si no veis nada raro es que no estara tan mal...

EDITO:

Los errores...

( ! ) Warning: mysql_num_rows() expects parameter 1 to be resource, boolean given in C:\wamp\www\zulo\includes\funciones.php on line 333
Call Stack
#   Time   Memory   Function   Location
1   0.0013   415120   {main}( )   ..\alta_usuario.php:0
2   0.0053   529240   comprobaruser( )   ..\alta_usuario.php:49
3   0.0085   529584   mysql_num_rows ( )   ..\funciones.php:333

( ! ) Warning: mysql_num_rows() expects parameter 1 to be resource, boolean given in C:\wamp\www\zulo\includes\funciones.php on line 348
Call Stack
#   Time   Memory   Function   Location
1   0.0013   415120   {main}( )   ..\alta_usuario.php:0
2   0.0090   529328   comprobarmail( )   ..\alta_usuario.php:50
3   0.0100   529672   mysql_num_rows ( )   ..\funciones.php:348

Las funciones:


// +++++++++++++++++++++++++++++++++++++++++++
// +++++++++++++++++++++++++++++++++++++++++++

function comprobaruser($user)
   {
      global $database_conexionzulo, $conexionzulo;
      mysql_select_db($database_conexionzulo, $conexionzulo);
      $query_ConsultaFuncion = sprintf("SELECT strNick FROM tblusuarios WHERE idUsuario = %s",$user);
      $ConsultaFuncion = mysql_query($query_ConsultaFuncion, $conexionzulo);
      $totalRows_ConsultaFuncion = mysql_num_rows($ConsultaFuncion);
      if ($totalRows_ConsultaFuncion == 0)
      return 0;

   }      

// +++++++++++++++++++++++++++++++++++++++++++
// +++++++++++++++++++++++++++++++++++++++++++

function comprobarmail($user)
   {
      global $database_conexionzulo, $conexionzulo;
      mysql_select_db($database_conexionzulo, $conexionzulo);
      $query_ConsultaFuncion = sprintf("SELECT strEmail FROM tblusuarios WHERE idUsuario = %s",$user);
      $ConsultaFuncion = mysql_query($query_ConsultaFuncion, $conexionzulo);
      $totalRows_ConsultaFuncion = mysql_num_rows($ConsultaFuncion);
      if ($totalRows_ConsultaFuncion == 0)
      return 0;

   }

Shell Root

1. Miedo de qué?
2. Los IF's son básicos, si no los sabes usar, aprende!
Por eso no duermo, por si tras mi ventana hay un cuervo. Cuelgo de hilos sueltos sabiendo que hay veneno en el aire.

nobo

Era una broma lo de miedo xDD Los errores esos no son de if...