DIP（Dependency Injection Principle）に従うためにこのクラスをどのように変更する必要がありますか？

Question

コンストラクタで2つのArrayListの依存関係を削除するために、このクラスをDIP（Dependency Inversion Principle）に従うように変更する方法はありますか？インターフェイスはどのようにすべきですか？

私を混乱させる1つのことは、新しいリファレンスがクラスのコンストラクタだけでなく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

依存反転-wayでそれを行うには、次のいずれかを実行できます。リストへ

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; 
   } 
   

Btw、私はリストにArrayListsを変更しました。 Listの実装に依存関係を導入する必要はありません。

Answer 2

それは本当にあなたが削除しようとしているものの依存関係は明らかではないのですが、あなたは明示的なArrayListのインスタンスを削除したい場合、あなたはコンストラクタ・インジェクションを使用することができます...そのような状況に対処する方法がわかりません：

Answer 3

私はリストをオブジェクトに挿入しないで、リストをモックして単体テストするためにカプセル化を中断します。リストは外部依存関係ではありません。それはクラスの内部の一部です。

このクラスを単体テストする場合は、リストに格納されているオブジェクトを使用するメソッドが、実際にそれらを使用するかどうかをテストします。また、コントローラに追加したモデルのプロパティを変更するときに、propertyChangeメソッドが呼び出されることをテストする必要があります。

または、追加が期待どおりに機能するかどうかをテストするために、getView()とgetModels()メソッド（潜在的に保護されている）を追加できます。