Armar restriccion para consulta con GET del formulario

[morfasto]

Hola, que tal?

Hice una pagina en la que muestro fotos cargando la info de la base de datos, tambien he hecho un filtrado de informacion enviando un formulario via GET.

El problema es que creo que estoy haciendo algo mal ya que mi solucion para recoger la informacion del formulario y procesarla para armar la restriccion para hacer la consulta a la base de datos es demasiado larga, tal vez hay una manera mucho mas sencilla de hacerlo, pero no se me ocurre.

Queria saber si ustedes me podrian ayudar a hacerlo mucho mas sensillo.

Este es mi codigo:

Código PHP:

  <?php
if(isset($_GET['titulo'])){
    $buscar_titulo=$_GET['titulo'];
}else{
    $buscar_titulo=NULL;
}
if(isset($_GET['autor'])){
    $buscar_autor=$_GET['autor'];
}else{
    $buscar_autor=NULL;
}
if(isset($_GET['formato'])){
    $buscar_formato=$_GET['formato'];
}else{
    $buscar_formato=NULL;
}
if(isset($_GET['visitas'])){
    $buscar_visitas=$_GET['visitas'];
}else{
    $buscar_visitas=NULL;
}
if(isset($_GET['categoria'])){
    $buscar_categoria=$_GET['categoria'];
}else{
    $buscar_categoria=NULL;
}
if(isset($_GET['tamano'])){
    $buscar_tamano=$_GET['tamano'];
}else{
    $buscar_tamano=NULL;
}
$restriccion="";
if($buscar_titulo!=NULL){
    $restriccion=" titulo like '".$buscar_titulo."%'";
    if($buscar_autor!=NULL){
        $restriccion=$restriccion." and autor = '".$buscar_autor."'";
        if($buscar_formato!=NULL){
            $restriccion=$restriccion." and formato = '".$buscar_formato."'";
            if($buscar_visitas!=NULL){
                $restriccion=$restriccion." and visitas >= '".$buscar_visitas."'";
                if($buscar_categoria!=NULL){
                    $restriccion=$restriccion." and categoria = '".$buscar_categoria."'";
                    if($buscar_tamano!=NULL){
                        $restriccion=$restriccion." and tamano >= '".$buscar_tamano."'";
                    }
                }else{
                    if($buscar_tamano!=NULL){
                        $restriccion=$restriccion." and tamano >= '".$buscar_tamano."'";
                    }
                }
            }else{
                if($buscar_categoria!=NULL){
                    $restriccion=$restriccion." and categoria = '".$buscar_categoria."'";
                    if($buscar_tamano!=NULL){
                        $restriccion=$restriccion." and tamano >= '".$buscar_tamano."'";
                    }
                }else{
                    if($buscar_tamano!=NULL){
                        $restriccion=$restriccion." and tamano >= '".$buscar_tamano."'";
                    }
                }
            }
        }else{
            if($buscar_visitas!=NULL){
                $restriccion=$restriccion." and visitas >= '".$buscar_visitas."'";
                if($buscar_categoria!=NULL){
                    $restriccion=$restriccion." and categoria = '".$buscar_categoria."'";
                    if($buscar_tamano!=NULL){
                        $restriccion=$restriccion." and tamano >= '".$buscar_tamano."'";
                    }
                }else{
                    if($buscar_tamano!=NULL){
                        $restriccion=$restriccion." and tamano >= '".$buscar_tamano."'";
                    }
                }
            }else{
                if($buscar_categoria!=NULL){
                    $restriccion=$restriccion." and categoria = '".$buscar_categoria."'";
                    if($buscar_tamano!=NULL){
                        $restriccion=$restriccion." and tamano >= '".$buscar_tamano."'";
                    }
                }else{
                    if($buscar_tamano!=NULL){
                        $restriccion=$restriccion." and tamano >= '".$buscar_tamano."'";
                    }
                }
            }
        }
    }else{
        if($buscar_formato!=NULL){
            $restriccion=$restriccion." and formato = '".$buscar_formato."'";
            if($buscar_visitas!=NULL){
                $restriccion=$restriccion." and visitas >= '".$buscar_visitas."'";
                if($buscar_categoria!=NULL){
                    $restriccion=$restriccion." and categoria = '".$buscar_categoria."'";
                    if($buscar_tamano!=NULL){
                        $restriccion=$restriccion." and tamano >= '".$buscar_tamano."'";
                    }
                }else{
                    if($buscar_tamano!=NULL){
                        $restriccion=$restriccion." and tamano >= '".$buscar_tamano."'";
                    }
                }
            }else{
                if($buscar_categoria!=NULL){
                    $restriccion=$restriccion." and categoria '".$buscar_categoria."'";
                    if($buscar_tamano!=NULL){
                        $restriccion=$restriccion." and tamano >= '".$buscar_tamano."'";
                    }
                }else{
                    if($buscar_tamano!=NULL){
                        $restriccion=$restriccion." and tamano >= '".$buscar_tamano."'";
                    }
                }
            }
        }else{
            if($buscar_visitas!=NULL){
                $restriccion=$restriccion." and visitas >= '".$buscar_visitas."'";
                if($buscar_categoria!=NULL){
                    $restriccion=$restriccion." and categoria = '".$buscar_categoria."'";
                    if($buscar_tamano!=NULL){
                        $restriccion=$restriccion." and tamano >= '".$buscar_tamano."'";
                    }
                }else{
                    if($buscar_tamano!=NULL){
                        $restriccion=$restriccion." and tamano >= '".$buscar_tamano."'";
                    }
                }
            }else{
                if($buscar_categoria!=NULL){
                    $restriccion=$restriccion." and categoria = '".$buscar_categoria."'";
                    if($buscar_tamano!=NULL){
                        $restriccion=$restriccion." and tamano >= '".$buscar_tamano."'";
                    }
                }else{
                    if($buscar_tamano!=NULL){
                        $restriccion=$restriccion." and tamano >= '".$buscar_tamano."'";
                    }
                }
            }
        }
    }
}else{
    if($buscar_autor!=NULL){
        $restriccion=" autor = '".$buscar_autor."'";
        if($buscar_formato!=NULL){
            $restriccion=$restriccion." and formato = '".$buscar_formato."'";
            if($buscar_visitas!=NULL){
                $restriccion=$restriccion." and visitas >= '".$buscar_visitas."'";
                if($buscar_categoria!=NULL){
                    $restriccion=$restriccion." and categoria = '".$buscar_categoria."'";
                    if($buscar_tamano!=NULL){
                        $restriccion=$restriccion." and tamano >= '".$buscar_tamano."'";
                    }
                }else{
                    if($buscar_tamano!=NULL){
                        $restriccion=$restriccion." and tamano >= '".$buscar_tamano."'";
                    }
                }
            }else{
                if($buscar_categoria!=NULL){
                    $restriccion=$restriccion." and categoria = '".$buscar_categoria."'";
                    if($buscar_tamano!=NULL){
                        $restriccion=$restriccion." and tamano >= '".$buscar_tamano."'";
                    }
                }else{
                    if($buscar_tamano!=NULL){
                        $restriccion=$restriccion." and tamano >= '".$buscar_tamano."'";
                    }
                }
            }
        }else{
            if($buscar_visitas!=NULL){
                $restriccion=$restriccion." and visitas >= '".$buscar_visitas."'";
                if($buscar_categoria!=NULL){
                    $restriccion=$restriccion." and categoria = '".$buscar_categoria."'";
                    if($buscar_tamano!=NULL){
                        $restriccion=$restriccion." and tamano >= '".$buscar_tamano."'";
                    }
                }else{
                    if($buscar_tamano!=NULL){
                        $restriccion=$restriccion." and tamano >= '".$buscar_tamano."'";
                    }
                }
            }else{
                if($buscar_categoria!=NULL){
                    $restriccion=$restriccion." and categoria = '".$buscar_categoria."'";
                    if($buscar_tamano!=NULL){
                        $restriccion=$restriccion." and tamano >= '".$buscar_tamano."'";
                    }
                }else{
                    if($buscar_tamano!=NULL){
                        $restriccion=$restriccion." and tamano >= '".$buscar_tamano."'";
                    }
                }
            }
        }
    }else{
        if($buscar_formato!=NULL){
            $restriccion=" formato = '".$buscar_formato."'";
            if($buscar_visitas!=NULL){
                $restriccion=$restriccion." and visitas >= '".$buscar_visitas."'";
                if($buscar_categoria!=NULL){
                    $restriccion=$restriccion." and categoria = '".$buscar_categoria."'";
                    if($buscar_tamano!=NULL){
                        $restriccion=$restriccion." and tamano >= '".$buscar_tamano."'";
                    }
                }else{
                    if($buscar_tamano!=NULL){
                        $restriccion=$restriccion." and tamano >= '".$buscar_tamano."'";
                    }
                }
            }else{
                if($buscar_categoria!=NULL){
                    $restriccion=$restriccion." and categoria = '".$buscar_categoria."'";
                    if($buscar_tamano!=NULL){
                        $restriccion=$restriccion." and tamano >= '".$buscar_tamano."'";
                    }
                }else{
                    if($buscar_tamano!=NULL){
                        $restriccion=$restriccion." and tamano >= '".$buscar_tamano."'";
                    }
                }
            }
        }else{
            if($buscar_visitas!=NULL){
                $restriccion=" visitas >= '".$buscar_visitas."'";
                if($buscar_categoria!=NULL){
                    $restriccion=$restriccion." and categoria = '".$buscar_categoria."'";
                    if($buscar_tamano!=NULL){
                        $restriccion=$restriccion." and tamano >= '".$buscar_tamano."'";
                    }
                }else{
                    if($buscar_tamano!=NULL){
                        $restriccion=$restriccion." and tamano >= '".$buscar_tamano."'";
                    }
                }
            }else{
                if($buscar_categoria!=NULL){
                    $restriccion=" categoria = '".$buscar_categoria."'";
                    if($buscar_tamano!=NULL){
                        $restriccion=$restriccion." and tamano >= '".$buscar_tamano."'";
                    }
                }else{
                    if($buscar_tamano!=NULL){
                        $restriccion=" tamano >= '".$buscar_tamano."'";
                    }
                }
            }
        }
    }
}
if(!isset($_GET['resultados_pagina']))
{
    $_GET['resultados_pagina']=10;
}
if(isset($_GET['pagina'])){
    $pagina= $_GET['pagina'];
}else{
    $pagina=1;
}
$limite = " LIMIT ".($pagina-1)*$_GET['resultados_pagina'].",".$_GET['resultados_pagina'];
$ruta = "SELECT foto_id, tamano, categoria, visitas, autor, titulo, formato FROM fotos WHERE $restriccion ORDER BY fecha DESC $limite";
?>   

