в структурку положить id, имя автора, дату, количество удаленных и добавленных строк, количство changesetChanges для каждого changset'a.
Т.е. есть метод, который получает все changeset'ы репозитория, затем для каждого changeset'а получает changesetChanges, затем считает количество удаленных и добавленных строк.
Вопрос такой: Правильно ли разделить этот метод на 4 маленьких (GetChangeSets, GetChangeSetChanges, GetAddedRows, GetDeletedRows)? С одной стороны каждый метод будет отвечать за более маленькую задачу, с другой стороны эти методы никогда не будут переиспользованы для других задач.
Если они больше одной строки, то вполне можно. Методы не только для переиспрльзования создаются, но и для структурирования кода, чтоб лучше понимать что какая строка делает. Плюс эти методы как минимум могут быть «переиспользованы» в тестах)
Чтобы переиспользовать, функция должна быть чистой
переюз это не единственная причина для распиливания
Обсуждают сегодня