Comparer deux chaines en c#

**lamia89** · 20/12/2010, 14h56

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é · 20/12/2010, 15h21

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.

**Bluedeep** · 20/12/2010, 15h22

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** · 20/12/2010, 15h23

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

**lamia89** · 20/12/2010, 15h27

les colonnes sont de type varchar

**Arnaud F.** · 20/12/2010, 20h22

Envoyé par h2s84

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é · 20/12/2010, 21h03

Envoyé par Arnaud F.

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

pas de problème.

as-tu testé ?

**Arnaud F.** · 21/12/2010, 19h26

Je dois avouer que non

**hegros** · 21/12/2010, 21h38

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 ;
}

Invité · 21/12/2010, 21h42

Envoyé par 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 ;
}

je pense que c'est du Linq To DataSet que tu fais et pas du Linq To Object

**hegros** · 21/12/2010, 21h45

Envoyé par h2s84

je pense que c'est du Linq To DataSet que tu fais et pas du Linq To Object

Oui mais pour moi un DataSet est un objet

Dans la théorie je sais que les noms ont quelques peu changé et qu'il y a des linq to quelque chose qui ont disparu ou qui ont été renommée.

De plus le AsEnumerable() renvoit plutôt des objets (une collection). C'est le foutu dataset qui fait que je passe par Linq to Dataset plutôt que Linq To Object ?

Invité · 21/12/2010, 21h57

Envoyé par hegros

Oui mais pour moi un DataSet est un objet

Dans la théorie je sais que les noms ont quelques peu changé et qu'il y a des linq to quelque chose qui ont disparu ou qui ont été renommée.

De plus le AsEnumerable() renvoit plutôt des objets (une collection). C'est le foutu dataset qui fait que je passe par Linq to Dataset plutôt que Linq To Object ?

un lien

**hegros** · 21/12/2010, 22h11

Envoyé par h2s84

un lien

Oui merci j'avais vu. En fait la confusion vient du fait que la requête Linq ci-dessus utilise les 2 (linq to dataset/object)

Les IEnumerable c'est le boulot de Linq To Objects et c'est bien ce qui est fait avec AsEnumerable()

Invité · 21/12/2010, 22h14

Envoyé par hegros

Oui merci j'avais vu. En fait la confusion vient du fait que la requête Linq ci-dessus utilise les 2 (linq to dataset/object)

Les IEnumerable c'est le boulot de Linq To Objects et c'est bien ce qui est fait avec AsEnumerable()

Ben t'as raison, mais Microsoft qui dis le contraire.

**hegros** · 21/12/2010, 22h27

Envoyé par h2s84

Ben t'as raison, mais Microsoft qui dis le contraire.

Non je dis peut-être une connerie en interprétant ce que dis Microsoft.

Envoyé par msdn

LINQ to Objects fait référence à l'utilisation directe de requêtes LINQ avec n'importe quelle collection IEnumerable ou IEnumerable<T>

Il me semble bien que AsEnumerable() sur un dataset renvoit une collection de IEnumerable, d'où le mélange/confusion entre linq object/dataset

**Nathanael Marchand** · 22/12/2010, 00h03

Hahaha! Ma question préférée lorsque je fais passer un entretien d'embauche!
Alors, la différence entre IEnumerable et IQueryable.
Tant qu'on utilise IQueryable, toutes les clauses LINQ seront traduites en SQL. C'est seulement lors du AsEnumerable que les données sont fetchées de la base.
Point de détails un peu tricky et qui peut changer totalement les perfs!

**hegros** · 22/12/2010, 00h13

Envoyé par PitMaverick78

Hahaha! Ma question préférée lorsque je fais passer un entretien d'embauche!

Elles sont pointues tes questions quand même, tu recherches quel niveau ?

Alors, la différence entre IEnumerable et IQueryable.
Tant qu'on utilise IQueryable, toutes les clauses LINQ seront traduites en SQL. C'est seulement lors du AsEnumerable que les données sont fetchées de la base.
Point de détails un peu tricky et qui peut changer totalement les perfs!

Est-ce que tu as un article (sur développez) qui compare clairement les 2 ?

Et pour répondre à la question clairement : c'est du linq to object ou linq to dataset ou les 2 pour l'exemple de requête de mon post précédent.

J'ai trouvé ce premier article mais c'est relativement pauvre et ce second article. Et un troisième pour la route

Si je comprends bien c'est plus performant avec IEnumerable selon l'article 1 et le IQueryable pour l'article 2 ? Tu as un article ?

**Nathanael Marchand** · 22/12/2010, 00h58

Envoyé par hegros

Elles sont pointues tes questions quand même, tu recherches quel niveau ?

C'est juste pour ouvrir le débat

Même si le candidat ne sait pas (surtout si il ne sait pas en fait) je regarde le raisonnement qu'il tient.

Envoyé par hegros

Est-ce que tu as un article (sur développez) qui compare clairement les 2 ?

Et pour répondre à la question clairement : c'est du linq to object ou linq to dataset ou les 2 pour l'exemple de requête de mon post précédent.

J'ai trouvé ce premier article mais c'est relativement pauvre et ce second article. Et un troisième pour la route

Si je comprends bien c'est plus performant avec IEnumerable selon l'article 1 et le IQueryable pour l'article 2 ? Tu as un article ?

Essaie de faire un test sur un table de 100lignes dont 20 lignes contiennent abc comme valeur et intercepte les requêtes SQL des trois requêtes:

Code :

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

1
2
3
MaTable.AsQueryable().Where(l=>l.Field=="abc").First()
MaTable.AsQueryable().Where(l=>l.Field=="abc").AsEnumerable().First()
MaTable.AsEnumerable().Where(l=>l.Field=="abc").First()

Première ligne tu rappatries 1 ligne (clause where executée sur sql, top en sql)
Seconde ligne tu rappatries 20 lignes (clause where executée sur sql, top en mémoire)
Dernière ligne tu rappatries ta table entière(clause where executée en mémoire, top en mémoire)

**hegros** · 22/12/2010, 01h20

Envoyé par PitMaverick78

Essaie de faire un test sur un table de 100lignes dont 20 lignes contiennent abc comme valeur et intercepte les requêtes SQL des trois requêtes:

Code :

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

1
2
3
MaTable.AsQueryable().Where(l=>l.Field=="abc").First()
MaTable.AsQueryable().Where(l=>l.Field=="abc").AsEnumerable().First()
MaTable.AsEnumerable().Where(l=>l.Field=="abc").First()

Première ligne tu rappatries 1 ligne (clause where executée sur sql, top en sql)
Seconde ligne tu rappatries 20 lignes (clause where executée sur sql, top en mémoire)
Dernière ligne tu rappatries ta table entière(clause where executée en mémoire, top en mémoire)

Ok mais AsQueryable n'existe pas pour une DataTable donc ton code ne compile pas. A moins que tu sois en .NET 4 ?

Pour me défendre je dirais que d'une part je ne sais pas si la DataTable utilisée est connectée à une base de données (qu'est-ce qui permet de l'affirmer) et d'autre part cette datatable peut être remplie et filtrée à une liste d'utilisateur d'un groupe ou autre chose provenant d'une autre requête déjà executé en base.Soit le problème de performance est à relativiser.

**Nathanael Marchand** · 22/12/2010, 08h03

J'avoue ne jamais utiliser de Dataset ni de Datatable.
Je travaille avec Linq to SQL ou EF

Comparer deux chaines en c#

C#

Discussions similaires

Partager

Partager