3

I am making an ASP.NET MVC website with an Onion architecture. I have implemented a repository pattern and am now creating a Controller method in my project. This is my architecture:

  1. Domain : Entities / Domain Interfaces
  2. Repository : Generic repository (for now) using Entity Framework Code First
  3. Service : Generic Service that calls the Repository
  4. MVC

Now in my MVC project I need to access repository but according to Jeffrey Palermo, MVC is dependant from Service and Domain, but nothing is said of Repository.

Can i make my UI dependant of Repository?

I cannot do anything without the dependency right now, because Service creation depends on Repository. I have tried modifying Startup.cs to not directly use Repository objects in my Controllers, but it still means I need to add a reference to Repository (see http://dotnetliberty.com/index.php/2015/10/15/asp-net-5-mvc6-dependency-injection-in-6-steps/ )

Flexabust Bergson
  • 732
  • 14
  • 34
  • 1
    At this point MVC (UI) is the composition root, which would need to know all the layers in order to register them with the DI container. Can you show some code to demonstrate what it is that is causing problems. – Nkosi Jun 20 '17 at 15:09
  • @Nkosi ok, so I thought there could be pbs with that but I guess not. Thanks alot! – Flexabust Bergson Jun 20 '17 at 15:13

1 Answers1

3

At this point MVC (UI) is the composition root, which would need to know all the layers in order to register them with the DI container.

The start up would either refer to the implementations directly when registering them or indirectly via extension methods exposed from that layer. Either way the composition root needs to reference all lower layers if they are to be added to DI container.

The controllers should also be dependent on abstractions from lower layers and not concretions.

If injecting repositories directly into controllers then I suggest reviewing the design as that may indicate mixing of responsibilities.

Nkosi
  • 235,767
  • 35
  • 427
  • 472
  • Ok thanks! I just need Repos to pass them as parameters to instantiate Services, I will not access their methods from the controller (my service is in charge of this). So if I can do this in Startup, all is solved ! – Flexabust Bergson Jun 21 '17 at 08:19