Test Foro de elhacker.net SMF 2.1

Programación => Desarrollo Web => PHP => Mensaje iniciado por: nobo en 20 Febrero 2012, 00:49 AM

Título: Este formulario de registro estaria bien ?
Publicado por: nobo en 20 Febrero 2012, 00:49 AM
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);}}
Título: Re: Este formulario de registro estaria bien ?
Publicado por: Shell Root en 20 Febrero 2012, 01:08 AM
Volvemos a lo mismo, no somos adivinos, tienes que decirnos que es lo que pasa, errores, dudas, etc?
Título: Re: Este formulario de registro estaria bien ?
Publicado por: nobo en 20 Febrero 2012, 01:10 AM
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;

   }
Título: Re: Este formulario de registro estaria bien ?
Publicado por: Shell Root en 20 Febrero 2012, 02:14 AM
1. Miedo de qué?
2. Los IF's son básicos, si no los sabes usar, aprende!
Título: Re: Este formulario de registro estaria bien ?
Publicado por: nobo en 21 Febrero 2012, 00:03 AM
Era una broma lo de miedo xDD Los errores esos no son de if...