모범 사례 : 여러 종속성 저장소를 ActionController에 가져 오는 방법은 무엇입니까?

Question

나는 IInventoryRepository를 inevected로 가져온 InventoryController를 가지고 있지만, 나의 필요성이 바뀌었고 이제는 컨트롤러 메소드 중 또 하나의 저장소 인 ILoansRepository (빌려준 인벤토리 아이템에 대한 정보 얻기)와 또 다른 하나를 사용해야한다. 일부 통계 및 추가 정보가 있습니다.

실제로 작동하는 방식은 InventoryController의 ActionMethod에서 호출되는 ViewModelBuilder 클래스입니다. 즉 실제로 필요한 부품 인 ViewModelBuilder 클래스입니다. 현재 IInventoryRepository를 컨트롤러에서 빌더로 전달했지만 현재 어떻게해야합니까? 컨트롤러에 3 개의 레포지토리를 주입시켜야하고, 지금 내가했던 것처럼 빌더에 전달해야합니까? 아니면 그냥 IoC.GetInstance()를 수행해야합니까? (비록 그것이 안티 패턴이라고 생각하지 않습니까?)

감사합니다!

Answer 1

는 다음 지침은 놀이에 와서 :

• 너무 많은 의존성 당신이 Single Responsibility Principle을 위반 냄새입니다.
• 4 개를 초과하는 종속성이 없습니다. 이것은 상대적 가이드 라인입니다. 나는 개인적으로 덜 노력하도록 노력한다. 세 번째 종속성을 추가하자마자 불안해 지지만 (위의 첫 번째 항목 참조) 최대 4 개까지 살 수 있습니다. 그 이상으로 나는 리팩토링해야한다.
• 종속성을 전달하기 만하면 안됩니다.

세 가지 종속성이있는 한, 특정 디자인 측면을 더 신중하게 관찰해야하지만, 종속성의 수에 관해서는 안전 영역 내에서 다소 차이가 있습니다.

그러나 현재 구현을 이해하고 있으면 ViewModelBuilder에 의존성을 전달하기 만하면됩니다 (따라서 세 번째 글 머리 기호를 위반 함). 약간 더 나은 옵션은 (예를 들어, IViewModelBuilder와 같은) 추상화를 정의하고 이것을 세 곳의 저장소가 아닌 컨트롤러에 주입하는 것입니다.

서비스 로케이터 방지 패턴 (IoC.GetInstance())을 사용하지 않아야합니다.

Answer 2

컨트롤러에 대한 많은 책임이 있습니다.

어쩌면 당신은 그것을 처리하기위한 특별한 서비스를 만들어야하고 그 서비스는 생성자에 의해 자동 배선 된 저장소 (IoC를 통해)를 사용해야합니다.

Answer 3

1. 컨트롤러가 너무 많은 작업을 수행하는 경우 여러 컨트롤러로 분할하십시오.

2. ViewModelBinder를 만들기 위해 3 개의 리포지토리를 주입하는 경우 :(I) ViewModelBinder 대신 삽입하십시오. IoC 컨테이너가 그 일을하고 당신을위한 의존성을 해결하도록하십시오; 또한 이것은 아키텍처, 테스트 등을 단순화합니다. ServiceLocator/GetInstance는 항상 피할 수있는 것은 아닙니다. 때로는 MVC 컨트롤러 팩토리와 같은 "루트"주입 지점이 없으며 모델 생성기와 같은 객체 생성을 제어 할 수없는 경우가 있습니다. 그래서 모델 빌더 (빌더가 아님)가 GetInstance를 호출하게하지만 직접 "루트"를 만듭니다. 예를 들어, GetInstance를 호출합니다. <IModelResolver>, GetInstance가 아닌 < 엔티티 >. 이와 같은 상황에서