Discussion :

[farfouch00]

Bonjour,

Ma Liste contient des instances de ma classe ObjetPerso définie comme :

Code :

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

1
2
3
4
5
6
7
8
public class ObjetPerso
{
private object realObject;
private int niveau,
.
.(Constructeur, accesseurs... Qui vont bien)
.
}

Ensuite j'ai deux autre classes :

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 class MaClasse1
{
private int ID,
.
.(Constructeur, accesseurs... Qui vont bien)
.
}
public class MaClasse2
{
private int ID,
.
.(Constructeur, accesseurs... Qui vont bien)
.
}

chacune a un attribut ID.

Ensuite j'ai rempli cette liste :

Code :

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

1
2
3
4
5
List<ObjetPerso> maListe = new List<ObjetPerso>();
maList.Add(new ObjetPerso(new MaClasse1(1), 1));
maList.Add(new ObjetPerso(new MaClasse1(2), 1));
maList.Add(new ObjetPerso(new MaClasse2(1), 2));
maList.Add(new ObjetPerso(new MaClasse2(2), 2));

Mon problème, c'est que quand j'essaie de récupérer l'objet de niveau 2, et ID 1 (Par exemple), sachant que dans mon programme il ne peux être qu'une instance de la classe maClasse2 :

Code :

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

objetClass2 = maList.Single(t => t.niveau == 2 && ((objetClass2)t).ID == 1);

J'ai une exception InvalidCastException :

Impossible d effectuer un cast de l objet de type 'MaClasse1'en type 'MaClasse2'.

Vous auriez une idée sur la source de ce problème? Y a-t-il un truc qui cloche dans ma manière de procéder?

Merci!

[BenoitM]

ben logique tu lui demandes de caster des "objetClass1" en "ObjetClass2"

donc soit tu mets Id en commun au 2 classes ou alors tu ajoutes dans ton Single() un test sur le type d'object

Code :

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

Single(t => t.niveau == 2 && t Is ObjetClass2  && ((objetClass2)t).ID == 1);

[asmduty]

Met un espion mais vraisemblablement c'est toi qui te trompes.
Si il veut pas caster classe1 en classe 2 c'est qu'effectivement tu as d'un côté classe1 et de l'autre classe2

[farfouch00]

Merci BenoitM,

le contrôle sur le type d'objet a résolu mon problème. Bonne journée!

Merci asmduty aussi (;

[Nathanael Marchand]

Je chipote mais je suggère ca:

Code :

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

objetClass2 = maList.OfType<objetClass2>().Single(t => t.niveau == 2 && t.ID == 1);

[BenoitM]

Je chipote mais je suggère ca:

Code :

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

objetClass2 = maList.OfType<objetClass2>().Single(t => t.niveau == 2 && t.ID == 1);

Ben non ca permet d'apprendre de nouvelles instructions du framework

part contre ca doit être un peu plus lent vu qu'il fait 2x la boucle non? (une fois pour crée une liste d'"ObjetClass2" ensuite pour faire le select)

Tiens et si tu es sur de toi, l'utilisation de First() peut-être plus rapide aussi que Single()
Single parcourt tous les élements et lance une execption si aucun élément ou si plus d'un élément répond au(x) critère(s)

[Nathanael Marchand]

Ben non ca permet d'apprendre de nouvelles instructions du framework

part contre ca doit être un peu plus lent vu qu'il fait 2x la boucle non? (une fois pour crée une liste d'"ObjetClass2" ensuite pour faire le select)

Tiens et si tu es sur de toi, l'utilisation de First() peut-être plus rapide aussi que Single()
Single parcourt tous les élements et lance une execption si aucun élément ou si plus d'un élément répond au(x) critère(s)

Nop ca n'est pas plus long, il n'y a pas de création de liste intermédiaire, c'est du Defered Streaming (instruction yield toussa toussa)
Par contre, oui je confirme pour la différence Single/First mais y'a des cas ou on veut s'assurer de l'unicité. Si ca n'est pas le cas ici, il faut effectivment privilégier le First().

Une petite table bien utile:
http://msdn.microsoft.com/en-us/library/bb882641.aspx