[APORTE] Registro de usuario PHP y ajax

T4ke0veR · #1 (**permalink**) 18/06/2008, 13:05

Bueno señores este formulario fue gracias a la colaboracion de personas q dia a dia entran a este foro, ya q aclarando mis dudas pude culminarlo.
Aun pueden pulirlo un poco mas!!

base de datos
CREATE TABLE `clientes` (
`id` int(11) NOT NULL auto_increment,
`usuario` text NOT NULL,
`pass` text NOT NULL,
`nombre` text NOT NULL,
`apellido` text NOT NULL,
`mail` text NOT NULL,
`cedula` text NOT NULL,
`telefono` text NOT NULL,
`fecha` text NOT NULL,
PRIMARY KEY (`id`)
) ENGINE=InnoDB DEFAULT CHARSET=latin1 AUTO_INCREMENT=10 ;

primero hacemos la conexion a la base de datos
conect.php

Código PHP:

  <?php 
 
$host="localhost"; 
 
$QQ="usuario"; 
 
$p="pass"; 
 
$db=mysql_connect($host,$QQ,$p) 
 
or die ("No pude conectarme a la base de datos"); 
 
mysql_select_db("nombre_de_la_base_de_datos") 
 
or die ("No puedo acceder a la base de datos del sistema"); 
 
?>

luego vamos al form

Código PHP:

  <div id="log"> 
        <div id="log_res"> 
        <!-- SPANNER --> 
        </div> 
    </div><div id="container"> 
                                        <form method="post" id="form" action="register.php"> 
                                          <table align="center" cellpadding="2" cellspacing="0"> 
                                            <tr> 
                                              <td width="120" style="width:120px"><div align="right" class="textoResto">Nombre: </div></td> 
                <td width="172"><div align="left" class="string"> 
                  <input name="nombre" type="text" class="input" id="nombre" value="" size="32" /> 
                  </div>              </td> 
              </tr> 
                                            <tr> 
                                              <td class="textoResto" style="width:120px"><div align="right">Apellido: </div></td> 
                <td><div align="left"> 
                  <input name="apellido" class="input" type="text" id="apellido" value="" size="32" /> 
                  </div></td> 
              </tr> 
                                            <tr> 
                                              <td class="textoResto" style="width:120px"><div align="right">Usuario: </div></td> 
                <td><div align="left"> 
                  <input name="usuario" type="text" class="input" id="usuario" value="" size="32" /> 
                  </div>              </td> 
              </tr> 
                                            <tr> 
                                              <td class="textoResto" style="width:120px"><div align="right">Contraseña: </div></td> 
                <td><div align="left"> 
                  <input name="pass" type="password" class="input" id="pass" value="" size="32" /> 
                  </div></td> 
              </tr> 
                                            <tr> 
                                               
                                              <td class="textoResto" style="width:120px"><div align="right">Repetir Contraseña: </div></td> 
                <td><div align="left"> 
                  <input name="re_pass" type="password" class="input" id="re_pass" value="" size="32" /> 
                  </div></td> 
              </tr> 
                                            <tr> 
                                               
                                              <td class="textoResto" style="width:120px"><div align="right">E-Mail: </div></td> 
                <td><div align="left"> 
                  <input name="mail" type="text" class="input" id="mail" value="" size="32" /> 
                  </div></td> 
              </tr> 
                                            <tr> 
                                              <td style="width:120px"><div align="right" class="textoResto">C.I. / RIF.:</div></td> 
                <td><input name="cedula" type="text" class="input" id="cedula" value="" size="32" /></td> 
              </tr> 
                                            <tr> 
                                              <td style="width:120px"><div align="right" class="textoResto">Teléfono: </div></td> 
                <td><input name="telefono" type="text" class="input" id="telefono" value="" size="32" /></td> 
              </tr> 
                                            <tr> 
                                              <td style="width:120px"><div align="right" class="textoResto">Fecha: </div></td> 
                <td><div align="left"> 
                  <input name="fecha" type="text" class="input" id="fecha" value="" size="25" /> 
                </div></td> 
              </tr> 
                                            <tr> 
                                              <td colspan="2"><div align="right"> 
                                                <input type="image" name="register" class="submit-btn" src="btn.gif" alt="submit" title="submit" /> 
                                                </div></td> 
              </tr> 
                                            </table> 
          </form>

