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

Version imprimable

Bonsoir tout le monde :) ,

C'est encore moi (à croire que je passe ma vie sur les forums... Presque :roll:), 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:

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

Code:

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

04/02/2015, 11h07
Limoen

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"...
04/02/2015, 11h27
Dokho1000

dans mon script j'ai oublié le "@" devant {1}
Tu la lances apres le foreach...
04/02/2015, 11h53
Limoen

Citation:

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 :? :( ...
04/02/2015, 12h37
Dokho1000

Met ton code modifié...

Oui, pardon, voici le code:

Code:

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

04/02/2015, 15h33
Dokho1000

le TextBoxSearch.Text représente un champ dans ta base ???
04/02/2015, 15h49
Limoen

Citation:

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"?

alors le script devrait plutot être comme ceci :

Code:

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

04/02/2015, 16h43
Limoen

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? 8O
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....
04/02/2015, 17h50
Dokho1000

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

Faire une store proc serait mieux...
05/02/2015, 09h21
Limoen

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

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

Essaye plutot :

Code:

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

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:

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

05/02/2015, 13h00
Dokho1000

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

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

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

Code:

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+

05/02/2015, 17h13
Limoen

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

Code:

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à...
06/02/2015, 09h37
Dokho1000

C''est p-e le string.format qui pose problème. essaye en concaténant la chaine à la place ?
Quel resultat obtient tu dans le Recherche.CommandeText ?
06/02/2015, 10h58
Limoen

Quand je lance l'appli, Resultat.CommandText vaut "SELECT UserNom, UserPrenom FROM User WHERE 1=1 OR UserNom LIKE @ OR UserPrenom LIKE @;"
Du coup, c'est vraiment le fait qu'il prend pas en valeur les éléments de la textbox... Donc un soucis au niveau du "varname" je pense... Alors que le tableau "words" rassemble bien les "word" de ce TextBox...

Je ne peux pas concaténer la chaine directement, sinon cela me souligne mon "UserNom", "UserPrenom" et varname avec comme erreur: " ; attendue " , mais quand je change les virgules en points-virgules (j'ai testé au cas où même si je trouve ça bizarre), cela me fait l'erreur: "Seule une assignation, un appel,..., peuvent être utilisés comme instruction". De plus, si je met un point d'arrêt au niveau du Recherche.CommandText juste après ce changement, cela ne s'arrête même plus et me met directement l'erreur: " La variable scalaire "@" doit être déclarée. " ...
06/02/2015, 12h05
Dokho1000

La seul explication est que varname est vide. Je ne vois plus dans ton code l'initialisation de la variable avec le compteur...
06/02/2015, 12h15
Limoen

Citation:

Envoyé par Dokho1000

La seul explication est que varname est vide. Je ne vois plus dans ton code l'initialisation de la variable avec le compteur...

Justement, j'ai posé la question un peu plus haut de ce qu'on faisait du compteur dans le code car je ne savais plus où l'initialiser, et du coup je ne l'utilisais pas dans le dernier code...

2 pièce(s) jointe(s)

Bon voici un code écrit à la va vite mais testé (même si je pense qu'une procédure stockée serai plus appropriée) :
Requête complète de la table :
Pièce jointe 168265

Code :

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
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
            using (var connection = new System.Data.SqlClient.SqlConnection(
                @"Data Source=(localdb)\v11.0;Initial Catalog=VetOffice;Integrated Security=True;"))
            {
                using (var command = new System.Data.SqlClient.SqlCommand())
                {
                    command.Connection = connection;
                    command.CommandType = System.Data.CommandType.Text;
                    string textToSearch = "Cat Dar Joe";
                    //On construit la requête :
                    var builder = new System.Text.StringBuilder();
                    builder.Append("SELECT * FROM dbo.Patients WHERE 1=1 AND ");
                    var words = textToSearch.Split(' ');
                    int indexWord = 0;
                    foreach (var word in words)
                    {
                        if (indexWord != 0)
                        {
                            builder.Append(" OR ");
                        }
                        builder.AppendFormat(" Name LIKE @{0}{1} OR AnimalType_Species LIKE @{0}{1} ", "word", indexWord);
                        indexWord++;
                    }
 
                    command.CommandText = builder.ToString();
 
                    //On ajoute les paramètres :
                    for (int i = 0; i < indexWord; i++)
                    {
                        var parameterName = "word" + i;
                        command.Parameters.Add(parameterName, System.Data.SqlDbType.VarChar).Value = "%" + words[i] + "%";
                    }
 
                    try
                    {
                        connection.Open();
                        var dataReader = command.ExecuteReader();
 
                        while(dataReader.Read())
                        {
                            Console.WriteLine("{0} - {1} | {2}", dataReader[dataReader.GetOrdinal("Id")],
                                dataReader[dataReader.GetOrdinal("Name")], dataReader[dataReader.GetOrdinal("AnimalType_Species")]);
                        }
                    }
                    catch (Exception ex)
                    {
                        Console.BackgroundColor = ConsoleColor.Red;
                        Console.WriteLine(ex.Message);
                    }
                    finally
                    {
                        connection.Close();
                        Console.ResetColor();
                    }
                }

Il faut bien faire attention à mettre des espaces au début et à la fin des append et appendFormat pour éviter de mal concaténer les chaines (éviter que "AND" et "Name" se retrouve comme ceci : "ANDName") .

On a retrouvé :

les chats qui contiennent "Cat" dans la colonne "AnimalType_Species"
"Darwin" et "Radar" qui contiennent "Dar" dans la colonne Name
"JoeFish" qui contient "Joe" dans la colonne Name

Pièce jointe 168266

06/02/2015, 12h37
Dokho1000

Tant mieux si ca marche, attention stringBuilder est intéressant à partir du moment ou il y a un grand nombre de concaténations (centaines voir milliers) sinon une concaténation simple est plus indiqué...
06/02/2015, 16h47
Limoen

Je vous remercie tous les deux pour votre aide et de m'avoir aidé jusque là.

Je cherchais vraiment à comprendre ce code, comment il se déroulait et pourquoi... Merci, car j'ai réussi à comprendre chaque ligne. J'y ai passé l'après-midi, mais je suis parvenue à faire "marcher" le code.

Mon but maintenant est de passer par une procédure stockée. Je vous avoue que je suis encore dans le flou. Si jamais vous connaissez des tutos ou autre, n'hésitez pas... J'ai cherché des exemples sur le net par rapport à mon cas mais j'avoue pour le moment ne pas trouver.

Bref, encore un GRAND merci à vous deux!