Alguna idea?

Muchisimas gracias!

[morfasto]

Si es que tengo mas campos en el form, mi codigo se duplica por cada campo.

Alguna idea?

[Eleazan]

Yo tengo una pequeña funcion, del estilo

Código PHP:

  function getGET($campo, $pre = 0) {
    if(isset($_GET[$campo])) return $_GET[$campo];
    else return $pre;
} 
   

Así, puedes recoger los datos del estilo

Código PHP:

  buscar_titulo = getGET('titulo', NULL); 
   

Por ejemplo... así ahorras un par de lineas con el isset ;)

[Eleazan]

Y para armar las consultas, te pego un trozo de mi código, q se entiende más o menos bien:

Código PHP:

      //Así armo la clausula where!
    $where = '';
    //Creamos la consulta de búsqueda!
    $where = doWhere($where, 'estado', $vEstado, 0);
    $where = doWhere($where, 'osalida', $vOficina, 0);
    $where = doWhereLike($where, 'nombre+apellidos', $vConductor);
    $where = doWhereLike($where, 'localizador', $vLocalizador);
    $where = doWhereLike($where, 'nreserva', $vNreserva);
    $where = doWhere($where, 'fsalida', invertDate($vDesde), 1, '>=');
    $where = doWhere($where, 'fsalida', invertDate($vHasta), 1, '<=');
    $where = doWhere($where, 'id_grupo', $vGrupo, 0); 
   

