tipo nullable e ReSharper avvertimento

Question

Ho il seguente codice:

private static LogLevel? _logLevel = null; 

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

     return _logLevel.Value; 
    } 
} 

private static LogLevel readLogLevelFromFile() { ... } 

Ho ricevuto un avviso ReSharper sulla dichiarazione return su un possibile System.InvalidOperationException e suggerisce verifico _logLevel per vedere se è null prima. Tuttavia, readLogLevelFromFile restituisce LogLevel, non LogLevel?, quindi non è possibile raggiungere la dichiarazione return quando _logLevel è null. È solo una svista di ReSharper o mi manchi qualcosa?

Answer 1

Sembra un errore in Resharper.

Si noti, tuttavia, che non è thread-safe.

The best way to do this is to use a static initializer, in questo modo:

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

Si potrebbe refactoring in qualcosa di simile:

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

In alternativa è possibile utilizzare il costruito nel tipo pigro (Richiede .NET 4.0 o si può rotolare il tuo.):

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

Answer 3

Resharper non era "intelligente" abbastanza da capirlo per te u. È una specie di cosa complessa da capire, immagino.

Preferisco il refactoring di @ ChaosPandion comunque ...