PDO distintas BD pero y los LIMIT 0,40

[anacona16]

He estado utilizando PDO para trabajar con las BD por varias razones, es POO, inmune a inyeccion SQL segun he leido, etc... y tambien que si se cambiar el driver puedes facilmente cambiar de BD.

Habia trabajado PDO con MySQL, me han dicho: "Trabaja con Postgres, es mas robusto y ahora con Oracle no se sabe que va a pasar con MySQL" y me dije bueno como estan facil cambiar de BD utilizando PDO pues voy a hacer una prueba, cambie el driver en la conecion de my por pg se conecto bien, cuando voy a listar tengo una consulta con LIMIT 0,40 (para paginacion) y me salta error, busque el LIMIT aca en FW y para postgres hay que hacer asi LIMIT 40 OFFSET 0 y en un modelo se puede cambiar pero ¿si tienes muchos modelos?

Entonces al fin si funciona lo mismo para todas las BD soportadas o no...

[masterpuppet]

Que tal anacona16,

Cuando se dice que es portable, se refiere a que el API, es la misma para todos los motores, por ejemplo, para ejecutar una query independientemente de si es mysql, postgres u oracle, utilizas execute, no tiene que ver con la sintaxis SQL de cada motor.

Con respecto al cambio en los modelos, primero el paginador debería ser una clase y segundo si estas repitiendo el código en varios modelos, tenes un problema de diseño.

Saludos.

[anacona16]

Hola masterpuppet gracias por tu aclaracion

Cuando se dice que es portable, se refiere a que el API, es la misma para todos los motores, por ejemplo, para ejecutar una query independientemente de si es mysql, postgres u oracle, utilizas execute, no tiene que ver con la sintaxis SQL de cada motor.

Ahora entiendo!

Con respecto al cambio en los modelos, primero el paginador debería ser una clase y segundo si estas repitiendo el código en varios modelos, tenes un problema de diseño.

Ahora si me hiciste preocupar, para paginacion si utilizo una clase, bastante buena por cierto http://objetivophp.com/?p=101 simplemete donde la necesito pues le paso los parametros y ya.

Ahora con repecto a tu segunda observacion de problema de diseño mas me preocupo, mira este es el modelo de clientes para listarlo (Donde esta la paginacion):

Código PHP:

function listarProveedores($db)
{
    $consultaProveedores    = $db->query("SELECT * FROM tb_proveedores");       
    $cantRegistros          = $consultaProveedores->rowCount();
    $limit              = 20;
    
    $pag = new Paginador();
    
    $pgnActual          = isset($_GET['pgn'])? $_GET['pgn'] : 0;        
    
    $datos              = $pag->paginar($pgnActual, $cantRegistros);
    $inicio                 = $pgnActual * $limit;
    
    $consultaProveedores    = $db->query("SELECT * FROM tb_proveedores LIMIT $inicio, $limit");
    
    return $datos[]         = array("consultaProveedores"   => $consultaProveedores->fetchAll(),
                                    "cantRegistros"         => $cantRegistros, "datos" => $datos, "pag" => $pag->getCantidadPaginas());        
}
 

Ahora el modelo de proveedores:

Código PHP:

function listarClientes($db)
{
    $consultaClientes   = $db->query("SELECT * FROM tb_clientes");      
    $cantRegistros      = $consultaClientes->rowCount();
    $limit              = 20;       
    
    $pag = new Paginador();
    
    $pgnActual      = isset($_GET['pgn'])? $_GET['pgn'] : 0;
    
    $datos          = $pag->paginar($pgnActual, $cantRegistros);
    $inicio             = $pgnActual * $limit;
    
    $consultaClientes   = $db->query("SELECT * FROM tb_clientes LIMIT $inicio, $limit");
    
    return $datos[]     = array("consultaClientes" => $consultaClientes->fetchAll(),
                                "cantRegistros" => $cantRegistros, "datos" => $datos, "paginas" => $pag->getCantidadPaginas());        
}
 

Como veras son modelos diferentes, cada uno en un script php separado y cada uno con el resto de tareas a BD.

Como son modelos diferentes pues cada uno tienes su LIMIT 0,20.

¿Esta bien asi o si estoy cometiendo el error de diseño?

Gracias.

[masterpuppet]

A ver, estas trabajando OOP ?, estas utilizando MVC ?, porque no veo la visibilidad declarada en los métodos y los modelos acceden directamente al GET.

Y creo que tenes un problema en tu diseño, las dos funciones son prácticamente iguales, la diferencia es la tabla que atacan, podes postear las dos clases completas ? o al menos un resumen de las mismas.

Saludos.

[anacona16]

Estoy trabajando con MVC pero no con OOP completamente, solo la clase paginador, esas funciones la llama el controlador que dependiendo de la accion muestra una vista.

[_ssx]

Hola

Creo que necesitas trabajar más aún con tu modelo de manejo de datos con SQL, ya que estas manejando tu SQL "A mano" en mi parecer necesitas un objeto "intermedio" o adaptador que se ocupe te acomodar las consultas y convertirlas al MODELO de base de datos adecuado..

El método query debería de ser un método "wrapper" del método query de PDO para que puedas aplicar reglas especificas del motor que estas usando antes de enviar la query al motor de base de datos.

Código PHP:

  
interface Adapter{
    public function query($consulta,$params,$limit);
}

class MySql implements Adapter{
   public function query($consulta,$params,$limit){
          //Aqui creas tu cadena SQL conforme lo hace Mysql con los paremetros enviados.
          return $consulta.$params.$limit;
  }
}

class PostGres implements Adapter{
   public function query($consulta,$params,$limit){
          //Aqui creas tu cadena SQL conforme lo hace Postgres con los paremetros enviados.
          return $consulta.$params.$limit.' OFFSET ';
  }
}

class Db{
     private $_adapter;
     private $_db;

     public function setAdapter(Adapter $adapter){
          $this->_adapter = $adapter;
     }
     
     public function query($consulta,$params,$limits){
           $this->_db->query($this->_adapter('SELECT','campos',array(0,4)));
     }
} 
   

Algo así el chiste es que tengas un "wrapper" antes de dejar caer los datos sobre PDO

Ahora si cumplirias tu objetivo de SOLO cambiar la base de datos en el setAdapter ;)

Por que MySQL, Postgress o ORACLE estarian cumpliendo con el CONTRATO de implementación de la interface.

Código PHP:

  $db->setAdapter(new Mysql());
$db->setAdapter(new Postgress()); 
   

Cabe mencionar que los frameworks actuales tienen ya super bien definidos sus adaptadores para que no te manejes el SQL a mano