Discussion :

[Msysteme]

bonjour, j'ai un form (une page de recherche) avec plusieurs critere de recherche 6 criteres, je veux construire ma requête en mettant uniquement les critères remplis, j'ai procédé comme suit :

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
        string rqFin = "";
 
            string rubrique = txtChap.Text+txtArt.Text+txtReg.Text+txtParag.Text+txtLigne.Text;
 
            if(!string.IsNullOrEmpty(rubrique))
                rqFin = "where .... like '"+rubrique+"%'";
 
            if (cmbService.SelectedIndex != 0 && !string.IsNullOrEmpty(rubrique))
                rqFin += "AND ....= '" + cmbService.Text + "'";
            else if (cmbService.SelectedIndex != 0 && string.IsNullOrEmpty(rubrique))
                rqFin += "WHERE .... = '" + cmbService.Text + "'";
 
            if ((cmbNatureAO.SelectedIndex != 0 && cmbService.SelectedIndex != 0 && !string.IsNullOrEmpty(rubrique)) || (cmbNatureAO.SelectedIndex != 0 && cmbService.SelectedIndex != 0))
                rqFin = "AND .......... = '" + cmbNatureAO.Text + "'";
            else if (cmbNatureAO.SelectedIndex != 0 && cmbService.SelectedIndex == 0 && string.IsNullOrEmpty(rubrique))
                rqFin += "WHERE ...........= '" + cmbNatureAO.Text + "'";
            else if (cmbNatureAO.SelectedIndex != 0 && cmbService.SelectedIndex == 0 && !string.IsNullOrEmpty(rubrique))
                rqFin += "AND ..............= '" + cmbNatureAO.Text + "'";
 
            if (!string.IsNullOrEmpty(rqFin))
            {
                try
                {
                    SqlDataAdapter adaptR = new SqlDataAdapter("SELECT Rubriques.IDR, ...................... " + rqFin, connection);
                    adaptR.Fill(dt);
                    dgRésultatRecherche.DataSource = dt;
                }
                catch (Exception ex)
                {
                    MessageBox.Show(ex.Message);
                }
            }

mais je veux ajouter d'autre requete, la gestion du where et du AND ca serait un peut difficile, une idée ?

Merci

[ZenZiTone]

Pour simplifier ta création de requête, je te conseillerais de créer une fonction prenant en paramètre un filtre. Le filtre correspondrait aux valeurs saisies dans l'IHM. Ainsi, le requêteur aurait juste à déduire les clauses where en fonction de ce filtre (et non de l'ihm).

Côté fonctionnement ce sera sensiblement la même chose que tu as actuellement, à la différence près que tu retires la dépendance IHM/requête et que tu facilite grandement la lecture du code.

[Msysteme]

je comprends mais c'est pratiquement la meme chose

[Jaco67]

Salut,

En plus de ce que propose ZenZiTone, tu peux construire ta clause where de la manière suivante :

Code :

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

1
2
3
4
5
6
7
 
string rqFin = "where 1 = 1 "
if(!string.IsNullOrEmpty(param1))
    rqFin += "and .... like '"+param1+"%'";
if(!string.IsNullOrEmpty(param2))
    rqFin += "and .... like '"+param2+"%'";
...

[Msysteme]

mais si tu as une dizaine de critere tu ne peux pas gerer ta requete facilement
parfois tu va te retrouver avec plusieurs where dans la requete

[ZenZiTone]

mais si tu as une dizaine de critere tu ne peux pas gerer ta requete facilement
parfois tu va te retrouver avec plusieurs where dans la requete

C'est le principe des filtres oui.. Je ne vois pas ce qui dérange là

[Jaco67]

Justement, avec ma solution tu n'as plus ce problème de "plusieurs where".
Si aucun filtre n'est activé, tu auras :

Code :

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

1
2
3
select ...
from ...
where 1=1

Si tu as un filtre :

Code :

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

1
2
3
4
select ...
from ...
where 1=1
and critere1 = ...

Si tu as deux filtres :

Code :

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

1
2
3
4
5
select ...
from ...
where 1=1
and critere1 = ...
and critere2 = ...

[Pol63]

en plus propre (mais ca revient au même) tu peux ajouter chaque filtre dans une list<string>
pour chaque test tu fais juste un conditions.add ("telcolonne like ...")
puis faire

Code :

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

1
2
3
4
if (conditions.count > 0)
{
rq += " where " + string.join(" and ", list)
}

par contre ce n'est pas best practice d'utiliser des valeurs contenues dans des controles pour les concaténer dans la requête
ca expose à des failles de sécurité et des bugs
normalement on utilise des DbParameter

ce qui donnerait dans l'idéal

Code :

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

1
2
3
var rq = @"select ... from table where col1 (like '%' + @col1 + '%' OR @col1 is null) and (col2 = @col2 or @col2 is null) ...";
cmd.parameters.Add("@col1").value = txtb1.txt is null ? dbnull.Value : txt1.txt;
cmd.parameters.Add("@col2").value = cmb1.selecteditem is null ? dbnull.Value : smb1.selectedvalue;

ici plus de if, c'est le sgbdr qui s'occupera d'ignorer les trucs inutiles

(je n'écris pas du c# tous les jours donc la syntaxe est peut être à adapter)

[Msysteme]

Merci Pol63 , c'est tres propre comme solution, merci pour votre passage

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
List<string> Lstconditions = new List<string>();
            string rqFin = "";
 
            string rubrique = txtChap.Text+txtArt.Text+txtReg.Text+txtParag.Text+txtLigne.Text;
 
            if(!string.IsNullOrEmpty(rubrique))
                Lstconditions.Add("Rubriques.Chap + Rubriques.Art + Rubriques.Reg + Rubriques.Para + Rubriques.Ligne like '"+rubrique+"%'");
 
            if (cmbService.SelectedIndex!=0)
                Lstconditions.Add("AX.ServiceAO = '" + cmbService.Text + "'");
 
            if (cmbNatureAO.SelectedIndex !=0)
                Lstconditions.Add("AX.NatureAO = '" + cmbNatureAO.Text + "'");
 
            if (cmbTypeImp.SelectedIndex != 0)
                Lstconditions.Add("Rubriques.typeImputation = '" + cmbTypeImp.Text + "'");
 
            if (cmbAnnée.SelectedIndex != 0)
                Lstconditions.Add("Rubriques.AnneeImputation = '" + cmbAnnée.Text + "'");
 
            if (Lstconditions.Count > 0)
            {
                rqFin += " WHERE " + string.Join(" and ", Lstconditions);
}

[Pol63]

par contre ce n'est pas best practice d'utiliser des valeurs contenues dans des controles pour les concaténer dans la requête
ca expose à des failles de sécurité et des bugs
normalement on utilise des DbParameter