Y las dos funciones que uso (invertDate sólo hace eso... invertir la fecha xD), son estas:

Código PHP:

  
function doWhere($where, $campo, $value, $text = 1, $comp = '=') {
    if(isset($where) && !empty($where)) {
        if(isset($value) && !empty($value)) {
            $where .= ' AND ';
            if($text) $where .= "`$campo`$comp'$value'";
            else $where .= "`$campo`$comp$value";
        }
        
    }
    else {
        if(isset($value) && !empty($value) ) {
            $where = 'WHERE ';
            if($text) $where .= "`$campo`$comp'$value'";
            else $where .= "`$campo`$comp$value";
        }
    }
    
    return $where;
}

function doWhereLike($where, $campo, $value) {
   if(strpos($campo, '+')) {
       $campos = explode('+', $campo);
       $campo = " CONCAT(`".$campos[0]."`,' ',`".$campos[1]."`) ";
   }
    if(isset($where) && !empty($where) ) {
        if(isset($value) && !empty($value)) {
            $where .= ' AND ';
            if(!isset($campos)) $where .= "`$campo` LIKE '%$value%'";
            else $where .= "$campo LIKE '%$value%'";
          
        }
        
    }
    else {
        if(isset($value) && !empty($value)) {
            $where = 'WHERE ';
            if(!isset($campos)) $where .= "`$campo` LIKE '%$value'%";
            else $where .= "$campo LIKE '%$value%'";
            
        }
    }
    
    return $where;
} 
   

Básicamente, a explicar:
Ambas funciones añaden comparaciones (en caso de existir alguna previa) o arman la comparación a partir del $where q les pasas como parámetro.
$campo es el nombre del campo SQL
$value es el valor que le das
$text es 1 si es texto, 0 si es falso
$comp es el tipo de comparacion. Por defecto, es "=" (puedes poner >, <, <=, >=...)

