поправка методов

Pizzeria/PizzeriaBusinessLogic/OrderLogic.cs

@@ -17,74 +17,14 @@ namespace PizzeriaBusinessLogic
     {
         private readonly ILogger _logger;
         private readonly IOrderStorage _orderStorage;
-
         public OrderLogic(ILogger<OrderLogic> 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<OrderViewModel>? 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 CreateOrder(OrderBindingModel model)
-        public bool TakeOrderInWork(OrderBindingModel model)
         {
-            CheckModel(model, false);
+            CheckModel(model);
-            var element = _orderStorage.GetElement(new OrderSearchModel
+            if (model.Status != OrderStatus.Неизвестен)
+                return false;
+            model.Status = OrderStatus.Принят;
+            if (_orderStorage.Insert(model) == null)
             {
-                Id = model.Id
+                _logger.LogWarning("Insert operation failed");
-            });
-            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 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);
+            if (model.Sum < 1)
-            var element = _orderStorage.GetElement(new OrderSearchModel
+            {
+                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}");
         }
     }
 }