Limiter les conditions

Version imprimable

Bonjour,

Je souhaite savoir si il y a une façon pour éviter de saturer le code avec les if.

Voici un exemple :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 if((!($code1 == NULL)) and (!($zone1 == NULL)) and(!($values == NULL)) ) { foreach($mots as $mot) { $sql.="SELECT * FROM afficher,doc WHERE afficher.idqrqc = doc.idqrqc and code='$code1' and zone='$zone1' $sql_types and titre LIKE '%$mot%' UNION "; } $sql=substr($sql,0,-6); $req = mysql_query($sql, $cnx) or die("Pb dans la requête : " . mysql_error($cnx)); }
Evidement c'est pour un formulaire de recherche par critère.

Si je met pas de conditions, quand la personne ne rempli pas un champ, il y aura aucun résultat qui s'affiche parce que il compare (je pense) avec le vide.

Merci à vous

'Saturer le code avec des if', Je ne vois pas en quoi c'est un problème. Les conditions, comme les boucles, si elles sont utiles, sont l'alpha et l'omega de la programmation !

Par contre une remarque, moi, pour rendre les choses pratiques et lisibles, je code ainsi cette problématique classique des développements PHP, imaginons qu'aucun champ ne doive rester vierge :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 if(isset($_POST){ $complet=true; foreach($_POST as $input){ if($input==NULL || $input==''){//toujours penser au champ saisi puis effacé $complet=false; } } if($complet){ //je traite } else{ //je bloque et je retourne un message en rouge. } }

15/06/2013, 14h09
yotman

Oui, mais je n'arrive pas trop à gérer tout ça !

C'est un formulaire de recherche et je veux rien imposer. Genre dans le cas ou la personne ne coche rien, le résultat est : toutes les données de la BD. etc

Le problème est que j'ai plusieurs champ et donc si je continue avec cette méthode, pour chaque nouveau champ il faut ajouter d'autres conditions

nbre de requète = X puissance 2 (X est le nombre de champ)

je sais pas si c'est claire ou pas
15/06/2013, 15h45
sabotage

Je t'ai déjà indiqué d'utiliser des AND/OR plutôt que de faire des UNION.
Ca te ferra une seule requête quelque que soit le nombre de critère.
15/06/2013, 15h49
yotman

La je parle d'autre chose !
Union concerne la recherche par mot clé

je parle de :

Code:

if((!($code1 == NULL)) and (!($zone1 == NULL)) and(!($values == NULL)) )

@Dendrite

