静态初始化器的合法用途?

时间:2015-04-06 17:37:36

标签: java guava static-initialization event-bus

我记得几年前我使用静态初始化程序来调用类级别的设置操作。我记得它有非常奇怪的行为,我只是决定避开他们。也许是因为我弄乱了最上层的订单或成为新手。但我遇到了重新审视它们的需要,我想确保没有一种更简洁的方法。

我知道这不流行,但我经常使用数据驱动的类来维护从数据库导入的静态实例列表。

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
}

当我有几十个像这样的表驱动类时,这个模式非常简洁(是的,我知道它将类与一个数据/实例源紧密结合)。

但是,当我发现Google Guava的优点时,我希望在发布特定事件时使用EventBus更新静态列表。我会创建一个静态的最终布尔变量,只是为了调用初始化注册的静态方法。

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

      private final int minRange;
      private final int maxRange;

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

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

这很烦人,因为编译器会对永远不会使用的订阅变量发出警告。而且,它只是增加了混乱。所以我想知道使用静态初始化器是否是犹太教,如果我不将它分成两个或更多类,那么真的没有更好的方法。思考?

 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


    }

1 个答案:

答案 0 :(得分:3)

  

所以我想知道使用静态初始化器是否犹豫不决

有趣的是

private static final boolean subscribed = subscribe();

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

编译成完全相同的字节码。所以使用不必要的静态变量严格来说更糟糕。


  

但在我们准备扩展到DI驱动的框架之前,

发现Guice。不要称之为框架(尽管它是)。它易于使用,让你摆脱static

或者手动完成。通过删除所有静态修饰符重写您的类,并将其传递到您需要的任何位置。它有时候相当冗长,但明确说明依赖关系允许您单独测试类。

就这样,无论测试方法多么微不足道,都无法在不命中数据库的情况下测试StratBand。问题是每个StratBand实例与所有StratBand列表的耦合。

此外,您无法测试依赖于stratBands内容的行为,因为它始终从数据库加载(当然,您可以相应地填充您的数据库,但这是一个很大的痛苦)。

对于初学者,我会创建StratBandManager(或StratBands或您喜欢的任何名称)并将所有静态功能移动到它。为了方便过渡,我创建了一个带有静态助手的临时类,如

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

然后弃用所有内容并将其替换为DI(使用Guice或手动执行)。


I find Guice即使对于小型项目也很有用。由于通常没有或几乎没有任何配置,因此开销很小。