Inline tworzenie instancji stałej List

Próbuję zrobić coś takiego:

public const List<String> METRICS = new List<String>()
        {
            SourceFile.LOC,
            SourceFile.MCCABE,
            SourceFile.NOM,
            SourceFile.NOA,
            SourceFile.FANOUT,
            SourceFile.FANIN,
            SourceFile.NOPAR,
            SourceFile.NDC,
            SourceFile.CALLS
        };

Ale niestety to nie działa:

FileStorer.METRICS' is of type 'System.Collections.Generic.List<string>'. A const field of a reference type other than string can only be initialized with null.

Jak mogę rozwiązać ten problem?

constdotyczy stałych czasu kompilacji. Można to po prostu zrobić static readonly, ale dotyczyłoby to tylko METRICSsamej zmiennej (która zwykle powinna być Metryki, zgodnie z konwencjami nazewnictwa .NET ). Nie uczyniłoby to listy niezmienną - więc ktoś mógłby zadzwonićMETRICS.Add("shouldn't be here");

Możesz użyć a, ReadOnlyCollection<T>aby go owinąć. Na przykład:

public static readonly IList<String> Metrics = new ReadOnlyCollection<string>
    (new List<String> { 
         SourceFile.LoC, SourceFile.McCabe, SourceFile.NoM,
         SourceFile.NoA, SourceFile.FanOut, SourceFile.FanIn, 
         SourceFile.Par, SourceFile.Ndc, SourceFile.Calls });

ReadOnlyCollection<T>po prostu otacza potencjalnie zmienną kolekcję, ale ponieważ nic innego nie będzie miało dostępu do List<T>późniejszej, możesz uznać całą kolekcję za niezmienną.

(Wielkie litery to głównie domysły - użycie pełniejszych nazw sprawiłoby, że byłyby jaśniejsze, IMO).

Czy zadeklarować ją jako IList<string>, IEnumerable<string>, ReadOnlyCollection<string>lub coś innego jest do ciebie ... jeśli można oczekiwać, że powinien być traktowany jedynie jako ciąg, a następnie IEnumerable<string>będzie prawdopodobnie najbardziej odpowiednie. Jeśli kolejność ma znaczenie i chcesz, aby ludzie mieli do niego dostęp za pomocą indeksu, IList<T>może być odpowiednie. Jeśli chcesz, aby niezmienność była widoczna, zadeklaruj ją jako ReadOnlyCollection<T>przydatną - ale nieelastyczną.

Zamiast tego musisz użyć static readonlylisty. A jeśli chcesz, aby lista była niezmienna, możesz rozważyć użycie ReadOnlyCollection<T>zamiast List<T>.

private static readonly ReadOnlyCollection<string> _metrics =
    new ReadOnlyCollection<string>(new[]
        {
            SourceFile.LOC,
            SourceFile.MCCABE,
            SourceFile.NOM,
            SourceFile.NOA,
            SourceFile.FANOUT,
            SourceFile.FANIN,
            SourceFile.NOPAR,
            SourceFile.NDC,
            SourceFile.CALLS
        });

public static ReadOnlyCollection<string> Metrics
{
    get { return _metrics; }
}

Szukasz prostego kodu, takiego jak ten:

    List<string> tagList = new List<string>(new[]
    {
         "A"
        ,"B"
        ,"C"
        ,"D"
        ,"E"
    });