problema con registro de usuarios

[WinderJerter]

Hola que tal, lo que pasa es que ando haciendo un formulario para registrar usuarios y lo estoy haciendo de esta maner

Código PHP:

<?php
 
$username = $_POST['username'];
$email = $_POST['email'];
$pass = $_POST['pass'];
$repass = $_POST['repass'];
 
if($_POST['submit'] == "register") {
    
    if(preg_match("/[^a-zA-Z0-9]/", $username)) {
        $error['number/letters'] = '<div style="color: red;">Please use only letters (a-z), and numbers.</div>';
    } else {
        if(!empty($username)) {
            if((strlen($username)) < 6 || (strlen($username)) > 18) {
                $error['min/max_c'] = '<div style="color: red;">Please use between 6 and 18 characters.</div>';
            }
        }
    }
    
    if(!empty($email)) {
        if(!preg_match("/^([a-zA-Z0-9])+([\.a-zA-Z0-9_-])*@([a-zA-Z0-9_-])+(\.[a-zA-Z0-9_-]+)*\.([a-zA-Z]{2,6})/", $email)) {
            $error['email'] = '<div style="color: red;">Enter your correct email address.</div>';
        }
    }
    
    if(!empty($pass)) {
        if((strlen($pass)) < 6 || (strlen($pass)) > 18) {
            $error['pass_min/max'] = '<div style="color: red;">Please use between 6 and 18 characters.</div>';
        } else {
            if(!empty($repass)) {
                if($pass != $repass) {
                    $error['pass'] = '<div style="color: red;">These passwords don\'t match.</div>';
                }
            }
        }
    }
    
        
}
 
?>
 

El problema es que no se como hacerle para que si no se produce ningún error se guarden los datos.

Saludos y gracias de antemano

[Nemutagk]

o.O no se te ocurrió revisar si $error tiene items dentro...

Código PHP:

if (count($error) == 0) {
echo 'No hay errores, a insertar!';
}else {
echo 'Errores:';
foreach($error as $e) {
echo $e;
}
}
 

[WinderJerter]

Iniciado por Nemutagk

o.O no se te ocurrió revisar si $error tiene items dentro...

Código PHP:

if (count($error) == 0) {
echo 'No hay errores, a insertar!';
}else {
echo 'Errores:';
foreach($error as $e) {
echo $e;
}
}
 

Hola que tal Nemutagk, gracias por responder.

Hace rato estuve tratando de hacerlo y lo hice de otras formas parecidas a la tuya pero de todas las formas que lo intente me encontré el mismo problema, lo que ocurre es que cuando mando el formulario sin llenar ningún campo me sale el echo 'No hay errores, a insertar!';

Saludos!

[Nemutagk]

eso es cuestión de tus validaciones de los campos, no se si te has dado cuenta, pero todos los campos solo son validados si no están vacíos! por lo tanto si llegan vacíos no se validan, en tus if debes agregar un else donde agregas un error indicando que están vacíos...

Código PHP:

if(!empty($email)) {
        if(!preg_match("/^([a-zA-Z0-9])+([\.a-zA-Z0-9_-])*@([a-zA-Z0-9_-])+(\.[a-zA-Z0-9_-]+)*\.([a-zA-Z]{2,6})/", $email)) {
            $error['email'] = '<div style="color: red;">Enter your correct email address.</div>';
        }
    }else {
        $error['email'] = '<div style="color: red;">Is necessary to indicate an email address.</div>';
    }
 

[WinderJerter]

Iniciado por Nemutagk

eso es cuestión de tus validaciones de los campos, no se si te has dado cuenta, pero todos los campos solo son validados si no están vacíos! por lo tanto si llegan vacíos no se validan, en tus if debes agregar un else donde agregas un error indicando que están vacíos...

Código PHP:

if(!empty($email)) {
        if(!preg_match("/^([a-zA-Z0-9])+([\.a-zA-Z0-9_-])*@([a-zA-Z0-9_-])+(\.[a-zA-Z0-9_-]+)*\.([a-zA-Z]{2,6})/", $email)) {
            $error['email'] = '<div style="color: red;">Enter your correct email address.</div>';
        }
    }else {
        $error['email'] = '<div style="color: red;">Is necessary to indicate an email address.</div>';
    }
 

Se me olvido agregar la otra parte de mi formulario que se procesa en la misma pagina

Código HTML:

<form method="post" action="register.php" enctype="multipart/form-data">
 
<div class="panel">
<h2>Choose a username:</h2>
<input type="text" class="inputtext" autocomplete="off" name="username">
<?php if(empty($_POST['username']) && isset($_POST['username'])) { echo '<div style="color: red;">You can\'t leave this empty.</div>'; } ?>
<?=$error['user_exist']?>
<?=$error['min/max_c']?>
<?=$error['number/letters']?>
</div>
 
