Java在对象构造函数中启动一个线程池

Question

在对象构造函数中启动线程池是否安全？我知道您不应该从构造函数启动线程，这是关于“this”指针转义的事情(我并不完全理解这一点，但会进行更多搜索以尝试找出答案)。

代码看起来像这样:

private ExecutorService pool;

public handler()
{
  pool = Executors.newCachedThreadPool();
}

public void queueInstructionSet(InstructionSet set)
{
  pool.submit(new Runnable that handles this instruction set);
}

如果这不起作用，我可以将此类创建为 Runnable 并在新线程中启动它。然而，这似乎是在它并不真正需要的地方向程序添加了一个不必要的线程。

谢谢。

编辑:

感谢大家的回复，他们肯定帮助理解了这一点。

根据代码，在我看来这个构造函数创建线程池是有道理的，但让我解释一下这段代码具体做了什么，因为我可能会以一种奇怪的方式思考这个问题。

这个对象的全部意义在于采用“指令集”对象，并相应地对其进行操作。指令集来自连接到服务器的客户端。一旦从客户端接收到完整的指令集，该指令集就会发送到此对象(处理程序)进行处理。

此处理程序对象包含对指令集可以操作的每个对象的引用。它将指令集提交给线程池，线程池将找到该指令集要与哪个对象交互，然后处理该对象上的指令集。

我可以处理 IO 服务器中的指令集对象，但我的想法是有一个单独的类，因为它使整个代码更具可读性，因为每个类只专注于做一件特定的事情。

想法？有什么建议吗？

谢谢

Answer 1

您的示例代码根本不会让“this”转义。在构造函数中启动一个新线程是相当安全的(甚至使用 this 作为 Runnable，在这个例子中你没有这样做)只要你确定就新线程需要它而言，您已经初始化了该对象。例如，设置新线程在启动线程后将依赖的 final 字段将是一个非常糟糕的主意:)

基本上让“this”引用转义是通常令人讨厌的，但并非普遍如此。在某些情况下它是安全的。请小心。

话虽如此，让构造函数启动一个线程可能会被视为在构造函数中做了太多事情。很难说它在这种情况下是否合适 - 我们对您的代码在做什么还不够了解。

编辑:是的，阅读了额外的信息后，我认为这没问题。您可能还应该有一个方法来关闭线程池。