데이터베이스에 액세스하는 메소드를 두는 곳

Question

현재 코드 액세스가 데이터 액세스 레이어로 프로토 타입으로 ASP .NET MVC 3 웹 응용 프로그램을 개발 중입니다. 지금 당장 나는 코드를 어디에 넣을 지 혼란 스럽다.

고객 및 수업 프로젝트 클래스가 있습니다. 프로젝트는 고객에게 네비게이션 속성을 제공합니다.

public class Customer 
{ 
    public int Id { get; set; } 
    public string Name { get; set; } 
} 

public class Project 
{ 
    public int Id { get; set; } 
    public string Name { get; set; } 
    public virtual Customer Customer { get; set; } 
} 

나는 두 가지 클래스를 codefirst를 사용하여 데이터 액세스에 사용합니다.

public class MyContext : DbContext 
{ 
    public DbSet<Customer> Customers { get; set; } 
    public DbSet<Project> Projects { get; set; } 
} 

가 나는 직접 내 MVC보기에 이러한 데이터 액세스 클래스를 사용하지 에 가장 좋은 방법은 있다고 배웠습니다 - 그래서 나는 단일 고객에 대한 뷰 모델 (CustomerListModel)와 DTO를 만들었습니다. 내 컨트롤러에서

public class CustomerDto 
{   
    public int Id { get; set; } 
    public string Name { get; set; } 
    public int ProjectCount { get; set; } <== this property is in question 
} 

public class CustomerListModel 
{ 
    public List<CustomerDto> Customers; 
} 

나는 실제 데이터 액세스를 수행하는 (서비스 -) 클래스에서 고객을 가져 오는거야.

public class CustomerService : IDisposable 
{ 
    private MyContext db; 
    public CustomerService() 
    { 
     db = new MyContext(); 
    } 

    public IEnumerable<Customer> GetAllCustomers() 
    { 
     return db.Customers.ToList<Customer>(); 
    } 
} 

내 컨트롤러에서 모든 고객 확보 방법을 호출합니다.

public ViewResult Index() 
    { 
     //Mapper.CreateMap<Customer, CustomerDto>(); 

     var customerList = new List<CustomerDto>(); 
     foreach (var customer in rep.GetAllCustomers()) 
     { 
      var cust = new CustomerDto(); 
      cust.Id = customer.Id; 
      cust.Name = customer.Name; 
      cust.Rate = customer.Rate; 
==> cust.ProjectCount = customer.ProjectCount; <===== 
      customerList.Add(cust); 
     } 

     var viewModel = new CustomerListModel() 
     {     
      Customers = customerList //Mapper.Map<IEnumerable<Customer>, List<CustomerDto>>(rep.GetAllCustomers()) 
     }; 
     return View(viewModel);    
    } 

저는 단 한 명의 고객을 위해 ProjectCount를 어디에 넣어야하는지 묻습니다. 나는 그것을 고객 클래스에 넣어 줄 수있다.

public int ProjectCount 
    { 
     var db = new MyContext(); 
     return db.Projects.Where(x => x.Customer.Id == this.Id).Count(); 
    } 

... 그러나 서비스 클래스와 고객 클래스는 데이터 액세스가 가능한 두 개의 장소가있다.

public int GetProjectCount(Customer customer) 
    { 
     return db.Projects.Where(x => x.Customer.Id == customer.Id).Count(); 
    } 

을 ...하지만 그때 컨트롤러에서 호출했다 :

나는 또한 내 ServiceClass에이 코드를 넣을 수

 foreach (var customer in rep.GetAllCustomers()) 
     { 
      var cust = new CustomerDto(); 
      cust.Id = customer.Id; 
      cust.Name = customer.Name; 
      cust.Rate = customer.Rate; 
      cust.ProjectCount = rep.GetProjectCount(customer); <== 
      customerList.Add(cust); 
     } 

... 나는이 부를 수 메서드를 내 고객 클래스의 getter에서 내 서비스 클래스에서 가져옵니다.

public int ProjectCount 
    { 
     get 
     { 
      return new CustomerService().GetProjectCount(this); 
     } 
    } 

설명 된 방법의 모든 일을하고 나에게 정확한 결과를 제공 - 하지만 내가 올바른 방법으로 수행 할 - 당신이 그것을 할 것 어떻게 - 또는 나는 완전히 트랙 떨어져입니다 ;-) ?

감사합니다.

Answer 1

다음은 필자가 주석에 쓴 내용을 요약 한 것입니다. 나는 그것이 "가는 길"이라고 말할 수는 없지만 그것이 가능한 해결책이라고 생각하고 싶습니다. 당신이 (내가 언급 한 바와 같이)는 제네릭이 아닌 접근 방식을 사용하는 경우

그래서, 간단한 관리가 용이하고 몇 테이블 작은 데이터베이스에 대한 디버그, 당신은 다음과 같이 작업을 수행 할 수 있습니다

public interface ICustomerService 
{ 
    IEnumerable<Customer> GetAllCustomers(); 
    // and other methods you'd like to use 
} 

public sealed class CustomerService : ICustomerService 
{ 
    private YourContext context = new YourContext(); 
    // or, alternatively, even something along these lines 
    private YourContext context; 
    public CustomerService() 
    { 
     context = ContextFactory.GetDBContext(); 
    } 
    //---- 

    public IEnumerable<Customer> GetAllCustomers() 
    { 
     // here goes the implementation 
    } 
} 

public class YourController : Controller 
{ 
    private ICustomerService customerService; 
    public YourController(ICustomerService service) 
    { 
      customerService = service; 
    } 
    // and now you can call customerService.GetAllCustomers() or whatever other methods you put in the interface. 
} 

디그 레션 (Digression) - 예를 들어 데이터베이스 컨텍스트 팩토리가 그 것처럼 보일 수 있습니다.나중에 데이터베이스 연결을 변경

같은 쉬운 설치는 선택의 IoC 컨테이너와 함께 작동 물론

public static ContextFactory 
{ 
    public static YourContext GetDBContext() 
    { 
     return new YourContext(); 
    }   
} 

, 광산이이 Global.asax에 Application_Start 간다 웹 애플 리케이션을위한 Autofac입니다 :

var builder = new ContainerBuilder(); 
    builder.Register(service => new CustomerService()).As<ICustomerService>().InstancePerHttpRequest(); 
    builder.RegisterControllers(typeof(MvcApplication).Assembly); 
    var container = builder.Build(); 
    DependencyResolver.SetResolver(new AutofacDependencyResolver(container)); 

의존성 주입은 나중에 모의 도우미 중 하나를 사용하여 데이터베이스 액세스를 모의 할 수 있으므로 (예 : 준비된 항목 목록) 컨트롤러를 나중에 테스트하는 것이 더 쉽습니다.

그러나 대부분의 리포지토리 메소드가 비슷하다면 코드를 반복 할 필요가 없으므로 사용자의 요구에 더 적합한 general solution (it's for database-first, but can be applied to code-first too)을 더보고 싶을 수 있습니다. 코드가 좀 더 복잡해 보일 수 있습니다. :)