Skip to main content
2 de 3
se añadieron 389 caracteres en el cuerpo
A. Cedano
  • 95.2k
  • 31
  • 145
  • 257

Puedes crear una función de verificación, la cual invocarías antes de insertar.

Propongo esta función que además construirá una consulta con WHERE dinámico.

function dataExists($theTable,$theData)
{ 
    $theWhere = array();
    /*Agregamos cada nombre de columna y marcadores ?*/
    foreach($theData as $k=>$v)
    {
        array_push($theWhere, $k . "=?");
    }
    $sql="SELECT COUNT(*) total FROM $theTable";
    $sql .= " WHERE " . implode(" AND ", $theWhere);
    $theParams=array_values($theData);
    $stmt = Conexion::conectar()->prepare($sql);
    /*En PDO podemos pasar los parámetros en el execute como un array :)*/
    $stmt -> execute($theParams);
    $theTotal=$stmt->fetchColumn();
    $theStatus=($theTotal>0) ? TRUE : FALSE;
    return $theStatus;
}

##¿Cómo usarla?

Lo primero que debes respetar de forma estricta es que en el array que la pasas, las claves deben ser exactamente los nombres de columnas que servirán de criterio en el WHERE, pues la función ha sido pensada para que sea dinámica, de forma que funcione con cualquier tabla / columnas.

Por lo tanto, deberás pasarle un array parecido a esto:

$theData = array(
                     'id_usuario'=>1,
                     'id_producto'=>7
                );

Igualmente, como es lógico, el parámetro $theTable debe ser el nombre de la tabla sobre la que quieres consultar, por ejemplo:

$theTable="nombre_tabla";

La consulta preparada que creará la función en este caso será algo así:

SELECT COUNT(*) total FROM nombre_tabla WHERE id_usuario=? AND id_producto=?

Nótese que en PDO podemos usar marcadores de :nombre, o marcadores de posición ?. Aquí he optado por esta segunda posibilidad por conveniencia, practicidad y para darle flexibilidad al código, permitiendo un WHERE dinámico.

Entonces, mdlAgregarDeseo quedaría más o menos así:

static public function mdlAgregarDeseo($tabla, $datos){
    /*Vamos a recoger con esta variable para hacer un solo return*/
    $msgInfo="";
    /*Pasamos la información a nuestra función, la cual devolverá FALSE si no encuentra datos*/
    if (!dataExists($tabla,$datos)){
        $stmt = Conexion::conectar()->prepare("INSERT INTO $tabla (id_usuario, id_producto) VALUES (:id_usuario, :id_producto)");
        $stmt->bindParam(":id_usuario", $datos["idUsuario"], PDO::PARAM_INT);
        $stmt->bindParam(":id_producto", $datos["idProducto"], PDO::PARAM_INT); 
        if($stmt -> execute()){
            $msgInfo="ok";
        }else{
            $msgInfo="error";
        }   
        $stmt-> close();
    }else{
        $msgInfo="Datos repetidos, no se puede insertar";
    }
    return $msgInfo;
}

Espero te sea de utilidad.

##NB: Sería interesante seguir el criterio de dataExists para tus otros métodos. Si al array $data le dieras como nombres de clave los nombres exactos de las columnas en la base de datos podrías construir fácilmente consultas de inserción (o de otro tipo) de una manera dinámica. De ese modo tendrías un CRUD que te puede servir para cualquier tabla de tu sistema.

A. Cedano
  • 95.2k
  • 31
  • 145
  • 257