单例类的Java静态初始化中的死锁

时间:2018-01-13 03:56:45

标签: java singleton singleton-methods

我正在用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个不同的单例管理器类几乎都需要以某种方式进行通信时,问题就会混乱。最简单的解决方案显然不会使用这种方法,但这需要我重写我的大部分代码。

我无法弄清楚如何使用这种单例类方法而不会遇到这个问题,因为大多数类需要构造函数中其他类的信息才能运行。
从单例类的构造函数中删除所有代码,还是执行其他操作?

2 个答案:

答案 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;
    }
}