diff --git a/Pizzeria/PizzeriaBusinessLogic/OrderLogic.cs b/Pizzeria/PizzeriaBusinessLogic/OrderLogic.cs index f9a47a8..7f173dc 100644 --- a/Pizzeria/PizzeriaBusinessLogic/OrderLogic.cs +++ b/Pizzeria/PizzeriaBusinessLogic/OrderLogic.cs @@ -17,74 +17,14 @@ namespace PizzeriaBusinessLogic { private readonly ILogger _logger; private readonly IOrderStorage _orderStorage; - public OrderLogic(ILogger logger, IOrderStorage orderStorage) { _logger = logger; _orderStorage = orderStorage; } - - public bool CreateOrder(OrderBindingModel model) - { - CheckModel(model); - if (model.Status != OrderStatus.Неизвестен) return false; - model.Status = OrderStatus.Принят; - if (_orderStorage.Insert(model) == null) - { - _logger.LogWarning("Insert operation failed"); - return false; - } - return true; - } - - public bool DeliveryOrder(OrderBindingModel model) - { - CheckModel(model, false); - var element = _orderStorage.GetElement(new OrderSearchModel - { - Id = model.Id - }); - if (element == null) - { - _logger.LogWarning("Read operation failed"); - return false; - } - if (element.Status != OrderStatus.Готов) - { - _logger.LogWarning("Status change operation failed"); - throw new InvalidOperationException("Заказ должен быть переведен в статус готовности перед выдачей!"); - } - model.Status = OrderStatus.Выдан; - model.DateImplement = DateTime.Now; - _orderStorage.Update(model); - return true; - } - - public bool FinishOrder(OrderBindingModel model) - { - CheckModel(model, false); - var element = _orderStorage.GetElement(new OrderSearchModel - { - Id = model.Id - }); - if (element == null) - { - _logger.LogWarning("Read operation failed"); - return false; - } - if (element.Status != OrderStatus.Выполняется) - { - _logger.LogWarning("Status change operation failed"); - throw new InvalidOperationException("Заказ должен быть переведен в статус выполнения перед готовностью!"); - } - model.Status = OrderStatus.Готов; - _orderStorage.Update(model); - return true; - } - public List? ReadList(OrderSearchModel? model) { - _logger.LogInformation("ReadList. Id:{Id}", model?.Id); + _logger.LogInformation("ReadList. OrderId:{Id}", model?.Id); var list = model == null ? _orderStorage.GetFullList() : _orderStorage.GetFilteredList(model); if (list == null) { @@ -94,29 +34,31 @@ namespace PizzeriaBusinessLogic _logger.LogInformation("ReadList. Count:{Count}", list.Count); return list; } - - public bool TakeOrderInWork(OrderBindingModel model) + public bool CreateOrder(OrderBindingModel model) { - CheckModel(model, false); - var element = _orderStorage.GetElement(new OrderSearchModel + CheckModel(model); + if (model.Status != OrderStatus.Неизвестен) + return false; + model.Status = OrderStatus.Принят; + if (_orderStorage.Insert(model) == null) { - Id = model.Id - }); - if (element == null) - { - _logger.LogWarning("Read operation failed"); + _logger.LogWarning("Insert operation failed"); return false; } - if (element.Status != OrderStatus.Принят) - { - _logger.LogWarning("Status change operation failed"); - throw new InvalidOperationException("Заказ должен быть переведен в статус принятого перед его выполнением!"); - } - model.Status = OrderStatus.Выполняется; - _orderStorage.Update(model); return true; } - + public bool TakeOrderInWork(OrderBindingModel model) + { + return ChangeStatus(model, OrderStatus.Выполняется); + } + public bool FinishOrder(OrderBindingModel model) + { + return ChangeStatus(model, OrderStatus.Готов); + } + public bool DeliveryOrder(OrderBindingModel model) + { + return ChangeStatus(model, OrderStatus.Выдан); + } private void CheckModel(OrderBindingModel model, bool withParams = true) { if (model == null) @@ -127,23 +69,52 @@ namespace PizzeriaBusinessLogic { return; } - if (model.Sum <= 0) - { - throw new ArgumentNullException("Сумма заказа должна быть больше 0", nameof(model.Sum)); - } if (model.Count <= 0) { - throw new ArgumentNullException("Количество изделий должно быть больше 0", nameof(model.Count)); + throw new ArgumentException("Количество пиццы в заказе не может быть меньше 1", nameof(model.Count)); } - _logger.LogInformation("Order. Sum:{Sum}. Id:{Id}", model.Sum, model.Id); - var element = _orderStorage.GetElement(new OrderSearchModel + if (model.Sum < 1) + { + throw new ArgumentException("Стоимость заказа на может быть меньше 1", nameof(model.Sum)); + } + if (model.DateImplement.HasValue && model.DateImplement < model.DateCreate) + { + throw new ArithmeticException($"Дата выдачи заказа {model.DateImplement} не может быть раньше даты его создания {model.DateCreate}"); + } + _logger.LogInformation("Pizza. PizzaId:{PizzaId}.Count:{Count}.Sum:{Sum}Id:{Id}", + model.PizzaId, model.Count, model.Sum, model.Id); + } + private bool ChangeStatus(OrderBindingModel model, OrderStatus requiredStatus) + { + CheckModel(model, false); + var element = _orderStorage.GetElement(new OrderSearchModel() { Id = model.Id }); - if (element != null && element.Id != model.Id) + if (element == null) { - throw new InvalidOperationException("Заказ с таким ID уже существует"); + throw new ArgumentNullException(nameof(element)); } + model.DateCreate = element.DateCreate; + model.PizzaId = element.PizzaId; + model.DateImplement = element.DateImplement; + model.Status = element.Status; + model.Count = element.Count; + model.Sum = element.Sum; + if (requiredStatus - model.Status == 1) + { + model.Status = requiredStatus; + if (model.Status == OrderStatus.Выдан) + model.DateImplement = DateTime.Now; + if (_orderStorage.Update(model) == null) + { + _logger.LogWarning("Update operation failed"); + return false; + } + return true; + } + _logger.LogWarning("Changing status operation faled: Current-{Status}:required-{requiredStatus}.", model.Status, requiredStatus); + throw new ArgumentException($"Невозможно присвоить статус {requiredStatus} заказу с текущим статусом {model.Status}"); } } }