Discussion :

[DenisBuzz]

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 :

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

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

au niveau de ma ligne:

Code :

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

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.

[nova-id]

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

Code :

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

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 :

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

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

[Robotech]

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

Code :

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

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 :

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

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