两个常见的并发错误

作为Baeldung的编辑，我很高兴与一位作者一起撰写有关Java通用并发陷阱的文章。这是一本不错的书，但是假设开发人员具有一定的能力。

我已经看到了几件即时并发失败的事情。它们很容易添加到代码中，并保证为您提供奇怪的结果。开发人员仍会提交这些事实，这是对我们如何对OO和并发进行教育的一种评论，如果使用不当，这将非常危险。

除了代码审查

作为代码审查员，这些年来我已经开发了一些速记。这些帮助我发现了在较大的代码更改中需要更详细地研究的区域。它们包括红旗的事情，我希望出问题。训练自己去发现关键的反模式或潜在的反模式是一个好主意，因为它们可以是有效的代码，但会导致无效的行为。

Bean中的请求状态

在Java应用程序中，服务，控制器，处理程序和存储库通常是单例的。它们是在应用启动时创建的，然后请求通常通过多个线程传递给它们。

考虑如下代码：

 public void processOrder(Order order) { ... currentLineItem = order.getLine( 0 ); processLineItem();  }  private void processLineItem() { myService.store(currentLineItem);  }

在这种情况下，该类的作者已决定该对象可以记住其当前正在处理的项目，从而节省了将该项目传递给下一个函数的工作。

这违反了两个原则：线程安全和有意义的对象状态。订单处理者不太可能真正了解其正在处理的订单。您可能会想像一些有状态地遍历某个订单，某种游标，阅读器或构建器的项目，但是将所有这些项目混合到一个对象中则很麻烦。

不过，最重要的是，有一个明确的定义可以解释为什么这是错误的。如果将请求的每个属性放入该请求的接收者中，那么您将有两个风险：

在多线程执行中的请求之间出血
如果事情没有完全整理，则在单线程的请求之间流血

简而言之，永不做！

疯狂的懒惰初始化

延迟初始化允许：

由于更快的启动
必要时及时加载资源
如果不需要，则不加载资源（例如，无服务器Lambda，在其生命周期中可能永远不会被要求执行特定的代码路径）
定制如何通过较早发生的活动加载资源

所有这些都很好。但是，此代码：

 private LazyService getLazyService() { if (lazyService != null ) { return lazyService; } LazyService newLazyService = connectToLazyService(); registerWithServiceRegistry(newLazyService); lazyService = newLazyService; return newLazyService;  }

尽管它可以工作，但可以同时调用并出错。它的错误程度取决于各种各样的事情。在示例中，我试图暗示我们正在处理的事情：