optimizar código

[charlyta]

Hola. Actualmente útilizo este script para realizar búsquedas, pero desearía optimizarlo, ya que para implementarlo correctamente debo repetirlo 30 veces!! titulo, autor, editorial, etcc..

os lo pongo y os cuento mejor:

Código PHP:

if (!isset($_GET['buscar'])){
$_pagi_sql = "";
 
if (isset ($autor) && !($autor == "")) {
$_pagi_sql .= " autor LIKE '%$autor%'";
}
if (isset ($titulo) && !($titulo == "")) {
  $palabras=explode(" ",$titulo);
  foreach ($palabras as $palabra){
    if (! ($_pagi_sql == "")) {$_pagi_sql .= " AND "; } //tendras que poner esto aquí
    $_pagi_sql .= " titulo LIKE '%$palabra%'"; 
  }
}
 

Debería modificar todas las líneas como:

Código PHP:

if (isset ($autor) && !($autor == "")) {
$_pagi_sql .= " autor LIKE '%$autor%'";
}
 

por:

Código PHP:

if (isset ($titulo) && !($titulo == "")) {
  $palabras=explode(" ",$titulo);
  foreach ($palabras as $palabra){
    if (! ($_pagi_sql == "")) {$_pagi_sql .= " AND "; } //tendras que poner esto aquí
    $_pagi_sql .= " titulo LIKE '%$palabra%'"; 
  }
}
 

con el consiguiente lío para asignar a todas las variables lo mismo una y otra vez y repetirla instrucción constantemente tantísimas veces. Por otro lado desconozco si aplicar en todas las intrucciones esto me daría problemas ya que asignaría siempre:

Código PHP:

LIKE '%$palabra%'";
 

[IEKK]

Hay una cosa que se llama function. Básico en los lenguajes de programación

[pochos]

Como dice IEKK usa funciones, y tambien te recomiendo que uses indices full text de mysql. visita este enlace: http://dev.mysql.com/doc/refman/5.0/...xt-search.html, quizas te ayude para no usar tanto los like en las consultas sql

saludos!

[charlyta]

Hola y gracias por las respuestas.

actualmente he visto una función que podría adaptar, la pregunta es: ¿cómo puedo introducir el array de todos los campos del $_GET?

Código PHP:

function buscar($sTabla,$aCampos,$cDb,$iPaginas = 10){
  global $iTotal,$aResultados;
  $_GET['pag'] *= 1;
  $aPalabras = split(' ',addslashes($_GET['q']));
  foreach($aPalabras as $sPalabra)if($sPalabra != ''){
    $preq .= " AND ( 0";
    foreach($aCampos as $sCampo)$preq .= " OR $sCampo LIKE '%".$sPalabra."%'";
    $preq .= ")";
  }
  $iTotal = mysql_result(mysql_query("SELECT COUNT(".$aCampos[0].") FROM `$sTabla` WHERE 1$preq",$cDb),0);
  $aResultados=mysql_query("SELECT * FROM `$sTabla` WHERE 1$preq LIMIT ".($_GET['pag']*$iPaginas).",$iPaginas",$cDb);
}
 

[Ronruby]

Yo te recomiendo usar lo que te recomendó pochos. Es fácil de implementar y es mucho mas eficiente.
Nota: Los indices FULLTEXT solo funcionan con el engine MyISAM de MySQL, si usas InnoDB o algún otro tendrás que usar un servidor aparte e instalarte algo como Sphinx o Lucene.

[charlyta]

Hola. Ya utilizo full text, tan sólo quiero mejorar esta parte porque con full text no es fiable:

Código PHP:

if(isset($_POST['submitted'])) {
if ($clave<>''){  
 
{ 
 
   $trozos=explode(" ",$clave); 
   $numero=count($trozos); 
  if ($numero==1)
   { 
  $buscar="SELECT * FROM $db WHERE NOMB LIKE '%$clave%' OR AUTOR LIKE '%$clave%' 
OR TEMA LIKE '%$clave%' OR COLECCION LIKE '%$clave%' OR EDITORIAL LIKE '%$clave%' 
OR LIBRE LIKE '%$clave%' OR OBSERVA LIKE '%$clave%' order by $ordenarpor $orden"; 
  }
  elseif ($numero>1)
   { 
 
   $buscar= "SELECT * , MATCH (NOMB, AUTOR, TEMA, COLECCION, EDITORIAL, LIBRE, OBSERVA) 
AGAINST ('$clave' IN BOOLEAN MODE) AS Score FROM $db WHERE MATCH (NOMB, AUTOR, TEMA, 
COLECCION, EDITORIAL, LIBRE, OBSERVA) AGAINST ('$clave' IN BOOLEAN MODE) ORDER BY Score DESC ";
}
} 
}}else
 
 { 
 $buscar = "SELECT * FROM $db WHERE NOMB LIKE '%$NOMB%' AND AUTOR LIKE '%$AUTOR%' 
AND TEMA LIKE '%$TEMA%'  AND EDITORIAL LIKE '%$EDITORIAL%'  OR TEMA LIKE '%$TEMA%' AND
 (AUTOR LIKE '%$AUTOR%' AND EDITORIAL LIKE '%$EDITORIAL%' AND NOMB LIKE '%$NOMB%') OR 
NOMB LIKE '%$NOMB%' AND (AUTOR LIKE '%$AUTOR%' AND EDITORIAL LIKE '%$EDITORIAL%' AND 
TEMA LIKE '%$TEMA%') order by $ordenarpor $orden";
 }
 

utilizando lo anterior a este post, pudo encontrar busquedas multicriterio con frases en cada campo..

[iovan]

Me quede un poco confundido. Si deseas repetirlo 30 veces, aunque esta es la forma menos recomendada en el mundo pero te ayudará a entender como usar una funcion para repetir las instrucciones dentro un determinado numero de veces.

Código PHP:

for($i=0;$i<30;$i++){
 
echo "El interior de éste for se ha ejecutado ".$i." veces <br>";
 
}
 

Resultado:

Código texto:

El interior de éste for se ha ejecutado 0 veces
El interior de éste for se ha ejecutado 1 veces
El interior de éste for se ha ejecutado 2 veces
El interior de éste for se ha ejecutado 3 veces
El interior de éste for se ha ejecutado 4 veces
.
.//Sucesivamente hasta 29
.
El interior de éste for se ha ejecutado 29 veces
 

Contado que empieza en 0 y termina en 29, son un total de 30 ejecuciones

Saludos.

[charlyta]

Gracias por las respuestas.

Perdonadme pero creo que no me he explicado bien.. Imaginaros que tengo esta instrucción:

Código PHP:

$fields=array('tema', 'autor', 'titulo');
$sql_where="";
for ($i=0;$i<count($fields);$i++)
{
 
    ($sq_where="")?"":" , ";
    $sql_where.=$fields[$i]."='".$_GET[$fields[$i]]>"' ";
 
}
$sql="SELECT * from $table where ".$sql_where." limit 20";
 

Lo que necesito es introducir en ella el GET con las variables recibidas
(yo suelo hacer:

Código PHP:

foreach ($_GET as $key => $value)
 

e implementar AND/OR

es decir sustituir $_GET[$fields[$i]]>" por las variables recibidas


Sé que es todo demasiado tonto pero me ayuda a aprender.. Gracias de nuevo


que me quedase parecido a esto

Código PHP:

foreach ($_GET as $key => $value) if ($key != ''){
      $preq .= " AND ( 0";
    foreach($Campos as $Campo)$preq .= " OR $Campo LIKE '%".$value."%'";
    $preq .= ")";
 

o lo que es lo mismo solucionar esta función correctamente:

Código PHP:

function buscar($Tabla,$Campos,$db,$con){
foreach ($_GET as $key => $value) if ($key != ''){
      $preq .= " AND ( 0";
    foreach($Campos as $Campo)$preq .= " OR $Campo LIKE '%".$value."%'";
    $preq .= ")";
    
  }
  $result=mysql_query("SELECT * FROM '$Tabla' WHERE $preq",$con);
          
}
 

[charlyta]

Código PHP:

<?php
function buscar($Tabla,$Campos,$db,$con){
    $numero = count($_GET);
    $tags = array_keys($_GET);
    $valores = array_values($_GET);
    for($i=0;$i<$numero;$i++) 
    {
        $$tags[$i]=$valores[$i];
    }
foreach ($_GET as $key => $value) if ($key != ''){
      $preq .= " AND ( 0";
    foreach($Campos as $Campo)$preq .= " OR $Campo LIKE '%".$value."%'";
    $preq .= ")";
    
  }
  $result=mysql_query("SELECT * FROM '$Tabla' WHERE $preq",$con);
          
}
?>
<html><head> 
<title>Buscando ...</title> 
</head><body><? 
?> 
<form method="GET" action="recibir.php">
<input type="text" name="autor" id="autor" value="">
<input type="text" name="titulo" id="titulo" value="">
<input type="submit" value="Buscar"><br> </form>
<? 
 
 
 
 
$db = mysql_connect('localhost','root','')or die (mysql_error()); ; 
 
$con = mysql_select_db('libros',$db)or die (mysql_error()); ; 
 
$Campos = array ('tema','autor','titulo'); 
 
buscar('libros',$Campos,$db,$con); 
 while ($row = mysql_fetch_array($result)){
 
echo $row['autor'];
 
 }
 

esta es mi función que no va bien