Variable non unique dans un traitement de requête suite à une boucle

**Limoen** · 03/02/2015, 23h14

Bonsoir tout le monde

,

C'est encore moi (à croire que je passe ma vie sur les forums... Presque

), suite à ce topic...
Alors voilà, j'ai une barre de recherche sur mon site web en ASP.NET, qui permet d'afficher dans un tableau le résultat de ce qu'à taper l'utilisateur. Mon code fonctionne quand l'utilisateur ne rentre qu'un seul mot, par contre dès qu'il veut en rentrer plusieurs, c'est la cata.

Je cherche depuis le début de l'aprem une solution: J'ai regardé s'il était possible de mettre plusieurs valeurs à un "LIKE" comme on peut le faire avec un "IN" mais nada.. Enfin on ne peut pas d'après ce que j'ai lu. Et remplacer mon "LIKE" par un "IN" changerait toute la donne... Voici mon code (en C#) pour le moment, qui se déroule lors du clique du bouton "Rechercher", une fois que l'utilisateur a tapé les mots recherchés:

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
Connexion = new SqlConnection("Data Source=**;Initial Catalog=**;Integrated Security=True");
                SqlCommand Recherche = new SqlCommand("Select UserID, UserNom, UserPrenom FROM User WHERE UserNom LIKE @motclef OR UserPrenom LIKE @motclef", Connexion);
                Connexion.Open();
 
                String[] words = TextBoxSearch.Text.Split(' ');
 
 
                foreach (String word in words)
               {
                    Recherche.Parameters.Add("motclef", SqlDbType.NVarChar).Value = '%' + word + '%';
                }
 
                SqlDataReader Lecture = Recherche.ExecuteReader();
                Gridview.DataSource = Lecture;
                Gridview.DataBind();
                Lecture.Close();

Mon Gridview correspond donc à un tableau récapitulatif de ma recherche, et le TextBoxSearch, aux mots rentrés par l'utilisateur. Quand je teste avec un seul mot, pas d'erreur. Par contre, quand je teste avec deux, j'ai l'erreur suivante:
"Le nom de variable '@motclef' a déjà été déclaré. Les noms de variables doivent être uniques dans un traitement de requêtes ou une procédure stockée."
Ce qui est normal puisque dans ma boucle ma variable est "créée" plusieurs fois. Or, je n'ai pas d'autres idées de comment faire... J'ai tenté avec une boucle While mais idem.

Comment je pourrai stocker cette variable et la reprendre ensuite dans ma requête? Je m'y prend peut être pas comme il faut...

**Dokho1000** · 04/02/2015, 10h39

Code :

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

1
2
3
4
5
6
7
8
9
 
int cpt = 0;
string sql = "SELECT....WHERE 1=1 ";
 foreach (String word in words)
               {
                    string varname = "motclef" + (++cpt);
                    Recherche.Parameters.Add(varname , SqlDbType.NVarChar).Value = '%' + word + '%';
                    sql += String.Format(" OR {0} like {1}","tonfieldname",varname); 
                }

Voici un exemple...

**Limoen** · 04/02/2015, 11h07

Bonjour Dokho1000,

