Discussion :

[PaulP]

Bonjour,

Je cherche a élaborer une classe contenant 5 propriétés dont une read only.

Les 4 premières propriétés sont des "filtres" servant à l'exécution de la méthode "GetResults". Le but étant que seule cette méthode puisse affecter le contenu de results.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
            public class MaClasse
            {
                public string Criteria1 { get; set; }
                public string Criteria2 { get; set; }
                public DateTime Date1 { get; set; }
                public DateTime Date2 { get; set; }
                public readonly List<string> Results { get; set; }
 
                public void GetResults()
                {
                    //ICI LE CODE QUI DOIT ALIMENTER RESULTS
                }
 
            }

En écrivant mon code comme ceci j'obtiens le message :
"Le modificateur 'readonly' n'est pas valide pour cet élément".

Je m'y prends donc mal.
Pouvez-vous s'il vous plaît m'aider?

D'avance merci.

[antoine.debyser]

Bonjour,

Le modificateur "readonly" est réserver au attribut. Pour avoir une propriété readonly, il suffit de ne pas mettre de setter.

Pour ton cas je ferai les choses suivantes:
1) ne pas mettre de "Setter" sur Results
2) Renommer GetResults en quelques chose de plus parlant comme ComputeResults
3) Changer le type "Results" pour une collection readonly comme ReadOnlyCollection<T>

Ce qui donnerai

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
            public class MaClasse
            {
                private readonly List<string> _results = new List<string();
                private readonly ReadOnlyCollection<string> _resultsReadonly = _results.AsReadOnly();
 
                public string Criteria1 { get; set; }
                public string Criteria2 { get; set; }
                public DateTime Date1 { get; set; }
                public DateTime Date2 { get; set; }
                public readonly IList<string> Results { get { return _resultsReadonly; } }
 
                public void ComputeResults()
                {
                    //ICI LE CODE QUI DOIT ALIMENTER RESULTS
                }
 
            }

[Babyneedle]

Ça dépend...

Si les propriétés Criteria et Date n'ont d'utilité que pour alimenter la requête alors c'est un mauvais design.

Il vaudrait mieux créer une méthode avec des paramètres. De cette manière, tu te retrouves avec une classe service qui est stateless.

Sinon, si les propriétés en question sont utilisées dans d'autres contextes, alors l'intervention précédente est à-propos.

[PaulP]

@Antoine, cela fonctionne, j'ai juste dû corriger 2 niveaux d'accesibilité

- Ajout de static sur _results sinon innacessible pour _resultsReadonly
- Suppression de readonly sur Results sinon même erreur qu'au début.

Ce qui donne :

Code :

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

1
2
3
4
5
6
7
8
            private static List<string> _results = new List<string>();
            private readonly ReadOnlyCollection<string> _resultsReadonly = _results.AsReadOnly();
 
            public string Criteria1 { get; set; }
            public string Criteria2 { get; set; }
            public DateTime Date1 { get; set; }
            public DateTime Date2 { get; set; }
            public IList<string> Results { get { return _resultsReadonly; } }

@BabyNeedle, il s'agit donc d'un mauvais design. Je vais transformer tout ça en méthode et garder quelque part la façon de procéder d'Antoine, cela me reservira surement.

Merci beaucoup à vous deux.