Считается ли это плохим кодом?

Question

[Twitt]

Есть класс, который должен ходить в хранилище, и записывать/доставать данные туда/оттуда.
Вот его методы:
get() - получает данные,
preparePath() (он просто указывает путь, где именно брать данные (это не бд)),
validate(),
moveToStorage() - инсертит данные,
format(),
paginate()
Правильно ли я понимаю, что это грубое нарушение Single Responsibility Principle?
Т.е. этот класс нужно разделить на 3 других, например:
Formatter (вынести туда format метод, и возможно, preparePath)
Validator (вынести validate)
Paginator (вынести paginate)
Ну а в самом изначальном классе оставить moveToStorage + get методы.

Интересно мнение, не перебарщиваю ли я в этом плане, где для каждого метода надо отдельный класс или "и так норм"?

Comments

[Twitt]

> Причем тут SRP
класс по идее делает больше, чем надо. он сразу: форматирует, инсертит, получает, делает пагинацию
SRP же говорит о том, что класс должен делать только одну задачу

Answer 1

[Евгений Ромашкан]

get()

Чего он делает? И где типы и имена параметров, где возвращаемые типы?

preparePath() (он просто указывает путь, где именно брать данные (это не бд)),

Ничего не понятно даже с комментарием.

validate()

Кого validate?

paginate()

И чего оно делает?

Правильно ли я понимаю, что это грубое нарушение SRP?
...
Ну а в самом изначальном классе оставить moveToStorage + get методы.

Оставьте гейтвей к базе который умеет из неё данные доставать и в нужном формате возвращать, остальное в отдельные классы.