armar consulta dinámica con múltiples parametros

[catpaw]

que tal forer@s!!!

Estoy tratando de optimizar y mejorar una consulta dinamica que voy armando depende de las variables que me manden, espero que me puedan aconsejar y dar su punto vista sobre mi codigo y si se puede mejorar se los agradecere.

Veran, tengo un formulario con los siguientes campos:

año: [combo de años desde 1990 a 2012]
trimestre: [combo de los 4 trimestres del año]
o Seleccione la fecha: [input con formato de fecha 2012-10-29]

después puede haber o no (uno o muchos) los siguientes combos:

Estatus: [combo con los valores, por ejemplo: en mantenimiento, acabado]
Tipo: [combo con los valores, por ejemplo: mantenimiento, soporte]
...pueden haber mas combos o uno o ninguno => los valores estos combos los meto en un arreglo, por ejemplo si tengo dos combos queda, dependiendo de la opción que elijan [1,2]

entonces mis parámetros son:

año, trimestre, fecha y arreglo de combos (en todos los campos si no son seleccionados se les pone el valor s por default)

por el momento la consulta la hago con la siguiente lógica:

Código PHP:

<?php
//recibo variables
$fecha = $_GET['fecha'];
$anio = $_GET['anio'];
$trim = $_GET['trim'];
$clases = $_GET['clases']; //arreglo
 
$query = "select * from tabla where id='$id' and "; //esta linea siempre sera igual
 
if($fecha!="s"){ //si el input de fecha no esta vació
   $query .= "fecha='$fecha' ";
} else{ 
    //si el usuario elige año y trimestre
    if($anio!="s" && $trim!="s"){ 
         //entonces yo armo la fecha de inicio y la fecha final en base al trimestre correspondiente
         $trim = explode("/",$trim); //recibo el trimestre: mm-dd/mm-dd
         $trim_ini = $trim[0];
         $trim_fin = $trim[1];
         $fecha_ini = $anio."-".$trim_ini;
         $fecha_fin = $anio."-".$trim_fin;
         //hago la condición de la consulta
         $query .= "fecha between '$fecha_ini' and '$fecha_fin' ";
   }
   else{
       //si solo eligio el año
       if($anio!="s"){
            $query .= "left(fecha, 4)='$anio' ";
       }
   }
}
?>
 

ahora si el arreglo es diferente de s, que significa que esta vacio, a lo anterior le sumo:

Código PHP:

<?php
$clase = explode(",",$clases);
$NroClases = count($clase)
    for($i=0;$i<$NroClases;$i++){
    if($clase[$i]!="s"){
        $query .= " and ";
        $query .= "clase='".$clase[$i]."' ";
    }
}
 
//finalmente
$query .= "order by id ASC";
$rs3 = mysql_query($query) or die(mysql_error());
while($row3 = mysql_fetch_array($rs3, MYSQL_ASSOC)){
      //imprimo los valores
      echo $row3['id'];
      ....
}
?>
 

Bueno los detalles que le veo a mi código actualmente es que la consulta forzosamente tiene que pasar por tener fecha o año para desplegar la consulta, es decir si yo quiero consultar solo los combos que forman el arreglo la consulta no se hace por que al armar el query les pongo que empiecen con "and" y entonces ya hay error en la sintaxis

me gustaría poder seleccionar primero cualquiera de la dos alternativas (año-trimestre/fecha o arreglo de combos) para que la consulta se lance sin problemas.

Tambien espero que me digan si habra mejor forma de hacer esto, o de optimizar mi codigo o de plano si es una aberración a la programacion lo que estoy haciendo

o si voy bien

bueno cabe aclarar que como lo tengo si funciona bien, salvo que me gustaria mejorar lo que ya les comente

gracias

[Triby]

Lo más fácil sería crear un arreglo con los filtros que aplicarás, ejemplo:

Código PHP:

$filtros = array();
if(isset($_GET['id']) && $_GET['id'] != 0) {
    $filtros[] = "id={$_GET['id']}";
}
if(isset($_GET['combo1']) && $_GET['combo1'] != 's') {
    $filtros[] = "campo1='{$_GET['combo1']}'";
}
if(isset($_GET['comboN']) && $_GET['comboN'] != 's') {
    $filtros[] = "campoN='{$_GET['comboN']}'";
}
 
// Armas la consulta
$sql = 'SELECT * FROM tabla';
 
// Si hay filtros a aplicar, los agregas:
if(count($filtros) > 0) {
    $sql .= ' WHERE ' . implode(' and ', $filtros);
}
 

Sólo recuerda verificar y, en su caso, escapar cada dato recibido para evitar errores y/o posibles ataques.

[catpaw]

hola triby gracias por tu respuesta y disculpa la tardanza de la mia,

Es verdad que usar filtros es lo mas recomendable como me planteas, pero el problema es que yo antes de armar la consulta tengo que validar las variables que llegan para decidir como crear la consulta, aunque me parece de maravilla como usaste el implode

pues resulta que hice algunos cambios y ya me da el resultado que esperaba, aunque sigo con mis dudas sobre la programacion, en fin hice lo siguiente:

En la primer parte hice un pequeño cambio, en lugar de concatenar en la cadena que guarda la consulta, concateno en una cadena auxiliar

Código PHP:

