Discussion :

[solaar]

Bonjour.
j'ai une table Etiquette(N° Etiquette,Etiquette);
je veux créer une fonction pour afficher mes étiquettes sur certains contrôle comme bouton et autres mes j'ai une erreur qui me fatigue sur ma fonction.
can you help me?

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
 
 public string Afficher_Etiquette_par_Default(int valeur)
        {
 
            string queryString = "SELECT [N° Etiquette],[Etiquette]FROM [GestionDB].[dbo].[Etiquettes]";
 
            using (SqlConnection connection = new SqlConnection(GetConnectionString()))
            {
                SqlCommand command = new SqlCommand(queryString, connection);
                connection.Open();
                SqlDataReader reader = command.ExecuteReader();
                try
                {
                    while (reader.Read())
                    {
                        if (valeur==(int)reader[0])
                        {
                            return reader[1].ToString(); 
                        }
                    }
                }
                finally
                {
                      reader.Close();
                }
 
            }
 
 
        }

l'erreur est affichée est :

Erreur 1 '.Sécurités___Ouverture.Afficher_Etiquette_par_Default(int)' : tous les chemins de code ne retournent pas nécessairement une valeur

pour l'affiche sur mon bouton:

Code :

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

1
2
 
bt_Annuler.Text = Afficher_Etiquette_par_Default(10);

[Max]

Salut.

Ton problème est le suivant : si la condition valeur == (int)reader[0] n'est jamais rencontrée, alors l'instruction return n'est jamais appelée, et le programme va en fin de fonction et n'a rien à retourner : il faut donc que tu mettes une sorte de return par défaut en fin de fonction :

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
public string Afficher_Etiquette_par_Default(int valeur) {    
    string queryString = "SELECT [N° Etiquette],[Etiquette]FROM [GestionDB].[dbo].[Etiquettes]";
    using (SqlConnection connection = new SqlConnection(GetConnectionString())) {
        SqlCommand command = new SqlCommand(queryString, connection);
        connection.Open();
        SqlDataReader reader = command.ExecuteReader();
        try {
            while (reader.Read()) {
                if (valeur == (int)reader[0]) {
                    return reader[1].ToString(); 
                }
            }
        }
        finally {
              reader.Close();
        }
    }
    // Le programme arrive ici
    return -1;
}

[calagan99]

Ton erreur est assez parlante...
Tu es dans une fonction, quoi qu'il arrive tu dois donc forcément retourner une valeur.
Or, si ton SQLDataReader ne contient aucun élément, tu ne renverra rien.

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
 public string Afficher_Etiquette_par_Default(int valeur)
        {
            string etiquetteLibelle = string.Empty;
 
            string queryString = "SELECT [N° Etiquette],[Etiquette]FROM [GestionDB].[dbo].[Etiquettes]";
 
            using (SqlConnection connection = new SqlConnection(GetConnectionString()))
            {
                SqlCommand command = new SqlCommand(queryString, connection);
                connection.Open();
                SqlDataReader reader = command.ExecuteReader();
                try
                {
                    while (reader.Read())
                    {
                        if (valeur==(int)reader[0])
                        {
                            etiquetteLibelle = reader[1].ToString(); 
                        }
                    }
                }
                finally
                {
                      reader.Close();
                }
 
            }
            return etiquetteLibelle;
 
        }

ainsi, on initialise systématiquement une variable dans ton type de retour (string). On lui affecte une valeur si on trouve la valeur voulue dans le DataReader et on retourne cette variable.

A ton IHM par la suite de gérer le fait que la valeur retournée puisse être une chaine vide.

[solaar]

super c'est cool.
j'ai pris l'exemple de calagan99 que j'ai bien compris, par contre celui de eusebe19 il marche bien mais je comprend pas pour quoi return -1 au lieu de 1 ou 0.

[Max]

super c'est cool.
j'ai pris l'exemple de calagan99 que j'ai bien compris, par contre celui de eusebe19 il marche bien mais je comprend pas pour quoi return -1 au lieu de 1 ou 0.

Bah déjà, je me suis gouré j'avais cru que tu retournais un int et pas un string

Et -1 c'est souvent la valeur de retour par défaut pour un int quand rien n'a été trouvé (comme string.IndexOf, etc.)