Discussion :

[xizvak]

Bonjour,

Voici ce que je veux faire, 2 listes d'objets (une pour les armes, une pour les armures) mais je veux répertorier tout ça dans une seule liste pour gérer un inventaire par la suite.

L'erreur est volontairement commise car je ne sais pas comment faire

Voici le code en question

https://dotnetfiddle.net/q0EEYn

Cordialement,

G.

[PixelJuice]

Bonjour,

je ne connais pas ton niveau donc excuse moi si je parle de choses que tu connais déjà :

Tu utilises déjà l'héritage donc utilise le pour faire ce que tu veux, c'est à dire du polymorphisme. C'est grossièrement le fait d’appeler une seule et même méthode et de laisser l'héritage faire la différence comme un grand.

Voici une des façons de faire, obliger les classes découlant de Item a définir une méthode qui décrit l'objet en question :

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
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
public abstract class Item
{
    public string type;
 
    public Item(string type)
    {
        this.type = type;
    }
 
    public abstract string Description();
}
 
public class Arme : Item
{
    int attaque;
 
    public Arme(int attaque) : base("Arme")
    {
        this.attaque = attaque;
    }
 
    public override string Description()
    {
        return attaque.ToString();
    }
 
    public int Attaque
    {
        get { return attaque; }
    }
}
 
public class Armure : Item
{
    int defense;
 
    public Armure(int defense) : base("Armure")
    {
        this.defense = defense;
    }
 
    public override string Description()
    {
        return defense.ToString();
    }
 
    public int Defense
    {
        get { return defense; }
    }
}

• J'ai passé Item en abstract, pour un meilleur héritage. En effet tu n'as pas envie de créer un objet de type Item qui ne représentera rien de concret. Et cela te permet d'y ajouter une méthode Description() qui sera redéfini par les classes filles.
• Même une classe abstraite peut avoir un constructeur, mais ça ne sert qu'a alléger les arguments. Tu peux l’appeler a l'aide du mot-clé base. Comme ça, la classe mère sait exactement ce qu'il faut faire et ça t'évite d'écrire du code.
• J'ai fait quelques changements, c'était juste pour y voir plus claire, à toi de garder ce que tu as besoin.

Il est courant qu'avec un exemple assez simple comme celui là, ça ne parait pas très utile mais crois-moi dès que tu commences a avoir beaucoup de code, ça aide.

Du coup, le code devient beaucoup plus simple :

Code :

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

1
2
3
4
        foreach (Item i in inventaire)
        {
            Console.WriteLine(i.Description());
        }

[xizvak]

Un grand merci a toi PixelJuice pour ces explications claires et precises, je vais pouvoir l'appliquer a mon projet Unity.

Bonne fin de journee,

Cordialement

[Themacleod1980]

Salut,
le c# est assez puissant et vu que les armes et armures héritent de inventaire :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
        foreach (Item i in inventaire)
        {
            if (i is Arme)
            {
                (Arme)i.FaisUnTruc();
            }
            if (i is Armure)
            {
                (Armure)i.FaisUnAutreTruc();
            }
        }

Si la solution de PixelJuice ou la mienne te convient, n'oublie pas de cliquer sur 'résolu'

[xizvak]

Merci egalement de ta réponse,

PixelJuice avait résolu mon soucis, entre temps j'ai bouquiné la programmation orienté objet et maintenant je ne rencontre plus ce genre de problèmes. Je suis opérationnel a ce niveau.

Par contre je ne vois pas ou est le bouton résolu sur mobile ? Le modo peut le faire pour moi svp.

Bonne journée a tous.