From f899455329a2bf271459eeb5b7a2e35fdd7c53e1 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=D0=98=D0=BB=D1=8C=D1=8F=20=D0=A4=D0=B5=D0=B4=D0=BE=D1=82?=
 =?UTF-8?q?=D0=BE=D0=B2?= <ilf.for.programming@gmail.com>
Date: Sun, 28 Apr 2024 23:36:49 +0400
Subject: [PATCH] fixed 0.3 :/

---
 .../BusinessLogic/UserLogic.cs                | 62 +++++++++++--------
 1 file changed, 37 insertions(+), 25 deletions(-)

diff --git a/ElectronicsShop/ElectronicsShopBusinessLogic/BusinessLogic/UserLogic.cs b/ElectronicsShop/ElectronicsShopBusinessLogic/BusinessLogic/UserLogic.cs
index 58bb163..a3106cf 100644
--- a/ElectronicsShop/ElectronicsShopBusinessLogic/BusinessLogic/UserLogic.cs
+++ b/ElectronicsShop/ElectronicsShopBusinessLogic/BusinessLogic/UserLogic.cs
@@ -5,22 +5,20 @@ using ElectronicsShopContracts.ViewModels;
 using Microsoft.Extensions.Logging;
 using System;
 using System.Collections.Generic;
-using System.ComponentModel.DataAnnotations;
-using System.Data;
 using System.Linq;
-using System.Runtime.CompilerServices;
 using System.Text;
 using System.Threading.Tasks;
 
 namespace ElectronicsShopBusinessLogic.BusinessLogic
 {
-    public class ClientLogic : IUserLogic
+    internal class UserLogic : IUserLogic
     {
         private readonly ILogger _logger;
         //private readonly IClientStorage _storage;
 
         // todo нет интерфейса хранилища  
-        public UserLogic(ILogger<UserLogic> logger) {
+        public UserLogic(ILogger<UserLogic> logger)
+        {
             _logger = logger;
             //storage = _storage;
         }
@@ -29,7 +27,8 @@ namespace ElectronicsShopBusinessLogic.BusinessLogic
         {
             CheckModel(model);
             // todo логика добавления в _clientStorage:_clientStorage.Insert(model) == null
-            if (model == null) {
+            if (model == null)
+            {
                 _logger.LogWarning("Add operation failed");
                 return false;
             }
@@ -51,9 +50,10 @@ namespace ElectronicsShopBusinessLogic.BusinessLogic
         public bool Delete(UserBindingModel model)
         {
             CheckModel(model, false);
-            _logger.LogInformation($"Delete.ID:{model.UserID}");
+            _logger.LogInformation($"Delete.ID:{model.ID}");
             // todo логика добавления в _clientStorage:_clientStorage.Delete(model) == null
-            if (model == null) {
+            if (model == null)
+            {
                 _logger.LogWarning("Delete operation failes");
                 return false;
             }
@@ -63,28 +63,31 @@ namespace ElectronicsShopBusinessLogic.BusinessLogic
 
         public UserViewModel? ReadElemet(UserSearchModel? model)
         {
-            if (model == null) {
+            if (model == null)
+            {
                 throw new ArgumentNullException(nameof(model));
             }
-            _logger.LogInformation($"ReadElement: logint:{ model.Login}.ID:{model.UserID}");
+            _logger.LogInformation($"ReadElement: logint:{model.Login}.ID:{model.ID}");
             // todo element = _clientStorage.GetElement(model);
             var element = model;
-            if (element == null) {
+            if (element == null)
+            {
                 _logger.LogWarning("ReadElement element not fount");
                 return null;
             }
-            _logger.LogInformation($"ReadElement: find.ID:{element.UserID}");
+            _logger.LogInformation($"ReadElement: find.ID:{element.ID}");
             // todo retun element;
             return null;
         }
 
         public List<UserViewModel>? ReadList(UserSearchModel? model)
         {
-            
-            _logger.LogInformation($"ReadList: ClientID:{model?.UserID}");
+
+            _logger.LogInformation($"ReadList: ClientID:{model?.ID}");
             // todo получение списка из хранилища,  model == null ? _clientStorage.GetFullList() : _clientStorage.GetFilteredList(model);
             var list = model;
-            if (list == null) {
+            if (list == null)
+            {
                 _logger.LogWarning("ReadList: return null list");
                 return null;
             }
@@ -94,29 +97,38 @@ namespace ElectronicsShopBusinessLogic.BusinessLogic
             return null;
         }
 
-        private void CheckModel(UserBindingModel model, bool withParams = true) { 
-            if (model == null) {
+        private void CheckModel(UserBindingModel model, bool withParams = true)
+        {
+            if (model == null)
+            {
                 throw new ArgumentNullException(nameof(model));
             }
-            if (!withParams) {
+            if (!withParams)
+            {
                 return;
             }
-            if (string.IsNullOrEmpty(model.Login)) {
+            if (string.IsNullOrEmpty(model.Login))
+            {
                 throw new ArgumentNullException("Нет логина пользователя", nameof(model.Login));
             }
-            if (string.IsNullOrEmpty(model.FirstName)) {
+            if (string.IsNullOrEmpty(model.FirstName))
+            {
                 throw new ArgumentNullException("Нет имени пользователя", nameof(model.FirstName));
             }
-            if (string.IsNullOrEmpty(model.LastName)) {
+            if (string.IsNullOrEmpty(model.LastName))
+            {
                 throw new ArgumentNullException("Нет фамилии пользоватея", nameof(model.LastName));
             }
-            if (string.IsNullOrEmpty(model.Email)) {
+            if (string.IsNullOrEmpty(model.Email))
+            {
                 throw new ArgumentNullException("Нет почты клиента", nameof(model.Email));
             }
-            if (string.IsNullOrEmpty(model.Password)) { 
-                throw new ArgumentNullException("Нет пароля пользователя", nameof (model.Password));
+            if (string.IsNullOrEmpty(model.Password))
+            {
+                throw new ArgumentNullException("Нет пароля пользователя", nameof(model.Password));
             }
-            if (string.IsNullOrEmpty(model.PhoneNumber)) {
+            if (string.IsNullOrEmpty(model.PhoneNumber))
+            {
                 throw new ArgumentNullException("Нет номер телефона пользователя", nameof(model.PhoneNumber));
             }
             _logger.LogInformation($"Client. Login:{model.Login}.FirstName:{model.FirstName}.LastName:{model.LastName}.Email:{model.Email}." +