<?php
//recibo variables
$fecha = $_GET['fecha'];
$anio = $_GET['anio'];
$trim = $_GET['trim'];
$clases = $_GET['clases']; //arreglo
 
$query = "select * from tabla where id='$id' and "; //esta linea siempre sera igual
 
$cad1 = ""; //cadena auxiliar 1
if($fecha!="s"){ //si el input de fecha no esta vació
   $cad1 .= "fecha='$fecha' ";
} else{ 
    //si el usuario elige año y trimestre
    if($anio!="s" && $trim!="s"){ 
         //entonces yo armo la fecha de inicio y la fecha final en base al trimestre correspondiente
         $trim = explode("/",$trim); //recibo el trimestre: mm-dd/mm-dd
         $trim_ini = $trim[0];
         $trim_fin = $trim[1];
         $fecha_ini = $anio."-".$trim_ini;
         $fecha_fin = $anio."-".$trim_fin;
         //hago la condición de la consulta
         $cad1 .= "fecha between '$fecha_ini' and '$fecha_fin' ";
   }
   else{
       //si solo eligio el año
       if($anio!="s"){
            $cad1 .= "left(fecha, 4)='$anio' ";
       }
   }
}
?>
 

el problema era como armaba la segunda parte, que estaba atenida a la primera, por lo que, agregue una segunda cadena auxiliar para concatenar la parte de los arreglos:

Código PHP:

<?php
$clase = explode(",",$clases);
 
$cad2 = ""; //cadena auxiliar 2 
$NroClases = count($clase); //cuantos valores trae el arreglo
if($NroClases!="0"){ //si el arreglo no esta vacio
    for($i=0;$i<$NroClases;$i++){
        if($clase[$i]!="s"){
            //concateno en la cadena auxiliar
            $cad2 .= "id_".$array_clases[$i]."='".$clase[$i]."' ";
            $cad2 .= "and ";
        }    
    }
    $cad2 = substr($cad2,0, -4); //le quito el ultimo and    
}
 
//armo la otra parte del where de la consulta dependiendo de si hay cad1 y/o cad2
if($cad1!="" && $cad2!=""){
    $query .= $cad1." and ".$cad2;
} else{
    if($cad1!="" && $cad2==""){
        $query .= $cad1;
    } else{
        if($cad2!="" && $cad1==""){
            $query .= $cad2;
        }
    }
}
 
//concluyo la consulta con un order by
$query .= " order by id_arch ASC";
 
$rs3 = mysql_query($query) or die(mysql_error());
while($row3 = mysql_fetch_array($rs3, MYSQL_ASSOC)){
      //imprimo los valores
      echo $row3['id'];
      ....
}
?>
 

Con esos cambios ya se hace la consulta ya sea con los primeros parámetros que recibo (fecha o año y/o trimestre) o con los parámetros del arreglo

Aunque para otros casos la solución idónea debería ser como lo planteas Triby gracias

[Triby]

No veo la dependencia entre $cad1 y $cad2, lo único que te está complicando la consulta es que siempre incluyes and y en el for tienes que eliminar el agregado en la última iteración.

Código PHP:

<?php
//recibo variables
$fecha = $_GET['fecha'];
$anio = $_GET['anio'];
$trim = $_GET['trim'];
$clases = $_GET['clases']; //arreglo
 
$query = "select * from tabla where "; //esta linea siempre sera igual
$filtros = array("id='$id'"); // Inicializas filtros con ID
 
// Utiliza if's anidados, el código queda más legible
if($fecha!="s"){
   //si el input de fecha no esta vació
   $filtros[] = "fecha='$fecha' ";
} else if($anio!="s" && $trim!="s"){
    //si el usuario elige año y trimestre
         //entonces yo armo la fecha de inicio y la fecha final en base al trimestre correspondiente
         $trim = explode("/",$trim); //recibo el trimestre: mm-dd/mm-dd
         $trim_ini = $trim[0];
         $trim_fin = $trim[1];
         $fecha_ini = $anio."-".$trim_ini;
         $fecha_fin = $anio."-".$trim_fin;
         //hago la condición de la consulta
         $filtros[] = "fecha between '$fecha_ini' and '$fecha_fin' ";
} else if($anio!="s"){
       //si solo eligio el año
       $filtros[] = "left(fecha, 4)='$anio' ";
}
 
// Segunda parte
$clase = explode(",",$clases);
 
// Recorres $clase directamente, con foreach(), no necesitas contar los elementos
// $i es el índice que usabas en el for, $valor corresponde a $clase[$i]
foreach($clase as $i => $valor) {
    if($clase[$i] != 's') {
        $filtros[] = "id_{$array_clases[$i]} = '$valor'";
    }
}
 
// No necesitas comparar cad1 y cad2, sólo concatenas lo que haya en $filtros
$query .= implode(' and ', $filtros); 
 
//concluyo la consulta con un order by
$query .= " order by id_arch ASC";
 

La lógica siguie siendo la misma, excepto por la complicación que tenías con $cad1 y $cad2

[catpaw]

Hola triby, no,

la dependencia la quite cuando implemente las cadenas auxiliares $cad1 y $cad2

pero me quedo con tu solución del arreglo filtros, con tu ejemplo ya me queda claro como lo puedo manipular y es mas practico!!

muchas gracias!!