Quelques remarques :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
if(isset($_POST){ // isset inadapté car $_POST toujours définie
   $complet=true;
   foreach($_POST as $input){
      // $input ne vaudra jamais NULL : chaîne de caractère vide ou renseignée
      if($input==NULL || $input==''){//toujours penser au champ saisi puis effacé
         $complet=false;
      }
   }
   if($complet){
      //je traite
   }
   else{
      //je bloque et je retourne un message en rouge.
   }
}

Donc :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
if( ! empty($_POST) ){ // Form soumis ?
   $complet=true;
   foreach($_POST as $input){
      if($input==''){
         $complet=false;
      }
   }
   if($complet){
      //je traite
   } else{
      //je bloque et je retourne un message en rouge.
   }
}

15/06/2013, 15h54
sabotage

En quoi ce if augmente le nombre de requête ?
15/06/2013, 15h57
Séb.

Citation:

Le problème est que j'ai plusieurs champ et donc si je continue avec cette méthode, pour chaque nouveau champ il faut ajouter d'autres conditions

nbre de requète = X puissance 2 (X est le nombre de champ)

Un nombre de critères plus élevés ne devrait pas faire augmenter le nombre de requêtes.
Le nombre de requêtes doit rester ~le même : ~1.

Donne-nous ton schéma de base de données, un exemple de formulaire et stp présente/indente ton code correctement.
15/06/2013, 16h15
Dendrite

Merci Seb, c'est noté.

Voici le formulaire de recherche en mode simplifier

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
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
<body>
<table style="text-align: left; width: 100%;" border="0"
 cellpadding="2" cellspacing="2">
  <tbody>
    <tr>
      <td colspan="3" rowspan="1">  
 
	  <div>Code<input  name="code1" value="" type="text" placeholder="Ex: 15227"  pattern="[0-9]{5}" /></div>
 
	  </td>
    </tr>
    <tr>
      <td colspan="3" rowspan="1">
 
	  <div>Mots clés<input   name="titre1" value="" type="text"  " /></div>
 
	  </td>
    </tr>
    <tr>
      <td colspan="3" rowspan="1">
 
	  <div style="text-align: left;">Type de défaut</div>
	  </td>
    </tr>
    <tr>
      <td>
 
	  <div><input name="checkbox[]" value="Choqués" id="choques"  type="checkbox"> <label for="choques">Choqués</label>
</div>
<div><input name="checkbox[]" value="Décor_accroché" id="decor_accroche" type="checkbox"> <label for="decor_accroche">Décor accroché</label><br>
</div>
<div><input name="checkbox[]" value="Décor_décalé" id="decor_decale" type="checkbox"> <label for="decor_decale">Décor décalé</label><br>
</div>
<div><input name="checkbox[]" value="Décor_déformé" id="decor_deforme" type="checkbox"> <label for="decor_deforme">Décor déformé</label>
</div>
<div><input name="checkbox[]" value="Décor_rétracté" id="decor_retracte" type="checkbox"> <label for="decor_retracte">Décor rétracté</label><br>
</div>
<div><input name="checkbox[]" value="Décor_voilé" id="decor_voile" type="checkbox"> <label for="decor_voile">Décor voilé</label><br>
</div>
 
 
	  </td>
      <td>
 
	  </td>
    </tr>
    <tr>
      <td colspan="3" rowspan="1">Zone
<div><input type="radio" name="zone1" value="Décor" id="decor" /> <label for="decor">Decor</label><br />
</div>
<div>
                <input type="radio" name="zone1" value="Laquage" id="laquage" /> <label for="laquage">Laquage</label><br />
</div>				
<div>
				<input type="radio" name="zone1" value="Dépoli" id="depoli" /> <label for="depoli">Dépoli</label><br />
</div>				
<div>
				<input type="radio" name="zone1" value="Sablage" id="sablage" /> <label for="sablage">Sablage</label><br />
</div>
 
	  </td>
    </tr>
  </tbody>
</table>
 
 
<center><input type="submit" value="Chercher"  />

et voila le php (simplifié aussi)

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
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
if(isset($_POST['code1']))      $code1=$_POST['code1'];
else      $code1="";
 
 
if(isset($_POST['zone1']))      $zone1=$_POST['zone1'];
else      $zone1="";
 
 
if(isset($_POST['titre1']))                                                    
 
        $rec = ($_POST['titre1']);                             
 
 
if (isset($_POST['checkbox']))
foreach (
$_POST['checkbox'] as $checkbox){
$values[]=$checkbox;
}
else {
$values[0]="Choqués";
$values[1]="Décor_accroché";
$values[2]="Décor_décalé";
$values[3]="Décor_déformé";
$values[4]="Décor_rétracté";
$values[5]="Décor_voilé";
$values[6]="Ebréchés";
}
 
$arrlength=count($values);
if ($arrlength=='1'){
	$sql_types= " AND defaut='$values[0]'";
}elseif ($arrlength=='2'){
	$sql_types= " AND defaut in('$values[0]','$values[1]')";
}
elseif ($arrlength=='3'){
	$sql_types= " AND defaut in('$values[0]','$values[1]','$values[2]')";
}
elseif ($arrlength=='4'){
	$sql_types= " AND defaut in('$values[0]','$values[1]','$values[2]','$values[3]')";
}
elseif ($arrlength=='5'){
	$sql_types= " AND defaut in('$values[0]','$values[1]','$values[2]','$values[3]','$values[4]')";
}
elseif ($arrlength=='6'){
	$sql_types= " AND defaut in('$values[0]','$values[1]','$values[2]','$values[3]','$values[4]','$values[5]')";
}
elseif ($arrlength=='7'){
	$sql_types= " AND defaut in('$values[0]','$values[1]','$values[2]','$values[3]','$values[4]','$values[5]','$values[6]')";
}
 
 
f((!($code1 == NULL)) and (!($zone1 == NULL)) and(!($values == NULL)) )
{
 
 
 foreach($mots as $mot)
        {
$sql.="SELECT * FROM afficher,doc WHERE  afficher.idqrqc = doc.idqrqc and  code='$code1' and zone='$zone1'  $sql_types and titre LIKE '%$mot%' UNION ";
 }
//quand la boucle est terminée, il faut enlever le dernier union
$sql=substr($sql,0,-6);
 
 
$req = mysql_query($sql, $cnx) 
 		   or 
 		  die("Pb dans la requête : " . mysql_error($cnx));
 
 }
 
 
 
 
		  else if((($code1 == NULL)) and (!($zone1 == NULL)) and (!($values == NULL)))
{
 
foreach($mots as $mot)
        {
$sql.="SELECT * FROM afficher,doc WHERE  afficher.idqrqc = doc.idqrqc  and zone='$zone1'  $sql_types and titre LIKE '%$mot%'
 
UNION ";
 }
//quand la boucle est terminée, il faut enlever le dernier union
$sql=substr($sql,0,-6);
 
 
$req = mysql_query($sql, $cnx) 
 		   or 
 		  die("Pb dans la requête : " . mysql_error($cnx));
		  }

else if pour chaque cas

au total sa me donne 9 if pour 3 champ, et comme je vais avoir 6 ou 7 champ donc ....:cry:

15/06/2013, 16h20
Séb.

Citation:

Voici le formulaire de recherche en mode simplifier

Eh bien, qu'est-ce que ça aurait été sinon ? :mouarf:

Je regarde.

Le <form> nettoyé/simplifié un minimum :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
<form method="post">
 
<p>Code <input name="code1" type="text"></p>
 
<p>Mots clés <input name="titre1"></p>
 
<p>Type de défaut<br>
<input name="checkbox[]" value="Choqués"  type="checkbox">Choqués<br>
<input name="checkbox[]" value="Décor_accroché" type="checkbox">Décor accroché<br>
<input name="checkbox[]" value="Décor_décalé" type="checkbox">Décor décalé<br>
<input name="checkbox[]" value="Décor_déformé" type="checkbox">Décor déformé<br>
<input name="checkbox[]" value="Décor_rétracté" type="checkbox">Décor rétracté<br>
<input name="checkbox[]" value="Décor_voilé" type="checkbox">Décor voilé</p>
 
<p>Zone<br>
<input type="radio" name="zone1" value="Décor">Decor<br>
<input type="radio" name="zone1" value="Laquage">Laquage<br>
<input type="radio" name="zone1" value="Dépoli">Dépoli<br>
<input type="radio" name="zone1" value="Sablage">Sablage</p>
 
<p><input type="submit"></p>
 
</form>

15/06/2013, 16h34
Séb.
Bon donc apparemment ton problème concerne le traitement de $_POST['checkbox'].

Le plus simple :
Code:

1 2 3 4 5 6 $sql = 'SELECT ... FROM ... WHERE ...' ; if ( ! empty($_POST['checkbox']) ) { $defauts = implode('\', \'', $_POST['checkbox']) ; $sql .= " AND defaut IN('$defauts')" ; }
Ainsi ta requête s'adaptera automatiquement au nombre de critère sélectionné.

PS : faudra penser impérativement à renommer le champ "checkbox" en "defauts" par exemple
15/06/2013, 16h41
yotman

Merci Séb.

Mais je répète encore une fois ;) je parle des if

