0

我有一个类Singleton（为这个例子简化）

public class Singleton {
    private final static Lock METHOD_1_LOCK = new ReentrantLock();
    private final static Lock METHOD_2_LOCK = new ReentrantLock();
    static {
        try {
            init();
        }catch(InterruptedException ex) {
            throw new ExceptionInInitializerError(ex);
        }
    }

    public static void init() throws InterruptedException {
        ExecutorService executorService = Executors.newCachedThreadPool();
        executorService.submit(() -> {
            method1();
        });
        executorService.submit(() -> {
            method2();
        });
        executorService.shutdown();
        executorService.awaitTermination(Long.MAX_VALUE, TimeUnit.SECONDS);
    }

    public static List<String> method1() {
        METHOD_1_LOCK.lock();
        try {
            try {
                Thread.sleep(1000);
            } catch (InterruptedException e) {
                e.printStackTrace();
            }
            System.out.println("1");
            return Stream.of("b").collect(Collectors.toList());
        }finally {
            METHOD_1_LOCK.unlock();
        }
    }

    public static List<String> method2() {
        METHOD_2_LOCK.lock();
        try {
            try {
                Thread.sleep(500);
            } catch (InterruptedException e) {
                e.printStackTrace();
            }
            System.out.println("2");
            return Stream.of("a").collect(Collectors.toList());
        }finally {
            METHOD_2_LOCK.unlock();
        }
    }

    private Singleton() {
    }
}

Class.forName我想通过调用一个单独的线程来预初始化：

public static void main(String[] args) throws InterruptedException {
    Thread thread = new Thread(() -> {
        try {
            Class.forName(Singleton.class.getName());
        } catch (ClassNotFoundException ex) {
            ex.printStackTrace();
        }
        // working alternative:
        // try {
        //     Singleton.init();
        // }catch(InterruptedException ex) {
        //     ex.printStackTrace();
        // }
    });
    thread.start();
    thread.join();
}

这个构造永远不会从ExecutorService.awaitTermination.

如果我通过注释（并注释掉另一个）切换到标记为“工作替代”的代码并注释掉代码中的static块，则按Singleton预期执行（method1并method2根据输出以相反的顺序调用和返回）。

由于“工作替代方案”可以防止问题并且我可以忍受它，我正在寻找这种行为的解释。

我的意图是使用Class.forName而不是Singleton.init能够添加更多静态初始化任务，Singleton而无需考虑它们是否被预初始化例程覆盖。我同意整个设置并不理想。

4

1 回答 1

3

你正朝着错误的方向前进。首先，不要在类初始化时进行繁重的计算。在类初始化完成之前，类方法的调用受到限制。这个想法不是向类方法显示尚未初始化的变量。只有直接从静态初始化器调用的方法才能被执行，否则它们会被阻塞。在您的情况下，并行任务的调用method1和method2来自并行任务的调用被阻止。

通常，尽可能避免使用静态变量。而是创建对象的实例。对于给定的情况，创建一个类 Singleton 的实例，其中所有变量都从静态字段转换为实例字段。

最后，不要运行线程只是为了调用

  thread.start();
  thread.join();

最好直接调用传递给线程的方法作为Runnable.

于 2019-01-06T10:52:22.583 回答