[SQL] Recherche mutli critère [Résolu]

[masseur]

Bonjour.

J'aimerai avoir un petit eclaircissement.
Je possède un formulaire possédant 9 champs possibles.

AU minimum un champs doit être rempli et après le reste est optionnel.
Quel est le meilleur moyen de s'y prendre?

J'avais penser à faire comme ceci

$query="SELECT * FROM prospects";


$query.=" WHERE nom LIKE '$choixrechercher_nom' AND adresse1 LIKE '$choixrechercher_adresse' AND cp LIKE '$choixrechercher_cp' AND type_base LIKE '$choixrechercher_typebase' AND datecreation_base LIKE '$choixrechercher_datebase' AND profession LIKE '$choixrechercher_profession' AND societe LIKE '$choixrechercher_societe' AND CSP LIKE '$choixrechercher_CSP' ORDER BY nom";

Et les champs n'étant pas rempli vallait donc %.
Seul hic c'est lorsque il y a des champs non remplis dans la BDD et qu'ils valent donc NULL ben l'histoire du % ne marche plus.

Au final quel est la meilleure solution pour faire une recherche mutli critère sachant que 8 champs sur 9 sont optionnels.

Cordialement

[koopajah]

Il y a pas mal d'exemples de ce genre de problemes dans le forum.
Le mieux déjà est de constuire ta requete au fur et a mesure pour chaque critère indépendemment les uns des autres, exemple :

Code :

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
 
$req = "SELECT * FROM matable ";
$cond = " WHERE " ;
 
if(isset($_POST['critere1']))
{
$req .= $cond . " critere1 = '" . $_POST['critere1'] . "' ";
$cond = " AND ";
} 
 
if(isset($_POST['critere2']))
{
$req .= $cond . " critere2 = '" . $_POST['critere2'] . "' ";
$cond = " AND ";
}

etc.
tu notes que $cond permet de savoir quelle condition utiliser (par défaut WHERE et ensuite AND)

sinon il me semble que Asmodean (mais je confonds peut etre) utilise un WHERE 1 = 1 comme première condition, du coup tu sais que tu as toujours des AND et tu n'as pas de probleme de conditions.

[guigui5931]

ce que je fais dans ce cas la c'est que si le champs n'est pas rempli je ne le met pas dans la requete SQL. Je te fais un exemple avec 2 champs optionnel et un obligatoire

Code :

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
 
$obligatoire = $_POST['obligatoire'];
$optionnel1= $_POST['optionnel1'];
$optionnel2= $_POST['optionnel2'];
if (!empty($obligatoire  ))
{
$param1 = "obligatoire = '" .$obligatoire. "'";
}
else
{
$param1 = "1";
}
if (!empty($optionnel1))
{
$param2 = "optionnel1= '" .$optionnel1. "'";
}
else
{
$param2 = "1";
}
if (!empty($optionnel2))
{
$param3 = "optionnel2= '" .$optionnel2. "'";
}
else
{
$param3 = "1";
}
$sql="SELECT $ FROM table WHERE $param1 AND $param2 AND $param3"

Si tu as besoin de plus d'explication demande

[zulot]

Ouais et si ton deuxieme critiere y a rien il me semble que la requete finissant par AND plante non?
Donc pas tres malin

Pour reprendre l'exemple precedent :

Code :

1
2
3
4
5
6
7
8
9
10
11
12
 
if(isset($_POST['obligatoire'])){
$param1 = "obligatoire = ".$obligatoire. "'";
}
else{
 $param1 = 1;
}
if(isset($_POST['option1'])){
 $param2 = ' AND param2='.$_POST['option1'];
}
 
$sql="SELECT * FROM table WHERE $param1 $param2 "

Je trouve plus leger et tu peux mieux gérer les OR ou cas bizzard

[masseur]

Merci à vous deux, voila qui m'éclaire bien sur la facon de m'y prendre?
EN cas de besoin je reviendrai.

Merci

[guigui5931]

Avec le code de koopajah ça ne marchera pas si un champs est vide. Si tu met des 1 quand le champs est vide ça marche à tous les couts puisuqe 1 est interprété comme le boolean vrai (et n'a donc pas d'effet sur les AND)

EDIT : ça ne marchera pas si le dernier champs et vide pardon

[guigui5931]

Je viens de faire un EDIT mais il me semble que si le dernier de tes paramètres est vide tu vas te retrouver avec un AND puis rien après.
Mais je me trompe peut être

[zulot]

Envoyé par koopajah

Dans ma solution le AND n'est ajouté a la requete que si on met bel et bien un critere après donc je pense que mon message bien que pas forcément malin etait largement viable, je l'utilise tous les jours

Dans la solution de guigui c'est parfaitement viable aussi vu qu'on se retrouve avec dans 1 entre chaque AND si le critère n'est pas rempli. Le seul défaut étant qu'il faut connaitre a l'avance le nombre de paramètres.

Par contre j'aimerais savoir quelle est ta solution si intelligente que tu ne daignes meme pas proposer?

Si tu m'avais laissé le temps de modifier mon com

[koopajah]

Envoyé par guigui5931

Je viens de faire un EDIT mais il me semble que si le dernier de tes paramètres est vide tu vas te retrouver avec un AND puis rien après.
Mais je me trompe peut être

Et bien relis mon code qui ne fait pourtant que quelques lignes et tu verras bien que ca n'est pas le cas.

Dans chaque if on a deux lignes, la première ajoute a notre requete courante la condition courante (where si c'est le premier critère non vide, and sinon) et ensuite le critère lui meme.
La seconde ligne sert a dire "la condition courante de vient AND vu qu'on a mis un WHERE avant"

donc ce code marche très bien.

[zulot]

Envoyé par koopajah

Et bien relis mon code qui ne fait pourtant que quelques lignes et tu verras bien que ca n'est pas le cas.

Dans chaque if on a deux lignes, la première ajoute a notre requete courante la condition courante (where si c'est le premier critère non vide, and sinon) et ensuite le critère lui meme.
La seconde ligne sert a dire "la condition courante de vient AND vu qu'on a mis un WHERE avant"

donc ce code marche très bien.

Un code qui marche bien c'est cool, mais un code ou deux personnes se trompe au premier coup d'oeil c'est pas terrible...... Il est donc pas très claire.

[guigui5931]

J'ai été un poil vite, désolé

[koopajah]

Envoyé par zulot

Un code qui marche bien c'est cool, mais un code ou deux personnes se trompe au premier coup d'oeil c'est pas terrible...... Il est donc pas très claire.

Une ou deux personnes qui se trompent parce qu'elles ont survolé le code donné pour montrer LE code auquel elles se sont habituées.
Ca ne prouve en rien que mon code est faux surtout que c'était expliqué dans le message donc franchement je vois pas l'intéret de rabaisser ma solution juste parce que vous l'avez à peine lue.

Maintenant il a l'embarras du choix, il suffit qu'il utilise la plus pratique pour lui c'est tout.

[masseur]

J'ai utilisé la méthode de koopajah plus ou moins qui me convenait mieux et ca marche nickeL

Merci

[guigui5931]

Je ne suis pas d'accord avec toi zulot je pense que son code est le meilleur qui a été donné sur ce thread. Il est plus concit et clair (si on le lit plus de 10 seconde ).
Mais on va pas se battre