en la misma página debemos incluir los javascript y un estilo css

Código PHP:

  <script type="text/javascript" src="js/mootools.js"></script> 
    <script type="text/javascript"> 
        window.addEvent('domready', function(){ 
                    $('registerForm').addEvent('submit', function(e) { 
                        new Event(e).stop(); 
                        var log = $('log_res').empty().addClass('ajax-loading'); 
                        this.send({ 
                            update: log, 
                            onComplete: function() { 
                                log.removeClass('ajax-loading'); 
                            } 
                        }); 
                    }); 
                }); 
    </script> 
    <style type="text/css"> 
 
        body { 
            font:0.7em Arial, helvetica, sens-serif; 
            color:#567475; 
        } 
        .input { 
            border: 1px solid #99b3b4; 
            width: 170px; 
            background: #e4ebeb; 
            font: 11px verdana, sans-serif; 
            color:#443; 
            padding:3px; 
            margin-bottom:4px; 
            outline:none; 
        } 
        .input:focus { 
            border:0px solid #567475; 
            background: #e4ebeb; 
        } 
        .submit-btn { 
            width: 54px; 
            height: 20px; 
            background: #743 url(submit.gif) no-repeat; 
            outline: none; 
        } 
        .submit-btn:hover { 
            background: #069 url(submit.gif) no-repeat 0 -20px; 
        } 
        div#container { 
            border:0px solid #99b3b4; 
            padding:15px; 
            margin:auto; 
            width:245px; 
        } 
        #log_res { 
            height:auto; 
            padding:10px; 
            margin:10px auto 10px auto; 
            width:270px; 
        } 
        #log_res p { 
            margin:0; 
            padding:4px 0 4px 0; 
        } 
        #log_res.ajax-loading 
            {background: url(spinner.gif) no-repeat center; 
        } 
        .error { 
            color:red; 
            margin:0; 
            padding:0; 
        } 
 
    </style>

lugo leemos el archivo de verificacion de datos y la consulta a la db
register.php

Código PHP:

  <?php 
    include('funciones.php');//las validaciones de los campos 
 
    if ($_POST['nombre']=='' || strlen($_POST['nombre'])<3) 
    { 
        $errors[] = 'Nomnbre debe poseer al menos 3 letras.'; 
    } 
 
    if ($_POST['apellido']=='' || strlen($_POST['apellido'])<3) 
    { 
        $errors[] = 'Apellido debe poseer al menos 3 letras.'; 
    } 
 
    if ($_POST['usuario']=='' || alpha_numeric($_POST['usuario'])==FALSE) 
    { 
        $errors[] = 'Usuario debe contener valores alfa-numéricos'; 
    } 
 
    if ($_POST['pass']=='' || alpha_numeric($_POST['pass'])==FALSE) 
    { 
        $errors[] = 'A password is required and must be alpha-numeric'; 
    } 
 
    if ($_POST['pass']!=$_POST['re_pass']) 
    { 
        $errors[] = 'The two passwords must match'; 
    } 
 
    if (valid_email($_POST['mail'])==FALSE) 
    { 
        $errors[] = 'Por favor, use una direccioón válida de E-Mail'; 
    } 
    if ($_POST['cedula']=='' || strlen($_POST['cedula'])<6) 
    { 
        $errors[] = 'Cédula debe contener al menos 6 caracteres'; 
    } 
    if ($_POST['telefono']=='' || strlen($_POST['telefono'])<6) 
    { 
        $errors[] = 'Telefono debe contener al menos 6 caracteres'; 
    } 
    if ($_POST['fecha']=='' || strlen($_POST['fecha'])<8) 
    { 
        $errors[] = 'Fecha debe contener al menos 8 caracteres'; 
    } 
 
    if(is_array($errors)) 
    { 
        echo '<p class="error"><b>Han ocurrido los siguientes errores</b></p>'; 
        while (list($key,$value) = each($errors)) 
        { 
 
            echo '<span class="error">'.$value.'</span><br />'; 
        } 
    } 
    else { 
    //aqui colocas tu consulta o si lo vas a enciar via mail 
         include ('conect.php'); 
 
$nombre = $_POST['nombre']; 
$apellido = $_POST['apellido']; 
$pass = $_POST['pass']; 
$usuario = $_POST['usuario']; 
$mail = $_POST['mail']; 
$cedula = $_POST['cedula']; 
$telefono = $_POST['telefono']; 
$fecha = $_POST['fecha']; 
 
 
        $query="INSERT INTO clientes SET 
 
        nombre = '$nombre', 
 
        apellido = '$apellido', 
 
        usuario = '$usuario', 
         
        pass = '$pass', 
         
        mail = '$mail', 
         
        cedula = '$cedula', 
         
        telefono = '$telefono', 
         
        fecha = '$fecha'";     
 
         mysql_query($query) or die( "[ERROR] Query: $query, error: " . mysql_error() );   
 
 
 
                 
        echo '<p><b>Felicitaciones!</b></p>'; 
        echo '<span>Tu registro ha sido procesado con éxito. Ahora puedes ingresar usando tu usuario y contraseña. gracias por registrarte !</span>'; 
    } 
