Comment travailler avec Boolean

Version imprimable

Bonjour!
j'arrive pas à comprendre l'erreur:

Citation:

tout chemins de code ne retourne pas nécessairement une valeur

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
26
27
28
29
30
31
32
33
34
35
36
37
38
39
 
        public static  Boolean _Connexion_User (string plogin, string pmotpass)
        {
            Boolean Valeur = false;
            string SQL = "SELECT * FROM [Sécurités - Utilisateurs]";
 
 
            try
            {
 
                using (SqlConnection Conn = new SqlConnection(ConnectionString))
                {
                    Conn.Open();
                    SqlCommand Cmd = new SqlCommand(SQL, Conn);
                    SqlDataReader Reader = Cmd.ExecuteReader();
 
                    while (Reader.Read())
                    {
 
                        if ((Reader[0].ToString().Trim() == plogin.Trim()) && (Reader[2].ToString().Trim() == pmotpass.Trim()))
 
                            Valeur = true;
 
                        else
 
                            Valeur = false;
                    }
                    /// Fermer la connection .
                    Conn.Close();
                   }
                return Valeur;
            }
            catch (Exception pb)
            {
                pb.Message.ToString();
 
 
            }
        }

Merci pour l'aide.

04/09/2011, 23h16
MaximePalmisano

Bonjour,

Ici, ta fonction doit retourner un booléen. Tu fais bien un "return Valeur" à la fin de ton try, mais si jamais une exception arrive, aucune valeur n'est remontée dans le catch ou le finally.

Résultat, VS devine que si une exception arrive, aucune valeur ne sera retournée. Il ne veut donc pas compiler :)

dans le catch si ca te semble illogique de retourner true ou false, tu peux faire throw;
l'exception sera alors remontée à l'appelant (qui pourra donc agir en conséquence)

de plus, tu sembles chercher si le couple login/mot de passe existe dans la table, et pour ca il ne sert à rien de lire toute la table dans ton programme
en plus ton code ne peut pas marcher, quand ta boucle va trouver le couple, valeur va prendre true, mais à la ligne d'après le couple ne sera plus bon et tu vas réécrire false (il fallu aurait par exemple sortir de la boucle dans le valeur = true)
Code:

1 2 3 4 5 6 cmd.commandtext = "select 1 from [Sécurités - Utilisateurs] where login = @login and password = @password"; cmd.parameters.Add("@login", plogin.trim); cmd.parameters.Add("@password", pmotpass.trim); (...) reader = cmd.executereader; return reader.Read
si le couple existe, la requete retournera une ligne (avec une colonne contenant 1)
si le couple n'existe pas, la requete ne retournera pas de données, et .read retournera false

les données sont lues par le sgbdr sans les rapatrier donc c'est plus rapide, surtout si la table est indexée
au passage il y a l'utilisation des parameters, qui sont plus que conseillés pour les string et date

le conn.close n'est pas utile non plus, conn étant dans un bloc using, .dispose sera appelé, et .dispose fait .close

Oui cool!
le return après le catch ou finally.
mais j'ai un autre souci encore SVP.

dans ma base de donnée j'ai un champ avec type bit.

je ne sais comment le manipuler ce type.
j'ai comme erreur:

Citation:

l'opérateur == ne peut pas être appliqué aux opérandes de type objet et bool

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
      string SQL = "SELECT * FROM  [Sécurités - Accès Sociétés] WHERE( [Id Utilisateur]= '" + login + "')";
 
            try
            {
 
                using (SqlConnection Conn = new SqlConnection(ConnectionString))
                {
                    Conn.Open();
                    SqlCommand Cmd = new SqlCommand(SQL, Conn);
                    SqlDataReader Reader = Cmd.ExecuteReader();
 
                    while (Reader.Read())
 
                        if (Reader[3] == true) SOC = Reader[1].ToString();
                        else
                            SOC = null;
 
                }
 
            }
            catch (Exception pb){pb.Message.ToString();}
            return SOC;
      }

05/09/2011, 01h18
Pol63

