Comparer deux chaines en c#

Version imprimable

Voir 40 message(s) de cette discussion en une page

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:

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:

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

merci

SQL Server 2000
visual c# expess 2008

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

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:

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.

20/12/2010, 15h22
Bluedeep
Déjà on ne comprends pas pourquoi tu fais une boucle sur les lignes pour ensuite utiliser :
Code:

1 2 l = (string)datast.Tables["utilisateur"].Rows[n].ItemArray[0]
Il eut été plus logique de faire :
Code:

1 2 l = (string)row.ItemArray[0]
quoique ce dernier est inutilement illisible.

Le code normal est :
Code:

1 2 l = (string)row["NomDuChamp"];
Sinon, pour la comparaison de chaine utiliser la méthode statique Compare:
Code:

1 2 String.Compare(chaine1, chaine2, true);
(true pour comparaison case insensitive, false pour une comparaison case sensitive).
20/12/2010, 15h23
Arnard

Les colonnes en base de données sont de type char ou varchar ?
20/12/2010, 15h27
lamia89

les colonnes sont de type varchar

Citation:
Envoyé par h2s84
j'ai l'impression que tu te "pourris" la vie ! :lol:

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:

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 (') ? :ange:

20/12/2010, 21h03
Invité

Citation:

Envoyé par Arnaud F.

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

pas de problème.

as-tu testé ?
21/12/2010, 19h26
Arnaud F.

Je dois avouer que non :oops:

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

Code:

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

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

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

21/12/2010, 21h45
hegros

Citation:

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 ? :aie:
21/12/2010, 21h57
Invité

Citation:

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

un lien
21/12/2010, 22h11
hegros

Citation:

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()
21/12/2010, 22h14
Invité

Citation:

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.
21/12/2010, 22h27
hegros

Citation:

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.

Citation:

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
22/12/2010, 00h03
Nathanael Marchand

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!
22/12/2010, 00h13
hegros

Citation:

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

Citation:

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 ?
22/12/2010, 00h58
Nathanael Marchand
Citation:

Envoyé par hegros

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

C'est juste pour ouvrir le débat :D Même si le candidat ne sait pas (surtout si il ne sait pas en fait) je regarde le raisonnement qu'il tient.

Citation:

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:

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)
22/12/2010, 01h20
hegros
Citation:
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:

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.
22/12/2010, 08h03
Nathanael Marchand

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

Voir 40 message(s) de cette discussion en une page