Définition de classes et leur utilisation avec des List

Version imprimable

Bonjour à tous.

Je suis en train d'apprendre le c# en autodidacte, je me suis fixé pour but d'arriver à écrire une application dans le genre du "snake" (le jeu).
Je viens du delphi, je ne suis pas débutant en programmation, mais j'ai parfois du mal à mettre en code c# ce que je faisais en delphi, je bute sur des choses simples et je perds beaucoup de temps :(
Et faire un copier-coller des nombreux codes du genre présents sur le net n'est pas très formateur ...

Bref voila pour la courte présentation.

Pour mon projet, j'ai défini 2 classes :
- une qui contient la définition d'une coordonnée
- une qui contient la définition du corps du serpent, en utilisant la classe coordonnée

Ca nous donne donc
Code:

1 2 3 4 5 6 7 8 namespace Snake { class Coordonnee { public int X; public int Y; } }
et
Code:

1 2 3 4 5 6 7 namespace Snake { class Serpent { public List<Coordonnee> CorpsSerpent = new List<Coordonnee>(); } }
Ensuite arrive le moment où je veux utiliser ces classes.
Code:

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 namespace Snake { public partial class Form1 : Form { Serpent UnSerpent = new Serpent(); // Création du serpent Coordonnee UneCoordonnee = new Coordonnee(); // Création d'une coordonnée public Form1() { InitializeComponent(); CreerNouveauJeu(); // Initialisation d'un nouveau jeu } ... private void CreerNouveauJeu() { UnSerpent.CorpsSerpent.Clear(); // Nettoyage du corps du serpent UneCoordonnee.X = 16; // Initialisation coordonnée X UneCoordonnee.Y = 1; // Initialisation coordonnée Y UnSerpent.CorpsSerpent.Add(UneCoordonnee); // Ajout de ces coordonnées au corps du serpent UneCoordonnee.X = 1; // Initialisation coordonnée X UneCoordonnee.Y = 1; // Initialisation coordonnée Y UnSerpent.CorpsSerpent.Add(UneCoordonnee); // Ajout de ces coordonnées au corps du serpent } } }
Les deux éléments de la liste sont bien crées, mais ils ne sont pas initialisés correctement.
Le deuxième ajout de coordonnées semble écrasé les valeur du premier, le couple de valeur (x,y)=(16,1) disparait de la liste.

C'est peut être une incompréhension sur le fonctionnement des List, ou une partie du code mal placé (j'ai du mal avec ça aussi), ou autre chose ...
J'ai besoin d'aide.

Bonjour,

ton problème vient de ce bout de code :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 private void CreerNouveauJeu() { UnSerpent.CorpsSerpent.Clear(); // Nettoyage du corps du serpent UneCoordonnee.X = 16; // Initialisation coordonnée X UneCoordonnee.Y = 1; // Initialisation coordonnée Y UnSerpent.CorpsSerpent.Add(UneCoordonnee); // Ajout de ces coordonnées au corps du serpent UneCoordonnee.X = 1; // Initialisation coordonnée X UneCoordonnee.Y = 1; // Initialisation coordonnée Y UnSerpent.CorpsSerpent.Add(UneCoordonnee); // Ajout de ces coordonnées au corps du serpent }
Tu as défini Coordonnee comme une classe, et tu rajoutes toujours le même objet de cette classe. Du coup, si tu modifies les propriétés de cet objet, tout changera, car ta liste pour l'instant n'est rempli que du même et seul objet.

Pour y remédier, soit tu créer un nouvel objet de type Coordonnee pour autant de coordonnées. Soit tu passes Coordonee en structure. C'est selon tes besoins.

Si tu as des difficultés sur ce point, je te conseille de chercher la différence entre passage de valeur et passage de réference.

Citation:

Envoyé par PixelJuice

Tu as défini Coordonnee comme une classe, et tu rajoutes toujours le même objet de cette classe. Du coup, si tu modifies les propriétés de cet objet, tout changera, car ta liste pour l'instant n'est rempli que du même et seul objet.

Pour y remédier, soit tu créer un nouvel objet de type Coordonnee pour autant de coordonnées. Soit tu passes Coordonee en structure. C'est selon tes besoins.

Je dirais même que plutôt de passer par une (ou plusieurs) instance locale, il me semble préférable d'utiliser directement l'élément dans la liste.

1/ En créant un constructeur adéquat à la classe "Coordonnee" :

Code:

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
 
namespace Snake
{
    class Coordonnee
    {
        public int X { get; private set; }
        public int Y { get; private set; }
 
        public Coordonnee(int x, int y)
        {
             X = x;
             Y = y;
        }
    }
    ...
    public partial class Form1 : Form
    {
        private void CreerNouveauJeu()
        {
            UnSerpent.CorpsSerpent.Clear(); // Nettoyage du corps du serpent
 
            UnSerpent.CorpsSerpent.Add(new UneCoordonnee(16, 1)); // Ajout de ces coordonnées au corps du serpent
            UnSerpent.CorpsSerpent.Add(new UneCoordonnee(1, 1)); // Ajout de ces coordonnées au corps du serpent
        }
    }
}

2/ En utilisant un constructeur dynamique (plus moderne, mais moins lisible, je trouve)

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
 
namespace Snake
{
    class Coordonnee
    {
        public int X { get; private set; }
        public int Y { get; private set; }
    }
    ...
    public partial class Form1 : Form
    {
        private void CreerNouveauJeu()
        {
            UnSerpent.CorpsSerpent.Clear(); // Nettoyage du corps du serpent
 
            UnSerpent.CorpsSerpent.Add(new UneCoordonnee() { X = 16, Y = 1 }); // Ajout de ces coordonnées au corps du serpent
            UnSerpent.CorpsSerpent.Add(new UneCoordonnee() { X = 1, Y = 1 }); // Ajout de ces coordonnées au corps du serpent
        }
    }
}

2 bis/ En utilisant un constructeur dynamique (plus moderne, mais moins lisible, je trouve)

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
 
namespace Snake
{
    class Coordonnee
    {
        public int X { get; private set; }
        public int Y { get; private set; }
    }
    ...
    public partial class Form1 : Form
    {
        private void CreerNouveauJeu()
        {
            UnSerpent.CorpsSerpent.Clear(); // Nettoyage du corps du serpent
 
            UnSerpent.CorpsSerpent.AddRange(new Coordonnee[] { new Coordonnee() { X = 16, Y = 1 }, new Coordonnee() { X = 1, Y = 1 } }); // Ajout de ces coordonnées au corps du serpent
        }
    }
}

3/ En modifiant directement les propriétés des instances dans la liste (mais ça peut vite être le bordel) :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
 
namespace Snake
{
    class Coordonnee
    {
        public int X { get; set; }
        public int Y { get; set; }
    }
    ...
    public partial class Form1 : Form
    {
        private void CreerNouveauJeu()
        {
            UnSerpent.CorpsSerpent.Clear(); // Nettoyage du corps du serpent
 
            UnSerpent.CorpsSerpent.Add(new UneCoordonnee()); // Ajout de ces coordonnées au corps du serpent
            UnSerpent.CorpsSerpent.Add(new UneCoordonnee()); // Ajout de ces coordonnées au corps du serpent
            UnSerpent.CorpsSerpent[0].X = 16;
            UnSerpent.CorpsSerpent[0].Y = 1;
            UnSerpent.CorpsSerpent[1].X = 1;
            UnSerpent.CorpsSerpent[1].Y = 1;
        }
    }
}

13/11/2017, 22h38
Tahiti-Bob

Merci de vos réponses :)

@ StringBuilder
J'ai utilisé la solution 2, ça marche effectivement bien mieux comme ça

@ PixelJuice
J'ai encore un peu de mal avec cette notion d'objet, je vais me renseigner sur les passages de valeur et références

J'ai bien avancé du coup !
13/11/2017, 23h13
Pol63

l'objet c'est la base oui
en fait quand tu écris

Code:

var c = new Coordonnee();

new Coordonnee est exécuté en premier, ca créé un espace mémoire pour recevoir l'instance de cette classe (qui contient x et y, qui étant des int font 4 octets)
puis l'adresse mémoire de cette instance est placée dans c
si tu fais ensuite

Code:

c = new Coordonnee();

alors un nouvel espace mémoire est créé, et c pointe maintenant vers cette instance mémoire

l'ancienne instance mémoire n'est ici plus pointée par personne, elle sera supprimée de la mémoire (mais ce n'est pas le sujet ici ^^)

si tu fais

Code:

var d = c

d est un pointeur, qui va pointer vers la même instance que c ; ce n'est pas l'objet qui est dupliqué mais la référence vers celui ci
donc modifier d.x "modifie" indirectement c.x
il n'y a pas réellement de modification de c, c'est que vu que c'est la même instance ca va lire le même espace mémoire, donc ca reflète le changement

Citation:

Envoyé par StringBuilder

2/ En utilisant un constructeur dynamique (plus moderne, mais moins lisible, je trouve)

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
 
namespace Snake
{
    class Coordonnee
    {
        public int X { get; private set; }
        public int Y { get; private set; }
    }
    ...
    public partial class Form1 : Form
    {
        private void CreerNouveauJeu()
        {
            UnSerpent.CorpsSerpent.Clear(); // Nettoyage du corps du serpent
 
            UnSerpent.CorpsSerpent.Add(new UneCoordonnee() { X = 16, Y = 1 }); // Ajout de ces coordonnées au corps du serpent
            UnSerpent.CorpsSerpent.Add(new UneCoordonnee() { X = 1, Y = 1 }); // Ajout de ces coordonnées au corps du serpent
        }
    }
}

À partir de quelle version du langage c'est possible avec des setters en private ?

14/11/2017, 11h30
Max

Citation:

Envoyé par Noxen

À partir de quelle version du langage c'est possible avec des setters en private ?

C# 2 il me semble :)
14/11/2017, 12h20
Noxen
Citation:

Envoyé par Max

C# 2 il me semble :)

Cette syntaxe

Code:

var c = new UneCoordonnee() { X = 16, Y = 1 }

existe depuis C# 3 pour être plus précis, mais elle implique des setters publics puisqu'elle est équivalente à celle-ci :
Code:

1 2 3 var c = new UneCoordonnee(); c.X = 16; c.Y = 1;
C'est toujours comme ça en C# 6. Je me disais qu'une version ultérieure pourrait éventuellement devenir plus permissive sur l'accessibilité du setter dans son usage au sein d'un initialiseur, néanmoins ce serait quand-même une atteinte au principe d'encapsulation.
14/11/2017, 13h41
Max
Citation:
Envoyé par Noxen

Cette syntaxe

Code:

var c = new UneCoordonnee() { X = 16, Y = 1 }

existe depuis C# 3 pour être plus précis, mais elle implique des setters publics puisqu'elle est équivalente à celle-ci :

Code:

1 2 3 var c = new UneCoordonnee(); c.X = 16; c.Y = 1;

C'est toujours comme ça en C# 6. Je me disais qu'une version ultérieure pourrait éventuellement devenir plus permissive sur l'accessibilité du setter dans son usage au sein d'un initialiseur, néanmoins ce serait quand-même une atteinte au principe d'encapsulation.
Oula j'ai mal compris ta question et donc ai répondu à côté de la plaque :aie:
14/11/2017, 15h03
Noxen

Citation:

Envoyé par Max

Oula j'ai mal compris ta question et donc ai répondu à côté de la plaque :aie:

C'est ce que j'avais déduit, j'ai donc précisé ma pensée :-)
14/11/2017, 23h35
StringBuilder

Oups, désolé, tapé mon code trop vite :)