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

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

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

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

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

3 ответов

7 просмотров

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

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

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

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

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

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

А как старый хаскел с новым стыковать ? потому как тут работает https://play.haskell.org/saved/C3xpMzcd, а вот тут https://stepik.org/lesson/7602/step/9?unit=1473 нет ошибка C...
Fedor
131
Народ, кто шарит в расширенных разделах (EBR/EPR) на дисках с разметками MBR? Везде пишут (в вики рус/англ) в частности + другие источники смотрел, что первый сектор расширенн...
Eugene Krasnikov (ᴊɪɴ x)
1
что насчет пагинга? на осдеве непонятно(
Vi Chapmann 🪙
26
Вопрос я правильно понимаю что в коде newtype ArrowMap k v = ArrowMap { getArrowMap :: k -> Maybe v } getArrowMap есть функция типа k -> Maybe v, если да, то не понимаю задач...
Fedor
64
Ребят, что лучше для реверса: гидра или ида?
En Vind Av Sorg
26
Делаю велосипед логгер. К сообщению хочу прикрутить некоторую информацию, типа, кем отправлено, какой уровень, и всякое такое. И тут подумалось мне, почему бы не хранить весь...
Serjone
24
Как Вы считаете нормально ли в двадцатых годах 21 века в ВУЗах Российской Федерации обучать студентов работе с TASM? Не слишком ли это "архаично"? (Если оффтоп или флейм для э...
Spiker01
52
Комрады, хотел уточнить. Проперть в OnDestroy юнита-хозяина по-прежнему доступна? И еще уточнение: finalization юнита наступает раньше или позже OnDestroy?
Ed Doc
48
Продолжая диалог про свифт в проде – сейчас возник вопрос в активном наборе бекендеров. В основном в нашей компании мы фанаты Java Spring и полностью ей довольны. Однако найм ...
Guseyn
27
Читаю сейчас [нет, уже больше не читаю!] курсовую о Булгакове, написанную, похоже, с помощью ChatGPT. Это удивительный психоделический опыт. Текст в основном написан в стиле б...
✨ Uni [🌊 В отпуске]
1
Карта сайта