CheckModels fix
This commit is contained in:
parent
79872191a8
commit
d76e6dc933
@ -198,8 +198,8 @@ namespace BankYouBankruptBusinessLogic.BusinessLogics
|
||||
throw new ArgumentNullException("Отсутствие номера у счёта", nameof(model.AccountNumber));
|
||||
}
|
||||
|
||||
//проверка на наличие id владельца
|
||||
if (model.CashierId < 0)
|
||||
//проверка на наличие id владельца
|
||||
if (model.CashierId < 0)
|
||||
{
|
||||
throw new ArgumentNullException("Некорректный Id владельца счёта", nameof(model.CashierId));
|
||||
}
|
||||
@ -211,11 +211,15 @@ namespace BankYouBankruptBusinessLogic.BusinessLogics
|
||||
}
|
||||
|
||||
//проверка на наличие пароля счёта
|
||||
if (string.IsNullOrEmpty(model.PasswordAccount))
|
||||
if (string.IsNullOrEmpty(model.PasswordAccount) )
|
||||
{
|
||||
throw new ArgumentNullException("Некорректный пароль счёта", nameof(model.PasswordAccount));
|
||||
}
|
||||
|
||||
if (model.Balance < 0) {
|
||||
throw new ArgumentNullException("Изначальный баланс аккаунта не может быть < 0", nameof(model.Balance));
|
||||
}
|
||||
|
||||
//проверка на корректную дату открытия счёта
|
||||
if (model.DateOpen > DateTime.Now)
|
||||
{
|
||||
|
@ -131,13 +131,13 @@ namespace BankYouBankruptBusinessLogic.BusinessLogics
|
||||
{
|
||||
return;
|
||||
}
|
||||
if (string.IsNullOrEmpty(model.Number))
|
||||
if (string.IsNullOrEmpty(model.Number) || model.Number.Length != 16)
|
||||
{
|
||||
throw new ArgumentNullException("Нет номера карты", nameof(model.Number));
|
||||
throw new ArgumentNullException("Неправильный номер карты", nameof(model.Number));
|
||||
}
|
||||
if (string.IsNullOrEmpty(model.CVC))
|
||||
if (string.IsNullOrEmpty(model.CVC) || model.CVC.Length != 16)
|
||||
{
|
||||
throw new ArgumentNullException("Нет СVC карты", nameof(model.CVC));
|
||||
throw new ArgumentNullException("Неправильный СVC карты", nameof(model.CVC));
|
||||
}
|
||||
if (model.Period < DateTime.Now)
|
||||
{
|
||||
|
@ -138,6 +138,7 @@ namespace BankYouBankruptBusinessLogic.BusinessLogics
|
||||
return false;
|
||||
}
|
||||
|
||||
|
||||
return true;
|
||||
}
|
||||
}
|
||||
|
Loading…
Reference in New Issue
Block a user