обнуляемым типа и ReSharper предупреждение

Question

У меня есть следующий код:

private static LogLevel? _logLevel = null; 

public static LogLevel LogLevel 
{ 
    get 
    { 
     if (!_logLevel.HasValue) 
     { 
      _logLevel = readLogLevelFromFile(); 
     } 

     return _logLevel.Value; 
    } 
} 

private static LogLevel readLogLevelFromFile() { ... } 

я получаю предупреждение ReSharper на return заявление о возможном System.InvalidOperationException и предлагает проверить _logLevel, чтобы увидеть, если это null первым. Однако readLogLevelFromFile возвращает LogLevel, а не LogLevel?, поэтому нет необходимости делать заявление return, если _logLevel - null. Это просто недосмотр ReSharper, или я что-то упускаю?

Answer 1

Это похоже на ошибку в Resharper.

Обратите внимание, что это не является потокобезопасным.

The best way to do this is to use a static initializer, как это:

public static LogLevel Instance { get { return Nested.level; } } 

class Nested { 
    // Explicit static constructor to tell C# compiler 
    // not to mark type as beforefieldinit 
    static Nested() { } 

    internal static readonly LogLevel level = readLogLevelFromFile(); 
} 

Answer 2

Вы можете реорганизовать его в нечто вроде этого:

return (_logLevel = _logLevel ?? readLogLevelFromFile()).Value; 

В качестве альтернативы вы можете использовать встроенный в ленивого типа (требуется .NET 4.0 или вы можете свернуть ваш собственный.):

public static LogLevel LogLevel 
{ 
    get { return _logLevel.Value; } 
} 
private static Lazy<LogLevel> _logLevel = new Lazy<LogLevel>(readLogLevelFromFile); 

Answer 3

Resharper не был «умным», чтобы понять его за лет и. Я бы предположил, что это сложная вещь.

Я предпочитаю @ рефакторинга ChaosPandion в любом случае ...