重新检查Spring中的注入字段

Question

我有Spring控制器，它使用构造函数注入来满足服务层依赖性。效果很好。现在，在组织中有一些旧的代码来双重检查依赖关系是否有效。我认为这是一个反模式，因为它是与横切关注点相关的代码，被注入的客户端不应该担心这一点。

如果无法解析依赖关系，则 IoC 容器 （Spring） 将引发相应的错误（UnSatisfiedLink 或其他），使用这些服务的客户端应该与如何或是否正确注入这些依赖关系无关。它应该假设所有依赖项都已初始化并准备就绪。这是首先使用IoC的本质。

例如，我看到这个：

  public MyController(MyService myService) {
     this.myService = myService;
  }

然后：

@PostConstruct
public void afterPropertiesSet() throws Exception {
 if(myService == null) {   // unecessary in my opinion
   throw new IllegalArgumentException("MyService is not set!");
 }
}

这些类型的“双重检查”是不必要的，并导致横切关注点代码混乱，意见？

Answer 1

我不会考虑这样的支票

if(myService == null) {
 //throw nice readable error
}

作为代码审查的一部分，我会删除这些杂乱的内容。我想更多地看到的问题是“抛出异常”部分，您现在必须将其作为此方法的一部分介绍。因此，每个调用此方法的方法现在都需要实现try/catch或re-throw实现。

我宁愿推荐一个继承自RunTimeException的Exception。

这样做的原因是，这是一个异常，存在严重的配置或设置错误。您不希望将其作为任何类型的常规功能的一部分发生。因此，我不认为任何其他方法需要“知情”，并且有一个特殊的尝试/捕获块来表现不同。应用程序基本上应该崩溃并显示异常。就在这一点上。全局服务级别上可能有一些 catch 处理程序捕获这些异常，然后（例如回滚事务）在 SoapError 等中转换异常。但一般来说，我不会考虑在抛出和显式异常中有任何额外的值，这些异常现在需要其他调用方法来实现一个特殊的 catch 块。