170 похожих чатов

Привет! Пишу библиотеку для работы с TFS-репозиториями из Azure Devops'a. Необходимо

в структурку положить id, имя автора, дату, количество удаленных и добавленных строк, количство changesetChanges для каждого changset'a.

Т.е. есть метод, который получает все changeset'ы репозитория, затем для каждого changeset'а получает changesetChanges, затем считает количество удаленных и добавленных строк.

Вопрос такой: Правильно ли разделить этот метод на 4 маленьких (GetChangeSets, GetChangeSetChanges, GetAddedRows, GetDeletedRows)? С одной стороны каждый метод будет отвечать за более маленькую задачу, с другой стороны эти методы никогда не будут переиспользованы для других задач.

3 ответов

18 просмотров

Если они больше одной строки, то вполне можно. Методы не только для переиспрльзования создаются, но и для структурирования кода, чтоб лучше понимать что какая строка делает. Плюс эти методы как минимум могут быть «переиспользованы» в тестах)

Андрей Васильев
Если они больше одной строки, то вполне можно. Мет...

Чтобы переиспользовать, функция должна быть чистой

переюз это не единственная причина для распиливания

Похожие вопросы

Обсуждают сегодня

Мужики и девушки, привет) в Вelphi xe7 в настройках во вкладке "Editor Options" далее " Color" есть список: "Elements", открыв который мы можем настраивать отображение разных...
Kraszx
14
Добрый вечер. Есть вопрос, а может и предложение. Был у меня диалог в другой группе о делфи и я задался вопросом: "А нельзя ли в делфи цвет //коментария и {комментария} сде...
Kraszx
24
Я вот подумал. SSE выполняет операции максимум с 64-битной точностью. А FPU - всегда с 80-битной. Разве не должно быть FPU точнее тогда?
The Bird of Hermes
13
как быть с принтером? такой подход прокатит?
zamtmn
12
Мдя, прикол, боевая сборка запускается (именно под отладчиком) после F9 примерно полторы минуты (97 секунд если быть точным). Начал копать - проблема детектится сразу - зависа...
Александр (Rouse_) Багель
38
Всем привет! Подскажи, пожалуйста, как передать в TComboBox сразу значение и id записи. На Delphi я делал так: ComboBox1.Items.AddObject('Какое-то значение', Pointer(id запис...
Евгений
13
Здравствуйте, вопрос по структурам данных. Были у вас случаи, когда пришлось писать деревья или двунаправленные списки?
/ /
50
Я не понимаю, это троллинг или что? Швабрика поддерживают, который буквально пишет на ассемблере взаимодействия с винапи. Я это ещё написал загрузчик и хоть что-то изучаю в о...
Shadow Akira
6
А вот это что за конструкция? Вернее, она тут нафига?
Serjone
10
Привет. Подскажите, как правильно сматчить лист фиксированного размера, чтобы компилятор не говорил мне о неполном паттерне? Допустим что-то такое [x', y'] = sort [x, y]?
Arseny
8
Карта сайта