Mẫu để thực hiện ủy quyền trong lớp kho lưu trữ của ứng dụng MVC

Question

Tôi có một ứng dụng MVC được chứng thực Windows với một lớp kho lưu trữ. Tất cả sự tương tác của bộ điều khiển với cơ sở dữ liệu được thực hiện thông qua kho lưu trữ. Mỗi bộ điều khiển có tham chiếu đến kho lưu trữ:

public class PostController : Controller 
{ 
    private Repository db = new Repository(); 

    [HttpPost] 
    public ActionResult DeletePost(int id) 
    { 
     // Authorize that the user is allowed to delete this post... 

     db.DeletePost(id); 
    } 
} 

Câu hỏi của tôi là liệu có cách nào tốt để chuyển logic ủy quyền của tôi vào tầng lưu trữ hay không. Tôi muốn chức năng Repository.DeletePost() từ chối xóa các bài đăng không được tạo bởi người dùng đã được xác thực. Vấn đề là kho lưu trữ của tôi không biết ai là người dùng được xác thực. Bộ điều khiển biết (thông qua Controller.User).

Chuyển số Controller.User vào hàm xây dựng Repository không hoạt động, bởi vì Controller.User dường như không được xác định tại thời điểm hàm tạo được gọi.

Tôi làm cách nào để thông báo cho Repository người dùng được xác thực là ai? Nó sẽ là tốt nhất để chỉ cần xây dựng các Repository trong mỗi hành động? Hoặc là một ý tưởng tồi để xử lý nó trong lớp kho lưu trữ?

Answer 1

Đề xuất tốt từ cả @BigDaddy và @ChrisPratt.

Tôi đã kết thúc việc giải quyết điều này bằng cách tạo bộ điều khiển cơ sở, tương tự như this answer. My lớp điều khiển cơ bản trông giống như:

public class BaseController : Controller 
{ 
    private ILog _log; 
    private Repository _db; 

    protected Repository Db 
    { 
     get 
     { 
      return _db ?? (_db = new Repository(User)); 
     } 
    } 

    protected ILog Log 
    { 
     get 
     { 
      return _log ?? (_log = LogManager.GetLogger(this.GetType())); 
     } 
    } 
} 

Tất cả các bộ điều khiển của tôi kế thừa từ lớp này, và đã xây dựng-trong việc tiếp cận một lười biếng-nạp Repository mà có một tài liệu tham khảo cho người dùng hiện chứng thực.

Answer 2

Chỉ cần làm một cái gì đó như:

db.DeletePostForUser(id, User.Identity.UserId); 

Sau đó, trong kho của bạn:

public void DeletePostForUser(int id, int userId) 
{ 
    var post = context.Posts.SingleOrDefault(m => m.PostId == id && m.User.UserId == userId) 
    if (post != null) 
    { 
     context.Posts.Remove(post); 
     context.SaveChanges(); 
    } 
} 

Answer 3

Hoặc là nó một ý tưởng tồi để xử lý nó trong lớp kho?

Tôi nghĩ Bộ điều khiển là nơi tốt hơn cho ủy quyền của bạn. Hãy để kho lưu trữ là một cổng vào dữ liệu và bộ điều khiển là một gatekeeper cho ứng dụng của bạn. Tôi hy vọng sẽ thấy logic ủy quyền/xác thực sớm nhất trong vòng đời càng tốt.