[Resolu] Recherche multi critères

Marsh Posté le 18-06-2012 à 15:16:56

Bonjour,

Je bloque actuellement sur la manière de coder mon moteur de recherche sur mon site.
Je fais une vérification sur chaque critère pour voir s'ils possèdent une valeur auquel cas je les ajoute dans un tableau qui les régroupera toute en ayant déjà la forme prête pour la futur requête SQL.

Puis gràce à une boucle, je rajoute les AND de ma requête.

Grâce au compteur utilisé auparavant pour le tableau, je verifie s'il y a eu au moins un critère et je lance la requete avec le WHERE regroupant toutes les critères.

Voilà l'adresse du site pour vous faire une idée.

http://hightechservices.free.fr/ordinateur.php

Lors de n'importe quelle recherche, je retrouve toujours le seul est unique enregistrement contenu dans la table Ordinateur de ma base de données.

Avez-vous une idée sur ce qui ne va pas?

D'avance merci.

Message édité par fmdcc le 19-06-2012 à 11:39:23

Reply

Marsh Posté le 18-06-2012 à 15:16:56

Reply

Marsh Posté le 18-06-2012 à 15:45:50

1) fais un echo de ta requête SQL pour voir ce qu'elle fait.

2) dans ton moteur de recherche, ça serait bien de réafficher les paramètres saisis correspondant au résultat de la requête

Edit : dans mon logiciel de help-desk ASTRES (cf ma signature), j'ai codé pleins de moteurs de recherche de ce type. Tu peux regarder comment j'ai fait

Message édité par rufo le 18-06-2012 à 15:46:48

---------------
Astres, outil de help-desk GPL : http://sourceforge.net/projects/astres, ICARE, gestion de conf : http://sourceforge.net/projects/icare, Outil Planeta Calandreta : https://framalibre.org/content/planeta-calandreta

Reply

Marsh Posté le 18-06-2012 à 15:53:23

Tes 2 remarques sont bonnes et je vais déjà les appliquer et voir ce que ça donne.

Je jeterai également un oeil sur ce logiciel.

Merci pour ton aide.

Message édité par fmdcc le 18-06-2012 à 16:04:50

Reply

Marsh Posté le 18-06-2012 à 17:35:18

Je n'y suis toujours pas parvenu.

Voilà un aperçu de mon code avec tout d'abord le formulaire puis le traitement php.

Code :

<form name="RechercheOrdinateur" action="rechercheOrdinateur.php" method="POST" onSubmit="return checkForm();">
<h4>Critères de recherche</h4>
<table border="0">
<tr>
<td>Processeur :</td>
<td><SELECT name="lst_processeur" style="width:12em">
<OPTION VALUE=null></OPTION>
<OPTION VALUE="coreI3">Intel Core i3</OPTION>
<OPTION VALUE="coreI5">Intel Core i5</OPTION>
<OPTION VALUE="coreI7">Intel Core i7</OPTION>
<OPTION VALUE="amdPhenom">AMD Phenom</OPTION>
<OPTION VALUE="amdFusion">AMM Fusion</OPTION>
</SELECT></td>
</tr>
...
<p align=center>
<input type=submit value="Rechercher">
</p>

Code :

<?
// connexion
include('secure/config.php');
pg_connect("host=$SQLhost user=$SQLlogin password=$SQLpass dbname=$SQLdbname" )
or die("Impossible de se connecter" );
// on récupère les critères sélectionnés
//extract($_POST);
$lst_processeur = $_POST["lst_processeur"];
$i = 0;
// si la variable est présente, on lui affecte une place dans le tableau 'choix[]', qui nous servira ensuite à construire le WHERE de la requête.
if(!empty($lst_processeur)) { $choix[$i++] = "processeur = '$lst_processeur'"; }
if(!empty($lst_hdd)) { $choix[$i++] = "hdd = '$lst_hdd'"; }
if(!empty($lst_ram)) { $choix[$i++] = "ram = '$lst_ram'"; }
if(!empty($lst_cg)) { $choix[$i++] = "cg = '$lst_cg'"; }
if(!empty($lst_ecran)) { $choix[$i++] = "cg = '$lst_ecran'"; }
// on insère les éléments remplis dans une variable $critere, en commençant par la première occurrence, puis on boucle
$critere = $choix[0]." ";
for($j=1;$j<$i;$j++)
{
$critere .= " AND ".$choix[$j]." ";
}
// enfin on fait la requête si $i >0, ça veut dire qu'il y a des critères
if($i > 0)
{
// requete de selection
$query = "SELECT * FROM ordinateur WHERE $critere ORDER BY prix";
}
// si $i = 0, alors l'utilisateur n'a pas saisi de critère, là soit on fait la même requete mais sans "WHERE $critere", soit on lui demande de saisir au moins un critère.
else
{
$query = "SELECT * FROM ordinateur ORDER BY prix";
}
$result = pg_query($query);
pg_close();

Message édité par fmdcc le 19-06-2012 à 11:01:58

Reply

Marsh Posté le 18-06-2012 à 17:43:44

