Belnik/CourseWork_Bank
Belnik/CourseWork_Bank
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Косметические правки BussinessLogic

Browse Source
This commit is contained in:
nikbel2004@outlook.com 2024-05-01 03:18:08 +04:00
parent 3f63e356d6
commit f303c2b85e
11 changed files with 46 additions and 38 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

22
Bank/BankBusinessLogic/BusinessLogic/Cashier/AccountLogic.cs
View File

@ -12,7 +12,8 @@ using System.Threading.Tasks;
namespace BankBusinessLogic.BusinessLogic.Cashier
{
public class AccountLogic : IAccountLogic
// Класс, реализующий бизнес-логику для счетов
public class AccountLogic : IAccountLogic
{
private readonly ILogger _logger;
@ -61,7 +62,7 @@ namespace BankBusinessLogic.BusinessLogic.Cashier
{
_logger.LogInformation("ReadList. AccountNumber:{AccountNumber}. Id:{Id}", model.AccountNumber, model?.Id);
//list хранит весь список в случае, если model пришло со значением null на вход метода
// list хранит весь список в случае, если model пришло со значением null на вход метода
var list = model == null ? _accountStorage.GetFullList() : _accountStorage.GetFilteredList(model);
if (list == null)
@ -169,17 +170,14 @@ namespace BankBusinessLogic.BusinessLogic.Cashier
throw new ArgumentNullException("Отсутствие номера у счёта", nameof(model.AccountNumber));
}
// Проверка на наличие id владельца
// Проверка на наличие id клиента (И всё-таки не будет лишним добавить Id клиента)
if (model.ClientId < 0)
{
throw new ArgumentNullException("Некорректный Id клиента", nameof(model.ClientId));
}
// СОЗВОН ИБО Я НЕ ПОН, А ЗАЧЕМ ЛИШКА ИЛИ МОЖЕТ ЭТО КЛИЕНТ ИМЕЛСЯ ВВИДУ???!
//if (model.CashierId < 0)
//{
// throw new ArgumentNullException("Некорректный Id владельца счёта", nameof(model.CashierId));
//}
// Проверка на наличие id кассира, создавшего счёт
if (model.CashierId < 0)
// Проверка на наличие id кассира, создавшего счёт
if (model.CashierId < 0)
{
throw new ArgumentNullException("Некорректный Id кассира, открывшего счёт", nameof(model.CashierId));
}

6
Bank/BankBusinessLogic/BusinessLogic/Cashier/CashWithdrawalLogic.cs
View File

@ -14,7 +14,8 @@ using System.Threading.Tasks;
namespace BankBusinessLogic.BusinessLogic.Cashier
{
public class CashWithdrawalLogic : ICashWithdrawalLogic
// Класс, реализующий бизнес-логику для снятия наличных
public class CashWithdrawalLogic : ICashWithdrawalLogic
{
private readonly ILogger _logger;
@ -23,8 +24,7 @@ namespace BankBusinessLogic.BusinessLogic.Cashier
private readonly IDebitingStorage _debitingStorage;
// Конструктор
public CashWithdrawalLogic(ILogger<CashWithdrawalLogic> logger, ICashWithdrawalStorage cashWithdrawalStorage,
IDebitingStorage debitingStorage)
public CashWithdrawalLogic(ILogger<CashWithdrawalLogic> logger, ICashWithdrawalStorage cashWithdrawalStorage, IDebitingStorage debitingStorage)
{
_logger = logger;
_cashWithdrawalStorage = cashWithdrawalStorage;

2
Bank/BankBusinessLogic/BusinessLogic/Cashier/CashierLogic.cs
View File

@ -175,7 +175,7 @@ namespace BankBusinessLogic.BusinessLogic.Cashier
Email = model.Email,
});
//если элемент найден и его Id не совпадает с Id переданного объекта
// Если элемент найден и его Id не совпадает с Id переданного объекта
if (element != null && element.Id != model.Id)
{
throw new InvalidOperationException("Аккаунт с таким логином уже есть");

7
Bank/BankBusinessLogic/BusinessLogic/Cashier/MoneyTransferLogic.cs
View File

@ -14,7 +14,8 @@ using System.Threading.Tasks;
namespace BankBusinessLogic.BusinessLogic.Cashier
{
public class MoneyTransferLogic : IMoneyTransferLogic
// Класс, реализующий бизнес-логику для перевода наличных
public class MoneyTransferLogic : IMoneyTransferLogic
{
private readonly ILogger _logger;
@ -55,7 +56,7 @@ namespace BankBusinessLogic.BusinessLogic.Cashier
return element;
}
// Вывод всего списка операций
// Вывод всего списка операций на перевод наличных
public List<MoneyTransferViewModel>? ReadList(MoneyTransferSearchModel? model)
{
_logger.LogInformation("ReadElement. AccountSenderId:{AccountSenderId}. AccountPayeeId:{AccountPayeeId}. Sum:{Sum}. Id:{Id}",
@ -87,7 +88,7 @@ namespace BankBusinessLogic.BusinessLogic.Cashier
return false;
}
//проверка на то, что это зачисление на карту, а не перевод между счетами
// Проверка на то, что это зачисление на карту, а не перевод между счетами
if (model.CreditingId.HasValue)
{
_creditingStorage.Update(new CreditingBindingModel

20
Bank/BankBusinessLogic/BusinessLogic/Client/CardLogic.cs
View File

@ -14,7 +14,8 @@ using System.Threading.Tasks;
namespace BankBusinessLogic.BusinessLogic.Client
{
public class CardLogic : ICardLogic
// Класс, реализующий бизнес-логику для банковских карт
public class CardLogic : ICardLogic
{
private readonly ILogger _logger;
@ -107,7 +108,6 @@ namespace BankBusinessLogic.BusinessLogic.Client
return true;
}
// Удаление банковской карты
public bool Delete(CardBindingModel model)
{
@ -144,15 +144,19 @@ namespace BankBusinessLogic.BusinessLogic.Client
throw new ArgumentNullException("Неправильный номер карты", nameof(model.Number));
}
// Здесь стоит обсудить про баланс !!!!!! И про добавление ID клиента ниже!
// Проверка баланса банковской карты (чтобы не было отрицательным)
if (model.Balance < 0)
{
throw new ArgumentNullException("Изначальный баланс карты не может быть < 0", nameof(model.Balance));
}
// Проверка на конкретный период действия карты
if (model.Period < DateTime.Now)
// Проверка на конкретный период действия карты
if (model.Period < DateTime.Now)
{
throw new ArgumentNullException("Нет периода действия", nameof(model.Period));
}
// Проверка на наличие id клиента, получеившего карту
// Проверка на наличие id клиента, получившего карту (лишним не будет)
if (model.ClientId < 0)
{
throw new ArgumentNullException("Некорректный Id кассира, открывшего счёт", nameof(model.ClientId));
@ -176,8 +180,8 @@ namespace BankBusinessLogic.BusinessLogic.Client
ClientId = model.ClientId
});
if (accountElement != null && accountElement.ClientId != model.ClientId)
// Проверка привязан ли счёт к данному клиенту
if (accountElement != null && accountElement.ClientId != model.ClientId)
{
throw new InvalidOperationException("Это не счёт данного клиента");
}

7
Bank/BankBusinessLogic/BusinessLogic/Client/ClientLogic.cs
View File

@ -36,6 +36,7 @@ namespace BankBusinessLogic.BusinessLogic.Client
_logger.LogInformation("ReadElement. Name:{Name}.Surname:{Surname}.Patronymic:{Patronymic}.Id:{Id}", model.Name, model.Surname, model.Patronymic, model.Id);
var element = _clientStorage.GetElement(model);
if (element == null)
{
_logger.LogWarning("Read element not found");
@ -68,6 +69,7 @@ namespace BankBusinessLogic.BusinessLogic.Client
public bool Create(ClientBindingModel model)
{
CheckModel(model);
if (_clientStorage.Insert(model) == null)
{
_logger.LogWarning("Insert operation failed");
@ -81,6 +83,7 @@ namespace BankBusinessLogic.BusinessLogic.Client
public bool Update(ClientBindingModel model)
{
CheckModel(model);
if (_clientStorage.Update(model) == null)
{
_logger.LogWarning("Update operation failed");
@ -94,7 +97,9 @@ namespace BankBusinessLogic.BusinessLogic.Client
public bool Delete(ClientBindingModel model)
{
CheckModel(model, false);
_logger.LogInformation("Delete. Id:{Id}", model.Id);
if (_clientStorage.Delete(model) == null)
{
_logger.LogWarning("Delete operation failed");
@ -136,7 +141,7 @@ namespace BankBusinessLogic.BusinessLogic.Client
throw new ArgumentNullException("Нет отчества пользователя", nameof(model.Patronymic));
}
// Проверка на наличие эл. почты
// Проверка на наличие электронной почты
if (string.IsNullOrEmpty(model.Email))
{
throw new ArgumentNullException("Нет почты пользователя", nameof(model.Email));

7
Bank/BankBusinessLogic/BusinessLogic/Client/CreditingLogic.cs
View File

@ -12,7 +12,8 @@ using System.Threading.Tasks;
namespace BankBusinessLogic.BusinessLogic.Client
{
public class CreditingLogic : ICreditingLogic
// Класс, реализующий бизнес-логику для пополнения карты
public class CreditingLogic : ICreditingLogic
{
private readonly ILogger _logger;
@ -129,10 +130,10 @@ namespace BankBusinessLogic.BusinessLogic.Client
return;
}
// Проверка на корректность Id банковской карты
// Проверка на корректность Id клиента банковской карты
if (model.ClientId < 0)
{
throw new ArgumentNullException("Некорректный Id счёта", nameof(model.ClientId));
throw new ArgumentNullException("Некорректный Id клиента", nameof(model.ClientId));
}
// Проверка на корректность суммы пополнения

7
Bank/BankBusinessLogic/BusinessLogic/Client/DebitingLogic.cs
View File

@ -12,7 +12,8 @@ using System.Threading.Tasks;
namespace BankBusinessLogic.BusinessLogic.Client
{
public class DebitingLogic : IDebitingLogic
// Класс, реализующий бизнес-логику для снятия наличных с карты
public class DebitingLogic : IDebitingLogic
{
private readonly ILogger _logger;
@ -129,10 +130,10 @@ namespace BankBusinessLogic.BusinessLogic.Client
return;
}
// Проверка на корректность Id банковской карты
// Проверка на корректность Id клиента банковской карты
if (model.ClientId < 0)
{
throw new ArgumentNullException("Некорректный Id счёта", nameof(model.ClientId));
throw new ArgumentNullException("Некорректный Id клиента", nameof(model.ClientId));
}
// Проверка на корректность снятия суммы

4
Bank/BankBusinessLogic/BusinessLogic/Reports/ReportCashierLogic.cs
View File

@ -49,7 +49,7 @@ namespace BankBusinessLogic.BusinessLogic.Reports
_saveToPdf = saveToPdf;
}
// Формирование списка переводов между счетами за период
// Формирование списка переводов между счетами
public List<ReportCashierViewModel>? GetMoneyTransfers(ReportBindingModel model)
{
return _moneyTransferStorage.GetFilteredList(new MoneyTransferSearchModel { ClientId = model.ClientId, DateTransfer = model.DateTo })
@ -64,7 +64,7 @@ namespace BankBusinessLogic.BusinessLogic.Reports
.ToList();
}
// Формирование списка выдачи наличных со счёта за период
// Формирование списка выдачи наличных со счёта
public List<ReportCashierViewModel>? GetCashWithrawals(ReportBindingModel model)
{
return _cashWithdrawalStorage.GetFilteredList(new CashWithdrawalSearchModel { ClientId = model.ClientId, DateWithdrawal = model.DateTo })

1
Bank/BankBusinessLogic/OfficePackage/AbstractSaveToPdf.cs
View File

@ -9,7 +9,6 @@ namespace BankBusinessLogic.OfficePackage
{
public abstract class AbstractSaveToPdf
{
/// Создание pdf-файла
protected abstract void CreatePdf(PdfInfo info);

1
Bank/BankBusinessLogic/OfficePackage/AbstractSaveToWord.cs
View File

@ -9,7 +9,6 @@ namespace BankBusinessLogic.OfficePackage
{
public abstract class AbstractSaveToWord
{
/// Создание doc-файла
protected abstract void CreateWord(WordInfo info);