Discussion :

[takinelinfo]

Bonjour à tous,

J'ai un petit soucis avec EF.
Voici l'EDMX:


Je voudrais récupérer la hiérarchie des niveaux.
Je fais donc :

Code c# :

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

1
2
3
4
5
6
7
8
 
public void NiveauGetAll()
{
	var db = new EntitiesDatabase();
	var listNiveau = db.niveau
		.Include("niveau_enfant")
		.Where(n => n.niveau_parent == null).ToList();
}

=> Je récupère le 1er niveau ainsi que c'est sous niveaux mais pas plus.
Pour récupérer les sous niveaux des sous niveaux il faut que je rajoute ça:

Code c# :

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

1
2
3
4
5
6
7
8
9
 
public void NiveauGetAll()
{
	var db = new EntitiesDatabase();
	var listNiveau = db.niveau
		.Include("niveau_enfant")
		.Include("niveau_enfant.niveau_enfant")
		.Where(n => n.niveau_parent == null).ToList();
}

Le problème c'est que je ne connais pas à l'avance le nombre de sous niveau qu'il peut y avoir. Peut on faire un .Include() récursif?
Merci d'avance.

[chrisdot]

Je vois le problème. Je ne sais pas si c'est lié à EF. C'est un problème d'ordre général. Si tu utilisais du SQL directement, tu aurais le même type de problème.

Dans le projet sur lequel j'ai travaillé, j'ai aussi eu à gérer ça, mais comme c'était une imbrication de catégories simples, je l'ai fait à la main...

En gros (pseudo-code, rien testé):

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
 
public void NiveauGetAll()
{
	var db = new EntitiesDatabase();
	var listNiveau = db.niveau.Where(n => n.niveau_parent == null);
 
             foreach(var elem in listNiveau)
             {
                  LoadChildren(db, elem);
             }
 
}
 
public void loadChildren(EntitiesDatabase db, niveau elem)
{
    db.Load<niveau>(elem, e=>e.niveau_enfant);
    foreach(var subelem in elem.niveau_enfant)
    {
                  LoadChildren(db, subelem);
    }
 
}

Je sais pas si c'est le plus efficace, mais si quelqu'un a une meilleure idée, je suis preneur...
Sachant qu'il est possible de tout charger et de construire l'arborescence en mémoire ensuite....

A+
Christophe

[anthyme]

db.niveau.ToList()

[chrisdot]

db.niveau.ToList()

Oui, en fait si tu veux charger l'arbre entier à chaque fois, c'est le plus simple. Pour ma part je n'ai pas fait ça car mes entités sont transformées en simples DTO pour voyager dans un web service...

Je nuancerai:

Code :

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

1
2
 
db.niveau.ToList().Where(n=>n.niveau_parent==null);

Comme ça j'ai la liste des racines, mais avec les enfant déjà chargés !!

Merci @anthyme !

[anthyme]

Quand on manipule des arbres - qui est une notion purement objet - il vaut mieux essayer de tout loader en un minimum d'appels et faire un maximum de traitement dans le code ensuite pour de bonnes performances.

Linq permet de mixer une partie SQL et objet séparé par le tolist() et donc de tout faire en une "requete logique"

Apres il faut voir s'il y a des filtres à faire coté sql pour ne pas tout ramener.

Bon courage pour la suite.

[takinelinfo]

Merci pour tous ces conseils. Cela fonctionne parfaitement.