如何用私有构造函数适配Spring并避免SonarLint的UnusedPrivateMethod问题？

首先咱们拆解下你遇到的两个问题根源：

• 第一种写法的NPE：字段初始化代码private final String keyValue = theComponent.getKey();是在对象实例化阶段执行的，这时候Spring还没完成theComponent的注入（依赖注入是在实例化之后、初始化方法之前完成的），所以theComponent还是null，自然触发空指针。
• 第二种写法的SonarLint警告：Sonar不知道Spring会通过反射调用你的私有构造函数，它只看到代码里没有显式调用这个构造，所以误报“未使用的私有构造方法”。

那怎么同时解决这两个问题呢？有两种靠谱的方案：

方案1：给私有构造函数添加@Autowired注解 #

Spring支持通过反射调用私有构造函数完成注入，而@Autowired注解可以明确告诉SonarLint：这个构造函数是被Spring的依赖注入机制使用的，不是无用代码。

示例代码：

@Service
public class TheService {
    private final String keyValue;

    @Autowired
    private TheService(TheComponent theComponent) {
        this.keyValue = theComponent.getKey();
    }
}

这里把keyValue声明为final是个好习惯——因为它在构造函数中一次性初始化，后续不会变更，符合Spring Bean的不可变性设计，也更安全。

方案2：用@SuppressWarnings("unused")抑制Sonar警告 #

如果你不想加@Autowired（Spring 4.3+之后，当类只有一个构造函数时，即使不加@Autowired，Spring也会自动用它做构造注入），可以直接给私有构造函数加上@SuppressWarnings("unused")注解，告诉静态代码分析工具：这个构造函数是有意保留的，虽然看起来没被显式调用，但实际被Spring反射使用。

示例代码：

@Service
public class TheService {
    private final String keyValue;

    @SuppressWarnings("unused")
    private TheService(TheComponent theComponent) {
        this.keyValue = theComponent.getKey();
    }
}

两种方案都能解决你的问题：既避免了NPE（因为构造注入保证theComponent在初始化keyValue时已经存在），又能让SonarLint不再误报未使用的构造函数。

内容的提问来源于stack exchange，提问作者ThCollignon