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

Version imprimable

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:

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:

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

Merci d'avance.

10/12/2008, 23h02
Gold Bug

La seconde version ne fonctionne qu'en C# 3 (Framework 3.5).
Seule la première version est compatible avec le Framework 2.0
10/12/2008, 23h07
Gold Bug
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:

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; } } }
10/12/2008, 23h09
smyley
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:

1 2 public string nbreChoix { get; set }
il écrit ça dans le programme
Code:

1 2 3 4 5 6 7 8 string backingFieldBidule = ""; public string nbrChoix { get {return backingFieldBidule;} set{backingFieldBidule = value;} }
D'où le résultat ...
10/12/2008, 23h10
Gold Bug

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.
11/12/2008, 00h32
Philippe Vialatte

Citation:

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 :) )
11/12/2008, 00h38
smyley

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).
11/12/2008, 01h18
tomlev

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.
11/12/2008, 09h00
Gold Bug

Citation:

Envoyé par pvialatte

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

Ben moi aussi je l'étais... jusqu'à ce qu'on décide dans ma boîte de passer StyleCop sur tous nos sources :cry:
11/12/2008, 09h37
Philippe Vialatte

eheh, nous aussi, on es passe a stylecop...mais la regle a vite ete remplacee pour s'adapter a notre style ;)
11/12/2008, 12h25
User.Anonymous

La 2ème ne marchera pas, il manque le ; après le set :mouarf:

Pour la 1ère, même remarque que Gold Bug, mais dans l'absolu les 2 doivent marcher de manière identique.