В asp.net-mvc проекте для проверки доступности того или иного метода в gui я использую фильтры действия, пример такого фильтра ниже:
public class DisallowChangingTheApprovedOperationsAttribute : ActionFilterAttribute
{
public IOperationService _operationService { get; set; }
public override void OnActionExecuting(ActionExecutingContext filterContext)
{
var valueResult = filterContext.Controller.ValueProvider.GetValue("operationId");
if (valueResult == null || string.IsNullOrWhiteSpace(valueResult.AttemptedValue))
{
throw new InvalidOperationException("OperationId is not be null");
}
int operationId = int.Parse(valueResult.AttemptedValue);
var result = _operationService.IsPublished(operationId);
if (result)
{
if (string.Equals(filterContext.HttpContext.Request.RequestType, "Get", System.StringComparison.InvariantCultureIgnoreCase))
{
filterContext.Result = new RedirectResult(filterContext.HttpContext.Request.UrlReferrer.PathAndQuery);
}
else
{
filterContext.Result = new RedirectToRouteResult(filterContext.ActionDescriptor.ControllerDescriptor.ControllerName,
new RouteValueDictionary { { "operationId", operationId } });
}
}
}
}
его предназначение не допустить изменение утвержденной Operation, методя для переданного параметра operationId
проверяет состояние var result = _operationService.IsPublished(operationId);
и если результат true
то происходит отмена действия
Данный фильтр я применяю к необходимым мне методам: EditOperation
, DeleteOperation
и т.п., пример метода для редактирования ниже:
[DisallowChangingTheApprovedOperations]
public ActionResult EditOperation(int operationId)
{
ViewBag.Products = new SelectList(_productService.GetProductList(), "Id", "Name");
ViewBag.Equipments = new SelectList(_equipmentService.GetEquipmentList(), "Id", "SelectListName");
ViewBag.Units = new SelectList(_unitService.GetAll(), "Id", "FullName");
var model = _operationService.GetOperation(operationId);
return View("EditOperation", model);
}
[HttpPost]
[DisallowChangingTheApprovedOperations]
public Actionresult EditOperation(int operationId, EditedOperation model)
{
}
Для того что бы данный метод работал мне приходится в каждый из методов таскать operationId
что мне кажется не удобным/не правильным, но это еще не все.
Так же у меня есть сущность Program
которая относится к Operation
в соотношении 1 операция много программ, для работы с Program
у меня есть методы: AddProgram
, EditProgram
и т.п. Мне необходимо запретить изменения операции на основании её состояния(_operationService.isPublished(int operationId)
) - нельзя добавить, изменить существующие программы, для этих целей я в методы редактирования, изменения, удаления программы передаю параметр operationId
и применяю к методу тот же фильтр DisallowChangingTheApprovedOperations
пример метода для редактирования программы:
[DisallowChangingTheApprovedOperations]
//здесь прочие методы
public ActionResult EditProgram(int operationId, programId)
{}
т.е. здесь я помимо id
программы мне приходися передавать еще и id
операции, как вариант можно реализовать метод от наследованный от ActionFilterAttribute
который на основании programId
будет получать состояние операции и в соответствии с некой логикой отменять или разрешать действие, т.е. что то типа
public class DisallowChangingTheApprovedOperationsForTheProgrammAttribute : ActionFilterAttribute
{
//получаем programId из контекста
var result = _operationService.GetOperationStateForTheProgramm(programId);
//отменяем действия в соответствии с логикой
}
Подскажите как поступить лучше/реализовать правильней возможность ограничения некоторых действий пользователя в соответствии с некой логикой?
get
методаEditOperation
, а в методы изменения программ я его передаю только для фильтра