Шаблон для выполнения авторизации в слое репозитория приложения MVC

Question

У меня есть Windows-аутентифицированное MVC-приложение с уровнем репозитория. Все взаимодействие контроллера с базой данных осуществляется через репозиторий. Каждый контроллер имеет ссылку на репозиторий:

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); 
    } 
} 

Мой вопрос заключается в том, есть ли хороший способ, чтобы переместить мою логику авторизации в слой хранилище. Я хотел бы, чтобы функция Repository.DeletePost() отказала в удалении сообщений, которые не были созданы аутентифицированным пользователем. Проблема в том, что мой репозиторий не знает, кем является аутентифицированный пользователь. Контроллер знает (через Controller.User).

Передача Controller.User в конструктор Repository не работает, потому что Controller.User, по-видимому, не определен в момент вызова конструктора.

Как я могу сообщить Repository, кто является аутентифицированным пользователем? Было бы лучше всего построить Repository в каждом действии? Или это плохая идея справиться с этим на уровне репозитория?

Answer 1

Хорошие рекомендации от @BigDaddy и @ChrisPratt.

Я решил это решить, создав базовый контроллер, аналогичный this answer. Мой базовый класс контроллер выглядит следующим образом:

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())); 
     } 
    } 
} 

Все мои контроллеры наследовать от этого класса, и встроенный доступ к ленивым загруженным Repository, который имеет ссылку на текущую проверку подлинности пользователя.

Answer 2

Просто сделать что-то вроде:

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

Тогда в вашем репозитории:

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

Или это плохая идея, чтобы справиться с этим в хранилище слой?

Я думаю, что контроллер лучше для вашего разрешения. Пусть репозиторий будет шлюзом данных, а контроллер - привратником для вашего приложения. Я ожидаю увидеть логику авторизации/аутентификации как можно раньше на жизненном цикле.