tu vois bien dans l'exemple que je vais avoir :

1 if
7 else if
1 else

et tt sa pour 3 champ (code, zone, type)

je vais avoir encore 4 champ, donc sa me fera (je pense) 49 if et c'est trop :cry:

je suis vraiment débutant et je sais pas si il y a une autre manière pour gérer tt sa

Autre piste de simplification :

Code:

1
2
3
4
5
6
$inputs = array('code1', 'zone1', 'titre1');
foreach ($inputs as $name) {
   if (!empty($_POST[$name]) {
         $criteres[$name] = mysql_real_escape_string($_POST[$name]);
   }
}

et ensuite pour la requête :

Code:

1
2
3
foreach ($criteres as $name => $value) {
     $sql .= ' AND ' . $name . ' = ' . $value;
}

Citation:

tu vois bien dans l'exemple que je vais avoir :

1 if
7 else if
1 else

Ceci (et même plus) :

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
30
31
32
33
34
35
36
if (isset($_POST['checkbox']))
foreach (
$_POST['checkbox'] as $checkbox){
$values[]=$checkbox;
}
else {
$values[0]="Choqués";
$values[1]="Décor_accroché";
$values[2]="Décor_décalé";
$values[3]="Décor_déformé";
$values[4]="Décor_rétracté";
$values[5]="Décor_voilé";
$values[6]="Ebréchés";
}
 
$arrlength=count($values);
if ($arrlength=='1'){
	$sql_types= " AND defaut='$values[0]'";
}elseif ($arrlength=='2'){
	$sql_types= " AND defaut in('$values[0]','$values[1]')";
}
elseif ($arrlength=='3'){
	$sql_types= " AND defaut in('$values[0]','$values[1]','$values[2]')";
}
elseif ($arrlength=='4'){
	$sql_types= " AND defaut in('$values[0]','$values[1]','$values[2]','$values[3]')";
}
elseif ($arrlength=='5'){
	$sql_types= " AND defaut in('$values[0]','$values[1]','$values[2]','$values[3]','$values[4]')";
}
elseif ($arrlength=='6'){
	$sql_types= " AND defaut in('$values[0]','$values[1]','$values[2]','$values[3]','$values[4]','$values[5]')";
}
elseif ($arrlength=='7'){
	$sql_types= " AND defaut in('$values[0]','$values[1]','$values[2]','$values[3]','$values[4]','$values[5]','$values[6]')";
}

A été remplacé avec ceci :

Code:

1
2
3
4
if ( ! empty($_POST['checkbox']) ) {
    $defauts = implode('\', \'', $_POST['checkbox']) ;
    $sql .= " AND defaut IN('$defauts')" ;
}

Si ça ne te suffit pas alors il faudra que tu sois plus précis dans ta demande.

15/06/2013, 16h53
yotman

Dans le code je dis :

Code:

if((!($code1 == NULL)) and (!($zone1 == NULL)) and(!($values == NULL)) )

dans le cas ou tt est coché

et

Code:

else if((($code1 == NULL)) and (!($zone1 == NULL)) and (!($values == NULL)))

si tt est coché sauf code

Code:

else if((!($code1 == NULL)) and (($zone1 == NULL)) and (!($values == NULL)))

si tt est coché sauf zone

etc

je sais que c'est pas une méthode propre c'est pour je cherche une méthode plus facile
15/06/2013, 16h57
Séb.

C'est quoi l'intérêt de faire ça ?

Quelles sont tes contraintes ?

Au lieu de faire :

Code:

if ( (!($code == NULL)) ) {

Fais :

Code:

if ( $code != NULL ) {

Déjà t'y verras plus clair.
15/06/2013, 17h02
yotman

Si je fais pas ça, et que le champ n'est pas rempli, il va chercher dans la colonne un champ vide et il m'affiche rien, c'est pour ça si il est vide je le met pas dans la requète
15/06/2013, 17h05
Séb.

Citation:

Envoyé par yotman

Si je fais pas ça, et que le champ n'est pas rempli, il va chercher dans la colonne un champ vide et il m'affiche rien, c'est pour ça si il est vide je le met pas dans la requète

Combien de champs ? Quels sont leurs noms ?
15/06/2013, 17h09
yotman

code1, zone1, et le checkbox ($values)

la table contient une colonne code, zone, defaut (voir la requète)

Je ne vois pas comment tu pourrais avoir beaucoup plus de 3 if avec 3 champs à vérifier. Déjà ce que je t'ai donné plus haut raccourci pas mal ton script.

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
$sql = 'SELECT ... FROM ... WHERE 1 = 1 ' ;
 
if ( critère saisi sur 1er champ ) {
    $sql .= ' AND champ1 = "(filtre)"' ;
}
 
if ( critère saisi sur 2e champ ) {
    $sql .= ' AND champ2 = "(filtre)"' ;
}
 
if ( ! empty($_POST['checkbox']) ) { // Critère sur 3e champ
    $defauts = implode('\', \'', $_POST['checkbox']) ;
    $sql .= " AND defaut IN('$defauts')" ;
}
 
echo $sql ;

15/06/2013, 17h20
yotman
merci pour l'astuce des checkbox :ccool:

pour les if j'aurai bien 9 et pas 3
```
1  1  1
1  1  0
1  0  1
0  1  1
1  0  0
0  1  0
0  0  1
0  0  0
```
15/06/2013, 17h21
Séb.

Pourquoi ? Quelles requêtes veux-tu obtenir au final ?
15/06/2013, 17h26
yotman

une requète pour chaque if

exmple 1

code : 12457
Zone : décor
defaut : " "

la requète est

Code:

SELECT * FROM afficher,doc WHERE afficher.idqrqc = doc.idqrqc and code='$code1' and zone='$zone1' and titre LIKE '%$mot%' UNION ";

exmple 2

code : " "
Zone : décor
defaut : " manques, empâtés"

la requète est

Code:

SELECT * FROM afficher,doc WHERE afficher.idqrqc = doc.idqrqc and zone='$zone1' $sql_types and titre LIKE '%$mot%' UNION ";

exmple 3

code : " "
Zone : " "
defaut : " manques, empâtés"

la requète est

Code:

SELECT * FROM afficher,doc WHERE afficher.idqrqc = doc.idqrqc and $sql_types and titre LIKE '%$mot%' UNION ";

etc

si je fais pas sa, il va comparer le vide et il me renvoi rien
15/06/2013, 17h28
Séb.

Citation:

une requète pour chaque if

Non. Tu as une requête de base à laquelle tu rajoutes des filtres dans le WHERE, comme montré plus haut.
Au final 1 seule requête construite dynamiquement selon les saisies.

PS : les jointures sont opérées avec INNER JOIN et pas dans le WHERE ( http://dev.mysql.com/doc/refman/5.0/fr/join.html ).