<OPTION VALUE=null></OPTION> -> déjà, ça, c'est pas bon, vaudrait mieux mettre <OPTION VALUE=""></OPTION>

Pour info, pas besoin de faire $choix[$i++] = "....";
Tu peux faire direct : $choix[] = "...";

Pour savoir si t'as des critères, juste un if (!empty($choix)).

Et pour les and à ajouter entre les critères :
$critere = implode(" and ", $choix);

---------------
Astres, outil de help-desk GPL : http://sourceforge.net/projects/astres, ICARE, gestion de conf : http://sourceforge.net/projects/icare, Outil Planeta Calandreta : https://framalibre.org/content/planeta-calandreta

Reply

Marsh Posté le 18-06-2012 à 17:55:58

Je vais mettre tout ça au point.

merci

Reply

Marsh Posté le 18-06-2012 à 18:05:54

Le problème, c'est que dès le début, j'ai l'impression que mes critères sont toujours nuls

Code :

<? echo $lst_processeur?>/<? echo $lst_hdd?>/<? echo $lst_ram?>/<? echo $lst_cg?>/<? echo $lst_ecran?>

J'ai ajouté cette ligne pour voir les critères demandé et rien n'en sors comme critère.

Pour la fonction implode avec les AND, j'ai ce message d'erreur malgré que ce soit les bons arguments.
Warning: implode() [function.implode]: Bad arguments.

Message édité par fmdcc le 18-06-2012 à 18:13:31

Reply

Marsh Posté le 19-06-2012 à 09:57:05

T'as quoi quand tu fais print_r($_POST); ?

Cela dit, lignes 8 et 10 du script rechercheOrdinateur.php me paraissent louches

Message édité par rufo le 19-06-2012 à 09:58:14

---------------
Astres, outil de help-desk GPL : http://sourceforge.net/projects/astres, ICARE, gestion de conf : http://sourceforge.net/projects/icare, Outil Planeta Calandreta : https://framalibre.org/content/planeta-calandreta

Reply

Marsh Posté le 19-06-2012 à 10:33:07

Avec print_r($_POST);, j'obtiens bien les critères sélectionnés.

Si je choisis par exemple disque dur 320 Go, j'ai:

Array ( [Processeur] => [DisqueDur] => 320 [Ram] => [CG] => [ecran] => )

Concernant les lignes 8 et 10, il s'agissait de test que j'avais oublié de supprimer.

Code :

