Discussion :

[koktel_dfr]

Bonjour,

Dans mon projet j'ai créé une classe DatabaseConnexion implémentant une méthode executeCommand recevant en paramètre la commande sql a exécuter.

Je souhaiterais que cette méthode me renvoie un string ayant pour valeur le résultat de la requête sql. Le problème c'est que je ne sais pas comment faire concernant la gestion des exceptions. La clause catch n'a pas besoins de me renvoyer de string car sont seul but est d'afficher un message d'erreur et de quitter l'application. Elle ne contient donc aucun "return" mais forcément Visual Studio gueule et m'affiche l'erreur suivante : "tous les chemins de code ne retournent pas nécessairement une valeur".

J'ai pensé à remplacer le environment.exit() par un truc du genre return "erreur" et tester la valeur de retour de la méthode par la suite mais je souhaiterais savoir si il existe un meilleur moyen de gérer cela car ca me parait un peu alambiqué...

Ci dessous le code de ma classe :

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
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
 
 
class DatabaseConnexion
    {
        private string connectionString;
 
        public DatabaseConnexion(string connStr)
        {
            connectionString = connStr;
        }
 
        public string executeCommand(string sqlCommand)
        {
            OdbcConnection myOdbcConnection = null;
            OdbcDataReader myReader = null;
 
            string line = "";
            int i;
 
            try
            {
                myOdbcConnection = new OdbcConnection(connectionString);
                myOdbcConnection.Open();
                OdbcCommand myOdbcCommand = new OdbcCommand(sqlCommand);
                myOdbcCommand.Connection = myOdbcConnection;
                myReader = myOdbcCommand.ExecuteReader();
 
                while (myReader.Read())
                {
                    line = "";
                    for (i = 0; i < myReader.FieldCount; i++)
                    {
                        line += myReader[i];
                    }
                    Console.Out.WriteLine(line);
                }
 
                return line;
            }
 
            catch (Exception ex)
            {
                Console.Error.WriteLine("Erreur d'exploitation de la base de données " + ex.Message + ")");
                Environment.Exit(2);
            }
 
            finally
            {
                try
                {
                    myReader.Close();
                    myOdbcConnection.Close();
                }
                catch { }
            } 
        }
    }

PS : si vous avez des remarques pertinentes concernant le contenu de ma classe je suis bien sur preneur... sachant que cette classe n'est destiné pour le moment qu'a faire des select dans la base de données.


Merci

[fally]

Salut!
Comme tu as fait un return dans le try, tu dois forcément le faire aussi dans le catch (puisque quand il y a une erreur, il vient dan sle catch et donc doit pouvoir donner une valeur à ton string)!
Pour éviter cela, il faut faire un return entre le try et le catch
mais là faudra être sur que ton code ne pourrait pas bugger

[koktel_dfr]

C'est bien ce que je pensais, cependant je préfère exclure le return entre le try et le catch car je préfère partir du principe que mon code peut toujours foirer à un moment ou a un autre (comme dans 99% des cas...).

Merci de ton aide, je vais donc me résoudre a renvoyer un string et a le tester par la suite... a moins que je trouve un autre moyen en organisant ma classe différemment.

[bossun]

ben ce que tu peux faire c'est renvoyer l'exception que tu interceptes pour finalement l'intercepter à un autre niveau. Normalement c'est pas dans ta couche d'accès aux données que tu sort du programme..


Dans ton cas, je te conseille de créer une classe d'exceptions qui t'es propre et qui hérite de la classe Exception

Par ex:

Code :

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

1
2
3
4
5
    public class DBWrapperException : Exception
    {
        public DBWrapperException(string Message) : base(Message) { }
        public DBWrapperException(string Message, Exception inerException) : base(Message, inerException) { }
    }

et dans ta classe DatabaseConnexion tu fais ça

Code :

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

1
2
3
4
                catch(Exception ex)
                {
throw new DBWRapperException(ex.Message,ex.InerException);
                }

Du coup il ne te reste plus qu'à intercepter les DBWrapperException et à sortir du programme...

[koktel_dfr]

Un grand merci à toi Bossun, ta solution m'a l'air très interessante

Je vais d'ailleurs l'adopter de ce pas