Как инициализировать статическую переменную readonly с помощью анонимного метода?

Question

Я пытаюсь очистить свой код для инициализации статических переменных readonly.

Оригинал:

public static readonly List<int> MyList; 

//Initialize MyList in the static constructor 
static MyObject() { ... } 

---

Я решил очистить его, потому что CodeAnalysis сказал, что не следует использовать статический конструктор (CA1810).

Cleanup:

public static readonly List<int> MyList = GetMyList(); 

//Returns the list 
private static List<int> GetMyList() { ... } 

---

Я действительно не как дополнительный метод, поэтому я решил попробовать, чтобы заставить его быть все инлайн, но это не будет работать. Я не уверен, что я делаю неправильно здесь ...

public static readonly List<int> MyList = 
    () => 
     { 
      ... 

      return list; 
     }; 

Я попытался взять код в методе GetMyList() и поместить его в анонимный делегат возвращает список, чтобы назначить, но он говорит, я Я пытаюсь преобразовать delegate в List<int>?

Answer 1

Это выглядит немного странно, но попробуйте это:

public static readonly List<int> MyList = new Func<List<int>>(
() => 
{ 
    // Create your list here 
    return new List<int>(); 
})(); 

Уловка создания нового Func<List<int>> и ссылающегося на него.

Answer 2

Это говорит правильно. () => ... возвращает делегата. Но вам нужен список. По этой причине вы должны позвонить этому делегату и получить результат.

Answer 3

Причина заключается в том, что когда вы пишете

() => 
    { 
     ... 

     return list; 
    }; 

Вы фактически объявляя делегат - функция, которая возвращает List<int>, но вы на самом деле не вызывающую эту функцию, и, следовательно, это оценивает в Func<List<int>> и не до List<int>.

Я не совсем понимаю, почему вы пытаетесь удалить свой статический конструктор. Статические конструкторы используют их, а в некоторых случаях даже делают код более понятным и понятным.

Answer 4

он говорит, что я пытаюсь преобразовать делегата в List<int>

Это потому, что вы. Помните, что определение лямбда - это не то же самое, что вызывать лямбду. Вы должны были бы делать что-то больше, как это:

Func<List<int>> createList =() => { ... return list; }; 
MyList = createList(); 

Answer 5

Вы должны немедленно вызвать его, чтобы он стал список:

public static readonly List<int> MyList = new Func<List<int>>(() => { 
    return new List<int>(); 
})(); //<-- the parentheses that invoke 

с синтаксисом в C# для этого настолько сомнительны, что это, вероятно, лучше пойти с отдельным вызовом или метода.

Answer 6

Как многие заявили, простой () => { ... } возвращает делегат или выражение.Я думаю, что создание отдельного метода private static для инициализации списка делает немного более читаемый код, чем () => { ... }() или () => { ... }.Invoke()

Answer 7

Вы забыли скобки. Используйте это:

public static readonly List<int> MyList = (() => 
{ 
    ... 
    return list; 
});