0

Necesito verificar si a la hora de insertar un nuevo cliente, registrar.php, existe el email que se ha insertado en el formulario o no.

  • Si existe, mensaje de alerta, manda el foco al campo input email y no envía datos.

  • Si NO existe, lo insertamos en la base de datos al cliente.

Función insertarCliente(...,...,...):

    static public function insertarCliente($email, $nombre, $apellidos, $dni, $telefono, $password){
        $ejecucion = self::Conexion();
        $sql = "INSERT INTO clientes VALUES ('".$email."', '".$nombre."', '".$apellidos."', '".$dni."', '".$telefono."', '".$password."', 1)";
        $ok = $ejecucion->exec($sql);
        if($ok==1){
            return true;
        }else{
            return false;
        }
    }

Función obtenerEmails():

static public function obtenerEmails(){
    //Realizamos la consulta.
    $ejecucion = self::Conexion();
    $sql = "SELECT idemail FROM clientes;";
    $registro = $ejecucion->query($sql);
    //Creamos una array de emails.
    $emails = array();
    //Mientras haya email los guardamos...
    while($email = $registro->fetch()){
        //Array asociativo: al array $emails le pasamos el email concreto.
        array_push($emails, $email);
    }
    //Devolvemos el array de emails.
    return $emails;
}

Código HTML/PHP:

<!DOCTYPE html>
<html lang="es">
    <head>
        <meta http-equiv="Content-Type" content="text/html; charset=UTF-8" />
        <title>Formulario de Registro</title>
        <link rel="stylesheet" href="css/estilos_registrar.css">
        <script type="text/javascript" src="js/validar_formulario.js"></script>
    </head>

    <body background="imagenes/fondo_campo2.jpg">
        <!--<h1>Formulario de Registro</h1>-->
        <form name="miformulario" id="miformulario" action="registrar.php" method="POST" class="form-register" onsubmit="return validar_formulario();">
            <h2 class="form-titulo">CREA UNA CUENTA</h2>
            <div class="contenedor-inputs">
                <!-- Por cada etiqueta de <input> <label for="id_mismo_que_input">XXXX: </label> -->             <!-- required para HTML5 -->
                <input type="text" name="nombre" id="nombre" placeholder="Nombre" tabindex="1" class="input-2" onkeypress="return soloLetras(event);">
                <input type="text" name="apellidos" id="apellidos" placeholder="Apellidos" tabindex="2" class="input-2">
                <br/>
                <input type="text" name="idemail" id="email" placeholder="Email" tabindex="3" class="input-1">
                <br/>
                <input type="text" name="dni" id="dni" placeholder="DNI" maxlength="9" tabindex="4" class="input-2" onkeypress="return soloLetrasNumeros(event);">
                <input type="text" name="telefono" id="telefono" placeholder="Teléfono" maxlength="9" tabindex="5" class="input-2" onkeypress="return soloNumeros(event);">
                <br/>
                <input type="password" name="password" id="password1" placeholder="Contraseña" tabindex="6" class="input-2">
                <input type="password" name="password" id="password2" placeholder="Repetir contraseña" tabindex="7" class="input-2">
                <br/>
                <input type="submit" value="Registrar" name="registrar" class="registrar" tabindex="8"/>
                <?php
                include "Clases/BD.php";
                //Si pulsamos el botón "Registrar"...
                if(isset($_POST["registrar"])){
                    $nombre = $_POST["nombre"];
                    $apellidos = $_POST["apellidos"];
                    $dni = $_POST["dni"];
                    $telefono = $_POST["telefono"];
                    $idemail = $_POST["idemail"];
                    $password = $_POST["password"];

                    $idemails = BD::obtenerEmails();
                    if(!in_array($idemail, $idemails)){
                        //Llamamos al método "insertarCliente" y le pasamos los parámetros del formulario.
                        BD::insertarCliente($idemail, $nombre, $apellidos, $dni, $telefono, $password);
                        header ("Location: si.php");
                    }else{
                        header ("Location: no.php");
                    }
                }
                ?>
                <p class="form-link">¿Ya tienes una cuenta? <a href="iniciar_sesion_cliente.php">Ingresa aquí</a></p>
            </div>
        </form>
        <!--<div class="footer">
            <p>Copyrigth</p>
        </div>-->
    </body>
