Effectuer une recherche dans base de donnée

**moonshine972** · 08/04/2011, 11h30

Bonjour,

J'aimerais que mon application puisse rechercher par "nom" ou "numéros" de téléphone dans ma base donnée MySql puis me les afficher.

j'ai trouvé ceci sur le net

La commande qui permet de faire cela

Code :

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

"SELECT * FROM matable WHERE nom LIKE + textBox1.Text + ";

Est ce vraiment cette ligne de commande car j'arrive pas a la faire fonctionner

Merci

**BuchsPa** · 08/04/2011, 11h38

Bonjour,

essaye de faire comme ça:

Code :

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

1
2
 
"SELECT * FROM matable WHERE nom LIKE '"+ textBox1.Text +"'";

normalement ça devrait marcher

EDIT:

pour sécuriser le tout c'est encore mieux de faire

Code :

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

1
2
 
"SELECT * FROM matable WHERE nom LIKE '"+ textBox1.Text.Replace("'","''") +"'";

**kheironn** · 08/04/2011, 11h40

Ce sujet aurais, plutôt, sa place dans une section SQL...

Avec la requête telle quelle, rien n'empêche les injections sql -> pbm de sécurité !

Sinon, le Like s'utilise avec un % (MySql ne doit pas être différent d'oracle ou sqlserver sur ce point).

Code :

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

select [tes champs] from [ta table] where [ton champ] like 'tavaleur'

où ta valeur doit être précédée de % ou suivie ou les deux, selon la partie constante connue.

Je m'explique :
Dans tototiti
toto% signifie que tu auras toutes les chaînes commençant par toto,
%titi toutes celles qui finnissent par titi
%toti% toute celles qui contiennent toti...

Si tu veux faire une ségrégation par nom exact, tu fais un

Code :

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

select [tes champs] from [ta table] where [ton champ] = 'tavaleur'

Ensuite ta requête doit rapporter un résultat sur le nom OU le N° tel ?
dans ce cas, il faut ajjouter une close OR :

Code :

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

select [tes champs] from [ta table] where nom = 'tavaleur' or telnum = 'tavaleur'

.

Ensuite, si tu postes ici, tu dois être en C# donc tu peux utiliser les MySqlParameters pour passer tes valeurs et non directement la lecture des txtBox !!!

P.S. like sans % revient à faire = ... aucun intérêt de ralentir la requête dans ce cas ! car Like est plus lent que = ...
En plus, il faut penser aux index de champs pour optimiser la recherche.

**ostenhard** · 08/04/2011, 11h40

Bonjour,

Normalement c'est très mal de faire ça !

Donc déjà un tour s'impose sur la FAQ et articles pour découvrir les requêtes paramétrées, la connexion à une base de données, ...

**kheironn** · 08/04/2011, 11h47

Envoyé par BuchsPa

Bonjour,

essaye de faire comme ça:

Code :

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

1
2
 
"SELECT * FROM matable WHERE nom LIKE '"+ textBox1.Text +"'";

normalement ça devrait marcher

EDIT:

pour sécuriser le tout c'est encore mieux de faire

Code :

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

1
2
 
"SELECT * FROM matable WHERE nom LIKE '"+ textBox1.Text.Replace("'","''") +"'";

Non BuchsPa !
la substitution de ' par '' est peut-être "moins pire", mais pas le top !

Requête paramétrée comme l'a dit mon camarade, ci-dessus !
Tu peux te permettre de te passer de sécurité pour des test, mais pas pour une VRAIE Appli !
Une bdd se doit d'être protégée, c'est le coeur de l'appli du point de vue client : Imagine amazone qui se fait hacker ses bases ???
tu récupères les N° de CB de tous les clients !!!
ou tu supprimes des infos primordiales pour leur commerce !!!

**BuchsPa** · 08/04/2011, 11h48

C'est déjà mieux que rien

**ostenhard** · 08/04/2011, 11h52

Envoyé par BuchsPa

C'est déjà mieux que rien

Non c'est le mal ! Il faut faire les choses correctement donc avec des requêtes paramétrées !

Car le cas fonctionne ici mais prenons un decimal qui aura une virgule ou un autre cas, la façon que tu proposes ne résoud pas d'avance les problèmes qui se poseront.

**Jean-Michel Ormes** · 08/04/2011, 11h55

Je plussoie entièrement les remarques de kheironn et ostenhard. Utilises les requêtes paramétrées.

**moonshine972** · 08/04/2011, 12h08

Effectiviement je suis en c#

