Есть вот такая сущность:
public class Product
{
public int Id { get; set; }
public string Name { get; set; }
public string CategoryName { get; set; } //доступно для редактирования только пользователю «admin»
// в реальном проекте свойств больше
// и среди них также есть доступные для редактирования только для admin
}
Необходимо на стороне сервера создать метод PUT
, который обеспечил бы редактирование свойства CategoryName
только для авторизованного пользователя admin.
Какие практики REST существуют для реализации подобном логики?
Моя реализация такова:
public class ProductDto
{
[Required]
public string Name { get; set; }
[Required]
public string CategoryName { get; set; }
}
[Authorize]
[RoutePrefix("api/products")]
public class ProductsController : ApiController
{
private readonly IProductService _productService;
public ProductsController(IProductService productService)
{
_productService = productService;
}
[HttpPut]
[Route("{id}")]
public HttpResponseMessage Put([FromBody] ProductDto productDto)
{
if (!ModelState.IsValid)
{
throw new ArgumentException("dto");
}
var productId = RequestContext.RouteData.Values["id"].ToString();
var userName = User.Identity.Name; //извлекаем имя авторизованного пользователя
var isAdmin = userName == "admin";
_productService.Update(productId, productDto, isAdmin);
return Request.CreateResponse(HttpStatusCode.OK);
}
}
public interface IProductService
{
void Update(string id, ProductDto dto, bool isAdmin);
}
public class ProductService : IProductService
{
private readonly IGenericRepository<Product> _productRepository;
public ProductService(IGenericRepository<Product> productRepository)
{
_productRepository = productRepository;
}
public void Update(string id, ProductDto dto, bool isAdmin)
{
if (dto == null || id == null)
{
throw new ArgumentNullException();
}
var product = _productRepository.GetById(id);
if (product == null)
{
throw new ArgumentException(nameof(id));
}
product.Name = dto.Name;
if(isAdmin)
{
product.CategoryName = dto.CategoryName;
}
_productRepository.Update(product);
}
}
- Какими изъянами обладает данный код, учитывая то, что входе масштабирования у сущности
Product
будут появляться новые свойства, среди которых также могут быть те которые редактировать может только admin. - Или стоило попробовать использовать атрибуты валидации/перезагрузку PUT-метода и т.п?