(C#) itérer sur un membre de collection privée en lecture seule

Question

J'ai une classe qui a deux collections HashSet<String> en tant que membres privés. Les autres classes de mon code aimeraient pouvoir parcourir ces HashSets et lire leur contenu. Je ne veux pas écrire un getter standard car une autre classe pourrait encore faire quelque chose comme myClass.getHashSet().Clear(); Existe-t-il un autre moyen d'exposer les éléments de mes HashSets à l'itération sans exposer la référence au HashSet lui-même? J'aimerais pouvoir le faire d'une manière compatible avec chaque boucle.

Answer 1

Expose une IEnumerable<T> propriété:

public IEnumerable<whatevertype> MyHashSet { 
    get { 
     return this.myHashSet; 
    } 
} 

Bien sûr, l'utilisateur de ce code peut lancer cette IEnumerable<T> à un HashSet<T> et éléments modifier, afin d'être sur le côté sûr (tout mal performance), vous pouvez faire:

public IEnumerable<whatevertype> MyHashSet { 
    get { 
     return this.myHashSet.ToArray(); 
    } 
} 

ou:

public IEnumerable<whatevertype> MyHashSet { 
    get { 
     foreach(var item in this.myHashSet) { 
      yield return item; 
     } 
    } 
} 

Une méthode plus performante de protection, mais moins pratique à l'appelant, est de retourner un IEnumerator<T>:

public IEnumerator<whatevertype> GetMyHashSetEnumerator() { 
    return this.myHashSet.GetEnumerator(); 
} 

Answer 2

Ajouter une méthode/propriété comme celui-ci pour éviter d'exposer le récipient réel:

public IEnumerable EnumerateFirst() 
{ 
    foreach(var item in hashSet) 
     yield return item; 
} 

Answer 3

Marque votre getter expose le HashSet comme IEnumerable.

private HashSet<string> _mine; 

public IEnumerable<string> Yours 
{ 
    get { return _mine; } 
} 

Si le type générique est mutable, alors que peut encore être modifié, mais aucun élément peuvent être ajoutés ou retirés de votre HashSet.

Answer 4

Vous pouvez également fournir une séquence comme ceci:

public IEnumerable<string> GetHashSetOneValues() 
{ 
    foreach (string value in hashSetOne) 
     yield return value; 
} 

Cette méthode peut alors être appelée dans une boucle foreach:

foreach (string value in myObject.GetHashSetOneValues()) 
    DoSomething(value); 

Answer 5

Vous pouvez également utiliser la méthode Select pour créer une enveloppe de ne peut pas être rejetterait à HashSet<T>:

public IEnumerable<int> Values 
{ 
    get { return _values.Select(value => value); 
} 

Th évite d'itéter plus de deux fois _values, comme vous le feriez avec .ToArray(), mais conserve l'implémentation sur une seule ligne propre.

Answer 6

En supposant que vous utilisez .NET 3.5, une alternative à l'écriture du code de rendement vous-même est d'appeler une méthode LINQ. Par exemple:

public IEnumerable<string> HashSet 
{ 
    get { return privateMember.Select(x => x); } 
} 

ou

public IEnumerable<string> HashSet 
{ 
    get { return privateMember.Skip(0); } 
} 

Il existe différents opérateurs de LINQ qui pourraient être utilisés comme celui-ci - en utilisant Skip(0) est probablement le plus efficace, comme après la première « sauter 0 valeurs » boucle, il est probablement juste la boucle foreach/yield return montrée dans les autres réponses. La version Select appellera le délégué de projection no-op pour chaque élément généré.Les chances que cette différence soit significative sont astronomiquement faibles, cependant - je suggère que vous alliez avec ce qui rend le code le plus clair pour vous.

Answer 7

Cela peut être un peu trop tard pour la fête mais la façon la plus simple aujourd'hui serait d'utiliser Linq. Au lieu d'écrire

public IEnumerable<string> GetValues() 
{ 
    foreach(var elem in list) 
     yield return elem; 
} 

vous pouvez écrire

public IEnumerable<string> GetValues() => list;