?>

y por ultimo el archivo que define las funcion de verificacion
funciones.php

Código PHP:

  <?php 
 
    function alpha_numeric($str) 
    { 
        return ( ! preg_match("/^([-a-z0-9])+$/i", $str)) ? FALSE : TRUE; 
    } 
 
    function valid_email($str) 
    { 
        return ( ! preg_match("/^([a-z0-9\+_\-]+)(\.[a-z0-9\+_\-]+)*@([a-z0-9\-]+\.)+[a-z]{2,6}$/ix", $str)) ? FALSE : TRUE; 
    } 
 
?>

solo les faltaria el moontools.js q pueden descargarselo d aqui
http://mad4milk.net

Gracias a quienes me ayudaron y a quienes se tomaron la molestia d leer.
Y gracias en general a los q hoy en dia hacen d este foro algo util "los usuarios"

Habran muchas personas q podran mejorarlo!!

Espero les sea util a muchos.

Saludos

T4ke0veR · #2 (**permalink**) 18/06/2008, 14:06

anexo...
le aplique una verificacion de user en la base de datos

register.php

Código PHP:

  <?php 
    include('funciones.php');//las validaciones de los campos 
 
    if ($_POST['nombre']=='' || strlen($_POST['nombre'])<3) 
    { 
        $errors[] = 'Nomnbre debe poseer al menos 3 letras.'; 
    } 
 
    if ($_POST['apellido']=='' || strlen($_POST['apellido'])<3) 
    { 
        $errors[] = 'Apellido debe poseer al menos 3 letras.'; 
    } 
 
    if ($_POST['usuario']=='' || alpha_numeric($_POST['usuario'])==FALSE) 
    { 
        $errors[] = 'Usuario debe contener valores alfa-numéricos'; 
    } 
 
    if ($_POST['pass']=='' || alpha_numeric($_POST['pass'])==FALSE) 
    { 
        $errors[] = 'A password is required and must be alpha-numeric'; 
    } 
 
    if ($_POST['pass']!=$_POST['re_pass']) 
    { 
        $errors[] = 'The two passwords must match'; 
    } 
 
    if (valid_email($_POST['mail'])==FALSE) 
    { 
        $errors[] = 'Por favor, use una direccioón válida de E-Mail'; 
    } 
    if ($_POST['cedula']=='' || strlen($_POST['cedula'])<6) 
    { 
        $errors[] = 'Cédula debe contener al menos 6 caracteres'; 
    } 
    if ($_POST['telefono']=='' || strlen($_POST['telefono'])<6) 
    { 
        $errors[] = 'Telefono debe contener al menos 6 caracteres'; 
    } 
    if ($_POST['fecha']=='' || strlen($_POST['fecha'])<8) 
    { 
        $errors[] = 'Fecha debe contener al menos 8 caracteres'; 
    } 
 
    if(is_array($errors)) 
    { 
        echo '<p class="error"><b>Han ocurrido los siguientes errores</b></p>'; 
        while (list($key,$value) = each($errors)) 
        { 
 
            echo '<span class="error">'.$value.'</span><br />'; 
        } 
    } 
     
    $conexion = mysql_connect("localhost", "root", "");  
