Error creando formulario de búsqueda avanzada con select option

[kahlito]

Buenas.

Estoy creando un formulario de búsqueda avanzada con varios select donde algunos option tienen la opción "todas" con valor vacio (value="") y tambien algunos campos de texto vacios.

Si lo hago con dos opciones me sale perfectamente pero al meter 3, 4 o 5 ya me da error.

El formulario es este:

Código PHP:

 <form action="busqueda-avanzada-kites.php" method="post" name="busqueda avanzada kites"> 
Marca:<select id="marca" name="marca"> 
<option value=""  >Todas las marcas</option> 
<?php 
$marcas=array("F-one", "North", "Nobile", "Naish", "Best", "Slingshots", "Airush", "Wipika", "Liquidforce", "Takoon", "Advance", "Dakine", "Ozone", "Star", "RRD", "Cabrinha", "Gastar", "Ocean Rodeo", "Flysurfer", "Crazyfly", "Griffin", "Zeeko", "Flexifoil", "Bull", "Kiteloose", "Waiman", "Otra"); 
//$alfabetico=sort($marcas); 
sort($marcas);   
foreach ($marcas as $ma => $value) 
{     
    echo '<option value="'.$marcas[$ma].'" '; 
    if($marcas[$ma] == $_POST['marca']) 
    { 
        echo 'selected="selected"';          
    }     
    echo '>'.$marcas[$ma].'</option>';     
} 
?> 
</select><br /> 
Modelo: <input name="modelo" type="text" value="<?php if(isset($_POST["modelo"])) {echo $_POST["modelo"];} ?>" /><br /> 
Medidas:<select id="medidas" name="medidas"> 
<option value=""  >Todas</option> 
<?php 
$medidas=array("4", "5", "6", "7", "8", "9", "10", "11", "12", "13", "14", "15", "16"); 
foreach($medidas as $me => $value) 
{ 
    echo '<option value="'.$medidas[$me].'"'; 
    if($medidas[$me]== $_POST['medidas']) 
    { 
        echo 'selected="selected"';     
    } 
    echo '>'.$medidas[$me].'</option>'; 
} 
?> 
</select><br /> 
<input type="hidden" name="id_kite" value="<?php echo $id_kite ?>" /> 
<input type="submit" name="buscar-kite" value="Buscar" /><br /> 
</form>  

Y la búsqueda que uso es esta:

Código PHP:

  ?<php 
if($_POST["marca"] !="" and $_POST["modelo"] != "") 
{ 
    $busqueda = "Select * from kites where marca='".$_POST["marca"]."' and modelo='".$_POST["modelo"]."'"; 
} 
elseif($_POST["marca"] !="" or $_POST["modelo"] != "") 
{ 
    $busqueda = "Select * from kites where marca='".$_POST["marca"]."' or modelo='".$_POST["modelo"]."'"; 
} 
elseif($_POST["marca"] =="" and $_POST["modelo"] == "") 
{     
    $busqueda = "Select * from kites";     
} 
?>  
  

Con dos campos me va bien ya que:

- si todos estan completos me hace la busqueda y me muestra el resultado con todos los campos
- si uno u otro está marcado o completo pero el otro no me muestra el resultado con el campo que se ha marcado.
- si todos están vacios me lo muestra todo.

Hasta aquí todo bien, pero si meto uno nuevo como medidas ya no completa los criterios anteriores:

He usado varias maneras pero por ahora no lo consigo, por ejemplo asi:

Código PHP:

  if($_POST["marca"] !="" and $_POST["modelo"] != "" and $_POST["medidas"] != "") 
{ 
    $busqueda = "Select * from kites where marca='".$_POST["marca"]."' and modelo='".$_POST["modelo"]."' and medidas='".$_POST["medidas"]."'"; 
} 
elseif($_POST["marca"] !="" or $_POST["modelo"] != "" or $_POST["medidas"] != "") 
{ 
    $busqueda = "Select * from kites where marca='".$_POST["marca"]."' or modelo='".$_POST["modelo"]."' or medidas='".$_POST["medidas"]."'"; 
} 
else 
{     
    $busqueda = "Select * from kites";     
}  
   

¿Como podría hacerlo entonces con más de 2 campos?

[GatorV]