<?
// connexion
include('secure/config.php');
pg_connect("host=$SQLhost user=$SQLlogin password=$SQLpass dbname=$SQLdbname" )
or die("Impossible de se connecter" );
// on récupère les critères sélectionnés
extract($_POST);
$i = 0;
// si la variable est présente, on lui affecte une place dans le tableau 'choix[]', qui nous servira ensuite à construire le WHERE de la requête.
if(!empty($lst_processeur)) { $choix[] = "processeur = '$lst_processeur'"; }
if(!empty($lst_hdd)) { $choix[] = "hdd = '$lst_hdd'"; }
if(!empty($lst_ram)) { $choix[] = "ram = '$lst_ram'"; }
if(!empty($lst_cg)) { $choix[] = "cg = '$lst_cg'"; }
if(!empty($lst_ecran)) { $choix[] = "cg = '$lst_ecran'"; }
// on insère les éléments remplis dans une variable $critere, en commençant par la première occurrence, puis on boucle
$critere = $choix[0]." ";
for($j=1;$j<$i;$j++)
{
$critere .= " AND ".$choix[$j]." ";
}
// $critere = implode(" and ", $choix);
// enfin on fait la requête si $i >0, ça veut dire qu'il y a des critères
if($i > 0)
{
// requete de selection
$query = "SELECT * FROM ordinateur WHERE $critere ORDER BY prix";
}
// si $i = 0, alors l'utilisateur n'a pas saisi de critère, là soit on fait la même requete mais sans "WHERE $critere", soit on lui demande de saisir au moins un critère.
else
{
$query = "SELECT * FROM ordinateur ORDER BY prix";
}
print_r($_POST);
$result = pg_query($query);
pg_close();
while($ligne = pg_fetch_array($result)){
?>

Message édité par fmdcc le 19-06-2012 à 10:35:44

Reply

Marsh Posté le 19-06-2012 à 11:31:52

Problème résolu.

Tout d'abord dans mon code sur le formulaire, je faisais une grosse confusion entre l'attribut name et id et je fais appelle au nom donné dans l'id pour le sript php avec POST...

Rien ne pouvez en sortir donc. Ca m'a toujours rendu confu ces 2 attributs name et id.

Puis pour la condition qui commence en ligne 29, j'ai ajouté une incrémentation du compteur i pour chaque critère remplis.

Ce qui nous donne au final:

Code :

<?
// connexion
include('secure/config.php');
pg_connect("host=$SQLhost user=$SQLlogin password=$SQLpass dbname=$SQLdbname" )
or die("Impossible de se connecter" );
// on récupère les critères sélectionnés
extract($_POST);
$i = 0;
// si la variable est présente, on lui affecte une place dans le tableau 'choix[]', qui nous servira ensuite à construire le WHERE de la requête.
if(!empty($lst_processeur))
{
$choix[] = "processeur = '$lst_processeur'";
$i++;
}
if(!empty($lst_hdd))
{
$choix[] = "hdd = '$lst_hdd'";
$i++;
}
if(!empty($lst_ram))
{
$choix[] = "ram = '$lst_ram'";
$i++;
}
if(!empty($lst_cg))
{
$choix[] = "cg = '$lst_cg'";
$i++;
}
if(!empty($lst_ecran))
{
$choix[] = "ecran = '$lst_ecran'";
$i++;
}
// on insère les éléments remplis dans une variable $critere, en commençant par la première occurrence, puis on boucle
$critere = implode(" and ", $choix);
// enfin on fait la requête si $i >0, ça veut dire qu'il y a des critères
if($i > 0)
{
// requete de selection
$query = "SELECT * FROM ordinateur WHERE $critere ORDER BY prix";
}
// si $i = 0, alors l'utilisateur n'a pas saisi de critère, là soit on fait la même requete mais sans "WHERE $critere", soit on lui demande de saisir au moins un critère.
else
{
$query = "SELECT * FROM ordinateur ORDER BY prix";
}
print_r($_POST);
print_r($critere);
print_r($query);
$result = pg_query($query);
pg_close();
while($ligne = pg_fetch_array($result)){
?>

Merci pour ton aide rufo.

Message édité par fmdcc le 19-06-2012 à 11:40:10

Reply

Marsh Posté le 19-06-2012 à 11:31:52

Reply

Marsh Posté le 19-06-2012 à 11:43:41

name => c'est le nom du champ dans le formulaire, récupéré côté serveur dans $_GET ou $_POST. Y'a que ds balise de formulaire qui peuvent avoir un name.

id => n'importe quelle balise peut avoir un id. Sert surtout pour le JS ( getElementById() ) ou le CSS.

Une balise peut avoir un name ET un id (et en plus, la valeur de ces 2 attributs peut être identique, mais c'est pas obligé).

Et pour info, pas de style dans le HTML (utilisation de l'attribut "style" dans le html à éviter au maximum -> utiliser css). Pareil, éviter le JS intrusif dans le html. JS uniquement dans des fichiers js.

---------------
Astres, outil de help-desk GPL : http://sourceforge.net/projects/astres, ICARE, gestion de conf : http://sourceforge.net/projects/icare, Outil Planeta Calandreta : https://framalibre.org/content/planeta-calandreta

Reply

Marsh Posté le 19-06-2012 à 11:43:47

plop
tu peux commencer par envoyer un array associatif directement, ca t'evitera de galerer ensuite et c'est plus clean

Code :

<select multiple id="criteriaProcessor" name="criteria[processor][]">
<option value="">-</option>
<option value="i5">i5</option>
...
</select>
<select id="criteriaRam" name="criteria[ram]">
<option value="">-</option>
<option value="4">4gb</option>
...
</select>

ca te permet de recup directement un array $criteria

Code :

// query de fallback. si pas de criteres elle sera lancée telle qu'elle
$sqlquery = 'SELECT * FROM ordinateur WHERE 1 ';
$criteria = (isset($_POST['criteria'])&&is_array($_POST['criteria']))?$_POST['criteria']:array();
if (!empty($criteria)){
// pour chaque critere envoyé on recupere la key (ie: processor ou ram) et la value
foreach ($criteria as $key=>$value){
// si la value est un tableau ca veut dire qu'on a utilisé un select multiple d'ou la notation name='criteria[processor][]' en html
// la dans mon ex j'utilise un 'OR' pour chaque sous valeur
if (is_array($value)){
$sqlquery .= ' AND (
$subCount = 0;
foreach ($value as $subvalue){
if($subcount){
$sqlquery .= ' OR ';
}
$sqlquery .= '\''. $key .'\' = '. $subvalue ';
$subCount++;
}
$sqlquery .= ')';
}else{
// si ce n'est pas un tableau c'est un critere 'simple'
$sqlquery .= ' AND \''. $key .'\' = '. $value ';
}
}
}

code a l'arrache et pas testé mais le principe est normalement ok.

Message édité par pop-pan le 19-06-2012 à 12:12:57

---------------
[VDS] rail vesa, bras ecran, support TV / [ACH] des machins

Reply

Marsh Posté le 19-06-2012 à 11:46:06

Merci pour vos précisions et remarques.

Reply

Marsh Posté le 19-06-2012 à 12:02:02

je t'ai commenté un peu tout ca.
sinon n'oublie pas d'echaper correctemment tes valeurs
mysql_real_escape($key) ou mysql_real_escape($value)
pour limiter les injections SQL (je sais plus c'est lequel pour postgres...)

tu peux aussi utiliser PDO et des statements pour faire ca propre.

---------------
[VDS] rail vesa, bras ecran, support TV / [ACH] des machins

Reply

Marsh Posté le 19-06-2012 à 12:10:35

ah et sinon 'extract()' c'est le mal..

a la limite 'extract($_POST, EXTR_PREFIX_ALL, "extracted_" );' pour prefixer ces nouvelles variables.