mysql_select_db("pintor", $conexion);  
          
$sqlex2 = mysql_query("SELECT usuario FROM clientes WHERE usuario='".$_POST['usuario']."'", $conexion);    
$num_rows = mysql_num_rows($sqlex2);   
$usuari = ""; 
if(mysql_num_rows($sqlex2)>0)   
{  
echo 'Este usuario se encuentra registrado en el sistema'; }  
    else { 
    //aqui colocas tu consulta o si lo vas a enciar via mail 
         include ('conect.php'); 
 
  
$nombre = $_POST['nombre']; 
$apellido = $_POST['apellido']; 
$pass = $_POST['pass']; 
$usuario = $_POST['usuario']; 
$mail = $_POST['mail']; 
$cedula = $_POST['cedula']; 
$telefono = $_POST['telefono']; 
$fecha = $_POST['fecha']; 
 
 
        $query="INSERT INTO clientes SET 
 
        nombre = '$nombre', 
 
        apellido = '$apellido', 
 
        usuario = '$usuario', 
         
        pass = '$pass', 
         
        mail = '$mail', 
         
        cedula = '$cedula', 
         
        telefono = '$telefono', 
         
        fecha = '$fecha'";     
 
         mysql_query($query) or die( "[ERROR] Query: $query, error: " . mysql_error() );   
 
 
 
     
                 
        echo '<p><b>Felicitaciones!</b></p>'; 
        echo '<span>Tu registro ha sido procesado con éxito. Ahora puedes ingresar usando tu usuario y contraseña. gracias por registrarte !</span>'; 
        echo $usuari; 
    } 
?>

jaronu · #3 (**permalink**) 18/06/2008, 15:53

Hola T4ke0veR

Gracias por el aporte, seguro que a alguien le salva de romperse la cabeza para registrar usuarios.

Pero una cosa, tambien podrias verificar que el correo no exista en la BBDD para evitar tener usuarios con dos emails repetidos en la BBDD y dos cuentas.

Un saludo

T4ke0veR · #4 (**permalink**) 19/06/2008, 06:55

gracias jaronu lo hare de inmediato y lo publicare!!!
y bueno pues hice una modificacion en el código pues la comprobacion de si existia unuario me estaba saltando las demas verificaciones

Código PHP:

  <?php  
    include('funciones.php');//las validaciones de los campos  
     
        $conexion = mysql_connect("localhost", "root", "");   
        mysql_select_db("pintor", $conexion);   
           
        $sqlex2 = mysql_query("SELECT usuario FROM clientes WHERE usuario='".$_POST['usuario']."'", $conexion);     
        $num_rows = mysql_num_rows($sqlex2);    
 
 
    if ($_POST['nombre']=='' || strlen($_POST['nombre'])<3)  
    {  
        $errors[] = 'Nomnbre debe poseer al menos 3 letras.';  
    }  
 
    if ($_POST['apellido']=='' || strlen($_POST['apellido'])<3)  
    {  
        $errors[] = 'Apellido debe poseer al menos 3 letras.';  
    }  
 
    if ($_POST['usuario']=='' || alpha_numeric($_POST['usuario'])==FALSE)  
    {  
        $errors[] = 'Usuario debe contener valores alfa-numéricos';  
    }  
 
    if ($_POST['pass']=='' || alpha_numeric($_POST['pass'])==FALSE)  
    {  
        $errors[] = 'A password is required and must be alpha-numeric';  
    }  
 
    if ($_POST['pass']!=$_POST['re_pass'])  
    {  
        $errors[] = 'The two passwords must match';  
    }  
 
    if (valid_email($_POST['mail'])==FALSE)  
    {  
        $errors[] = 'Por favor, use una direccioón válida de E-Mail';  
    }  
    if ($_POST['cedula']=='' || strlen($_POST['cedula'])<6)  
    {  
        $errors[] = 'Cédula debe contener al menos 6 caracteres';  
    }  
    if ($_POST['telefono']=='' || strlen($_POST['telefono'])<6)  
    {  
        $errors[] = 'Telefono debe contener al menos 6 caracteres';  
    }  
    if ($_POST['fecha']=='' || strlen($_POST['fecha'])<8)  
    {  
        $errors[] = 'Fecha debe contener al menos 8 caracteres';  
    }  
    if(mysql_num_rows($sqlex2)>0) { 
        $errors[] = 'usuario ya registrado';} 
          
    if(is_array($errors))  
    {  
        echo '<p class="error"><b>Han ocurrido los siguientes errores</b></p>';  
        while (list($key,$value) = each($errors))  
        {  
 
            echo '<span class="error">'.$value.'</span><br />';  
        }  
    }  
      
 
   
