C# Copia variable de instancia de variable local en funciones de la misma clase

Question

He estado mirando a través de un cierto código en un proyecto de código abierto recientemente y encontraron muchas ocurrencias de este tipo de código:

class SomeClass 
{ 
    private int SomeNumber = 42; 

    public ReturnValue UseSomeNumber(...) 
    { 
     int someNumberCopy = this.SomeNumber; 
     if (someNumberCopy > ...) 
     { 
      // ... do some work with someNumberCopy 
     } 
     else 
     { 
      // ... do something else with someNumberCopy 
     } 
    } 
} 

¿Hay algún beneficio real para hacer una copia de la variable de instancia?

Answer 1

No, a menos que no desee cambiar el valor de SomeNumber y tenga la intención de actualizar someNumberCopy. Como si fuera a repetir el número de veces e hiciera decrecer algunaNumberCopy a cero para hacer un seguimiento del recuento.

Supongo que copiar la variable de esa manera podría protegerlo de alguna función externa que modifique SomeNumber y la cambie sin su conocimiento mientras realiza una operación. Podría ver esto si se supone que la clase debe usarse en una aplicación de subprocesos múltiples. Tal vez no de la forma en que lo haría, pero podría haber sido la intención del autor.

Answer 2

La única ventaja que veo es por tener una "versión de sólo lectura" de la variable sólo para ese método

Answer 3

no. deja este trabajo para el optimizador.

Answer 4

¿El // ... do some work with someNumberCopy está haciendo algo con someNumberCopy? ¿Está cambiando el valor? ¿Se está transfiriendo a un método que podría cambiar el valor?

De lo contrario, no, no hay ningún beneficio.

Answer 5

Posiblemente esto es parte del programa de subprocesos múltiples. Aunque este código no es seguro para subprocesos, garantiza que una vez que se asigna la variable de copia, no se modifique por otros subprocesos, y todo el código de función después de esto se ejecuta de forma coherente.

código similar con los eventos se vuelve crítica en el entorno multi-hilo:

 
MyEvent e = this.myEvent; 

if (e != null) 
{ 
    e(); 
} 

Aquí, sin hacer una copia local, es posible conseguir una excepción nula-puntero, si el evento será nulo después de evaluar la nula, y antes de invocar

Answer 6

Olvidó algunos detalles importantes.
Si tiene varios hilos con el acceso y someNumber este escenario:

int someNumberCopy = this.SomeNumber; 
    if (someNumberCopy > x) 
    { 
     // do some work with someNumberCopy 
     // that relies on (someNumberCopy > x) == true 
    } 

entonces es importante hacer una copia.

Answer 7

Podría ser útil si this.SomeNumber pueden ser modificados por otro hilo, pero durante la duración de este método es de vital importancia que el someNumberCopy no se puede cambiar (que puede estar fuera de sincronía con SomeNumber), entonces esto podría ser viable, de lo contrario No veo una razón para eso.