应该如何修改这个类才能遵循 DIP（依赖注入原则）？

Question

如何修改此类以遵循 DIP（依赖倒置原则），以便删除构造函数中的两个 ArrayList 依赖项？接口应该如何？

让我困惑的一件事是，新引用指向 ArrayList而不仅仅是类的构造函数。而且我不知道如何处理这种情况...

package mvc.controllers;

import java.util.ArrayList;
import mvc.models.AbstractModel;
import mvc.views.AbstractViewPanel;

public abstract class AbstractController {

    private ArrayList<AbstractViewPanel> registeredViews;
    private ArrayList<AbstractModel> registeredModels;

    public AbstractController() {
        registeredViews = new ArrayList<AbstractViewPanel>();
        registeredModels = new ArrayList<AbstractModel>();
    }

    public void addModel(AbstractModel model) {
        registeredModels.add(model);
        model.addPropertyChangeListener(this);
    }

    public void removeModel(AbstractModel model) {
        registeredModels.remove(model);
        model.removePropertyChangeListener(this);
    }

    public void addView(AbstractViewPanel view) {
        registeredViews.add(view);
    }

    public void removeView(AbstractViewPanel view) {
        registeredViews.remove(view);
    }
    ...
}

Answer 1

要以依赖关系反转方式执行此操作，您可以执行以下操作之一：

1. 在构造函数中给出对列表的依赖关系：

   公共抽象类 AbstractController {
   
   私有列表注册观看次数；
   私有列表注册模型；
   
   公共 AbstractController(ListregisteredViews, ListregisteredModels) {
       this.registeredViews = 注册视图；
       this.registeredModels = 注册模型;
   }
   

2. 为列表添加修改器（setter）：

   公共抽象类 AbstractController {
   
   私有列表注册视图；
   私有列表注册模型；
   
   公共抽象控制器（）{
   }
   
   公共无效setRegisteredViews（列表视图）{
      this.registeredViews = 视图；
   }
   
   公共无效setRegisteredModels（列表模型）{
      this.registeredModels = 模型；
   }
   

顺便说一句，我将 ArrayList 更改为 List。无需引入对 List 实现的依赖。

Answer 2

目前尚不清楚您要删除什么依赖项，但如果您想删除显式 ArrayList 实例化，您可以使用构造函数注入：

private final List<AbstractViewPanel> registeredViews;
private final List<AbstractModel> registeredModels;

public AbstractController(final List<AbstractViewPanel> registeredViews,
        final List<AbstractModel> registeredModels) {
    this.registeredViews = registeredViews;
    this.registeredModels= registeredModels;
}

Answer 3

我不会将列表注入到对象中，从而破坏封装，只是为了通过模拟列表来对其进行单元测试。该列表不是外部依赖项。它是班级内部的一部分。

如果您想对此类进行单元测试，请测试使用列表中存储的对象的方法是否确实按应有的方式使用它们。您还应该测试当您更改已添加到控制器的模型的属性时是否调用 propertyChange 方法。

或者，您可以添加 getView() 和 getModels() 方法（可能受保护），以测试添加是否按预期工作。