Discussion :

[Iloyo]

Bonjour à tous,

Je suis en train de concevoir un programme en C# qui utilise des structures.

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
Exemple de structure :
        public struct Personne
        {
            public int id;
            public string name;

            public Personne(int id, string name)
            {
                this.name= name;
                this.id= id;
            }

            public int ID
            {
                get { return this.id; }
                set { this.id= value; }
            }

            public string Name
            {
                get { return this.name; }
                set { this.name= value; }
            }
        }

Je parviens bien à créer une instance de ma structure...

Code :

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

1
2
3
 
 System.Collections.Generic.Dictionary<int, Personne> pers = new System.Collections.Generic.Dictionary<int, Personne>();
 pers.Add(1,new Personne(1,"toto");

... toutefois, je ne parviens pas à modifier les variables qui la compose :

Code :

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

1
2
3
 
pers[1].ID = 2;
pers[1].Name = "titi";

Erreur : Impossible de modifier la valeur de retour de 'System.Collections.Generic.Dictionary<int, Personne>.this[int]', car il ne s'agit pas d'une variable.

Quelqu'un pourrait-il m'expliquer et/ou m'orienter vers une solution afin de pouvoir modifier les variables ?

Merci

[emiscool]

tu pourra modifier les éléments de ton objet Dictionary via la méthode suivante:

Code :

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

1
2
Personne p = new Personne(2, "titi");
pers[1] = p;

pour plus d'information sur l'utilisation du type Dictionary , voila un lien http://msdn.microsoft.com/fr-fr/libr...08(VS.80).aspx

[Jérôme Lambert]

La raison pour laquelle il n'est pas possible de faire ce que tu fais, c'est tout simplement que l'indexeur ([x]) te renvoie une copie de ton objet (étant donné qu'il s'agit d'un type valeur).

Le compilateur c# interdit donc cette opération car l'affectation d'une nouvelle valeur sur une propriété de l'objet retourné ne servira à rien, vu que tu vas le faire sur une copie.

La bonne façon de faire est bien celle indiquée par emiscool :

Code :

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

1
2
3
4
Personne newPer1 = new Personne ();
newPer1 .ID = 2;
newPer1 .Name = "titi";
pers[1] = newPer1;

Pour plus d'information sur l'erreur générée par le compilateur : http://msdn.microsoft.com/fr-fr/library/wydkhw2c.aspx

[tomlev]

Je confirme l'explication de Jérôme, je voudrais juste ajouter quelques commentaires...

- Pourquoi utiliser une structure dans ce cas ? Les structures sont des types valeurs, elles sont à utiliser quand tu veux donner à ton type une sémantique de valeur. Une personne ne correspond pas vraiment à ce cas d'utilisation... Si tu veux passer une personne en paramètre d'une méthode, ça n'a pas de sens de passer une copie de la personne, il vaudrait mieux passer une référence. Si tu ne comprends pas vraiment ce qu'implique l'utilisation d'une structure par rapport à une classe, choisis toujours une classe, tu auras moins de surprises

- Evite de déclarer des champs publics dans une classe ou une structure. Les champs devraient toujours être privés (sauf à la rigueur les champs static readonly dans certains cas), et n'être accessibles que via des propriétés. Tu as déjà créé les propriétés, tu n'as donc plus qu'à rendre les champs privés.

[Iloyo]

Pour répondre à tomlev, c'est la première fois que j'utilise des structures en .NET. J'ai choisi cela car j'ai vu quelque part que les structures occupent moins de ressources que les classes.

Finalement, au vu de ce que je souhaite réaliser, je pense que je vais opter pour des classes.

Je vous remercie pour toutes vos réponses explicatives.

[tomlev]

J'ai choisi cela car j'ai vu quelque part que les structures occupent moins de ressources que les classes.

Je ne sais pas où tu as vu ça, mais ce n'est absolument pas vrai...

Il y a beaucoup d'idées fausses très répandues sur les types valeurs en .NET, genre "les types valeurs sont alloués sur la pile"... Si tu comprends l'anglais je te conseille de lire ces articles :
http://blogs.msdn.com/b/ericlippert/...lue-types.aspx
http://blogs.msdn.com/b/ericlippert/...lue-types.aspx

[nicolas.sitbon]

Je ne sais pas où tu as vu ça, mais ce n'est absolument pas vrai...

Ce n'est pas absolument faux non plus, au moins sur l'implémentation de Microsoft, si le value type n'est pas alloué sur le tas, il ne possède pas de sync block index, ni de Type Object Pointer. C'est peut être négligeable (on parle de quelques bytes) mais ça rend ton affirmation en partie fausse. Qui plus est, à la défense du PO, même des grands noms comme Jeffrey Richter font l'amalgame (ou plûtot la généralité) de dire que les value type sont alloués sur la pile à moins d'être boxé ou faire partie d'un reference type (voir CLR via C# 3ème édition, page 121).
Cela étant dit, il ne fait aucun doute dans ce cas précis que le type Personne doit être une classe et non une structure.
Cordialement.