Discussion :

[Will Fou]

Bonjour,

J'ai un petit souci d'héritage. Je vous expose le problème :

J'ai une classe Hand.

Code :

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

1
2
public class Hand : List<Card> 
[...]

Dans une autre classe, j'ai le code suivant :

Code :

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

1
2
3
4
 
Hand hand = new Hand();
[...]
hand = hand.OrderBy(c => c.Val).ToList();

Le code ne compile pas et j'ai l'erreur suivante : "Impossible de convertir implicitement le type 'System.Collections.Generic.List<Poker.Card>' en 'Poker.Hand'. Une conversion explicite existe (un cast est-il manquant*?)
". Ce que je ne comprends déjà pas puisque la classe Hand hérite de List<Card>.


J'essaye donc de faire le cast :

Code :

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

1
2
 
hand = (Hand)hand.OrderBy(c => c.Val).ToList();

Cette fois, le code compile mais me renvoie une erreur au Runtime :
"Impossible d'effectuer un cast d'un objet de type 'System.Collections.Generic.List`1[Poker.Card]' en type 'Poker.Hand'"

Si vous pouviez m'éclairer, je vous remercie.

[Vimaire]

Bonjour,

Code :

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

1
2
3
Hand hand = new Hand();
[...]
hand = hand.OrderBy(c => c.Val).ToList();

Te retourne une List<Card> or selon ta déclaration,

une Hand est une List<Card> par contre, le contraire n'est pas vrai.

Tu ne peux donc pas considérer une List<Card> comme une hand.

Pour pallier à ca, tu pourrais par exemple surcharger le constructeur avec enumerable:

Code :

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

1
2
3
4
public class Hand : List<Card> 
{
    public Hand(IEnumerable<Card> cards):base(cards){}
[...]

et l'utiliser comme suit :

Code :

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

hand = new Hand(hand.OrderBy(c => c.Val));

Cependant, je te conseille plutôt de créer un objet Hand qui soit composé d'une List<Card> plutôt que d'hériter de la List. Ce sera plus simple pour gérer les évolutions.


Bonne journée

[Will Fou]

Merci Effectivement, c'était logique.

Cependant, je te conseille plutôt de créer un objet Hand qui soit composé d'une List<Card> plutôt que d'hériter de la List. Ce sera plus simple pour gérer les évolutions.

C'est bien ce que j'avais fait au début avant de vouloir revenir vers l'héritage.

Bonne journée

[dkmix]

Bonjour,

Code :

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

1
2
3
4
 
Hand hand = new Hand();
[...]
hand = hand.OrderBy(c => c.Val).ToList();

La fonction OrderBy ne retourne pas un "Hand", elle retourne un "System.Linq.IOrderedEnumerable"

[Will Fou]

Non elle renvoie, dans ce cas, une List<Card>.

Elle retourne un "System.Linq.IOrderedEnumerable" si on enlève ToList().

[MaximePalmisano]

C'est ce qu'essaye de t'expliquer dkmix

La fonction OrderBy() renvoie un "System.Linq.IOrderedEnumerable".

C'est la fonction ToList() qui renvoie une liste (Logique :p)

En gros, voici les méthodes telles qu'elles pourraient être codées (C'est pas le cas en fait, mais passons)

Code :

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

1
2
3
4
public System.Linq.IOrderedEnumerable<T> OrderBy<T>(...)
{
    // Implémentation
}

Code :

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

1
2
3
4
public List<T> ToList<T>(...)
{
    // Implémentation
}

Voici une manière de 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
 
public class Hand
{
    private List<Card> hand = new List<Card>();
 
    public void OrderBy()
    {
        hand = hand.OrderBy(card => card.Value);
    }
 
    public void OrderByDesc()
    {
        hand = hand.OrderByDesc(card => card.Value);
    }
}

Il te suffira ensuite de faire uniquement :

Code :

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

1
2
3
 
hand.OrderBy(); // Tri du plus petit au plus grand
hand.OrderByDesc(); // Tri du plus grand au plus petit

My two cents

PS: Ceci est une des solutions possibles d'implémentation, à toi de voir comment tu veux que tes clients appellent ta méthode