[débutant]utilisation des propriétés

**isoman** · 10/12/2008, 22h32

Salut,
je commence le c# et j'aimerais vous posez une question .
parmi les 2 bout de code suivant quel est le code le plus correct ?

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
 
class test
{
String nbreChoix;
   public string NBRECHOIX
        {
            get
            {
                return nbreChoix;
            }
            set
            {
                nbreChoix = value;
            }
        }
}

Code :

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

1
2
3
4
5
6
 
class test
{
   public string nbreChoix
        { get; set }
}

Merci d'avance.

**Gold Bug** · 10/12/2008, 23h02

La seconde version ne fonctionne qu'en C# 3 (Framework 3.5).
Seule la première version est compatible avec le Framework 2.0

**Gold Bug** · 10/12/2008, 23h07

Si tu passes l'outil d'analyse de code StyleCop, il te fera faire plusieurs modifications :
- Le nom de ta classe doit commencer par une majuscule
- Ajouter explicitement le modificateur de portée sur ton champ (donc private)
- préfixer les accès à ton champ privé par this. pour spécifier que tu accèdes au champ d'une instance. Ca va en gros ressembler à ça

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
public class Test
{
    private string champ1;
 
    public string Champ1
    {
          get { return this.champ1; }
          set { this.champ1 = value; }
    }
}

**smyley** · 10/12/2008, 23h09

Il me semble que c'est plutôt Framework 3.0 pas 3.5
bref à partir du Frmwrk 3.0 les deux codes font la même chose. Quand le compilateur vois :

Code :

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

1
2
public string nbreChoix
        { get; set }

il écrit ça dans le programme

Code :

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

1
2
3
4
5
6
7
8
 
string backingFieldBidule = "";
 
public string nbrChoix
 {
  get {return backingFieldBidule;}
  set{backingFieldBidule = value;}
 }

D'où le résultat ...

**Gold Bug** · 10/12/2008, 23h10

Autre remarque, un champ qui s'intitule nbrechoix suppose qu'on va y stocker une valeur numérique (le nombre de choix). Or tu l'as déclaré de type string, ce n'est donc pas cohérent.
Je te conseillerai donc d'utiliser le type int pour stocker des entiers, ou float pour des décimaux.

**Philippe Vialatte** · 11/12/2008, 00h32

Envoyé par smyley

Il me semble que c'est plutôt Framework 3.0 pas 3.5

Framework 3.5 / C# 3.0

++ pour la remarque de goldbug.

Apres, pour les membres prives, tu as plusieurs conventions, entre les minuscules prefixees par this, les prefixes m_ et les prefixes _, c'est une guerre de religion qui dure depuis un bon moment

(personnelement, je suis dans le camp des _, mais chacun son truc

)

**smyley** · 11/12/2008, 00h38

Et puis non, c'est moi qui ai presque raison. En fait avec VS 2008 on peut utiliser ce genre de facilités même avec un projet en .NET 2.0 car c'est une fonctionnalité du compilateur (qui génère lui même ce qu'il faut), pas une fonctionnalité de la plateforme d'exécution (le CLR).
Donc voilà, ça a beau être du C# 3.0 on peut l'utiliser sur un projet en .NET 2.0 (pourvu d'utiliser le bon compilateur).

**tomlev** · 11/12/2008, 01h18

Pour répondre à la question initiale : les 2 écritures sont grosso-modo équivalentes. En fait, la 2e est pratique si tu n'as rien de particulier à faire dans le get et le set. Si, par exemple, tu as besoin d'implémenter un lazy loading dans le get, ou le déclenchement d'un évènement dans le set, tu es obligé d'utiliser la 1e écriture.