voila mon 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
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
55
56
57
58
59
60
61
62
private void button3_Click(object sender, RoutedEventArgs e)
        {
            using (Home.wConnexion)
            {
                // Instancier l’objet Command
                using (MySqlCommand wCommand = Home.wConnexion.CreateCommand())
                {
                    // Définition de la requête
                    wCommand.CommandText = "SELECT 'nom' 'nfixe' FROM test WHERE nom = " + textBox6.Text +"  OR nfixe = " + textBox17.Text ;
 
                    // Lecture des résultats
                    using (MySqlDataReader wReader = wCommand.ExecuteReader())
                    {
                        // Remplir le textbox
                        while (wReader.Read())
                        {
                            string wItem = wReader["nom"].ToString();
                            string wItems = wReader["prenom"].ToString();
                            string wItem3 = wReader["message"].ToString();
                            string wItem4 = wReader["sexe"].ToString();
                            string wItem5 = wReader["dateentre"].ToString();
                            string wItem6 = wReader["adresse"].ToString();
                            string wItem7 = wReader["cp"].ToString();
                            string wItem8 = wReader["ville"].ToString();
                            string wItem9 = wReader["nserie"].ToString();
                            string wItem10 = wReader["model"].ToString();
                            string wItem11 = wReader["marque"].ToString();
                            string wItem12 = wReader["typepc"].ToString();
                            string wItem13 = wReader["nfixe"].ToString();
                            string wItem14 = wReader["nmobile"].ToString();
                            string wItem15 = wReader["mail"].ToString();
                            string wItem16 = wReader["datesortie"].ToString();
                            string wItem17 =wReader["dcomplementaire"].ToString();
                            string wItem18 = wReader["sauprobleme"].ToString();
 
 
                            comboBox2.Items.Add(wItem4);
                            textBox2.Text = (wItem);
                            textBox3.Text = (wItems);
                            textBox4.Text = (wItem3);
                            textBox5.Text = (wItem5);
                            textBox7.Text = (wItem6);
                            textBox8.Text = (wItem7);
                            textBox12.Text = (wItem8);
                            textBox9.Text = (wItem11);
                            textBox10.Text = (wItem10);
                            textBox11.Text = (wItem9);
                            comboBox1.Items.Add(wItem12);
                            textBox13.Text = (wItem13);
                            textBox15.Text = (wItem14);
                            textBox14.Text = (wItem15);
                            textBox16.Text = (wItem16);
                            textBox1.Text = (wItem17);
                            textBox18.Text = (wItem18);
 
 
 
                        }
                    }
                }
            }
        }

**moonshine972** · 08/04/2011, 12h44

Merci a tous pour votre aide

j'ai trouvé grace a vous

Pour la sécurité ce nest pas grave l'application sera utiliser en local mais jamais sur le net

j'ai fait ceci :

Code :

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

"SELECT * FROM hibis_t WHERE nom LIKE '" + textBox6.Text + "' OR nfixe LIKE '" + textBox17.Text +"'";

Maintenant quand je clique sue le bouton afficher une premiere fois puis jefface la textbox et que je reclique sur afficher j'ai ce message d'erreur :

L'exception System.InvalidOperationException n'a pas été gérée
Message=Connection must be valid and open.

Que dois je faire?

**kheironn** · 08/04/2011, 13h32

utilises = à la place de like...

pour ton nouveau problème, passes en débug sur ton code... le message est clair : ta connection doit être fermée.

perso, je ne fais pas de using sur les cnx. Je préfère gérer moi-même les open, close et dispose.

**ostenhard** · 08/04/2011, 14h21

Envoyé par moonshine972

Pour la sécurité ce nest pas grave l'application sera utiliser en local mais jamais sur le net

Oui mais tes utilisateurs, tu ne sais pas à l'avance ce qu'ils vont saisir et un utilisateur par définition ça fait n'importe quoi !

Donc libre à toi de mal faire les choses, tu es prévenu !

**calagan99** · 08/04/2011, 14h22

Envoyé par moonshine972

Pour la sécurité ce nest pas grave l'application sera utiliser en local mais jamais sur le net

Oui, mais cela doit être une habitude!
Tu as vite fait de prendre de mauvais plis qui sont compliqués à oublier...

Je me réjouis en tout cas de voir que la ligue de promotion de la requête paramétrée s'allonge

**moonshine972** · 08/04/2011, 14h28

ok je vais essayer de rectifier le tir

Quant a l'autre problème je rencontre un souci de connexion ouverte mais quand je la ferme et que j'appuie sur le bouton j'ai ce message:

Invalid attempt to Read when reader is closed.

**ostenhard** · 08/04/2011, 14h33

