PHP OO Cómo resolver el siguiente requerimiento, respetando OCP

[Carxl]

Hola, cómo vamos?

Bien, tengo el siguiente requerimiento:

Hay una consulta sql que puede ir creciendo en cualquier momento, son casi como condiciones en el where.

Lo normal que se haría, sería algo como esto:

Código PHP:

$sql = $condiciones = $innerjoins = null;
 
$sql = "select * from tabla1 $innerjoins where table1.campo1 in (2,3,4) $condiciones;";
 
if ($tipoAlgo) {
innerjoins .= 'inner join tabla2 on tabla1.campo3=tabla2.campo1 ';
$condiciones .= 'and tabla2.tipo=4 ';
}
 
if ($maxAlgo) {
innerjoins .= 'inner join tabla2 on tabla3.campo3=tabla2.campo2 ';
 $condiciones .= 'and max_value=100 ';
}
.
.
.
$result = ejecutar_query($sql);
//mostrar resultados
 

Si lo hiciera así, estaría violando el principio Open/Close, porque cada vez que me pidan una nueva condición, tendría que estar escribiendo if y dando lugar a posibles errores.

Qué ideas se les ocurren para cubrir el requerimiento pero respetando OCP.

Quedo pendiente

Gracias!

[abimaelrc]

Que tal, almacenando los valores en un array y luego unirlo con implode algo como

Código PHP:

$a = array();
$a[] = 'foo = "valor1"';
$a[] = 'bar = "valor2"';
$a[] = 'baz = "valor3"';
 
$where = implode(' AND ', $a);
 
$qry = 'SELECT * FROM tabla where ' . $where;
/**
 * Mostraría algo como
 * SELECT * FROM tabla where foo = "valor1" AND bar = "valor2" AND baz = "valor3"
 */
 

También puedes usar array multidimensional y cosas así.

Código PHP:

$a = array();
$a[] = array('tt.foo = "valor1"', 'INNER JOIN tabla2 tt ON tt.id = t.id');
$a[] = array('ttt.bar = "valor2"', 'INNER JOIN tabla3 ttt ON ttt.id = t.id');
$a[] = array('tttt.baz = "valor3"', 'INNER JOIN tabla4 tttt ON tttt.id = t.id');
 
$b = array($a[0][0], $a[2][0]);
$c = array($a[0][1], $a[2][1]);
 
$where = implode(' AND ', $b);
$join = implode(PHP_EOL, $c)
 
$qry = 'SELECT * FROM tabla t ' . $join . ' where ' . $where;
/**
 * Mostraría algo como 
 * SELECT * FROM tabla t INNER JOIN tabla2 tt ON tt.id = t.id INNER JOIN tabla4 tttt ON tttt.id = t.id where tt.foo = "valor1" AND tttt.baz = "valor3"
 */
 

[Carxl]

Hola Abimael :) Cómo vas?

Pero volvemos al punto, tendría que seguir editando el archivo cada vez que haya una nueva condición.

La idea sería, si se puede, hacer algo como esto:

http://sebys.com.ar/2010/06/27/ocp-–...pio-openclose/

Saludos!

[abimaelrc]

Oh, bueno aún se puede usar, si te fijas, creas una variable que sea tipo array y luego creas las consultas desde otra clase añadiendo las necesarias. Digo no soy experto en los principios, tengo que aprender todavía, pero se puede usar lo que te muestro. Por si acaso, hice algunos cambios para que veas a lo que me refiero.

[Carxl]

Vale, vale, la tendré en cuenta si no llego a aplicarlo directamente con el principio

Gracias!

[Ronruby]

¿El patron Decorator no te sirve para tu caso?

[Carxl]

Qué tal RonRuby, cómo vamos?

Aunque nunca lo he usado, creo que podría intentarlo y creo que sí cubre lo que necesito. Voy a probar.

Vamos a ver si alguien tiene más ideas

Gracias!

[Ronruby]

Que tal Carxl ... Veo que andas bien ocupado con tus proyectos xD

En cuanto a tu duda, yo lo haria creando una clase llamada Query por ejemplo.
Luego creo varias clases mas para que decoren a Query, como Join, Where, etc ...
Asi harias:

Código PHP:

$q = new Query("SELECT * FROM tabla1");
$q = new Join($q, INNER, "tabla2", "tabla1.campo = tabla2.campo");
$q = new Join($q, INNER, "tabla3", "tabla2.campo = tabla3.campo");
$q = new Where($q, "tabla3.campo = 1");
echo $q->getQuery();
 

Resultado:

Código SQL:

SELECT * FROM tabla1 
INNER JOIN tabla2 ON (tabla1.campo = tabla2.campo)
INNER JOIN tabla3 ON (tabla2.campo = tabla3.campo)
WHERE 1=1 AND tabla3.campo = 1
 

Al menos veo yo que es la mejor forma de hacer lo que deseas sin hacer cambios invasivos en tu código.

EDIT: Si alguien tiene una idea mejor bienvenida sea. Asi tambien aprendo.

[masterpuppet]

Que tal Carxl,

Puedes utilizar el patrón Query Object, pero no es algo sencillo de implementar, hay muchos factores a tener en cuenta, si necesitas tanta flexibilidad deberías utilizar Doctrine(DQL) o Propel(Criteria).
Y hay algo que no entiendo muy bien, si lo que esta en el link te sirve, porque no lo utilizas ?

Saludos.

[Carxl]

Hola!

Gracias a todos por sus respuestas

Masterpuppet, cómo vamos? La idea de RonRuby la estoy aún digiriendo porque como dije, no he usado ese patrón por lo tanto tengo que estudiarlo y ver como aplicarlo de la mejor manera.

La tuya tampoco jajajaja xD

Sabía que preguntaba acá y surgían muchas posibilidades.

Gracias de nuevo!

[Dany_s]

creo que te referís a no tener que modificar un método agregando condicionales cada vez que surga un nuevo criterio, no? o sea que esas condiciones no estén en un método sino en una clase

fijate el patrón cadena de responsabilidades http://es.wikipedia.org/wiki/Chain_o...dise%C3%B1o%29
la idea es formar una cadena donde cada objeto realiza una acción o no y pasa al siguiente

no es para ayudar a formar sql sino para no tener la serie de if en el método, el sql vas formando en los manejadores de la cadena

[jahepi]

Iniciado por Carxl

Hola Abimael :) Cómo vas?

Pero volvemos al punto, tendría que seguir editando el archivo cada vez que haya una nueva condición.

La idea sería, si se puede, hacer algo como esto:

http://sebys.com.ar/2010/06/27/ocp-–...pio-openclose/

Saludos!

Hola Carxl !

Ese ejemplo del principio Abierto - Cerrado de manera muy clara aplican el patrón estrategia: http://en.wikipedia.org/wiki/Strategy_pattern

El patrón que menciona masterpuppet no lo conocía, le voy a dar un vistazo pero al igual que Ronruby había pensado en un decorador para la consulta.

Un saludo y suerte !

[Carxl]

Qué más, cómo vamos?

Ofrezco disculpas por aparecerme hasta ahora a comentar el resultado, tengo mucho trabajo para poderlos acompañar como quiero.

Al final lo que hice fue aplicar el principio Chain XD Ayudado del siguiente video tutorial:

http://www.davidrojas.net/index.php/...in-of-command/

Lo que hice en sí fue generar la consulta a partir de ese patrón, al final me retornaba los resultados y si necesito agregar una condicional más a la consulta, creo una clase nueva respetando la interface y así no modifico el código ya hecho.

Gracias a todos!