在线程安全类中返回一个对象

Question

我遇到的问题似乎表明 Java 对象不是在我预期的堆上创建的。简短的版本是我有一个 JMS onMessage() 方法，它调用另一个方法来实例化基于 TextMessage 的 JSon 有效负载的正确对象 (routingCommand)。

    public void onMessage(Message message) throws RuntimeException {

IRoutingCommand routingCommand = null;
routingCommand = instantiateRoutingCommand(message);
...
process(routingCommand);
}

private IRoutingCommand instantiateRoutingCommand(Message message)  {

    IRoutingCommand routingCommand = null;

    ... lots of code to build the correct type of RoutingCommand

    routingCommand = new IeRoutingCommand(ieNotification);
    return routingCommand;
}

问题在于，在我的负载测试期间，在极少数情况下，同一条 JMS 消息被多次“处理（）”。当我设置 maxConcurrentConsumers=1 时，这不会发生。

我在互联网上发现了这个我从不知道的智慧宝藏，但假设它是正确的，它可以解释我的问题：

对对象的本地引用有点不同。引用本身不共享。然而，引用的对象并不存储在每个线程的本地堆栈中。所有对象都存储在共享堆中。

instantiateRoutingCommand 方法大约有 50 行长——这就是我一开始就把它分解的原因。我可以充分检查 JMS 消息以确定它代表什么类型的对象，并在 onMessage() 方法中对其进行实例化，并将 instantiateRoutingCommand 转换为仅在传递的对象上使用 setter。即满足上述规定。但即使这样也不是微不足道的，而且会弄乱 onMessage() 方法。

我没有尝试使对象变得易失或使任何东西同步，因为如果上述情况属实，那么我看不到任何帮助。那么在多线程对象中处理传递对象的最佳方法是什么？

Answer 1

您必须同步对多个线程引用的对象的访问，或者为每个线程提供自己的副本。虽然后者可能是唯一有效的解决方案，但应注意复制的对象不包含也包含在被复制对象中的引用。

Answer 2

我最终为我的 RoutingCommand 的所有子类创建了克隆方法。

然后在类上同步的块中使用它。由于实例化正确子类的方法只是一个解析器（无 I/O），因此阻塞所有线程并不是特别繁重。

synchronized(FlowController.class) {
routingCommand = instantiateRoutingCommand(message).clone();  
}

这解决了我的线程安全问题。所以上面返回的对象在共享堆上的观点其实是对的。