Merci (encore

) pour ton aide. Le soucis est qu'il me demande de déclarer ma variable scalaire @motclef, comme je ne l'utilise plus dans mon "Recherche.Parameters.Add". J'ai tenté de le mettre en dure, sans passer par varname mais rien n'y fait.
De puis, je n'arrive pas à comprendre la requête SQL, enfin du moins l'utilité (même si je me doute que c'est en rapport avec le compteur...), ni même savoir quand est-ce qu'on l'a "lance"...

**Dokho1000** · 04/02/2015, 11h27

dans mon script j'ai oublié le "@" devant {1}
Tu la lances apres le foreach...

**Limoen** · 04/02/2015, 11h53

Envoyé par Dokho1000

dans mon script j'ai oublié le "@" devant {1}
Tu la lances apres le foreach...

Ok, je viens de rajouter mon @ devant le {1} de LIKE dans ma variable "sql". Il n'y a plus l'erreur du @motclef. Par contre, j'ai cette erreur:
"Le format de la chaîne d'entrée est incorrect.".

Pour la variable sql, ne vaut-il pas mieux que je la mette en SqlCommand plutôt que String? Car comment la lancer ensuite?
Et pour le "fieldname" (ligne 8 de ton code), je le remplace par "TextBoxSearch.Text"?

Excuse moi de te poser toutes ces questions, j'essaye de comprendre

...

**Dokho1000** · 04/02/2015, 12h37

Met ton code modifié...

**Limoen** · 04/02/2015, 13h56

Oui, pardon, voici le code:

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
Connexion = new SqlConnection("Data Source=**;Initial Catalog=**;Integrated Security=True");
SqlCommand Recherche = new SqlCommand("Select UserID, UserNom, UserPrenom FROM User WHERE UserNom LIKE @motclef OR UserPrenom LIKE @motclef", Connexion);
Connexion.Open();
 
String[] words = TextBoxSearch.Text.Split(' ');
int cpt =  0;
String sql = "SELECT UserID, UserNom, UserPrenom FROM User WHERE 1=1";
 
foreach (String word in words)
    {
        String varname = "motclef" + (++cpt);
        Recherche.Parameters.Add(varname, SqlDbType.NVarChar).Value = '%' + word + '%';
        sql += String.Format("OR {0} LIKE {@1}",TextBoxSearch.Text, varname);
     }
SqlDataReader Lecture = Recherche.ExecuteReader();
Gridview.DataSource = Lecture;
Gridview.DataBind();
Lecture.Close();

**Dokho1000** · 04/02/2015, 15h33

le TextBoxSearch.Text représente un champ dans ta base ???

**Limoen** · 04/02/2015, 15h49

Envoyé par Dokho1000

le TextBoxSearch.Text représente un champ dans ta base ???

Le TextBoxSearch.Text représente l'endroit où les utilisateurs écrivent les mots clefs recherchés...
En fait je n'ai aucune idée de ce qu'il faut mettre comme champ, car si c'est par rapport à ma base, il faudrait que je mette "UserNom" et "UserPrenom"?

**Dokho1000** · 04/02/2015, 16h38

alors le script devrait plutot être comme ceci :

Code :

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

1
2
3
4
5
6
7
 
foreach (String word in words)
    {
        String varname = "motclef" + (++cpt);
        Recherche.Parameters.Add(varname, SqlDbType.NVarChar).Value = '%' + word + '%';
        sql += String.Format("OR {0} LIKE {@2} OR {1} LIKE {@2}","UserNom", "UserPrenom", varname);
     }

Ceci dit, ce sera une requête pas performante du tout...

**Limoen** · 04/02/2015, 16h43

En fait, il faut que je définisse tous les champs qui sont susceptibles de "ressembler" à ce qu'à rentré l'utilisateur?
C'est ce que font par exemple les développeurs pour les barres de recherche dans les forums?

Parce qu'en fait mon but est à peu près le même... Comment faire pour que ma requête soit plus performante?

EDIT: J'ai toujours la même erreur de format....

**Dokho1000** · 04/02/2015, 17h50

Sorry les 2 @ doivent être devant l'accolade...

Faire une store proc serait mieux...

**Limoen** · 05/02/2015, 09h21

Je te remercie encore pour ton aide Dokho1000,
As-tu une idée du coup de comment faire avec Visual Studio pour faire une procédure stockée?

EDIT: Même comme cela, ils me demandent encore que la variable scalaire "@motclef" soit déclarée...

**Dokho1000** · 05/02/2015, 11h52

J'utilise le studio management de MS SQL...

Essaye plutot :

Code :

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

1
2
3
4
5
 
var param = Recherche.CreateParameter();
param.ParameterName = varname;
param.Value = '%' + word + '%';
Recherche.Parameters.Add(param );

**Limoen** · 05/02/2015, 12h12

En fait, je vois comment faire avec studio management pour ajouter une procédure stockée, ça y a pas de soucis. Par contre, dans ce cas-là, je n'ai aucune idée de quelle requête écrire (la mienne est-elle bonne?) ni comment l'aborder dans mon code C# (et comment y accéder???). Pourquoi cela règlerait le soucis puisque ce sera toujours le même problème avec cette variable qui se répète...?

Je viens également de tester ton code en remplaçant param par "motclef" mais rien n'y fait, j'ai toujours la même erreur...

Code :

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

1
2
3
4
5
6
7
8
9
 
foreach (String word in words)
               {
                   var "motclef" = Recherche.CreateParameter();
                   "motclef".ParameterName = varname;
                   "motclef".Value = '%' + word + '%';
                   Recherche.Parameters.Add(motclef);
                   sql += String.Format("OR {0} LIKE @{2} OR {1} LIKE @{2}","UserNom", "UserPrenom", varname);
               }

Puisque dans ma requête "Recherche" se trouve ce paramètre, et que celui-ci correspond à ce que contient la TextBoxSearch...

**Dokho1000** · 05/02/2015, 13h00

Peux tu mettre tout ton code car je ne comprend pas pq ca ne fonctionnerait pas...

Pour ce qui est de la store proc j'irais me renseigner dans un autre forum pour voir quel technique est la meilleur.
Moi j'aurais tenté de créer une table qui contient juste l'entité, la pk et le text afin d'effectuer ma recherche plus rapidemnt...

ca veux dire qu'a chaque fois que tu crées ou édite un record tu ne dois pas oublier d’éditer cette table aussi...

**Limoen** · 05/02/2015, 13h55

Voici tout mon code qui se déroule lors du clique de mon bouton:

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
 
public void RechercherUser()
        {
            SqlConnection Connexion = null;
            try
            {
 
                Connexion = new SqlConnection("Data Source=**;Initial Catalog=**;Integrated Security=True");
                SqlCommand Recherche = new SqlCommand("Select UserID, UserNom, UserPrenom FROM User WHERE UserNom LIKE @motclef OR UserPrenom LIKE @motclef", Connexion);
                Connexion.Open();
 
 
                String[] words = TextBoxSearch.Text.Split(' ');
                int cpt =  0;  // Du coup, ici il n'y a plus besoin du compteur?
                String sql = "SELECT UserID, UserNom, UserPrenom FROM User WHERE 1=1";
 
 
               foreach (String word in words)
               {
                   var "motclef" = Recherche.CreateParameter();
                   "motclef".ParameterName = varname;
                   "motclef".Value = '%' + word + '%';
                   Recherche.Parameters.Add(motclef);
                   sql += String.Format("OR {0} LIKE @{2} OR {1} LIKE @{2}","UserNom", "UserPrenom", varname);
               }
 
                SqlDataReader Lecture = Recherche.ExecuteReader();
                Gridview.DataSource = Lecture;
                Gridview.DataBind();
                Lecture.Close();
            }
            catch (Exception UneException)
            {
                LabelError.Text = UneException.Message;
            }
            finally
            {
                if ((Connexion != null))
                {
                    if (ConnexionState == Connexion.Open)
                    {
                        Connexion.Close();
                    }
                }
            }
        }
 protected void ImageButtonSearch_Click(object sender, ImageClickEventArgs e)
        {
            RechercherUser();
        }

Et donc toujours la même erreur: " La variable scalaire "@motclef" doit être déclarée. "

**Dokho1000** · 05/02/2015, 16h58

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
Connexion = new SqlConnection("Data Source=**;Initial Catalog=**;Integrated Security=True");
                SqlCommand Recherche = Connexion.CreateCommand();
                Connexion.Open();
 
 
                String[] words = TextBoxSearch.Text.Split(' ');
                int cpt =  0;  // Du coup, ici il n'y a plus besoin du compteur?
                String sql = "SELECT UserID, UserNom, UserPrenom FROM User WHERE 1=1";
 
 
               foreach (String word in words)
               {
                   var motclef = Recherche.CreateParameter();
                   motclef.ParameterName = varname;
                   motclef.Value = '%' + word + '%';
                   Recherche.Parameters.Add(motclef);
                   sql += String.Format("OR {0} LIKE @{2} OR {1} LIKE @{2}","UserNom", "UserPrenom", varname);
               }
 Recherche.CommentText = sql;
                SqlDataReader Lecture = Recherche.ExecuteReader();
                Gridview.DataSource = Lecture;
                Gridview.DataBind();
                Lecture.Close();

Voilà, vérifie la requete sql juste avant ton SqlDataReader afin de t'assurer qu'elle bien formée...

A+

**Limoen** · 05/02/2015, 17h13

J'ai recopié exactement le même code, j'ai juste changé:

Code :

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

 String varname = motclef.ParameterName;

Car sinon il ne reconnaissait pas la variable "varname".

Mais j'ai toujours le même soucis:
"La variable scalaire "@" doit être déclarée.".
C'est le @ devant les accolades qui pose soucis...

J'arrive complètement à comprendre ce code. Mais le soucis c'est le lien entre ma textbox et la requête je pense...

Enfin, je veux pas plus t'embêter que ça, et je te remercie de m'avoir déjà aidé jusque là...