//{   
//echo 'Este usuario se encuentra registrado en el sistema'; }   
    else {  
    //aqui colocas tu consulta o si lo vas a enciar via mail  
         include ('conect.php');  
 
   
$nombre = $_POST['nombre'];  
$apellido = $_POST['apellido'];  
$pass = $_POST['pass'];  
$usuario = $_POST['usuario'];  
$mail = $_POST['mail'];  
$cedula = $_POST['cedula'];  
$telefono = $_POST['telefono'];  
$fecha = $_POST['fecha'];  
 
 
        $query="INSERT INTO clientes SET  
 
        nombre = '$nombre',  
 
        apellido = '$apellido',  
 
        usuario = '$usuario',  
          
        pass = '$pass',  
          
        mail = '$mail',  
          
        cedula = '$cedula',  
          
        telefono = '$telefono',  
          
        fecha = '$fecha'";      
 
         mysql_query($query) or die( "[ERROR] Query: $query, error: " . mysql_error() );    
        
        echo '<p><b>Felicitaciones!</b></p>';  
        echo '<span>Tu registro ha sido procesado con éxito. Ahora puedes ingresar usando tu usuario y contraseña. gracias por registrarte !</span>';  
         
} 
?>

tucuta · #5 (**permalink**) 20/05/2013, 17:38

Una pregunta mi gente, como podría hacer para agregar la contraseña encriptada en md5...?

Muchas gracias y muy buen tutorial...

bulter · #6 (**permalink**) 20/05/2013, 18:20

Mmm solo voy a decir unas cuantas cositas que veo mal:

1: mysql_* es obsoleto mejor PDO.
2:

Cita:

Código PHP:

  $nombre = $_POST['nombre'];  
$apellido = $_POST['apellido'];  
$pass = $_POST['pass'];  
$usuario = $_POST['usuario'];  
$mail = $_POST['mail'];  
$cedula = $_POST['cedula'];  
$telefono = $_POST['telefono'];  
$fecha = $_POST['fecha'];

Si tienes error_reporting(E_ALL) eso pitara Nitificaciones como una sirena de ambulancia :P
No compruebas si estan definidas o si los indexes existen.

esto:

Código PHP:

  <?php 

    function alpha_numeric($str) 
    { 
        return ( ! preg_match("/^([-a-z0-9])+$/i", $str)) ? FALSE : TRUE; 
    } 

    function valid_email($str) 
    { 
        return ( ! preg_match("/^([a-z0-9\+_\-]+)(\.[a-z0-9\+_\-]+)*@([a-z0-9\-]+\.)+[a-z]{2,6}$/ix", $str)) ? FALSE : TRUE; 
    } 

?>

Lo puedes simplificar el IF este no sirve

Código PHP:

  <?php 

    function alpha_numeric($str) 
    { 
        return (preg_match("/^([-a-z0-9])+$/i", $str)); 
    } 

    function valid_email($str) 
    { 
        return (preg_match("/^([a-z0-9\+_\-]+)(\.[a-z0-9\+_\-]+)*@([a-z0-9\-]+\.)+[a-z]{2,6}$/ix", $str)); 
    } 

?>

Otra cosa:

Código PHP:

  $sqlex2 = mysql_query("SELECT usuario FROM clientes WHERE usuario='".$_POST['usuario']."'", $conexion);

