Discussion :

[lamia89]

Bonjour,

j'ai réalisé un code d'authentification permettant à un utilisateur de se connecter à l'application .

pour cela j'accède à la base de données et je vérifie si les informations saisies par l'utilisateur existent dans la base de données ou non.

voici le code que j'ai développer :

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
 
 public bool authentifier(string log, string pass) 
        {
            string l="", p="";
            bool trouve=false;
            int n=0;
 
            foreach (DataRow row in datast.Tables["utilisateur"].Rows)
            {
 
                l = (string) datast.Tables["utilisateur"].Rows[n].ItemArray[0];
                p = (string) datast.Tables["utilisateur"].Rows[n].ItemArray[1];
 
                if (l.Equals(log) == true) 
                {
                    if (p.Equals(pass) ==true) 
                    {
                        trouve = true; ;
                    }
                }
                n++;
            }
            return trouve;
 
        }

J'ai controlé la valeur du trouve dans le traitement suivant :

Code :

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

1
2
3
4
5
 
             bool valid;
            valid = user.authentifier(this.textBox_login.Text,      this.textBox_pass.Text);
            if (valid == true) { this.Close(); }
            else MessageBox.Show("Le nom d'utilisateur ou le mot de passe est invalide !");

Mais dans l'exécution , toujours le message " nom....invalide" est affiché.
quand j'ai fait quelques vérifications j'ai trouvé que le l et le p contiennent toujours des données et que le programme n'entre pas à l'intruction if même si la condition est réalisée .

quelqu'un peut m'aider à résoudre ce problème??

merci

SQL Server 2000
visual c# expess 2008

[Invité]

j'ai l'impression que tu te "pourris" la vie !

Bon je ne sais d'ou provient l'erreur.
mais je pense que la fonction ci-dessous est plus simple et plus rapide que tes boucles for imbriquées :

Code C# :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
public bool authentifier(string log, string pass)
{
    DataTable table = DataSet1.Tables["utilisateur"];
    string expression;
    expression = string.Format("Login = '{0}' and Password = '{1}'", log, pass);
    DataRow[] foundRows;
 
    foundRows = table.Select(expression);
 
    return foundRows.Length == 1;
}

Notes que Login et Password qui sont dans la variable expression doivent être des champs existant dans la table utilisateur.

[Arnaud F.]

j'ai l'impression que tu te "pourris" la vie !

Bon je ne sais d'ou provient l'erreur.
mais je pense que la fonction ci-dessous est plus simple et plus rapide que tes boucles for imbriquées :

Code C# :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
public bool authentifier(string log, string pass)
{
    DataTable table = DataSet1.Tables["utilisateur"];
    string expression;
    expression = string.Format("Login = '{0}' and Password = '{1}'", log, pass);
    DataRow[] foundRows;
 
    foundRows = table.Select(expression);
 
    return foundRows.Length == 1;
}

Notes que Login et Password qui sont dans la variable expression doivent être des champs existant dans la table utilisateur.

Et si le mot de passe contient une quote (') ?

[Invité]

Et si le mot de passe contient une quote (') ?

pas de problème.

as-tu testé ?

[Arnaud F.]

Je dois avouer que non

[hegros]

Je ne sais pas pour les quotes (pas testé avec Selection(expression)) sinon on peut faire du linq to object ce qui reviens à :

Code :

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

1
2
3
4
5
6
public bool authentifier(string log, string pass)
{
    return  (from ds in DataSet1.Tables["utilisateur"].AsEnumerable()
                     where ds["Login"].Equals(log) && ds["Password"].Equals(pass)
               select ds).Count()  == 1 ;
}

[Bluedeep]

Déjà on ne comprends pas pourquoi tu fais une boucle sur les lignes pour ensuite utiliser :

Code :

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

1
2
 
l = (string)datast.Tables["utilisateur"].Rows[n].ItemArray[0]

Il eut été plus logique de faire :

Code :

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

1
2
 
l = (string)row.ItemArray[0]

quoique ce dernier est inutilement illisible.

Le code normal est :

Code :

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

1
2
 
l = (string)row["NomDuChamp"];

Sinon, pour la comparaison de chaine utiliser la méthode statique Compare:

Code :

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

1
2
 
String.Compare(chaine1, chaine2, true);

(true pour comparaison case insensitive, false pour une comparaison case sensitive).

[Arnard]

Les colonnes en base de données sont de type char ou varchar ?

[lamia89]

les colonnes sont de type varchar