Lo mejor es armar dinámicamente el query:

Código PHP:

$query = 'SELECT * FROM foo';
$conditions = array();
if (foo()) {
       $conditions[] = "foo = 'foo'";
}
if (bar()) {
       $conditions[] = "bar= 'bar'";
}
if (baz()) {
       $conditions[] = "baz= 'baz'";
}
 
if (count($conditions) > 0)) {
       $query .= ' WHERE ' . implode(' AND ', $conditions);
}
 

Saludos.

[kahlito]

Hola GatorV, gracias por la respuseta aunque la verdad es que no termino de entender esa manera o el mecanismo para aplicarlo.

¿Tengo que usar cada campo del formulario como una condición? Por ejemplo asi:

Código PHP:

  $query = 'SELECT * FROM kites'; 
$condiciones = array(); 
if($_POST["marca"]) 
{ 
    $condiciones[] = "marca = 'marca'"; 
} 
if($_POST["modelo"]) 
{ 
    $condiciones[] = "modelo = 'modelo'"; 
} 
if($_POST["medidas"]) 
{ 
    $condiciones[] = "medidas = 'medidas'"; 
}  
   

He intentando esa opción pero claro luego me pide un valor en el contador:

Código PHP:

  if (count($condiciones[])>0)  
   

No termino de entenderlo.

[GatorV]

El count sirve para contar los elementos del array, por lo que debes de pasar la variable sin los corchetes:

Código PHP:

// mal
if (count($conditions[]) > 0) 
// bien
if (count($conditions) > 0)
 

Y sí puedes usar el campo pero es mejor usar isset() junto con empty para asegurarte que tenga algún valor ej:

Código PHP:

if (isset($_POST['campo']) && !empty($_POST['campo']) {
         // lo agregas al query
         $conditions[] = "campo='" . $_POST['campo'] . "'"; // Aquí sería bueno limpiar tu variable
}
 

Saludos.

[kahlito]

Hola de nuevo GatorV.

Lo he probado de dos maneras, una intentando hacerlo de la anterior manera pero la verdad ahi me sigo perdiendo sobre todo en esta parte:

Código PHP:

  if (marca()) 
{ 
    $condiciones[] = "marca = 'marca'"; 
} 
//if(marca) 
//if (marca($_POST['marca']))  
   

De ninguna de estas formas maneras me sale


En la segunda opción (usando el campo) he seguido tus aclaraciones y ahora parece que sí lo entiendo y además me va correcto, incluso he añadido un cuarto campo y también va perfecto:

Código PHP:

  $query = 'SELECT * FROM kites'; 
$condiciones = array(); 
if (isset($_POST['marca']) && !empty($_POST['marca'])) 
{ 
    // lo agregas al query 
    $condiciones[] = "marca='" . $_POST['marca'] . "'"; // Aquí sería bueno limpiar tu variable 
} 
if (isset($_POST['modelo']) && !empty($_POST['modelo'])) 
{ 
    // lo agregas al query 
    $condiciones[] = "modelo='" . $_POST['modelo'] . "'"; // Aquí sería bueno limpiar tu variable 
} 
if (isset($_POST['medidas']) && !empty($_POST['medidas'])) 
{ 
    // lo agregas al query 
    $condiciones[] = "medidas='" . $_POST['medidas'] . "'"; // Aquí sería bueno limpiar tu variable 
} 
if (isset($_POST['tipo']) && !empty($_POST['tipo'])) 
{ 
    // lo agregas al query 
    $condiciones[] = "tipo='" . $_POST['tipo'] . "'"; // Aquí sería bueno limpiar tu variable 
} 
if (count($condiciones) > 0) 
{ 
    $query .= ' WHERE ' . implode(' AND ', $condiciones); 
}  
   

Muchas gracias por tu ayuda

[GatorV]

Esto era un ejemplo:

Código PHP:

if (marca()) 
{ 
    $condiciones[] = "marca = 'marca'"; 
}
 

No para que lo usaras literal, la idea es que veas la estructura:

Código PHP:

if (secumplelacondicionquequiero()) {
      // Agregas al query el valor que quieres
}
 

Saludos.

[kahlito]

Ah ya decía yo.

Perfecto, muchas gracias por todo, un saludo