Раскидать метод на отдельные методы

@antoniogrid · Регистрация: 20.07.2014

Студворк — интернет-сервис помощи студентам

Всем привет
Я вижу, что здесь несколько логики в рамках одного метода
Метод проверяет и длину имени, и длину пароля, и устанавливает дату

А как это можно было бы вынести в отдельные методы, которые делают это по отдельности?

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
namespace Blalalal
{
    public class RegisterAccountAction
    {
        public IPasswordChecker PasswordChecker { get; set; }
        public IAccountManager AccountManager { get; set; }
 
        public void Register(IAccount account)
        {
            if (account.GetName().Length <= 7)
            {
                throw new WrongAccountNameException();
            }
            string password = account.GetPassword();
 
            if (password.Length <= 6)
            {
                if (PasswordChecker.Validate(password) != CheckStatus.Ok)
                {
                    throw new WrongPasswordException();
                }
            }
 
            account.SetCreatedDate(new DateTime());
            var addresses = new List<IAddress>
            {
                account.GetHomeAddress(),
                account.GetWorkAddress(),
                account.GetAdditionalAddress()
            };
            account.SetAddresses(addresses);
            AccountManager.CreateNewAccount(account);
        }
 
    }
}

@samana · 23.11.2019, 17:52

Да, я знаю, что советуют для каждого отдельно действия создавать отдельный метод, но всегда ли это нужно? Вынести в отдельные методы несложно, но стоит ли это делать в данном случае? Если это все необходимые проверки, то в данном случае удобно видеть сразу всё в одном месте.
Ещё мне кажется, что не стоит останавливать программу, выбрасывая исключения, если пользователь случайно сделал опечатку. Лучше просто запросить ввод данных ещё раз.

А разбить по методам можно так, например для ввода имени. Остальные методы можно сделать по аналогии.

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
public void Register(IAccount account)
{
    checkName(account);
 
    string password = account.GetPassword();
 
    if (password.Length <= 6)
    {
        if (PasswordChecker.Validate(password) != CheckStatus.Ok)
        {
            throw new WrongPasswordException();
        }
    }
 
    account.SetCreatedDate(new DateTime());
    var addresses = new List<IAddress>
    {
        account.GetHomeAddress(),
        account.GetWorkAddress(),
        account.GetAdditionalAddress()
    };
    account.SetAddresses(addresses);
    AccountManager.CreateNewAccount(account);
}
 
private void checkName(IAccount account)
{
    if (account.GetName().Length <= 7)
    {
        throw new WrongAccountNameException();
    }
}

@Элд Хасп · 23.11.2019, 20:56

antoniogrid, согласен с samana.
В данном случае не нужно разбивать на отдельные методы.

Имеет смысл только если такая же логика нужна ещё где-то.
Допустим, создать статический метод проверки логина.
Использовать его в методе Register.
А также появляется возможность его использования в представлении. Сразу при вводе логина - ещё до отправки на регистрацию.

Новые блоги и статьи Все статьи Все блоги /
1С: Программный отбор элементов справочника Номенклатура по группе Maks 22.03.2026 Установка программного отбора элементов справочника "Номенклатура" из модуля формы документа. В качестве фильтра для отбора справочника служит группа номенклатуры. Отбор по наименованию группы. . .	Как я обхитрил таблицу Word Alexander-7 21.03.2026 Когда мигает курсор у внешнего края таблицы, и нам надо перейти на новую строку, а при нажатии Enter создается новый ряд таблицы с ячейками, то мы вместо нервных нажатий Энтеров мы пишем любые буквы. . .	Krabik - рыболовный бот для WoW 3.3.5a AmbA 21.03.2026 без регистрации и смс. Это не торговля, приложение не содержит рекламы. Выполняет свою непосредственную задачу - автоматизацию рыбалки в WoW - и ничего более. Однако если админы будут против -. . .	1С: Программный отбор элементов справочника Сотрудники по значению перечисления Maks 21.03.2026 Установка программного отбора элементов справочника "Сотрудники" из модуля формы документа. В качестве фильтра для отбора служит предопределенное значение перечислений. Процедура. . .
Переходник USB-CAN-GPIO Eddy_Em 20.03.2026 Достаточно давно на работе возникла необходимость в переходнике CAN-USB с гальваноразвязкой, оный и был разработан. Однако, все меня терзала совесть, что аж 48-ногий МК используется так тупо: просто. . .	Оттенки серого Argus19 18.03.2026 Оттенки серого Нашёл в интернете 3 прекрасных модуля: Модуль класса открытия диалога открытия/ сохранения файла на Win32 API; Модуль класса быстрого перекодирования цветного изображения в оттенки. . .	SDL3 для Desktop (MinGW): Рисуем цветные прямоугольники с помощью рисовальщика SDL3 на Си и C++ 8Observer8 17.03.2026 Содержание блога Финальные проекты на Си и на C++: finish-rectangles-sdl3-c. zip finish-rectangles-sdl3-cpp. zip	Символические и жёсткие ссылки в Linux. algri14 15.03.2026 Существует два типа ссылок — символические и жёсткие. Ссылка в Linux — это запись в каталоге, которая может указывать либо на inode «файла-ИСТОЧНИКА», тогда это будет «жёсткая ссылка» (hard link),. . .

@Элд Хасп Модератор 16140 / 11264 / 2888 Регистрация: 21.04.2018 Сообщений: 33,110 Записей в блоге: 2
	23.11.2019, 20:56
	antoniogrid, согласен с samana. В данном случае не нужно разбивать на отдельные методы. Имеет смысл только если такая же логика нужна ещё где-то. Допустим, создать статический метод проверки логина. Использовать его в методе Register. А также появляется возможность его использования в представлении. Сразу при вводе логина - ещё до отправки на регистрацию. 2

Раскидать метод на отдельные методы

Решение