Pure SQL Injection Vulnerability

En vez de hacer un check en cada if para

Código PHP:

  if ($_POST['nombre']==''
 if ($_POST['apellido']==''
 if ($_POST['usuario']==''

Lo podrias hacer en una o dos lineas con 1 if general o con un loop.
A mi tantos IFs me ponen feo :D
Podrias hacer te una funcion que añade un error segun las condiciones.
Por ejemplo algo rapido podria ser:

Código PHP:

  <?php 

function validationAssert($bool, $ErrorMessage, &$ErrorMessageContainer)
{
    if($bool != false)
    {
        array_push($ErrorMessageContainer, $ErrorMessage);
    }
}

$Errors = array();

validationAssert((strlen($_POST['apellido'])<3), "Apellido debe poseer al menos 3 letras.", $Errors);
validationAssert((strlen($_POST['nombre'])<3), "Nomnbre debe poseer al menos 3 letras.", $Errors);
validationAssert(($_POST['pass'] != $_POST['re_pass']), "The two passwords must match", $Errors);

print_r($Errors);
?>

Otra cosita

Código PHP:

   if ($_POST['pass']=='' || alpha_numeric($_POST['pass'])==FALSE)

oO por que el pass tiene que ser alphanumeric ... todo tipo de pass que uso no cumple ese requisito :D es malo quitalo.

Código PHP:

  if(is_array($errors))

Esto no tiene sentido. Si no tienes ningun error $error simplemente no existira ya que no la defines por ningun lado, no puede ser ni array ni string , ni integer etc.

Hay mas cosas pero lo dejare hasta aqui, que no quiere que te lo tomes a mal :/

Saludos y suerte

bulter · #7 (**permalink**) 20/05/2013, 18:22

Cita:

Iniciado por tucuta

Una pregunta mi gente, como podría hacer para agregar la contraseña encriptada en md5...?

Muchas gracias y muy buen tutorial...

Código PHP:

  <?php 
    $password = "aig5821@#$nfkdfj@kyh9#&("; 
    $md5HashedPassword = md5($password); 
?>

Saludos

tucuta · #8 (**permalink**) 20/05/2013, 18:27

Muchas gracias Bulter, te agradezco en el alma...

;

Vengo nuevamente, felicidades por el codigo y gueno, mi pregunta:

No se puede hacer con Jquery...? porque vieras que el enlace para descargar mootools no funka y ya lo baje de la pagina oficial y tampoco funciona.

tucuta · #9 (**permalink**) 20/05/2013, 18:33

Hola Bulter, muchas gracias por la respuesta y de verdad te agradezco en el alma.

Pregunto y porque no posteas una versión mejorada del código, obviamente si @T4ke0veR pos lo permite.

Mejorarlo estaría genial.

Felicidades y nuevamente gracias...

Cita:

Iniciado por bulter

Mmm solo voy a decir unas cuantas cositas que veo mal:

1: mysql_* es obsoleto mejor PDO.
2:

Si tienes error_reporting(E_ALL) eso pitara Nitificaciones como una sirena de ambulancia :P
No compruebas si estan definidas o si los indexes existen.

esto:

Código PHP:

  <?php  
 
    function alpha_numeric($str)  
    {  
        return ( ! preg_match("/^([-a-z0-9])+$/i", $str)) ? FALSE : TRUE;  
    }  
 
    function valid_email($str)  
    {  
        return ( ! preg_match("/^([a-z0-9\+_\-]+)(\.[a-z0-9\+_\-]+)*@([a-z0-9\-]+\.)+[a-z]{2,6}$/ix", $str)) ? FALSE : TRUE;  
    }  
 
?>

Lo puedes simplificar el IF este no sirve

Código PHP:

  <?php  
 
    function alpha_numeric($str)  
    {  
        return (preg_match("/^([-a-z0-9])+$/i", $str));  
    }  
 
    function valid_email($str)  
    {  
        return (preg_match("/^([a-z0-9\+_\-]+)(\.[a-z0-9\+_\-]+)*@([a-z0-9\-]+\.)+[a-z]{2,6}$/ix", $str));  
    }  
 
?>

Otra cosa:

Código PHP:

  $sqlex2 = mysql_query("SELECT usuario FROM clientes WHERE usuario='".$_POST['usuario']."'", $conexion);

Pure SQL Injection Vulnerability

En vez de hacer un check en cada if para

Código PHP:

  if ($_POST['nombre']=='' 
 if ($_POST['apellido']=='' 
 if ($_POST['usuario']==''

Lo podrias hacer en una o dos lineas con 1 if general o con un loop.
A mi tantos IFs me ponen feo :D
Podrias hacer te una funcion que añade un error segun las condiciones.
Por ejemplo algo rapido podria ser:

Código PHP:

  <?php  
 
function validationAssert($bool, $ErrorMessage, &$ErrorMessageContainer) 
{ 
    if($bool != false) 
    { 
        array_push($ErrorMessageContainer, $ErrorMessage); 
    } 
} 
 
$Errors = array(); 
 
validationAssert((strlen($_POST['apellido'])<3), "Apellido debe poseer al menos 3 letras.", $Errors); 
validationAssert((strlen($_POST['nombre'])<3), "Nomnbre debe poseer al menos 3 letras.", $Errors); 
validationAssert(($_POST['pass'] != $_POST['re_pass']), "The two passwords must match", $Errors); 
 
print_r($Errors); 
?>

Otra cosita

Código PHP:

   if ($_POST['pass']=='' || alpha_numeric($_POST['pass'])==FALSE)

oO por que el pass tiene que ser alphanumeric ... todo tipo de pass que uso no cumple ese requisito :D es malo quitalo.

Código PHP:

  if(is_array($errors))

Esto no tiene sentido. Si no tienes ningun error $error simplemente no existira ya que no la defines por ningun lado, no puede ser ni array ni string , ni integer etc.

Hay mas cosas pero lo dejare hasta aqui, que no quiere que te lo tomes a mal :/

Saludos y suerte

Triby · #10 (**permalink**) 20/05/2013, 18:33

bulter, hace 5 años, cuando se publicó este aporte, mysql_* todavía no era obsoleta.

Las otras sugerencias de malas prácticas me parece que son adecuadas, sólo no acaba de gustarme la función validationAssert(), pero es cuestión de preferencias.

Me gusta más sin if y sin función:

Código PHP:

Ver original$errors = array();
(strlen($nombre) < 3 && $errors[] = 'El nombre debe contener al menos 3 caracteres');
(strlen($apellido) < 3 && $errors[] = 'El apellido debe contener al menos 3 caracteres');
($pass1 != $pass2 && $errors[] = 'Las contraseñas no coinciden.');

tucuta · #11 (**permalink**) 20/05/2013, 18:43

Wow, les digo algo bulter y Triby, geniales los dos.

Que bueno sería poder adaptarlo a hoy en día osea a corregir los errores que se han dicho incluyendo el uso de jquery.

Con gusto les puedo ayudar en lo que pueda, de verdad con todo el gusto mi gente.

Saludos a los dos

bulter · #12 (**permalink**) 20/05/2013, 19:04

Cita:

Iniciado por Triby

bulter, hace 5 años, cuando se publicó este aporte, mysql_* todavía no era obsoleta.

Las otras sugerencias de malas prácticas me parece que son adecuadas, sólo no acaba de gustarme la función validationAssert(), pero es cuestión de preferencias.

Me gusta más sin if y sin función:

Código PHP:

Ver original$errors = array();
(strlen($nombre) < 3 && $errors[] = 'El nombre debe contener al menos 3 caracteres');
(strlen($apellido) < 3 && $errors[] = 'El apellido debe contener al menos 3 caracteres');
($pass1 != $pass2 && $errors[] = 'Las contraseñas no coinciden.');

Wow ..... Tio juro que ni vi la fecha :O
Bueno lo de los ifs creo que es a gusto, porque pregiero la funccion ya que por si algun dia quiero Agregar o quitar algo en los principios o finales de los mensajes, no se me mola mas una funccion :D pero como dijiste es a gusto

Siento haber comentado un post tan vijo ni me fije :D

Saludos