</html>

Error:

¿Por qué siempre me entra en si.php exista o no exista el email en la BBDD?

10
  • El segundo parámetro del la función in_array espera un array y lo que contiene la variable $emails no lo es, revisa el tipo de dato que devuelve el método BD::obtenerEmails(); o agrégalo a tu pregunta y con gusto te ayudaremos. el 18 dic. 2017 a las 22:33
  • El tipo de dato de $emails es un array. Debido a que si hago esto me extrae todos los elementos. $emails = BD::obtenerEmails(); foreach ($emails as $email){ echo $email["idemail"]."<br/>"; }
    – omaza1990
    el 18 dic. 2017 a las 22:37
  • Intenta devolver los valores, en lugar de mostrarlos $correos = array(); foreach ($emails as $email){ $correos[] = $email["idemail"]; } return $correos; el 18 dic. 2017 a las 22:44
  • Los muestro en ese código del comentario para ver que sí están guardados, es decir, que el método obtenerEmails() funciona correctamente. Ahí no está el problema.
    – omaza1990
    el 18 dic. 2017 a las 22:46
  • 1
    Usa var_dump desde el front var_dump($emails) para un mejor detalle el 18 dic. 2017 a las 22:54

1 respuesta 1

2

El problema se debe a que tu función obtenerEmails está devolviendo un array de arrays, de la forma array( array('idemail' => xx), array('idemail' => yy), ...). Manteniendo tu código, la forma más sencilla de solucionarlo es cambiar la función así:

static public function obtenerEmails(){
    //Realizamos la consulta.
    $ejecucion = self::Conexion();
    $sql = "SELECT idemail FROM clientes;";
    $registro = $ejecucion->query($sql);
    //Creamos una array de emails.
    $emails = array();
    //Mientras haya email los guardamos...
    while($email = $registro->fetch()){
        //Array asociativo: al array $emails le pasamos el email concreto.
        array_push($emails, $email['idemail']);
    }
    //Devolvemos el array de emails.
    return $emails;
}

Sin embargo, también podrías desplazar el test a la BdDD directamente, cambiando a esto:

static public function mailExists($email) {
   $ejecucion = self::Conexion();
   $sql = "SELECT idemail FROM clientes WHERE idemail = ?";
   $stmt = $ejecucion->prepare($sql);
   $stmt->bind_param("e", $email);
   $stmt->execute();
   return ($stmt->get_results())->num_rows!==0;
}

Esta función (no testada, podría tener algún bug) debería devolver un booleano, verdadero si el mail ya existe en la base de datos y falso si no. Utiliza declaraciones preparadas para mayor seguridad.

2
  • Si quiero añadir un alert en plan "ventana emergente" con PHP, ¿habría una forma de realizarlo? Mensaje: "Cliente (no) guardado correctamente". Por ejemplo en java es con JOptionPane
    – omaza1990
    el 19 dic. 2017 a las 10:54
  • PHP es un lenguaje servidor. No interactúa directamente con el PC del usuario, sólo puede enviarle datos. Sin embargo, sería posible, mediante PHP, crear código HTML que, junto con un CSS apropiado, tuviese aspecto de popup. Otra opción sería hacer el test en Javascript, via una llamada al código PHP. Javascript es un lenguaje interpretado directamente en el navegador, lo que permitiría lanzar el popup en tiempo real. En cualquier caso, son problemas complejos y distintos del de esta pregunta.
    – delCano
    el 19 dic. 2017 a las 11:26

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.