Y en doWhereLike arma un Like para buscar coincidencias.
Automáticamente pone un % al principio y al final. Y para poner varios campos juntos (ej: nombre y apellidos) pondrías un 'nombre+apellidos' en el $campo.. con el + le indicas que son varios campos (de momento, sólo deja 2.. no he necesitado nunca más. Supongo que si algun dia necesito 3 lo adaptaré xD)

No sé si te servirá :)

Edit: La parte de validación es previa a este paso... eso corre a tu cuenta ;)


Edit 2:

He revisado tu código.. tenías los if's muy anidados, cosa innecesaria.

Podrías haber tenido algo así:

Código PHP:

  //Para cada valor
if($buscar_autor != NULL) {
    if(!isset($restriccion)) $restriccion = "`autor`='$buscar_autor' ";
    else $restriccion .= " AND `autor`='$buscar_autor' ";
} 
   

Y así sólo habrias tenido 5 if's, ninguno anidado...

De todas formas, te adjunto lo q tenías hecho "pasado" con mis funciones ;)

Código PHP:

  <?php 
$buscar_titulo = getGET('titulo', NULL);
$buscar_autor  = getGET('autor', NULL);
$buscar_formato = getGET('formato', NULL);
$buscar_visitas = getGET('visitas', NULL);
$buscar_categoria = getGET('categoria', NULL);
$buscar_tamano = getGET('tamano', NULL);


$restriccion=""; 

$restriccion = doWhereLike($restriccion, 'titulo', $buscar_titulo);
$restriccion = doWhere($restriccion,'autor', $buscar_autor);
$restriccion = doWhere($restriccion, 'formato', $buscar_formato);
$restriccion = doWhere($restriccion, 'visitas', $buscar_visitas, 1, '>=');
$restriccion = doWhere($restriccion, 'categoria', $buscar_categoria);
$restriccion = doWhere($restriccion, 'tamano', $buscar_tamano, 1, '>=');


if(!isset($_GET['resultados_pagina'])) 
{ 
    $_GET['resultados_pagina']=10; 
} 
if(isset($_GET['pagina'])){ 
    $pagina= $_GET['pagina']; 
}else{ 
    $pagina=1; 
} 
$limite = " LIMIT ".($pagina-1)*$_GET['resultados_pagina'].",".$_GET['resultados_pagina']; 
$ruta = "SELECT foto_id, tamano, categoria, visitas, autor, titulo, formato FROM fotos $restriccion ORDER BY fecha DESC $limite"; 
?>   

Quedaría algo así :)

Un saludo!

[morfasto]

Y si tengo varios OR:

consulta: SELECT foto_id FROM fotos WHERE categoria = 1 or categoria = 2 or categoria = 3 ORDER BY fecha DESC"

Esta consulta deberia de ser modificada para incluir parentesis:
consulta: SELECT foto_id FROM fotos WHERE (categoria = 1 or categoria = 2 or categoria = 3) ORDER BY fecha DESC"

Alguna recomendacion para añadir esos parentesis?

[Eleazan]

Yo esq en una búsqueda, solo pongo "AND", nunca "OR"... no sé, suele ser en plan de "o buscas todas las categorias, o una en concreto"...

Pero supongo que podrías, o bien adaptar el doWhere.. o hacer un doWhereIn, o doWhereOr xD

o bien, añadir un simple
"`categoria` IN (1,2,3)"

Ya depende de lo q necesites :)
A la consulta.. no sé.

[morfasto]

No sabia del IN(1,2,3) :), creo que con eso soluciono todo de una manera mas sencilla, gracias de nuevo! :)

[Sourcegeek]

No es por nada pero.. Dios, es uno de los códigos mas largos que he visto para una tarea tan sencilla
Veo que te falta mucho por aprender del mundo del php, usaste opciones demasiado básicas y te olvidaste de lo demás que ayuda muchísimo: funciones, arrays, bucles, etcétera.
No estoy seguro que ésto sea lo que busques, pero a lo que entendí, ésto te debe funcionar como deseas:

Código PHP:

<?php
 
$vars = array('titulo', 'autor',' formato', 'visitas', 'categoria', 'tamano');
 
foreach($vars as $var) {
    $var_var = "buscar_$var";
    $$var_var = (isset($_GET[$var])) ? $_GET[$var] : null;
}
 
if($buscar_titulo != null) {
    $restriccion = " titulo like '$buscar_titulo%'";
}elseif($buscar_autor != null) {
    $restriccion = " autor = '$buscar_autor'";
}
 
foreach($vars as $var) {
    $varr = $_GET[$var];
    $buscar_ = "buscar_$var";
        
    if($varr != null) {
        if($var == 'visitas' || $var == 'tamano')
            $restriccion .= " and $var >= '{$$buscar_}'";
        else
            $restriccion .= " and $var = '{$$buscar_}'";
    }
}
 

Si no funciona como quieres exactamente, te invito a modificar el código a tu gusto hasta que cumpla
con tus necesidades
Saludos