我记得几年前我使用静态初始化程序来调用类级别的设置操作。我记得它有非常奇怪的行为,我只是决定避开他们。也许是因为我弄乱了最上层的订单或成为新手。但我遇到了重新审视它们的需要,我想确保没有一种更简洁的方法。
我知道这不流行,但我经常使用数据驱动的类来维护从数据库导入的静态实例列表。
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
}
答案 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即使对于小型项目也很有用。由于通常没有或几乎没有任何配置,因此开销很小。