Discussion :

[meetsmile]

Bonjour tout le monde ,
J'ai un petit probléme de programmation d'un petit exemple :
En faite j'ai une classe Sprite ,
- avec un champ privé color de type Color
-et un champ public pour Color pour la lecture et l'écriture dans le champ précédant (avec get et set)

Code :

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

1
2
3
4
5
6
7
8
 
        Color color = Color.White;
 
        public Color Color
        {
            get { return color; }
            set { color = value; }
        }

-et une autre classe Batte qui hérite de la classe Sprite

Code :

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

class Barre : Sprite

alors voila mon probléme ,dans une méthode de la classe fille Batte dont l'objectif de changer de couleur (exemple très connu ) lorsque je voudrais lire le champ Color précédant pour comparaison dans if ça va ça marche très bien mais lorsque je voudrais écrire sur le champ (par exemple en incrémentant Color.R++) s'affiche une erreur comme quoi je ne peux pas al modifier car elle n'est pas une variable .

Code :

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

1
2
3
 
if (Color.R < nextcolor.R)
                Color.R++ ;

svp d'ou vient l'erreur merci infiniment et bonne journée .

[MoDDiB]

Ton problème n'a pas grand chose à voir avec XNA : cela se produit à chaque fois que tu utiliseras une struct en tant que propriété : il ne te sera pas permis d'en modifier les membres.

Par contre tu peux réaffecter la propriété :

Code :

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

Color = new Color(Color.R+1 , Color.G, Color.B);

Le mieux étant de directement définir dans la classe de base la propriété que tu veux pouvoir modifier :

Code :

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

1
2
3
4
5
public byte Red
{
            get { return color.R; }
            set { color.R = value; }
}

pour ainsi la modifier à ta guise dans la classe fille :

Code :

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

1
2
if (Color.R < nextcolor.R)
                Red++ ;

[meetsmile]

Merci infiniment MoDDiB ,
mais j'ai une petite question dans ce cas ,
pourquoi on a dans ce cas un set dans le champs color s'il n'a en faite aucune utilité et si on ne pourra pas le modifier et écrire dessus .(je précise que c'est un exemple pris et tiré avec une petite modification d'un livre célèbre de xna ).

[MoDDiB]

Merci infiniment MoDDiB ,
mais j'ai une petite question dans ce cas ,
pourquoi on a dans ce cas un set dans le champs color s'il n'a en faite aucune utilité et si on ne pourra pas le modifier et écrire dessus .(je précise que c'est un exemple pris et tiré avec une petite modification d'un livre célèbre de xna ).

Sisi tu peux le modifier comme je le montre dans le premier exemple, certes c'est gênant de tout reconstruire mais ça reste possible.

[Djakisback]

Ton problème n'a pas grand chose à voir avec XNA : cela se produit à chaque fois que tu utiliseras une struct en tant que propriété : il ne te sera pas permis d'en modifier les membres.

Salut, je ne comprends pas ce que tu veux dire par-là, peux-tu expliciter ?

@meetsmile :
Pourquoi tu ne déclares pas simplement Color color en protected, tu pourras y accéder directement depuis tes classes filles. Et je ne comprends pas non plus l'intérêt des getters-setters dans ton cas.

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
 
struct Color        {
       int R, G, B, A;
};
 
 
class Spr	{
public:
	Spr() {
		this->color.R = 0;
		this->color.G = 0;
		this->color.B = 0;
		this->color.A = 255;
	} 
 
	Color getColor() const	{
	  return this->color;
	} 
 
protected:
    Color color;
};
 
 
class BigSpr : Spr	{
public:
	BigSpr() {
		std::cout << this->color.R << "\r\n";
		this->color.R++;
		std::cout << this->color.R << "\r\n";
	} 
};
 
 
int main(int argc, char *argv[])   {
 
    BigSpr* spr = new BigSpr();
    delete spr;
}

[edit]
Oops, désolé pas fait attention que c'était du XNA, je laisse quand même le post car il me semble que le mieux est quand même de gérer cela avec protected.

[MoDDiB]

Salut, je ne comprends pas ce que tu veux dire par-là, peux-tu expliciter ?

Prenons cette struct :

Code :

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

1
2
3
4
5
struct Position2D
{
 public int X;
 public int Y;
}

Si ma struct deviens une propriété d'une classe :

Code :

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

1
2
3
4
5
class Entity
{
 public Position2D Position 
 { get ; set ; }
}

C# ne voudra pas compiler ceci :

Code :

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

1
2
Entity entity = new Entity();
entity.Position.X = 5;

Car au final ce que me retourne Position est une copie de valeur et non une référence, il n'y aurait aucun intérêt à le modifier....

Code :

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

Pourquoi tu ne déclares pas simplement Color color en protected, tu pourras y accéder directement depuis tes classes filles. Et je ne comprends pas non plus l'intérêt des getters-setters dans ton cas.

Normalement si l'on respecte l'encapsulation au maximum on évite de passer directement un champ en protected.

[Djakisback]

Ok, merci pour les infos, je comprends
Par contre, je ne suis pas d'accord pour l'encapsulation, protected sert justement à cela. Je t'explique pas la pile d'appel si tu ne l'utilises pas, surtout dans le JV où tu fais des tonnes de calcul à chaque frame, à moins que tout ça ne soit optimisé au niveau du compilo.

[MoDDiB]

Ok, merci pour les infos, je comprends
Par contre, je ne suis pas d'accord pour l'encapsulation, protected sert justement à cela. Je t'explique pas la pile d'appel si tu ne l'utilise pas.

C'est le même soucis que l'encapsulation classique, mais au final ça rend bien des services
Au final, même pour un jeu vidéo il y aura bien des choses à optimiser avant de se sentir ralentit par ça, et la sécurité du code à long terme c'est souvent le plus important.