Discussion :

[Msysteme]

Bonjour, j'ai une procedure stockée qui doit me retourner une seule valeur, au niveau de mon codage, j'ai pas pu lire cette seule valeur :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
public System.Data.SqlClient.SqlDataReader LoadMaxByYear(short annee)
        {
            System.Data.SqlClient. oOleDbDataReader;
            try
            {
                m_SqlClient.CreateCommand(System.Data.CommandType.StoredProcedure, "SelectMaxByYear");
                m_SqlClient.AddCommandParameter("@DateBV", System.Data.SqlDbType.Int, annee);
                oOleDbDataReader = ()m_SqlClient.ChargerValeur();            }
            catch (Exception ex)
            {
                throw ex;
            }
            return oOleDbDataReader;
        }

[koyot3]

Bonjour MSysteme,

Je crois qu'il te manque quelques lignes dans ton code

Code c# :

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

1
2
3
4
5
SqlCommand command = m_SqlClient.CreateCommand();
...
m_SqlClient.Open();
SqlDataReader reader = command.ExecuteReader();
return reader ;

En espérant que ça t'aide !

@+

[tomlev]

Je suis pas sûr de bien comprendre ton problème... En plus ton code n'a aucun sens . Qu'est-ce que c'est que ce "m_SqlClient" ? Une variable de type SqlConnection je suppose ? Si c'est ça, la méthode CreateCommand ne prend aucun paramètre... Et pourquoi est-ce que ta méthode renvoie un SqlDataReader si tu as juste besoin d'une valeur ? Elle devrait pas plutôt renvoyer directement la valeur ? Et que vient faire ce "oOleDbDataReader" dans l'histoire ?

Pour exécuter une requête (ou un appel à une procédure stockée) qui renvoie une seule valeur, on utilise la méthode ExecuteScalar() de la commande. Il faut ensuite caster le résultat vers le type attendu. Par exemple si ta procédure stockée renvoie un int :

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
        public int LoadMaxByYear(short annee)
        {
 
            try
            {
                SqlCommand command = m_SqlClient.CreateCommand();
                command.CommandType = CommandType.StoredProcedure;
                command.CommandText = "SelectMaxByYear";
                command.Parameters.Add("@DateBV", System.Data.SqlDbType.Int)).Value = annee;
                return (int)command.ExecuteNonQuery()
            }
            catch (Exception ex)
            {
                throw ex;
            }
            return oOleDbDataReader;
        }

[Msysteme]

Bonjour, j'ai modifié comme vous m'avez conseillé mais il me dit que le oOleDbDataReader n'existe pas dans le contexte actuel

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
         public int LoadMaxByYear(short annee)
        {
 
            try
            {
                SqlCommand command = m_SqlClient.CreateCommand(System.Data.CommandType.StoredProcedure, "SelectMaxByYear");
                command.Parameters.Add("@AnneeBV", System.Data.SqlDbType.Int).Value = annee;
                return (int)command.ExecuteNonQuery();
            }
            catch (Exception ex)
            {
                throw ex;
            }
            return oOleDbDataReader;
        }

[sisqo60]

retires cette ligne qui n'a plus lieu d'être...

Code :

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

 return oOleDbDataReader;

[tomlev]

Et au passage, enlève le try/catch. Il ne sert à rien puisque tu relances l'exception aussitôt, et en plus en faisant throw ex tu perds la pile originale de l'exception.

[Msysteme]

Merci et comment dois je définir mon datat reader, d'ailleurs je met quelque choses comme ca pour que je puisse appeler ma methose dans mon interface :

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 void LoadMaxBVByDate(short annee)
        {
            System.Data.SqlClient.SqlDataReader oDBDataReader;
 
            try
            {
                oDBDataReader = oBData.LoadDeclarationsByagrement(Agrement, annee);
                while (oDBDataReader.Read())
                {
                    Entity.DECLARARION oDeclaration;
                    oDeclaration = new Entity.DECLARARION();
                    oDeclaration.AgrCVT = Convert.ToString(oDBDataReader["AgrCVT"]);
                    oDeclaration.AnneeDec = Convert.ToInt16(oDBDataReader["AnneeDec"]);
                    oDeclaration.DateDec = Convert.ToDateTime(oDBDataReader["DateDec"]);
 
                    this.Add(oDeclaration);
                }
                oBData.CloseDataReader();
                oBData.CloseConnection();
            }
            catch (Exception ex)
            {
                throw ex;
            }
        }

Mais maintenant je n'ai qu'une seule valeur

[tomlev]

Je comprends rien à ce que tu racontes

Comme je l'ai dit plus haut, si tu n'as qu'une seule valeur tu n'as pas besoin d'un data reader, il y a la méthode ExecuteScalar qui te renvoie directement la valeur

[StringBuilder]

C'est pas ExecuteScalar plutôt ?

[tomlev]

C'est pas ExecuteScalar plutôt ?

Euh oui, c'est d'ailleurs ce que j'avais dit plus haut... merci, c'est corrigé