我有一个看起来与此类似的类,并且findbugz抱怨'从实例方法写入静态字段'(initialize()
和killStaticfield()
)。我无法在ctor中设置静态字段。
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;
}
}
保持尽可能接近原始设计......
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;
}
}
通过类名称引用静态变量,这将删除findbugz警告。将静态变量标记为volatile,这将使多线程环境中的引用更安全。
更好的是:
public class Something {
private static final SomeClass staticField = new SomeClass();
public Something() {
}
public static SomeClass getStaticField() {
return Something.staticField;
}
}
如果staticField不是静态的,则从staticField中删除静态。
自己使kill和getStaticField静态。并且您通常通过类名引用静态,而不是通过(隐式)引用静态,以明确它是静态的并且可能在其他thReads中导致意外后果。
如有疑问,请勿对非常量字段使用静态。
问题是你想用静态字段做什么。如果它对你创建的每个类都有所改变,那么将它静态化可能不是一个好主意。如果它只被初始化一次你应该懒得将它初始化为单例。
public class Something
{
private static SomeClass staticField = null;
public Something()
{
}
public static SomeClass getStaticField()
{
if(staticField == null)
staticField = new SomeClass();;
return staticField;
}
}