Question

I recently started a new webforms project and decided to separate the business classes from any DBML references. My business layer classes instead access discrete Data layer methods and are returned collections of DTO’s. So the data layer might project DTO’s like the following:

(from c in dataContext.Customers where c.Active == true  select new DTO.Customer {    CustomerID = c.CustomerID,    Name = c.CustomerName,    ... }).ToList() 

Although building the DTO objects adds work, this feels like a better approach to a tight binding between Business & Data layers and means I can test the Business layer without a database being present.

My question is, is this good practice?, Is there a way of generating the DTO’s (maybe via SQLMetal), and what other problems might I strike as the project progresses.

Answer 1

I don’t know if it’s best practice but I have written similar code in the not so recent past because I too felt that I could improve the separation of concerns by using my own classes instead of the LINQ-designer-generated ones within my application.

You may want to consider just returning an IQueryable<Customer> instead of an IList<Customer> from your data-access method. Since IQueryable<T> inherits from IEnumerable<T> the rest of your app should be able to deal with it quite well. You can also convert it to a List when you really need to.

The advantage of this is that you can dynamically modify your query quite easily and minimze the amount of data returned from SQL Server.

E.g. if your method signature is IQueryable<Customer> GetCustomers() you could get a single customer by calling GetCustomers().Where(c => c.CustomerID == 101).Single();

In this example only one record would be returned from the database whereas I imagine currently your code would return either all customers or you’d be required to write separate methods (and thus very repetitive code) to cater for all the different things you may want to filter by.