Gestion d'une requete paramétrable

Version imprimable

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:

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

16/11/2017, 12h29
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.
16/11/2017, 12h50
Msysteme

je comprends mais c'est pratiquement la meme chose

Salut,

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

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+"%'"; ...

16/11/2017, 15h11
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 :(
16/11/2017, 15h47
ZenZiTone

Citation:

Envoyé par 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 :(

C'est le principe des filtres oui.. Je ne vois pas ce qui dérange là :koi:
16/11/2017, 19h47
Jaco67
Justement, avec ma solution tu n'as plus ce problème de "plusieurs where".
Si aucun filtre n'est activé, tu auras :
Code:

1 2 3 select ... from ... where 1=1
Si tu as un filtre :
Code:

1 2 3 4 select ... from ... where 1=1 and critere1 = ...
Si tu as deux filtres :
Code:

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

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:

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:

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)

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

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

17/11/2017, 23h00
Pol63

Citation:

Envoyé par 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