5

Option 1:

At first I would make a call to my service layer, which served as an API for my core domain, to get a domain object or a list of domain objects and then pass them into the assembler which would construct the DTOs I needed for my view. The problem I have with this approach are cases where the domain object is large and I don't want to load in the whole object just to copy a few fields needed for the DTO (ie showing a list of summaries entities).

Option 2:

The next approach I used was to wire in a repository (for read-only purposes) into my assembler so that I could only query the database for the fields I need in the DTO. I can also use this repository when I get a DTO and need to use it to update and entity. For example, a DTO filled will values I need to update on my entity comes into my assembler, the assembler looks up the entity from a repository and then overlays the information from the DTO on the entity and then returns the entity. The controller then calls the service layer to save the entity that the assembler returned.

Option 3:

I could wire in the repositories directly into the controllers but something about exposing the repository in the controller seems wrong to me because the service layer should be handling transactions and security, but then again, if I put a repository in the assembler I am basically doing the same thing.

Any thoughts are welcome. I just want to understand pros and cons and what has worked well for others.

2
  • 1
    In the future, please do not cross-post questions between Stack Exchange sites. If you post a question that is off-topic on one site and you feel that you could get better answers on another site, flag it for moderator review and allow them to move it.
    – Thomas Owens
    Mar 31, 2015 at 14:11
  • 2
    Sounds good, I marked the other one to be closed and thought I would have to move it myself. I will do that (wait for moderator to move it) in the future.
    – testing123
    Mar 31, 2015 at 20:49

2 Answers 2

1

Ok. Problem 1: getting DTO's from entities:

Since your entities can expose their data publicly you can access their properties and instantiate DTO object or simply serialise the entity directly

Problem 2 : Entities from DTO's:

A constructor method which takes a list of the properties to be set can be called using the properties of the DTO

Problem 3 : large entities when you only need summary info

Create a new summary object which you can retrieve from the repository. Note I suggest you make strongly typed repositories with methods like Repo.GetMyObjectById(string id) rather than expose a generic ORM.

Problem 4 : where to orchestrate all this.

My recommendation is to have a service class one level below your hosting service/app/website.

This has access to the repositories DTO's and entities and its methods map to the controllers/service calls of your application so that you do not need any code at the top level and can host the same service in multiple ways.

Giving it access to the repos is not an issue when you can only use them to retireve entities rather than doing any query they like.

Putting this assembly/orchestration logic inside the entity is usually bad as you will want the entity to be reused for other purposes.

This top level service should be very light. Just, get these objects, call this method, create and return the result/DTO/viewmodel

Because its so light its not a massive prob if you skip the layer and put the code in your controller. But it will save you time if you change the hosting layer and help with testing etc

2
  • Isn't the service class you are talking about the same thing as an assembler?
    – testing123
    Jul 2, 2015 at 20:23
  • its doing the same job, I guess the difference is that the service is the thing you are making. You only have it at all to abstract the hosting layer
    – Ewan
    Jul 2, 2015 at 20:45
-2

Personally I would not use any of these options. I would skip the use of repositories all together, and just use ADO.NET and SQL queries to map data to my DTO's directly in the application layer. Or if you prefer behind a very thin query facade layer.

We use this method with great success currently. Querying against multiple domain models doesn't give you any real benefits and will generally give you poor performance. No real reason to get your domain entities involved at all in my opinion.

2
  • 3
    That works fine for smaller applications but for an enterprise application that gets really messy really fast. Occasionally I have to work in an older application that does just what you are suggesting and there is a ton of code duplication (same business logic in multiple places) not to mention the inability to have a core service library that you can call from a webservice because all you logic isn't in a shared service layer.
    – testing123
    Mar 31, 2015 at 5:28
  • How would you inject for testing?
    – Ewan
    Jun 29, 2015 at 7:01

Your Answer

By clicking “Post Your Answer”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Not the answer you're looking for? Browse other questions tagged or ask your own question.