Problème d'authentification c#

Version imprimable

Salut à tous,

Voilà je souhaite avoir une page de connexion.
J'ai réalisé ce code pour tester si le couple (login,mdp) existe dans ma base:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
 
 string sql = "Select COUNT(idclient) from clients where pseudo =" + login.Text + " and password =" + mdp.Text;
            SqlConnection conn = new SqlConnection(GetConnectionString());
            SqlCommand cmd = new SqlCommand(sql, conn);
            conn.Open();
            int compteur = (int)cmd.ExecuteScalar();
            conn.Close();
            if (compteur == 0)
            {
                Label4.Text = "L'identifiant et/ou le mot de passe est invalide";
            }
            else
            {
                string sql2 = "Select idclient from clients where pseudo =" + login.Text + " and password =" +  mdp.Text;
                SqlConnection conn2 = new SqlConnection(GetConnectionString());
                SqlCommand cmd2 = new SqlCommand(sql2, conn2);
                conn.Open();
                int idclient = (int)cmd.ExecuteScalar();
                conn.Close();
                Session["idclient"] = idclient;
                Response.Redirect("Accueil.aspx");
            }

Il me met comme erreur:

Citation:

System.Data.SqlClient.SqlException: Nom de colonne non valide : 'denisbuzz'.
Nom de colonne non valide : 'parisplouz'.

au niveau de ma ligne:

Code:

int compteur = (int)cmd.ExecuteScalar();

Je ne comprend pas trop où est mon erreur

Sachant que le login que je rentre est "denisbuzz" et le password est "parisplouz".
Ces données sont bien présente dans ma table.

Merci de m'aider.

26/10/2011, 00h22
nova-id

Utilise le séparateur de chaînes

Salut. Ton problème est que tu n'utilises pas le séparateur de chaînes pour construire ta requête SQL :

Code:

string sql = "Select COUNT(idclient) from clients where pseudo ='" + login.Text + "' and password ='" + mdp.Text + "'";

De plus, tu devrais utiliser un formatage ou un StringBuilder :

Code:

string sql = string.Format("Select COUNT(idclient) from clients where pseudo ='{0}' and password ='{1}'", login.Text, mdp.Text);

C'est juste à ceci prés qu'il vaut mieux utiliser des requêtes paramétrées
Code:

1 2 3 4 5 6 7 8 9 string sql = "Select COUNT(idclient) from clients where pseudo =@login and password =@mdp"; SqlConnection conn = new SqlConnection(GetConnectionString()); SqlCommand cmd = new SqlCommand(sql, conn); cmd.Parameters.AddWithValue("@login", login.Text); cmd.Parameters.AddWithValue("@mdp", mdp.Text); conn.Open(); int compteur = (int)cmd.ExecuteScalar(); etc...
Et ça fonctionnera ton erreur vient simplement du fait que tu as oublié les simples quotes lors de ton passage de valeurs dans la requête

par ex
SELECT COUNT(idclient) FROM clients WHERE pseudo='val 1' and password ='val 2'

donnera en c#

"SELECT COUNT(idclient) FROM clients WHERE pseudo='"+login.Text+"' and password ='"+mdp.Text+"'"

mais de toutes façons il ne faut non plus l' utiliser car ce n'est pas propre la solution consiste à définir des paramètres pour ta requête :
Code:

1 2 3 4 string sql = "Select COUNT(idclient) from clients where pseudo =@login and password =@mdp"; cmd.Parameters.AddWithValue("@login", login.Text); cmd.Parameters.AddWithValue("@mdp", mdp.Text);
voilou