Limiter les conditions

**yotman** · 15/06/2013, 13h43

Bonjour,

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

Voici un exemple :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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

**Dendrite** · 15/06/2013, 13h57

'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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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.
   }
}

**yotman** · 15/06/2013, 14h09

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

**sabotage** · 15/06/2013, 15h45

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.

**yotman** · 15/06/2013, 15h49

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

je parle de :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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

**Séb.** · 15/06/2013, 15h53

@Dendrite

Quelques remarques :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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.
   }
}

**sabotage** · 15/06/2013, 15h54

En quoi ce if augmente le nombre de requête ?

**Séb.** · 15/06/2013, 15h57

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.

**Dendrite** · 15/06/2013, 16h15

Merci Seb, c'est noté.

**yotman** · 15/06/2013, 16h17

Voici le formulaire de recherche en mode simplifier

Code html :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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 ....

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

Voici le formulaire de recherche en mode simplifier

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

Je regarde.

**Séb.** · 15/06/2013, 16h27

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

Code html :

Sélectionner tout - Visualiser dans une fenêtre à part

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>

**Séb.** · 15/06/2013, 16h34

Bon donc apparemment ton problème concerne le traitement de $_POST['checkbox'].

Le plus simple :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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

**yotman** · 15/06/2013, 16h41

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

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

**sabotage** · 15/06/2013, 16h41

Autre piste de simplification :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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

**Séb.** · 15/06/2013, 16h44

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

1 if
7 else if
1 else

Ceci (et même plus) :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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.

**yotman** · 15/06/2013, 16h53

Dans le code je dis :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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

dans le cas ou tt est coché

et

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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

si tt est coché sauf code

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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

**Séb.** · 15/06/2013, 16h57

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

Quelles sont tes contraintes ?

Au lieu de faire :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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

Fais :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

if ( $code != NULL ) {

Déjà t'y verras plus clair.

**yotman** · 15/06/2013, 17h02

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

**Séb.** · 15/06/2013, 17h05

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 ?

Limiter les conditions

PHP & Base de données

Discussions similaires

Partager

Partager