on ne fait pas un select * pour ramener un colonne c'est contre performant
il faut éviter les reader[3] mais plutot nommer les colonnes reader["telcolonnne"] sinon le code est illisible plusieurs mois plus tard
reader[] retourne une valeur as object, c'est à toi de vérifier que le type est ce que tu attends, ne l'occurrence un booléen (ou alors reader.getboolean)

en bref il faudrait apprendre un peu avant de coder
05/09/2011, 14h28

Bonjour,

Je ne conseille pas ce genre de requête qui sont sujette aux injections SQL:

Code:

select 1 from [Sécurités - Utilisateurs] where login = @login and password = @password

Il faut dans ce cas là utiliser le test sur le login

Code:

select password from [Sécurités - Utilisateurs] where login = @login

et ensuite tester dans le code le password.
05/09/2011, 14h36
Pol63

Citation:

Envoyé par darthmower

Bonjour,

Je ne conseille pas ce genre de requête qui sont sujette aux injections SQL:

Code:

select 1 from [Sécurités - Utilisateurs] where login = @login and password = @password

Il faut dans ce cas là utiliser le test sur le login

Code:

select password from [Sécurités - Utilisateurs] where login = @login

et ensuite tester dans le code le password.

il est impossible de faire une injection sql avec un dbparameter (d'où l'utilisation de variable sql @_ et non d'une concaténation)
et si tu veux parler de sécurité, ta méthode va faire transiter le mot de passe sur le réseau et le mettre en ram sur la machine :D
05/09/2011, 14h39
MaximePalmisano

De plus, vu que solaar est débutant, je ne suis pas sur que les injections sql soient vraiment un soucis pour le moment ;)
05/09/2011, 17h38

Citation:

Envoyé par Pol63

il est impossible de faire une injection sql avec un dbparameter (d'où l'utilisation de variable sql @_ et non d'une concaténation)
et si tu veux parler de sécurité, ta méthode va faire transiter le mot de passe sur le réseau et le mettre en ram sur la machine :D

Je me disais bien qu'il y avait quelque chose comme ça, cependant j'ai eu une petite flemme :oops:, pardon pour le post inutile.

Cependant je ne pense pas que parler des injections SQL soit réellement déplacé, ce n'est pas quelque chose de compliqué à comprendre lorsqu'on connait le SQL un minimum.
05/09/2011, 18h17
Pol63

oui c'est un truc à apprendre au début quand même
parce que dans ce monde, beaucoup de débutants font des logiciels professionnels (enfin vendus)
on le voit avec orange et la faille injection sql récemment trouvé sur leur boutique
05/09/2011, 18h59
solaar

Citation:

De plus, vu que solaar est débutant, je ne suis pas sur que les injections sql soient vraiment un soucis pour le moment

No c'est bon d'apprendre .... mais avec que petites explications puis google fait l'affaire ;).

Oké Pol63, j'ai repris mon code et sa marche bien:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
   public static Boolean _Connexion_User(string plogin, string pmotpass)
        {
                string SQL = "SELECT 1 FROM [Sécurités - Utilisateurs] where [Id Utilisateur] = @login and [PWD Utilisateur] = @password";
 
 
                    SqlConnection Conn = new SqlConnection(ConnectionString);
                    Conn.Open();
 
                    SqlCommand Cmd = new SqlCommand(SQL, Conn);
 
                    Cmd.Parameters.Add(new SqlParameter("@login", SqlDbType.NVarChar, 10));
                    Cmd.Parameters["@login"].Value = plogin.Trim();
                    Cmd.Parameters.Add(new SqlParameter("@password", SqlDbType.NVarChar, 10));
                    Cmd.Parameters["@password"].Value = pmotpass.Trim();
                    SqlDataReader Reader = Cmd.ExecuteReader();
                    return Reader.Read();
 
 
 
            }

Mais j'avais du mal à travailler avec using et catch .... pour quoi ? peut tu m'expliquer ... je me dis qu'ils sont important pour gérer mes erreurs.

06/09/2011, 07h57
Pol63

using sert à libérer la ram plus vite et de manière sûre (pour les variables implémentants IDisposable)
try sert à intercepter les erreurs

voir dans msdn (F1 ou sur le http://msdn.microsoft.com/library), tout doit être expliqué
06/09/2011, 11h36
solaar

Merci :ccool: