C# MySQL Requette WHERE ne fonctionne pas

Version imprimable

Bonjour,

J'ai un petit problème assé étrange...

J'ai crée une fonction pour vérifier si un nom existe déjà dans ma Bdd;
Tout fonctionnais bien, j'ai donc continuer à prog par si par la et la je m'aperçois que ma fonction ne me retourne plus true, quant je lui demande de vérifier un nom qui existe déjà ds ma bdd, alors que je n'ai rien toucher...

Voici ma requette
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 public bool VerifNomUtilisateur() { string query = "SELECT * FROM dtd_utilisateur WHERE dtd_nom='" + _nomUser + "'"; bool cnt = false; //Ont ouvre la connection if (this.OpenConnection() == true) { //Create Mysql Command MySqlCommand cmd = new MySqlCommand(query, connection); MySqlDataReader reader = cmd.ExecuteReader(); reader.Read(); if(reader.HasRows) { cnt = true; Debug.Log("Nom Utilisateur Existe Deja"); } connection.Close(); return cnt; } else { cnt = false; return cnt; } }
Et Voici la fonction qui l'appel
Code:

1 2 3 4 private bool Verif_NomExisteDeja() { return VerifNomUtilisateur(); }
celle-ci est situer dans un autre script.

Je travail sur Unity 3D D'ou les Debug Log, et ma query un peu spéciale peu être !?.

Je me souvient d'avoir ajouter un StringBuilder entre le moment ou tout allai bien et maintenant.
Possible que ce soit ça même si ça m’étonnerai ??

Je précise que lorsque je tape manuellement le nom à rechercher il me retourne toujours false (Le Nom n'a pas de doublon)...
Cela me désespère un peu en sachant que je n'ai aucun message d'erreur.

Merci d'avance à ceux qui pourront me venir en aide, et je vous souhaite une bonne nuitée, il ce fait déjà tard :)

16/04/2012, 08h37
GuruuMeditation

Est- ce que le OpenConnection renvoie bien true?
16/04/2012, 09h58
tomlev

Essaie de déboguer pas à pas en vérifiant l'état des variables, ça te montrera sûrement où est le problème...
16/04/2012, 11h05
Patmat44
Moi ce que je vois c'est que tu n'appelles jamais la fonction

Code:

connection.open();

car ce code :
Code:

1 2 //Ont ouvre la connection if (this.OpenConnection() == true)
n'ouvre pas la connexion comme tu l'écris en commentaire mais vérifie que la connexion est ouverte
16/04/2012, 11h15
merscos

Je pense que le Open connexion me renvoie bien true, car il l'enregistre bien mon nouveau compte, sans vérifier si le nom existe déjà.

J'ai essayer de mettre une bêtise Âpres WHERE et je reçois une erreur de syntaxe, Et si juste après L'erreur j’essaie à nouveau de lancer la requette, sa me dit erreur, la connexion est déjà ouverte ce qui prouve bien que ma connexion renvoie true.

Le debug ne fonctionne pas, je pense que cela est du faite que la solution est générer par Unity;

J'ai essayer de mettre le reader dans un using(){ }. même résultat, vérification du passe renvoie False;

Citation:

Envoyé par merscos

Et si juste après L'erreur j’essaie à nouveau de lancer la requette, sa me dit erreur, la connexion est déjà ouverte ce qui prouve bien que ma connexion renvoie true

c'est normale car si tu as eu une erreur avant,

Code:

connection.Close();

alors, il ne ferme pas ta connexion, dont ta connection reste ouverte.

ce qu'il faut faire, c'est ouvrir ta connexion au début de ta fonction ou ya ta requête sql et la fermer à la fin de ta fonction. Comme ça t'as plus besoin de vérifier si ta connexion est ouverte en plus :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 public bool VerifNomUtilisateur() { string query = "SELECT * FROM dtd_utilisateur WHERE dtd_nom='" + _nomUser + "'"; bool cnt = false; //Ont ouvre la connection if (connection.State != ConnectionState.Open) connection.Open(); //Create Mysql Command MySqlCommand cmd = new MySqlCommand(query, connection); MySqlDataReader reader = cmd.ExecuteReader(); reader.Read(); if(reader.HasRows) { cnt = true; Debug.Log("Nom Utilisateur Existe Deja"); } connection.Close(); return cnt; }

Bonjour ,

je rejoint l'avis de Patmat44, on ne vois pas le

Code:

connection.Open();

.

dans la mesure ou vous n'avez pas de debug, vous pouvez utiliser ça et nous dire si ce que ça affiche? pour voir justement si vous accédez a la boucle IF;

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
public bool VerifNomUtilisateur()
    {
        string query = "SELECT * FROM dtd_utilisateur WHERE dtd_nom='" + _nomUser + "'";
        bool cnt = false;
 
        //Ont ouvre la connection
        if (this.OpenConnection() == true) 
        {
            MessageBox.Show("true");
            connection.Close();
            return cnt;
        } 
 
        else
        {
            MessageBox.Show("false");
            cnt = false;
            return cnt;
        }
    }

16/04/2012, 13h46
olibara

Bonjour plusieurs petite choses

1- Que contient exactement : _nomUser
2- Essaye d'eviter les concaténation de chaine comme tu le fais particulierement pour une requete sql, utilise plutot les requetes parametrées
3- Quand tu execute ta requete en ligne de commande en récupérant le contenu de ton string query : quel est le résultat ?
16/04/2012, 19h26
merscos

Merci pour votre aide,

Patmat44

Code:

if (connection.State != ConnectionState.Open) connection.Open();

ConnectionState : Affiche Erreur

labib23dz
Je reçois false (Non il renvoie true, j'ai oublier de lancer wamp pour le test : / )

olibara

_nomUser Récupère La valeur String D'un autre Script
Comment faire ta suggestion ? il faudrait que je fasse dtd_nom=@USERNAME
Et que j'ajoute un paramètre @USERNAME, _usernom) En gros ?

Pour en revenir au test effectuer qui me renvoie false, je vais essayer en appelant la connections; Je vous tien au jus (Mais cela est étrange sachant que j'utilise le même procéder pour l’écriture du compte, et celui ci fonctionne nickel).

Test OpenConnection(); Placer avant le if co... == true; me renvoie connections déjà établie

Salut

Voici un exemple d'ajout de parametres

Attention en MySql le prefixe est ? plutot que @

Code:

1
2
3
4
5
6
7
8
9
      MySqlCommand MyCmd = new MySqlCommand();
      string strSQL = "INSERT INTO statgroup( "
                        + " Name )"
                        + " VALUES ( "
                        + " ?Name )";
 
 
      MyCmd.CommandText = strSQL;
      AddParams(MyCmd);

Code:

1
2
3
4
5
6
    // ************************************************************
    private void AddParams(MySqlCommand MyCmd)
    {
      MyCmd.Parameters.AddWithValue("?pkId", this.pkId);
      MyCmd.Parameters.AddWithValue("?Name", ToolsCvt.AddParamString(this.Name));
    }

16/04/2012, 20h00
tomlev

Citation:

Envoyé par olibara

Attention en MySql le prefixe est ? plutot que @

Euh, je sais pas si ça marche avec '?', mais moi j'ai toujours utilisé '@' en tous cas...

Je viens d'essayer ça , toujours le même problème, je reçois que le debuglog "avant read"

je vais essayer ta façon de faire.

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
    public bool VerifNomUtilisateur()
    {
        bool found = false;
 
        //On ouvre la connection
        if (this.OpenConnection() == true)
        {
            string query = "SELECT * FROM dtd_utilisateur WHERE dtd_nom=@nom";
            MySqlCommand cmd = new MySqlCommand(query, connection);
            cmd.Parameters.AddWithValue("@nom", "merscos");
            cmd.ExecuteNonQuery();
 
            using (MySqlDataReader reader = cmd.ExecuteReader())
            {
                Debug.Log("avant read");
 
                reader.Read();
 
                if (reader.Read())
                {
                    found = true;
                    Debug.Log("Nom Utilisateur Existe Deja");
                }
            }
 
            connection.Close();
        }
        else
        {
            Debug.Log("La connexion a la base a échoué");
        }
 
        return found;

16/04/2012, 20h09
tomlev

Ce serait sans doute plus facile de trouver le problème avec un try/catch qui log l'erreur...
16/04/2012, 20h15
merscos

Je ne sais pas faire...

J'ai essayer la méthode de olibara sa me fait la même chose :calim2:
16/04/2012, 21h10
olibara

Citation:

Envoyé par tomlev

Euh, je sais pas si ça marche avec '?', mais moi j'ai toujours utilisé '@' en tous cas...

Salut Tomlev

En Mysql jusqu'a la version 5.(quelque chose) le préfixe est ?
Ensuite je pense qu'ils ont ils ont compatibilisé et je pense que maintenant les deux doivent passer
En tout cas avec ma version 6.3.6 j'utilise encore ?

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
    public void VerifNomUtilisateur()
    {
        bool found = false;
 
        connection.Open();
 
        //On ouvre la connection
        using (connection)
        {
            using (MySqlCommand cmd = connection.CreateCommand()) 
            {
                cmd.CommandText = "SELECT COUNT(*) FROM dtd_utilisateur WHERE dtd_nom=@nom";
                cmd.Parameters.AddWithValue("@nom", _nomUser);
 
                cmd.ExecuteNonQuery();
 
                using (MySqlDataReader reader = cmd.ExecuteReader()) 
                {
                    if (reader.HasRows)
                    {
                        Debug.Log("Verification Nom Utilisateur Ok");
                        found = true;
                        connection.Close();
                    }
                    else
                    {
                        Debug.Log("Verification Nom Utilisateur Echouer");
                        found = false;
                        connection.Close();
                    }
                }
            }
        }
    }

Avec ce code je reçois Verification Nom Utilisateur Ok meme si j'entre un utilisateur qui existe

Citation:

Envoyé par merscos

Je ne sais pas faire

Tu sais pas faire un try/catch ? bah va falloir apprendre, sinon tu pourras jamais gérer les erreurs proprement... C'est pas bien compliqué :
Code:

1 2 3 4 5 6 7 8 9 try { // ton code // ... } catch (Exception ex) { Debug.Log("Erreur : " + ex); }
Cela dit, je suis pas sûr que ça va aider dans ton cas précis...

Citation:

Envoyé par merscos

Avec ce code je reçois Verification Nom Utilisateur Ok meme si j'entre un utilisateur qui existe

C'est normal, tu fais un SELECT COUNT(*), donc ça renvoie toujours quelque chose, donc HasRows est toujours true. Pour récupérer une valeur seule, il faut faire ExecuteScalar et non ExecuteReader, et tester la valeur retournée
Code:

1 2 3 4 5 6 7 8 9 10 11 int count = Convert.ToInt32(cmd.ExecuteScalar()); if (count > 0) { Debug.Log("Verification Nom Utilisateur Ok"); found = true; } else { Debug.Log("Verification Nom Utilisateur Echouer"); found = false; }
D'autre part, tu n'as pas besoin du connection.Close() puisque tu as un bloc using qui le gère déjà

17/04/2012, 09h22
Bluedeep

Citation:

Envoyé par tomlev

Euh, je sais pas si ça marche avec '?', mais moi j'ai toujours utilisé '@' en tous cas...

"@" marche avec SqlServer et PostGreSql
':' pour Oracle

Pour MySql, je n'en ai pas la moindre idée.

Merci tomlev

J'ai donc un probleme Avec le drowin un truc comme ça, mon system.data etait à changer un bug corriger, je 'nai plus qu'a coriger un petit probleme de Object Reference not set to ... à la ligne 69-connection.Open();

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
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
public class Connection_BDD : MonoBehaviour
{
 
    public MySqlConnection connection;
    private string server;
    private string database;
    private string uid;
    private string password;
 
    public string _nomUser;
    private string _passUser;
 
    //private int count = 0;
 
    #region Getter Setter
    public string GetNomUtilisateur
    {
        get { return _nomUser; }
        set { _nomUser = value; }
    }
    public string GetPassUtilisateur
    {
        get { return _passUser; }
        set { _passUser = value; }
    }
    #endregion
 
    //Constructor
    public Connection_BDD()
    {
        Initialize();
    }
 
    private void Start()
    {
        //_nomUser = "mersc77os";
 
        CloseConnection();
    }
 
 
    public void OnGUI() {
 
        if (GUI.Button(new Rect(50, 50, 50, 50), "ff")) {
 
            VerifNomUtilisateur();
        }
 
    }
    //Initialize values
    private void Initialize()
    {
        server = "localhost";
        database = "dodgethedead";
        uid = "root";
        password = "";
        string connectionString;
        connectionString = "SERVER=" + server + ";" + "DATABASE=" +
        database + ";" + "UID=" + uid + ";" + "PASSWORD=" + password + ";";
 
        connection = new MySqlConnection(connectionString);
    }
 
    //open connection to database
    private bool OpenConnection()
    {
        try
        {
            connection.Open();
            return true;
        }
        catch (MySqlException ex)
        {
            switch (ex.Number)
            {
                case 0:
                    Debug.Log("Cannot connect to server.  Contact administrator");
                    break;
 
                case 1045:
                    Debug.Log("Invalid username/password, please try again");
                    break;
            }
            return false;
        }
    }
 
    //Close connection
    private bool CloseConnection()
    {
        try
        {
            connection.Close();
            return true;
        }
        catch (MySqlException ex)
        {
            Debug.Log(ex.Message);
            return false;
        }
    }
 
    public void VerifNomUtilisateur()
    {
        //bool found = false;
 
        //MySqlConnection sqlConnection = new MySqlConnection();
        //sqlConnection.ConnectionString = "Server=" + server + ";Database=" + database + ";Uid=" + uid + ";Pwd=" + password + ";";
        //sqlConnection.Open();
 
        //On ouvre la connection
        if (this.OpenConnection() == true)
        {
            //using (MySqlCommand cmd = connection.CreateCommand())
            //{
                string query = "SELECT * FROM dtd_utilisateur WHERE dtd_nom=@nom";
                MySqlCommand cmd = new MySqlCommand(query, connection);
                cmd.Parameters.AddWithValue("@nom", _nomUser);
 
                //cmd.ExecuteNonQuery();
 
                int count = Convert.ToInt32(cmd.ExecuteScalar());
 
 
 
                    if (count > 0)
                    {
                        Debug.Log("Verification Nom Utilisateur Ok");
                        //found = true;
                        //sqlConnection.Close();
                    }
                    else
                    {
                        Debug.Log("Verification Nom Utilisateur Echouer");
                        //found = false;
                        //sqlConnection.Close();
                    }
                }
            //}
        }
 
    public void InsertNouvelleUtilisateur()
    {
        string query = "INSERT INTO dtd_utilisateur SET dtd_nom=' " + _nomUser + " ', dtd_password='" + _passUser + "'";
 
        //Open connection
        if (this.OpenConnection() == true)
        {
            //create mysql command
            MySqlCommand cmd = new MySqlCommand();
            //Assign the query using CommandText
            cmd.CommandText = query;
            //Assign the connection using Connection
            cmd.Connection = connection;
 
            //Execute query
            cmd.ExecuteNonQuery();
 
            //close connection
            this.CloseConnection();
        }
    }

Je corrigerai le reste de mon code qu'en tout sera fonctionnel ;)

17/04/2012, 14h41
merscos

Bon ben rebelote ... par contre plus du tout d'erreur, incroyable que ça ne fonctionne pas alors que j'ai réussi à le faire sur mon ancien projet..

Même en copiant collant cette ancien projet, ça ne fonctionne pas :aie:

Je pense qu'il serai préférable qu"ont m'aide via team viewer, si l'ont trouve la solution ensemble, je posterai la solution ici

PROBLÈME RÉSOLUT, erreur D'inattention.... Veuillez m'en excuser, programmé jusqu’à des 4 heur du matin mène qu'a des erreurs bête

Alors :
j'ai tout simplement laisser un espace

Code:

INSERT INTO dtd_utilisateur SET dtd_nom=' " + _nomUser + " ',

Sur ma requête d’insertion... :?
Honte à moi, sur ma bdd je ne voyais pas les espaces, cela ne m’étant jamais arrivé, bref ce topic m’aura appris plusieurs choses, je vous remercie tous pour votre aides et vos astuces, je vous souhaite bonne continuation à tous
18/04/2012, 22h48
olibara

Une requete parametrée t'aurais sans doute evité ce genre de desagrément

Et de toute façons j'estime que les concaténations de chaine a rallonge comme tu le fais
String truc="mon bout de machin "+" ' "+unbingz+" ' je ne sais pas quoi" + i+ etc ;

Sont des epouvantails a la compréhension et des horreurs de syntaxes et d'interpretation pour le compilateur

Je prefere de loin un string.Format
Ou un stringbuilder
Ou dans ton cas l'usage de parametres
18/04/2012, 23h32
tomlev

Citation:

Envoyé par olibara

~~Je prefere de loin un string.Format~~
~~Ou un stringbuilder~~
Ou dans ton cas l'usage de parametres

:ccool: