¿Está usando Lazy <T> malo para el rendimiento?

Question

Recientemente tuve algunos problemas con una clase de singelton que era perezoso al inicializar un diccionario donde un segundo hilo intentaba usarlo antes de que realmente se hubiera rellenado. Así que implementé la inicialización de la variable a través de la clase Lazy<T>.

Aquí está mi código:

private static Dictionary<string, string> GroupDefaults 
{ 
    get { return mGroupDefaults.Value; } 
} 
private static Lazy<Dictionary<string, string>> mGroupDefaults = 
    new Lazy<Dictionary<string,string>>(delegate 
    { 
     Dictionary<string, string> defaults = new Dictionary<string, string>(); 
     foreach (KeyValuePair<string, UnitGroup> groupDef in Groups) 
      defaults.Add(groupDef.Key, groupDef.Value.First().Key); 
     return defaults; 
    }); 

que esto esté arreglado el problema y ahora estoy considerando hacer de esto una práctica regular de la mina para utilizar la clase Lazy<T> en cualquier lugar que hago inicialización lenta para evitar posibles problemas de threads. Entonces, básicamente, me gustaría saber si esta es una buena/práctica común. ¿O será detremental para el rendimiento o algo así?

Answer 1

Es bastante difícil de decir sin saber qué tipo de restricciones de rendimiento tiene, pero en mi experiencia, la inicialización rara vez es un cuello de botella (ya que por definición solo ocurre una vez) Lazy<T> se escribió para proporcionarle este servicio exacto, entonces recomendaría usarlo.

Answer 2

Creo que puede estar utilizando Lazy para su uso no previsto. Lazy se debe usar para situaciones en las que algo tiene un gran costo de inicialización, pero existe la posibilidad de que no se use durante la vida del objeto.

Si siempre a GroupDefaults al menos una vez por su vida útil un método mejor sería para inicializar GroupDefaults en un subproceso en segundo plano en el inicio de la vida del envase y la esperanza de que se haga antes de que se realiza la inicialización (Sé que hay una clase para esto, pero necesito profundizar en el MSDN para encontrarlo)

Answer 3

Si esto es para un singleton, un static constructor podría ser lo que desea. Algo así como:

class MySingleton 
{ 
    static MySingleton() 
    { 
     Instance().InitDict(); 
    } 
} 

Answer 4

De la documentación, encuentro lo siguiente:

Si ningún delegado se pasa en el constructor perezoso, el tipo envuelta se crea mediante el uso de Activator.CreateInstance cuando el valor de la propiedad primero se accede. Si el tipo no tiene un constructor predeterminado, se lanza una excepción en tiempo de ejecución.

Activator.CreateInstance es un método que es notoriamente malo para el rendimiento. Sin embargo, eso no parece ser un problema en su caso, y en cualquier caso, como dijo dlev, invocar el método una vez no sería un problema. No he visto Lazy<T> usado muy a menudo, pero no veo ninguna razón para no usarlo en su caso.