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

Question

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

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

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

要以 Dependency Inversion 方式执行此操作，您可以执行以下操作之一：

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

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

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

   public abstract class AbstractController {
   
   private List<AbstractViewPanel> registeredViews;
   private List<AbstractModel> registeredModels;
   
   public AbstractController() {
   }
   
   public void setRegisteredViews(List<AbstractViewPanel> views) {
      this.registeredViews = views;
   }
   
   public void setRegisteredModels(List<AbstractModel> models) {
      this.registeredModels = models;
   }
   

顺便说一句，我将 ArrayLists 更改为 Lists。无需引入对 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

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

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

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