Spotbugs报MVEI2错误：传递可变对象至ExplosionView如何解决？

错误原因 #

Spotbugs报的MVEI2错误核心问题是：你将可变的Explosion实例（this）直接传给了ExplosionView的构造函数，View持有该引用后，可能直接修改Controller的内部状态，破坏了封装性。

虽然Explosion是final类，但它内部包含可变成员（比如explode列表），且实现了GameLoop接口（通常包含状态更新方法），属于可变对象。将这类对象的引用直接暴露给View，会导致Controller的内部状态可能被外部（View）意外篡改，违反了MVC架构中View应尽量只依赖只读数据的设计原则。

解决方案 #

最稳妥的修复方式是给Controller定义只读接口，让View只依赖这个接口，切断View对Controller可变方法的访问路径。

步骤1：定义只读接口 #

创建一个仅包含View所需只读方法的接口，比如ExplosionController：

public interface ExplosionController {
    List<Entity> getExplode();
    // 只添加View实际需要的只读方法，不要包含任何修改状态的方法
}

步骤2：让Controller实现该接口 #

修改Explosion类，实现刚定义的只读接口：

public final class Explosion implements GameLoop, ExplosionController {
    private final ExplosionView view;
    private final ExplosionImpl model;
    private List<Entity> explode;

    @Override
    public List<Entity> getExplode() {
        return new ArrayList<>(explode);
    }

    public Explosion() {
        // 这里将this以ExplosionController类型传入View
        view = new ExplosionView(this);
        model = new ExplosionImpl();
        explode = new ArrayList<>();
    }

    // 所有修改状态的方法保持私有/包私有，不对外暴露
}

步骤3：修改View依赖只读接口 #

更新ExplosionView的构造函数参数类型为只读接口，确保View只能调用安全的只读方法：

public final class ExplosionView implements GameLoop {
    private final ExplosionController controller;

    public ExplosionView(final ExplosionController controller) {
        this.controller = controller;
    }
}

补充说明 #

如果你的View确实需要调用Controller的可变方法（这种情况在MVC中应该尽量避免，建议通过事件/回调方式处理），可以：

• 确保Explosion类的所有可变方法都做了安全封装，比如对内部集合的修改仅通过受控方法进行，且不会被外部随意调用。
• 但仍推荐优先使用只读接口的方案，因为它能从根本上避免内部表示暴露的问题，同时更符合MVC的职责分离原则。

内容的提问来源于stack exchange，提问作者Niccolò Bertozzi