Discussion :

[pot2yaourt]

Bonjour à tous !

Voici mon problème, j'ai une classe d'accès aux données qui contient une méthode GetInfoClient() (voir ci-dessous) qui doit me renvoyer un objet Client.
La classe Client contient un seul constructeur qui prend en paramètre un SqlDataReader.

Le problème est que je n'arrive pas à compiler car il me dit que la variable locale client n'est pas assignée et le problème se porte sur return client;

Bon, je comprends bien le message d'erreur... mais je voudrais savoir quelle méthode dois-je utiliser pour contrer ce problème ?!

Suis-je obligé de définir un constructeur vide pour instancier mon objet Client avant le using et ensuite faire une méthode Add() qui prendrait en paramètre le SqlDataReader ??

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
 
public Client GetInfoClient(string sNumero
{
    try			
    {
        Client client;
 
        using (SqlCommand cmd = new SqlCommand())
        {
            cmd.Connection = this.DbConnection;
            cmd.CommandText = "getInfosClient";
            cmd.CommandType = CommandType.StoredProcedure;
 
            SqlParameter prmNumero= new SqlParameter("@numero", SqlDbType.VarChar, 50);
            prmNumero.Value = sNumero;
            cmd.Parameters.Add(prmNumero);
 
            using (SqlDataReader reader = cmd.ExecuteReader())
            {
                if(reader.Read())
                {
                    client = new Client(reader);
                }
                reader.Close();
            }
        }
        return client;
    }
    catch (Exception ex)
    {
        //
    }
}

Merci d'avance pour vos réponses !
Lionel.

[ppphil]

Code :

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

1
2
3
4
if(reader.Read())
{
   client = new Client(reader);
}

Si tu ne passe pas dans le if, ton client ne sera pas assigné. Et ça, le compilateur ne le veut pas.
Initialise ton client à null.
Ta valeur de retour sera un client ou null.....

[xian21]

Il faudrait aussi que ton return soit dans un finally comme ca tu es sur que ta methode te renvoye quelque chose

[ppphil]

Ouais, en plus....
Le mieux c'est d'initialiser le client à null avant tout try et de le retourner hors de tout try....

[pot2yaourt]

Il faudrait aussi que ton return soit dans un finally comme ca tu es sur que ta methode te renvoye quelque chose

Euh... à priori non, la clause finally n'accepte pas de return.

[pot2yaourt]

Voici du coup comment j'ai structuré ma méthode :

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
 
public Client GetInfoClient(string sNumeroCommande)
{
    //initialisation du client à null
    Client client = null;
 
    try
    {
        using (SqlCommand cmd = new SqlCommand())
        {
            cmd.Connection = this.DbConnection;
            cmd.CommandText = "getInfosClientColissimoByNumero";
            cmd.CommandType = CommandType.StoredProcedure;
 
            SqlParameter prmNumero = new SqlParameter("@numero", SqlDbType.VarChar, 50);
            prmNumero.Value = sNumeroCommande;
            cmd.Parameters.Add(prmNumero);
 
            using (SqlDataReader reader = cmd.ExecuteReader())
            {
                if (reader.Read())
                {
                    client = new Client(reader);
                }
                reader.Close();
            }
        }
    }
    catch (Exception ex)
    {
        //
    }
 
    return client;
}

Le fonctionnement me parait plus propre, reste à tester la valeur de retour de l'objet client dans la fonction appelante.

Merci pour votre aide !
Lionel.