Discussion :

[Error407]

Bonjour à tous,

Je développe en ce moment à titre personnel une application, utilisant EF et Linq.

Au lancement de l'application, je lance une requete en base (sql server compact, crée avec vs 2012, et que j'ai rempli par mes soins) afin de remplir une listBox.

Soucis : ma requête ne me retourne rien du tout : que faire ?

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
 
 
        public static IList<DAO.Categorie> GetAllCategorie()
        {
            List<DAO.Categorie> lstCategorie = new List<DAO.Categorie>();
 
            using (var db = new Entities())
            {
                IQueryable<Categorie> categories = from c in db.Categorie
                                                   select c;
 
                foreach (DAO.Categorie item in lstCategorie)
                {
                    lstCategorie = categories.ToList();
                }
            }
            return lstCategorie;
        }

A mon sens la requête est correcte, et je ne vois donc pas ce qui cloche ce niveau.
Comme mes connaissances sur le sujet sont encore menues, je me dirige vers vous pour avoir de potentielles pistes ...


Merci de votre aide.

Error

[callo]

Soucis : ma requête ne me retourne rien du tout : que faire ?

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
 
 
        public static IList<DAO.Categorie> GetAllCategorie()
        {
            List<DAO.Categorie> lstCategorie = new List<DAO.Categorie>();
 
            using (var db = new Entities())
            {
                IQueryable<Categorie> categories = from c in db.Categorie
                                                   select c;
 
                foreach (DAO.Categorie item in lstCategorie)
                {
                    lstCategorie = categories.ToList();
                }
            }
            return ;
        }

Woo, mais ça saute aux yeux! Ton problème est dans le foreach. Tu dois parcourir le jeux d'enregistrement de ta requête linq i.e parcourir categories et non lstCategorie. cette dernière est toujours vide donc tu ne rentres même pas dans le corps de ton foreach. C'est donc tout à fait normal que ta listbox soit vide.

Code :

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

1
2
3
4
foreach (DAO.Categorie item in lstCategorie) // tu dois parcourir  categories et non lstCategorienon 
   {
       lstCategorie = categories.ToList(); // ceci n'a aucun sens
   }

[Invité]

calo a tout à fait raison dans son diagnostic du foreach. En revanche, il n'est pas nécessaire de parcourir "categories" avec un foreach car la méthode "ToList" fait tout le travail nécessaire.

Il suffit d'un code tout simple pour faire ce que tu veux faire :

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
public static IList<DAO.Categorie> GetAllCategorie()
        {
            //Cette variable locale est inutile
            //List<DAO.Categorie> lstCategorie = new List<DAO.Categorie>();
 
            using (var db = new Entities())
            {
               //Pour ce que tu veux faire, "db.Categories" et
               //"from c in db.Categories select c" sont parfaitement identiques
               //autant prendre le plus court!
               //IQueryable<Categorie> categories = from c in db.Categorie
               //                                    select c;
 
                //Boucle inutile, "ToList" fait tout le travail nécessaire!
                //foreach (DAO.Categorie item in categories)
                //{
                //    lstCategorie.Add(item)
                //}
 
                return db.Categorie.ToList(); //C'est tout ce qu'il suffit de faire!
            }
            //return lstCategorie;
        }

Explication : la méthode "ToList" renvoie une nouvelle instance de type "List" avec les éléments de "db.Categories".

[callo]

Tu as tout à fait raison Ph_Gr . Je me disais que j'oubliais d'ajouter quelque chose. Et tu as bien fait de le mentionner.

[Error407]

oula ....


c'était surement la fatigue, une que l'on revient dessus cela parait trés clair.

Merci beaucoup,

Résolu !