ASP.NET MVC: HTTPContext và Dependency Injection

Question

Hiện tại tôi có một ActionFilter lấy tên người dùng hiện tại từ HttpContext và chuyển nó vào hành động sử dụng nó trên một phương thức dịch vụ. ví dụ:

Service.DoSomething(userName); 

Bây giờ tôi có lý do để làm điều này không phải ở cấp hành động mà là cấp độ bộ điều khiển bộ điều khiển. Hiện tại tôi đang sử dụng bản đồ cấu trúc để tạo bộ điều khiển và tiêm dịch vụ. Tôi đang xem một cái gì đó như:

public interface IUserProvider 
{ 
    string UserName { get; } 
} 

public class HttpContextUserProvider : IUserProvider 
{ 
    private HttpContext context; 

    public HttpContextUserProvider(HttpContext context) 
    { 
     this.context = context; 
    } 

    public string UserName 
    { 
     get 
     { 
      return context.User.Identity.Name; 
     } 
    } 
} 

Điều đó nói rằng, IoC foo của tôi thực sự yếu vì đây là dự án đầu tiên tôi đã sử dụng.

Vì vậy, câu hỏi của tôi là ... làm thế nào tôi có thể nói cho bản đồ cấu trúc để vượt qua trong HttpContext trong constructor cho HttpContextUserProvider? Điều này có vẻ lạ ... Tôi không chắc làm thế nào để nghĩ về HttpContext.

Answer 1

Có một giao diện trừu tượng HttpContext.Current. Chỉ trưng ra các phương thức bạn cần GetUserName() sẽ gọi HttpContext.Current.User.Identity.Name trong thực hiện, ví dụ. Hãy chắc rằng như mỏng càng tốt.

Hãy trừu tượng đó và tiêm nó vào lớp nhà cung cấp khác của bạn. Điều này sẽ cho phép bạn kiểm tra các nhà cung cấp bằng cách chế giễu http bối cảnh trừu tượng. Là một lợi ích phụ, bạn có thể làm những điều tiện lợi khác với HttpContext trừu tượng bên cạnh việc giả lập nó. Tái sử dụng nó, cho một điều. Thêm thông số loại chung cho túi xách, v.v.

Answer 2

Có lẽ tôi lại một cái gì đó, nhưng câu trả lời ở trên không làm việc cho tôi (từ đó đã được xóa - nó vẫn còn là một câu trả lời hữu ích mặc dù - nó đã cho thấy làm thế nào để nói với SM để vượt qua đối số nhà xây dựng). Thay vào đó, nếu tôi làm:

ObjectFactory.Initialize(x => 
{ 
    x.BuildInstancesOf<HttpContext>() 
     .TheDefault.Is.ConstructedBy(() => HttpContext.Current); 
    x.ForRequestedType<IUserProvider>() 
     .TheDefault.Is.OfConcreteType<HttpContextUserProvider>(); 
}); 

Tôi làm cho nó hoạt động. Tôi đã làm điều này sau khi tìm thấy: http://codebetter.com/blogs/jeremy.miller/archive/2008/03/20/if-you-need-something-in-structuremap-but-you-can-t-build-it-with-new.aspx

---

chỉnh sửa:

Nhờ câu trả lời của Brad Tôi nghĩ Tôi có một xử lý tốt hơn trên HttpContext. Câu trả lời của ông chắc chắn công trình, tôi chỉ không chắc chắn tôi thích có cuộc gọi đến HttpContext.Current bên trong một lớp (nó có vẻ như nó ẩn phụ thuộc, nhưng tôi xa một chuyên gia về công cụ này).

Mã trên sẽ hoạt động để tiêm HttpContext theo như tôi có thể biết. Matt Hinze sẽ trả thêm điểm rằng nếu tất cả những gì cần từ HttpContext là User.Identity.Name, thiết kế của tôi nên rõ ràng về điều đó (có một giao diện xung quanh HttpContext chỉ phơi bày những gì tôi cần). Tôi nghĩ rằng đây là một ý tưởng tốt.

Điều này là qua bữa ăn trưa Tôi kinda nhận ra dịch vụ của tôi thực sự chỉ cần phụ thuộc vào một chuỗi: userName. Có nó phụ thuộc vào IUserProvider có thể không có nhiều giá trị gia tăng. Vì vậy, tôi biết tôi không muốn nó phải phụ thuộc vào HttpContext, và tôi biết tất cả tôi cần là một chuỗi (userName) - Tôi cần phải xem liệu tôi có thể học đủ StructureMap foo có làm cho kết nối này cho tôi. (Câu trả lời của sirrocoo đưa ra gợi ý về nơi bắt đầu nhưng anh ta đã xóa nó: *().

Answer 3

Tôi không chắc tại sao bạn lại bận tâm. Bạn sẽ không bao giờ thay thế bằng một HttpContext khác ...

Answer 4

Có vẻ như bạn nên sử dụng HttpContextBase thay vì HttpContextUserProvider.Đây là một sự trừu tượng vượt trội của HttpContext và cho phép bạn tạo một mô hình, viết UnitTests và tiêm phụ thuộc của bạn.

public class SomethingWithDependenciesOnContext 
{ 
    public SomethingWithDependenciesOnContext(HttpContextBase context) { 
     ... 
    } 

    public string UserName 
    { 
     get {return context.User.Identity.Name;} 
    } 
} 

ObjectFactory.Initialize(x => 
      x.For<HttpContextBase>() 
      .HybridHttpOrThreadLocalScoped() 
      .Use(() => new HttpContextWrapper(HttpContext.Current));