Legitime Verwendung für statische Initialisierung?

Question

Ich erinnere mich, dass ich vor ein paar Jahren statische Initialisierer verwendet habe, um Setup-Operationen auf Klassenebene aufzurufen. Ich erinnere mich daran, dass ich sehr bizarre Verhaltensweisen hatte und ich beschloss einfach, mich von ihnen fern zu halten. Vielleicht lag es daran, dass ich die Bestellung von oben nach unten durcheinander brachte oder ein Neuling war. Aber ich stoße auf die Notwendigkeit, sie noch einmal zu besuchen, und ich möchte sicherstellen, dass es keinen besseren Weg gibt, der ebenso prägnant ist.

Ich weiß, dass es nicht modisch ist, aber ich habe oft datengesteuerte Klassen, die eine statische Liste von Instanzen verwalten, die aus einer Datenbank importiert werden.

public class StratBand { 
     private static volatile ImmutableList<StratBand> stratBands = importFromDb(); 

     private final int minRange; 
     private final int maxRange; 

     private static ImmutableList<StratBand> importFromDb() { 
      //construct list from database here 
     } 
     //constructors, methods, etc 
} 

Wenn ich Dutzende von tabellengesteuerten Klassen wie diese haben, ist dieses Muster sehr prägnant (ja, ich weiß, dass es eng koppelt die Klasse mit einer Datenquelle/Instanzen).

Als ich jedoch die Güte von Google Guava entdeckte, möchte ich den EventBus verwenden, um die statische Liste zu aktualisieren, wenn ein bestimmtes Ereignis gepostet wurde. Ich würde eine statische finale boolesche Variable erstellen, nur um eine statische Methode aufzurufen, die die Registrierung initialisiert hat.

Dies wurde sehr schnell nervig, weil der Compiler würde Warnungen über die abonnierte Variable nie verwendet werden. Außerdem fügte es einfach Unordnung hinzu. Ich frage mich, ob es koscher ist, den statischen Initialisierer zu verwenden, und es gibt wirklich keinen besseren Weg, wenn ich das nicht in zwei oder mehr Klassen entkopple. Gedanken?

public class StratBand { 
      private static volatile ImmutableList<StratBand> stratBands = importFromDb(); 

      static { 
      MyEventBus.get().register(new Object() { 
        @Subscribe 
        public void refresh(ParameterRefreshEvent e) { 
         stratBands = importFromDb(); 
        } 
       }); 
      } 

      private final int minRange; 
      private final int maxRange; 

      private static ImmutableList<StratBand> importFromDb() { 
       //construct list from database here 
      } 
      //constructors, methods, etc 


    } 

Answer 1

Also ich frage mich, ob es koscher ist der statische Initialisierer

Das Komische zu verwenden, ist, dass

private static final boolean subscribed = subscribe(); 

und

private static final boolean subscribed; 
static { 
    subscribed = subscribe(); 
} 

get kompiliert zu genau der gleicher Bytecode. Die Verwendung der unnötigen statischen Variablen ist also streng schlechter.

---

Aber bis wir bereit sind, zu einem DI-driven Rahmen zu skalieren,

Entdecken Guice. Nenne es nicht Framework (obwohl es ist). Es ist einfach zu bedienen und lassen Sie uns static loswerden.

Oder tun Sie es manuell. Schreiben Sie Ihre Klasse neu, indem Sie alle statischen Modifikatoren löschen und an allen Stellen übergeben, an denen Sie sie benötigen. Manchmal ist es ziemlich ausführlich, aber die Angabe von Abhängigkeiten erlaubt es Ihnen, Klassen isoliert zu testen.

So wie es ist, können Sie StratBand nicht testen, ohne die Datenbank zu treffen, egal wie trivial die zu testende Methode ist. Das Problem ist die Kopplung von jeder StratBand Instanz an die Liste aller StratBand s.

Darüber hinaus können Sie das Verhalten nicht abhängig von den stratBands Inhalt testen, wie es immer aus der DB geladen wird (sicher, Sie können Ihre DB entsprechend füllen, aber es ist ein großer Schmerz).

Für den Anfang würde ich StratBandManager (oder StratBands oder was auch immer Name, die Sie mögen) erstellen und verschieben Sie alle statische Funktionalität auf sie. Um einfach den Übergang, würde ich eine temporäre Klasse mit statischen Helfer wie

private static StratBandManager stratBandManager = new StratBandManager(); 
public static ImmutableList<StratBand> stratBands() { 
    return stratBandManager.stratBands(); 
} 

es schaffen Dann deprecate alle und ersetzen Sie es durch DI (mit Guice oder es manuell tun).

---

I find Guice nützlich auch für kleine Projekte. Der Overhead ist winzig, da es oft keine oder kaum Konfiguration gibt.