Discussion :

[yotman]

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]

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

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]

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]

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

[sabotage]

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

[Séb.]

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

[Séb.]

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]

Merci Seb, c'est noté.