1

Использую в проекте Spring и Hibernate, разные ServiceDAO, осуществляющие доступ к данным, имеют уже порядка 30 методов, мне стало интересно, стоит ли их разбивать ради повышения производительности, или кол-во методов в классе не влияет на его скорость загрузки?

1
  • 3
    Не думаю, что влияет. 24 окт 2018 в 4:49

2 ответа 2

5

Не влияет.

На скорость влияет код в методах, а не количество их в классе, т.к. код выполняется многократно, а классы загружаются один раз. Скорость загрузки байт-кода так высока что какие бы то ни было различия будут неощутимы. Выигрыша, даже незначительного, не будет в любом случае т.к. вместо одного класса с 30-ю методами будет загружено три по 10 в каждом.

Более общая мысль: если решили всерьез заняться оптимизацией, то первым делом нужно измерить скорость выполнения кода, найти самые медленные места и сосредоточиться на них. Навряд ли самой медленной частью будет работа загрузчика классов.

Вызовы методов, загрузка классов, JIT-компиляция, как правило, работают быстро. Запрос к БД либо обращение к сервисам займут в тысячи раз больше времени. Запуск сборщика мусора и загрузка сущности из кэша работают в сотни раз медленнее. Экономить на наносекундах можно, но в этом мало смысла когда секунды тратятся впустую.

стоит ли их разбивать ради повышения производительности

Не стоит. Разбивать классы стоит для организация логики и повышения удобочитаемости.

P.S. «Преждевременная оптимизация — корень всех зол.» Дональд Кнут

3
  • А если все методы типа getUserByLogin(), getUsersByRating(), getUsersByTags(). По какому принципу их можно разбить, я не совсем понимаю 24 окт 2018 в 15:38
  • @ЕльцовДанил Если честно, сложно дать какой-то дельный совет. Посмотрите где эти методы применяются, возможно можно разделить их относительно бизнес-логики. И да, не факт что класс нужно разбивать. 24 окт 2018 в 16:18
  • 1
    @ЕльцовДанил Кстати, можете попробовать подготовить пример кода и опубликовать его с меткой [инспекция-кода]. Возможно, видя код, местные знатоки подскажут какие-нибудь конкретные шаги для улучшения. 24 окт 2018 в 16:19
1

"Разбивать классы" следует в соответствии с архитектурными принципами, а не руководствуясь скоростью их загрузки. Если у вас 30 методов в одно классе, то вы явно нарушаете принцип единственной ответственности, как минимум.

2
  • разве принцип единственной ответственности хоть как-то ограничивает количество методов? 24 окт 2018 в 7:26
  • @VladMamaev Роберт Мартин в своём хрестоматийном труде "Чистый код" утверждает, что это один из "запахов" плохого кода и явный признак нарушение этого принципа. 24 окт 2018 в 7:31

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.