在对象构造函数中启动线程池是否安全?我知道你不应该从构造函数启动一个线程,关于“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 服务器中的指令集对象,但我的想法是拥有一个单独的类,因为它使整个代码更具可读性,因为每个类只专注于做一件特定的事情。
想法?建议?
谢谢
最佳答案
您的示例代码根本不允许“this”转义。只要您确定,在构造函数中启动一个新线程是相当安全的(甚至使用 this
作为 Runnable
,在本例中您没有这样做)您已经初始化了新线程需要的对象。例如,设置新线程在启动线程后将依赖的最终字段将是一个非常糟糕的主意:)
基本上,让“this”引用转义通常是令人讨厌的,但并非普遍如此。在某些情况下它是安全的。请小心。
话虽如此,让构造函数启动线程可能会被视为在构造函数内执行过多操作。很难说它在这种情况下是否合适 - 我们对您的代码在做什么了解不够。
编辑:是的,阅读了额外的信息后,我认为这没问题。您可能还应该有一种方法来关闭线程池。
关于Java在对象构造函数中启动线程池,我们在Stack Overflow上找到一个类似的问题: https://stackoverflow.com/questions/3495350/