<div class="panel">
<h2>Your current email address:</h2>
<input type="text" class="inputtext" autocomplete="off" name="email">
<?php if(empty($_POST['email']) && isset($_POST['email'])) { echo '<div style="color: red;">You can\'t leave this empty.</div>'; } ?>
<?=$error['email']?>
<?=$error['email_exist']?>
</div>
 
<div class="panel">
<h2>Choose a password:</h2>
<input type="password" class="inputtext" name="pass">
<?php if(empty($_POST['pass']) && isset($_POST['repass'])) { echo '<div style="color: red;">You can\'t leave this empty.</div>'; } ?>
<?=$error['pass_min/max']?>
</div>
 
<div class="panel">
<h2>Re-enter password:</h2>
<input type="password" class="inputtext" name="repass">
<?php if(empty($_POST['repass']) && isset($_POST['repass'])) { echo '<div style="color: red;">You can\'t leave this empty.</div>'; } ?>
<?=$error['pass']?>
</div>
 
<div class="panel_upload">
<input id="b-submit" type="submit" name="submit" value="register">
</div>
 
</form>
 

[Nemutagk]

o.O sin ofender, pero, realmente has comprendido las validaciones que haces en el primer script publicado y las que haces en el segundo script?... para que quede claro...en tu código del formulario (el segundo código publicado) validas si esta vacío el campo, PERO NO GUARDAS EL ERROR, solo imprimes el mensaje, por lo tanto es NO ES UNA VALIDACION, y por lo cual al verificar el array $error no encuentra dichos errores por lo tanto el script da por valido los datos enviados aun cuando estos no existan....

[WinderJerter]

Iniciado por Nemutagk

o.O sin ofender, pero, realmente has comprendido las validaciones que haces en el primer script publicado y las que haces en el segundo script?... para que quede claro...en tu código del formulario (el segundo código publicado) validas si esta vacío el campo, PERO NO GUARDAS EL ERROR, solo imprimes el mensaje, por lo tanto es NO ES UNA VALIDACION, y por lo cual al verificar el array $error no encuentra dichos errores por lo tanto el script da por valido los datos enviados aun cuando estos no existan....

Ok muchas gracias ya entendí el problema, ahorita lo arreglo y comento. Muchas gracias.

Saludos!

[WinderJerter]

Bueno muchas gracias por todo Nemutagk, aquí esta el código corregido por si alguien lo quiere utilizar, no se si sea la forma correcta de hacerlo y si alguien tiene una sugerencia para mejorarlo sera bien recibida.

Código PHP:

<?php
 
$username = $_POST['username'];
$email = $_POST['email'];
$pass = $_POST['pass'];
$repass = $_POST['repass'];
 
if($_POST['submit'] == "register") {
    
    if(preg_match("/[^a-zA-Z0-9]/", $username)) {
        $error['number/letters'] = '<div style="color: red;">Please use only letters (a-z), and numbers.</div>';
    } else {
        if(!empty($username)) {
            if((strlen($username)) < 6 || (strlen($username)) > 18) {
                $error['min/max_c'] = '<div style="color: red;">Please use between 6 and 18 characters.</div>';
            }
        } else {
            $error['username_null'] = '<div style="color: red;">You can\'t leave this empty.</div>';
        } 
    }
    
    if(!empty($email)) {
        if(!preg_match("/^([a-zA-Z0-9])+([\.a-zA-Z0-9_-])*@([a-zA-Z0-9_-])+(\.[a-zA-Z0-9_-]+)*\.([a-zA-Z]{2,6})/", $email)) {
            $error['email'] = '<div style="color: red;">Enter your correct email address.</div>';
        }
    } else {
        $error['email_null'] = '<div style="color: red;">You can\'t leave this empty.</div>';
    }
    
    if(!empty($pass)) {
        if((strlen($pass)) < 6 || (strlen($pass)) > 18) {
            $error['pass_min/max'] = '<div style="color: red;">Please use between 6 and 18 characters.</div>';
        }
    } else {
        $error['pass_null'] = '<div style="color: red;">You can\'t leave this empty.</div>';
    }
    
    if(empty($repass)) {
        $error['repass_null'] = '<div style="color: red;">You can\'t leave this empty.</div>';
    } else {
        if($pass != $repass) {
            $error['pass'] = '<div style="color: red;">These passwords don\'t match.</div>';
        }
    }
 
    if(count($error) == 0) {
        echo "no hay errores :)";
    }
        
}
 
?>
 

Saludos!