tipo anulable y una advertencia ReSharper

Question

Tengo el siguiente código:

private static LogLevel? _logLevel = null; 

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

     return _logLevel.Value; 
    } 
} 

private static LogLevel readLogLevelFromFile() { ... } 

recibo una advertencia ReSharper en la cuenta de return sobre un posible System.InvalidOperationException y sugiere verifico _logLevel para ver si es null primero. Sin embargo, readLogLevelFromFile devuelve LogLevel, no LogLevel?, por lo que no hay manera de que se llegue a la declaración return cuando _logLevel es _logLevel es . ¿Es solo un descuido de ReSharper, o me estoy perdiendo algo?

Answer 1

Parece un error en Resharper.

Tenga en cuenta, sin embargo, que eso no es seguro para subprocesos.

The best way to do this is to use a static initializer, así:

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

Usted podría refactorizar en algo como esto:

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

Alternativamente, puede utilizar el construido en el tipo perezoso (Requiere .NET 4.0 o se puede rodar la suya):

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

Answer 3

Resharper no era lo suficientemente "inteligente" como para resolverlo tú Es algo complejo de entender, me imagino.

prefiero @ refactorización de ChaosPandion de todos modos ...