我正在用libgdx设计一个游戏,我决定让某些经理类单例,因为我注意到我经常只使用一个类的一个实例,然后通过构造函数将同一个实例传递给许多其他类,这是非常痛苦。现在,我有一个管理器类,它在其构造函数中初始化了许多其他类。我通过为每个类使用静态块初始值设定器来完成此操作,如下所示:
public class Example{
private static Example instance;
static{
try{
synchronized(Example.class){
instance = new Example();
}
}catch(Exception e){
throw new RunTimeException("Failure to initialize Example instance");
}
public static Example getInstance(){
return instance;
}
在主管理器中,我通过getInstance方法创建每个类的实例。
出现的问题是:说我有静态单例类Example1和Example2 在Example1的构造函数中,我创建了一个名为:
的变量 example2 = Example2.getInstance();
但是因为example2和example1需要使用彼此的方法,所以在Example2的构造函数中我做了:
example1 = Example1.getInstance();
问题应该很容易看出来。因为example1正在等待example2完成初始化,而example2需要example1的实例,它最终会创建一个死锁并通过上面的代码RunTimeException崩溃。
这似乎很容易使用两个示例类来修复,但是当我有6个不同的单例管理器类几乎都需要以某种方式进行通信时,问题就会混乱。最简单的解决方案显然不会使用这种方法,但这需要我重写我的大部分代码。
我无法弄清楚如何使用这种单例类方法而不会遇到这个问题,因为大多数类需要构造函数中其他类的信息才能运行。
从单例类的构造函数中删除所有代码,还是执行其他操作?
答案 0 :(得分:1)
这不是死锁,它是无限递归。没有办法绕过它,你必须重构你的代码。
最好的事情不是在构造函数中有任何逻辑,只是初始化成员变量。由于您不需要将单例作为成员存储在类中,因此实际上不需要在构造函数中访问它们。只需使用适当的getInstance()方法从其他单例的方法内部访问单例。
答案 1 :(得分:0)
我不再使用很多单身人士了。我认为单例是一个用例,而不是“类的类”,然后依靠其他东西来管理它的“单例”(例如注入框架)。当我没有其中之一时,我创建了一个单独的“单例”来管理要作为单例使用的应用程序类。
因此,在这种情况下,您可以让此类为您管理构造和相互依赖性,而不是让类自己管理它们。
public class Singletons {
private Example1 example1;
private Example2 example2;
private Example3 example3;
private static Singletons instance;
static {
Example1 example1 = new Example1();
Example2 example2 = new Example2();
Example3 example3 = new Example3();
instance = new Singletons();
example1 = new Example1();
example2 = new Example2();
example3 = new Example3();
example1.setExample2(example2);
example2.setExample3(example3);
example3.setExample1(example1);
instance.setExample1(example1);
instance.setExample2(example2);
instance.setExample3(example3);
}
public Example1 getExample1() {
return example1;
}
private void setExample1(Example1 example1) {
this.example1 = example1;
}
public Example2 getExample2() {
return example2;
}
private void setExample2(Example2 example2) {
this.example2 = example2;
}
public Example3 getExample3() {
return example3;
}
private void setExample3(Example3 example3) {
this.example3 = example3;
}
public Singletons getInstance() {
return instance;
}
}