¿Cuál es la mejor manera de corregir este 'write to static field from instance method' findbugs warning?

Question

Tengo una clase que se parece a esto, y findbugz se queja del 'escribir en el campo estático del método de instancia' (initialize() y killStaticfield()). No puedo configurar el campo estático en el ctor.

• ¿Cuál es la mejor solución para este problema?

• ¿Sería suficiente con poner staticField en una AtomicReference?

  public class Something 
  { 
      private static SomeClass staticField = null; 
      private AnotherClass aClass; 
      public Something() 
      { 
  
      } 
  
      public void initialize() 
      { 
      //must be ctor'd in initialize 
      aClass = new AnotherClass(); 
      staticField = new SomeClass(aClass); 
      } 
  
      public void killStaticField() 
      { 
      staticField = null; 
      } 
  
      public static void getStaticField() 
      { 
      return staticField; 
      } 
  } 
  

Answer 1

permanecer lo más cerca posible a su diseño original ...

public class Something { 
    private static volatile SomeClass staticField = null; 

    public Something() { 
    } 

    public static SomeClass getStaticField() { 
    if(Something.staticField == null) 
     Something.staticField = new SomeClass();; 
    return Something.staticField; 
    } 
} 

referencia a su variable estática a través del nombre de la clase, que eliminará el aviso findbugz. Marque su variable estática como volátil, lo que hará que la referencia sea más segura en un entorno multiproceso.

Aún mejor sería:

public class Something { 
    private static final SomeClass staticField = new SomeClass(); 

    public Something() { 
    } 

    public static SomeClass getStaticField() { 
    return Something.staticField; 
    } 
} 

Answer 2

La cuestión es lo que quiere hacer con el campo estático. Si cambia para cada clase que crees, puede que no sea una buena idea tenerlo estático. Si se inicializa solo una vez, debería inicializarlo como un singleton.

public class Something 
{ 
    private static SomeClass staticField = null; 

    public Something() 
    { 

    } 

    public static SomeClass getStaticField() 
    { 
     if(staticField == null) 
      staticField = new SomeClass();; 
     return staticField; 
    } 
} 

Answer 3

Elimina la estática de staticField si no debería ser estática.

Make kill y getStaticField static. Y normalmente hace referencia a la estática por el nombre de la clase, no por una (implícita) esto, para dejar muy claro que es estático y puede causar consecuencias inesperadas en otros thReads.

En caso de duda, no utilice estadísticas para campos no constantes.