A quoi bon faire un While reader.Read() quand on ne lit qu'une seule ligne ?

**calagan99** · 08/04/2011, 14h34

Il faut que nous dise à quel endroit tu fermes ta connexion.
Montre nous ton code modifié.

**kheironn** · 08/04/2011, 14h36

ben oui, il ne faut pas fermer avant de lire...
Tu as déjà essayé de lire un livre fermé ? Je veux dire à part le résumé en fin de couverture ?
Ben, c'est pareil !

dans l'ordre, tu fais
- open
- execute
- read
- close
- dispose

remontre ton code modifié pour voir où est l'erreur.

**calagan99** · 08/04/2011, 14h46

@kheironn : c'est vendredi, je suis sûrement fatigué, mais tu vois où qu'il ferme sa connexion avant d'accèder au DataReader ?

**kheironn** · 08/04/2011, 14h51

Envoyé par calagan99

@kheironn : c'est vendredi, je suis sûrement fatigué, mais tu vois où qu'il ferme sa connexion avant d'accèder au DataReader ?

il a dit qu'il avait modifié des choses et vu le message d'erreur, c'est qu'un close est fait avant un read... d'où ma demande de reposter son ode en l'tat actuel pour confirmer mes dire et faire avancer le bleme...

je ne fais que supposer à partir de :

Envoyé par moonshine972

ok je vais essayer de rectifier le tir

Quant a l'autre problème je rencontre un souci de connexion ouverte mais quand je la ferme et que j'appuie sur le bouton j'ai ce message:

Invalid attempt to Read when reader is closed.

en l'occurence, dans le premier code ce ne devait pas être le cas...

humf, je n'aime vraiment pas les using pour l'ADO... je préfère rester maître de mes étapes de traitement DB...

**moonshine972** · 08/04/2011, 14h57

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
51
52
53
54
55
56
57
58
59
60
61
62
 
private void button3_Click(object sender, RoutedEventArgs e)
        {
                using (Home.wConnexion)
                {
                    // Instancier l’objet Command
                    using (MySqlCommand wCommand = Home.wConnexion.CreateCommand())
                    {
                        // Définition de la requête
                        wCommand.CommandText = "SELECT * FROM hibis_t WHERE nom = '" + textBox6.Text + "' OR nfixe = '" + textBox17.Text + "'";
 
                        // Lecture des résultats
                        using (MySqlDataReader wReader = wCommand.ExecuteReader())
                        {
                            // Remplir le textbox
                            while (wReader.Read())
                            {
                                string wItem = wReader["nom"].ToString();
                                string wItems = wReader["prenom"].ToString();
                                string wItem3 = wReader["message"].ToString();
                                string wItem4 = wReader["sexe"].ToString();
                                string wItem5 = wReader["dateentre"].ToString();
                                string wItem6 = wReader["adresse"].ToString();
                                string wItem7 = wReader["cp"].ToString();
                                string wItem8 = wReader["ville"].ToString();
                                string wItem9 = wReader["nserie"].ToString();
                                string wItem10 = wReader["model"].ToString();
                                string wItem11 = wReader["marque"].ToString();
                                string wItem12 = wReader["typepc"].ToString();
                                string wItem13 = wReader["nfixe"].ToString();
                                string wItem14 = wReader["nmobile"].ToString();
                                string wItem15 = wReader["mail"].ToString();
                                string wItem16 = wReader["datesortie"].ToString();
                                string wItem17 = wReader["dcomplementaire"].ToString();
                                string wItem18 = wReader["sauprobleme"].ToString();
 
 
                                comboBox2.Items.Add(wItem4);
                                textBox2.Text = (wItem);
                                textBox3.Text = (wItems);
                                textBox4.Text = (wItem3);
                                textBox5.Text = (wItem5);
                                textBox7.Text = (wItem6);
                                textBox8.Text = (wItem7);
                                textBox12.Text = (wItem8);
                                textBox9.Text = (wItem11);
                                textBox10.Text = (wItem10);
                                textBox11.Text = (wItem9);
                                comboBox1.Items.Add(wItem12);
                                textBox13.Text = (wItem13);
                                textBox15.Text = (wItem14);
                                textBox14.Text = (wItem15);
                                textBox16.Text = (wItem16);
                                textBox1.Text = (wItem17);
                                textBox18.Text = (wItem18);
 
                                Home.wConnexion.Close(); // j'ai rajouter ca d'ou j'obtiens ce message derreur
                            }
                        }
                    }
                }
            }

Effectuer une recherche dans base de donnée

C#

Discussions similaires

Partager

Partager