如何修复IndicesFactory中SonarLint检测到的泛型原始类型问题？

你的代码中SonarLint提示的泛型原始类型问题，根源是IndicesFactory里直接使用了无泛型参数的IndicesStrategy原始类型，这会绕过Java的泛型类型检查，触发代码质量警告。以下是兼顾类型安全和现有功能的最优修复方案：

1. 替换原始类型为通配符兼容类型 #

首先修改IndicesFactory中的成员变量、构造方法和getStrategy方法，使用通配符来兼容所有泛型参数的IndicesStrategy实现，消除原始类型警告：

public class IndicesFactory {
    // 用通配符替代原始类型，兼容所有泛型参数的策略实现
    private final Map<String, IndicesStrategy<?, ?, ? extends IndicesResponse>> indicesMap;

    public IndicesFactory(List<IndicesStrategy<?, ?, ? extends IndicesResponse>> indicesStrategyList) {
        this.indicesMap = new HashMap<>();
        indicesStrategyList.forEach(strategy -> 
            this.indicesMap.put(strategy.getOperation().toString(), strategy)
        );
    }

    private IndicesStrategy<?, ?, ? extends IndicesResponse> getStrategy(String operation) {
        return this.indicesMap.get(operation);
    }

2. 优化execute方法的类型安全性 #

原execute方法的强制类型转换(R)存在运行时类型转换风险，建议新增带类型令牌的类型安全版本，同时可选保留原方法做兼容：

// 推荐使用的类型安全execute方法，通过Class令牌做运行时类型校验
    @SuppressWarnings("unchecked")
    public <T, R extends IndicesResponse> R execute(String operation, T t, Class<R> responseType) {
        IndicesStrategy<?, ?, ? extends IndicesResponse> strategy = getStrategy(operation);
        // 强制转换为适配输入T的策略（编译期无法校验，故加SuppressWarnings）
        IndicesResponse rawResponse = ((IndicesStrategy<?, T, ? extends IndicesResponse>) strategy).execute(t);
        // 运行时校验返回类型是否匹配预期
        if (responseType.isInstance(rawResponse)) {
            return responseType.cast(rawResponse);
        }
        throw new IllegalArgumentException(
            String.format("返回类型不匹配：预期%s，实际%s", responseType.getName(), rawResponse.getClass().getName())
        );
    }

    // 兼容旧代码的原方法（可选保留）
    @SuppressWarnings("unchecked")
    public <T, R> R execute(String operation, T t) {
        try {
            return (R) getStrategy(operation).execute(t);
        } catch (ClassCastException e) {
            throw new IllegalArgumentException("类型转换失败，请使用带Class参数的execute方法", e);
        }
    }
}

3. 额外编码规范优化（可选） #

接口IndicesStrategy的泛型参数命名建议遵循Java规范，用大写开头的名称提高可读性：

public interface IndicesStrategy<Operation extends Enum<?>, T, R extends IndicesResponse> {
    R execute(T t);
    Operation getOperation();
}

方案说明 #

• 使用通配符IndicesStrategy<?, ?, ? extends IndicesResponse>既消除了原始类型警告，又能容纳所有实现该接口的策略类，不破坏工厂类的实例化逻辑。
• 带Class<R>参数的execute方法通过运行时类型检查，避免了无校验的强制转换带来的运行时错误，同时给调用方明确的类型约束。
• 保留原方法是为了兼容现有代码，建议逐步迁移到类型安全版本。

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