Закрадывалась ли вам в голову идея переписать своё жирное энтерпрайзное приложение с нуля? Если с нуля, то это ж ого-го. Как минимум кода будет раза в два меньше, верно? Но ведь пройдёт пара лет, и оно тоже обрастёт, станет легаси… времени и денег на переписывание не так много, чтобы делать идеально.
Успокойтесь, начальство всё равно не даст ничего переписать. Остаётся рефакторить. На что лучше всего потратить свои невеликие ресурсы? Как именно рефакторить, где проводить чистки?
Название этой статьи — в том числе отсылка к книге Дяди Боба «Чистая Архитектура», а сделана она на основе замечательного доклада Victor Rentea (твиттер, сайт) на JPoint (под катом он начнёт говорить от первого лица, но пока дочитайте вводную). Чтения умных книжек эта статья не заменит, но для такого короткого описания изложено весьма хорошо.
Идея в том, что популярные в народе вещи вроде «Clean Architecture» действительно являются полезными. Сюрприз. Если нужно решить вполне конкретную задачу, простой изящный код не требует сверхусилий и оверинжиниринга. Чистая архитектура говорит, что нужно защищать свою доменную модель от внешних эффектов, и подсказывает, как именно это можно сделать. Эволюционный подход к наращиванию объема микросервисов. Тесты, которые делают рефакторинг менее страшным. Вы ведь уже знаете всё это? Или знаете, но боитесь даже подумать об этом, ведь это же ужас что тогда делать придётся?
Кто хочет получить волшебную анти-прокрастинационную таблетку, которая поможет перестать трястись и начать рефакторить — добро пожаловать на видеозапись доклада или под кат.
Меня зовут Виктор, я из Румынии. Формально я являюсь консультантом, техлидом и ведущим архитектором в Румынском IBM. Но если бы меня попросили самому дать определение своей деятельности, то я евангелист чистого кода. Обожаю создавать красивый, чистый, поддерживаемый код — об этом, как правило, и рассказываю на докладах. Даже больше, меня вдохновляет преподавание: обучение разработчиков в областях Java EE, Spring, Dojo, Test Driven Development, Java Performance, а также в области упомянутого евангелизма — принципов чистоты паттернов кода и их разработки.
Опыт, на котором строится моя теория — в основном разработка корпоративных приложений для крупнейшего клиента IBM в Румынии — банковского сектора.
План на эту статью таков:
Но сначала давайте вспомним те главные принципы, о которых мы, как разработчики, должны помнить всегда.
![](https://habrastorage.org/r/w780q1/getpro/habr/post_images/353/dd0/cd5/353dd0cd58012749b70e379e37231265.jpg)
Иначе говоря, количество vs качество. Как правило, чем больше функциональности содержит ваш класс, тем хуже она оказывается в качественном отношении. Разрабатывая большие классы, программист начинает путаться, ошибаться в построении зависимостей, а большой код, помимо всего прочего, сложнее отладить. Лучше разбить такой класс на несколько более мелких, каждый из которых будет отвечать за некоторую подзадачу. Пусть лучше у вас будет несколько сильносвязанных модулей, чем один — крупный и неповоротливый. Модульность также даёт возможность повторного использования логики.
![](https://habrastorage.org/r/w780q1/getpro/habr/post_images/f72/1c8/0f7/f721c80f7b3184be2218a01b66f7440a.jpg)
Степень связывания — это характеристика того, насколько тесно взаимодействуют друг с другом ваши модули. Она показывает то, насколько широко способен распространиться эффект от изменений, вносимых вами в какой-то одной точке системы. Чем выше связывание, тем сложней осуществлять модификации: вы меняете что-то в одном модуле, а эффект распространяется далеко и не всегда ожидаемым образом. Поэтому показатель связывания должен быть как можно ниже — этим вы обеспечите больший контроль над системой, подвергающейся модификациям.
![](https://habrastorage.org/r/w780q1/getpro/habr/post_images/2b9/9de/5c0/2b99de5c00257d6123553c7bfaed0519.jpg)
Ваши собственные реализации могут быть хороши сегодня, но уже не так хороши завтра. Не разрешайте себе копировать свои собственные наработки и таким образом распространять их по кодовой базе. Вы можете копировать со StackOverflow, из книг — с любых авторитетных источников, которые (как вы точно знаете) предлагают идеальную (или близкую к тому) реализацию. Дорабатывать же свою собственную реализацию, встречающуюся не один раз, а размноженную по всей кодовой базе, может быть очень утомительно.
![](https://habrastorage.org/r/w780q1/getpro/habr/post_images/7d4/8d4/c85/7d48d4c85408c620ddcbe644c4acb6f9.jpg)
На мой взгляд, это главный принцип, который нужно соблюдать в инженерных и программных разработках. «Преждевременная инкапсуляция — корень зла», — говорил Адам Биен. Иначе говоря, корень зла заключается в «пере-инженерии». Автор цитаты Адам Биен одно время занимался тем, что брал легаси-приложения и, полностью переписывая их код, получал кодовую базу объемом в 2-3 раза меньше исходного. Откуда берётся столько лишнего кода? Он ведь возникает не просто так. Его порождают испытываемые нами страхи. Нам кажется, что, нагромождая в большом количестве паттерны, плодя косвенность и абстракции, мы обеспечиваем нашему коду защиту — защиту от неизвестностей завтрашнего дня и завтрашних требований. Ведь на самом-то деле сегодня ничего из этого нам не нужно, изобретаем мы всё это только ради каких-то «будущих нужд». И не исключено, что эти структуры данных впоследствии будут мешать. Скажу честно, когда ко мне подходит какой-нибудь мой разработчик и озвучивает, что он придумал кое-что интересное, что можно добавить в продакшн-код, я отвечаю всегда одинаково: «Парень, тебе это не пригодится».
Кода не должно быть много, а тот, что есть, должен быть простым — только так с ним можно будет нормально работать. Это забота о ваших разработчиках. Вы должны помнить, что именно они являются ключевыми фигурами для вашей системы. Постарайтесь снизить их энергозатраты, уменьшить те риски, с которыми им придётся работать. Это не значит, что вам придётся создать свой собственный фреймворк, более того, я бы не советовал вам это делать: в своём фреймворке всегда будут баги, всем нужно будет его изучать и т.д. Лучше пустите в ход существующие средства, коих сегодня имеется масса. Это должны быть простые решения. Пропишите глобальные обработчики ошибок, примените технологию аспектов, генераторы кода, расширения Spring или CDI, настройте области действия Request/Thread, используйте манипуляцию и генерацию на лету байткода и пр. Всё это будет вашим вкладом в поистине важнейшую вещь — в комфорт вашего разработчика.
В частности, я бы хотел продемонстрировать вам применение областей Request/Thread. Я не раз наблюдал, как эта вещь невероятным образом упрощала корпоративные приложения. Суть в том, что она даёт вам возможность, будучи залогиненным пользователем, сохранять данные RequestContext. Таким образом, RequestContext будет в компактном виде хранить данные о пользователе.
![](https://habrastorage.org/r/w780q1/getpro/habr/post_images/341/6c5/474/3416c54740aca3b8f514696bf9cf7f3d.jpg)
Как видите, реализация занимает всего пару строк кода. Прописав запрос в нужную аннотацию (несложно делается, если вы используете Spring или CDI), вы навсегда освободите себя от необходимости передавать методам пользовательский логин и что бы то ни было ещё: хранимые внутри контекста метаданные запроса будут прозрачным образом перемещаться по приложению. Scoped proxy же позволит вам в любой момент получить доступ к метаданным текущего запроса.
![](https://habrastorage.org/r/w780q1/getpro/habr/post_images/5c5/edf/d7a/5c5edfd7a14d760bc335d8e65f0d5666.jpg)
Обновляющихся требований разработчики боятся потому, что боятся процедуры рефакторинга (модификации кода). И самый простой способ помочь им — создать надёжный набор тестов для регрессионного тестирования. С помощью него разработчик будет иметь возможность в любой момент протестировать свою наработку — убедиться, что она не поломает систему.
Разработчик не должен бояться ничего сломать. Вы должны сделать всё, чтобы рефакторинг воспринимался как что-то хорошее.
Рефакторинг — важнейший аспект разработки. Помните, ровно в тот момент, когда ваши разработчики испугаются рефакторинга, приложение можно считать перешедшим в разряд «легаси».
![](https://habrastorage.org/r/w780q1/getpro/habr/post_images/4fb/8a2/08d/4fb8a208dc6d89279a663747de74369f.jpg)
Начиная реализацию любой системы (или компоненты системы), мы задаём себе вопрос: где лучше реализовать логику предметной области, то есть функциональные аспекты нашего приложения? Есть два противоположных подхода.
Первый из них основывается на философии Transaction Script. Здесь логика реализуется в процедурах, работающих с анемичными сущностями (то есть со структурами данных). Такой подход хорош тем, что в ходе его реализации можно опираться на сформулированные бизнес-задачи. Работая над приложениями для банковского сектора, я не раз наблюдал перевод бизнес-процедур в софт. Могу сказать, что это действительно очень естественно — соотносить сценарии с софтом.
Альтернативный подход — использовать принципы Domain-Driven Design. Здесь вам потребуется соотнести спецификации и требования с объектно-ориентированной методологией. Важно и тщательно продумать объекты, и обеспечить хорошую вовлеченность со стороны бизнеса. Плюс спроектированных таким образом систем в том, что в дальнейшем они легко поддерживаются. Однако по моему опыту, осваивать данную методологию достаточно непросто: более-менее смело вы почувствуете себя не раньше, чем через полгода её изучения.
Для своих разработок я всегда выбирал первый подход. Могу заверить, что в моем случае он работал идеально.
Как же нам моделировать данные? Как только приложение принимает более-менее приличные размеры, обязательно появляются персистентные данные. Это такие данные, которые вам требуется хранить дольше остальных — они являются доменными сущностями (domain entities) вашей системы. Где их хранить — в базе данных ли, в файле или напрямую управляя памятью — не имеет значения. Важно то, как вы будете их хранить — в каких структурах данных.
![](https://habrastorage.org/r/w780q1/getpro/habr/post_images/0a8/56b/127/0a856b127ec938a8c17eb345610b0ee8.jpg)
Вам как разработчику дан этот выбор, и только от вас зависит, будут ли эти структуры данных работать на вас или против вас при реализации функциональных требований в будущем. Чтобы всё было хорошо, вы должны реализовывать сущности, закладывая в них крупицы переиспользуемой доменной логики. Как конкретно? Продемонстрирую несколько способов на примере.
![](https://habrastorage.org/r/w780q1/getpro/habr/post_images/c73/e56/7da/c73e567da2fb69ceb3d627fb408530cd.jpg)
Давайте посмотрим, чем я снабдил сущность Customer. Во-первых, я реализовал синтетический геттер
Возможно, вы используете какой-нибудь ORM вроде Hibernate. Предположим, у вас имеется две сущности с двусторонней связью. Инициализацию необходимо выполнять с обеих сторон, иначе, как вы понимаете, у вас возникнут проблемы при обращении к этим данным в дальнейшем. Но разработчики зачастую забывают проинициализировать объект с какой-нибудь одной из сторон. Вы же, разрабатывая эти сущности, можете предусмотреть специальные методы, которые будут гарантировать двустороннюю инициализацию. Посмотрите на
![](https://habrastorage.org/r/w1560/getpro/habr/post_images/c55/7fe/273/c557fe273cc1f2658573f9e78a70ca1d.png)
Как видите, это вполне обычная сущность. Но внутрь неё заложена доменная логика. Такие сущности не должны быть скудными и поверхностными, но и не должны быть переполненными логикой. Переполненность логикой возникает чаще: если уж вы решили реализовать всю логику в домене, то для каждого use-case возникнет соблазн реализовать какой-нибудь специфический метод. А use-case-ов, как правило, бывает много. Вы получите не сущность, а одну большую кучу всевозможной логики. Старайтесь соблюдать здесь меру: в домен помещается только переиспользуемая логика и только в небольшом количестве.
Помимо сущностей вам, скорее всего, также понадобятся объекты-значения (object values). Это не что иное, как способ сгруппировать доменные данные, чтобы потом вместе перемещать их по системе.
Объект-значение должен быть:
![](https://habrastorage.org/r/w780q1/getpro/habr/post_images/a0e/765/284/a0e76528432de8c55dc146805a8b286d.jpg)
А если добавить в конструктор вызов метода
Объекты-значения отличаются от сущностей тем, что у них нет постоянного ID. В сущностях всегда будут поля, связанные с внешним ключом какой-нибудь таблицы (или другого хранилища). У объектов-значений таких полей нет. Возникает вопрос: отличаются ли процедуры проверки на равенство двух объектов-значений и двух сущностей? Поскольку у объектов-значений нет поля ID, чтобы заключить, что два таких объекта равны, придётся попарно сравнить значения всех их полей (то есть осмотреть всё содержимое). При сравнении же сущностей достаточно провести одно единственное сравнение — по полю ID. Именно в процедуре сравнения заключается главное отличие сущностей от объектов-значений.
![](https://habrastorage.org/r/w780q1/getpro/habr/post_images/72a/d31/681/72ad31681509cdef20509d4514b02139.jpg)
В чём заключается взаимодействие с пользовательским интерфейсом (UI)? Ему вы должны передать данные для отображения. Неужели понадобится ещё одна структура? Так и есть. А всё потому, что пользовательский интерфейс вам совсем не друг. У него свои запросы: ему нужно, чтобы данные хранились в соответствии с тем, как они должны отображаться. Это так чудно — что именно порой требуют от нас пользовательские интерфейсы и их разработчики. То им нужно достать данные для пяти строк; то им приходит в голову завести для объекта булево поле
Вопрос в том, можно ли вверить им в пользование наши сущности? Вероятней всего, они их изменят, причем самым нежелательным для нас образом. Поэтому предоставим им кое-что другое — Data Transfer Objects (DTO). Они будут приспособлены специально под внешние требования и под логику, отличную от нашей. Некоторые примеры структур DTO: Form/Request (поступают из UI), View/Response (отправляются в UI), SearchCriteria/SearchResult и пр. Можно в некотором смысле назвать это API-моделью.
Первый важный принцип: DTO должно содержать минимум логики.
Перед вами пример реализации
![](https://habrastorage.org/r/w780q1/getpro/habr/post_images/4ec/3bb/716/4ec3bb716c630f3b39ca82f8fe4c5e33.jpg)
Содержимое: private-поля, public-геттеры и сеттеры для них. Вроде бы всё супер. ООП во всей красе. Но одно плохо: в виде геттеров и сеттеров я реализовал слишком большое количество методов. В DTO же логики должно быть как можно меньше. И тогда какой мой выход? Я делаю поля публичными! Вы скажете, что такое плохо работает с method references из Java 8, что возникнут ограничения и пр. Но верите или нет, все свои проекты (10-11 штук) я делал вот с такими DTO. Брат жив. Теперь, поскольку мои поля — публичные, я имею возможность запросто присваивать значение
Итак, у нас есть задача: нам нужно преобразовать наши сущности в DTO. Реализуем преобразование так:
![](https://habrastorage.org/r/w780q1/getpro/habr/post_images/995/dfd/ca8/995dfdca8c1945fb92a64cee09851259.jpg)
Как видите, объявив DTO, мы переходим к операциям маппинга (присвоения значений). Нужно ли быть senior developer, чтобы писать в таком количестве обычные присвоения? Для некоторых это настолько непривычно, что они начинают переобуваться на ходу: например, копировать данные при помощи какого-нибудь фреймворка для маппинга, используя рефлекшн. Но они упускают главное — то, что рано или поздно произойдёт взаимодействие UI с DTO, в результате которого сущность и DTO разойдутся в своих значениях.
Можно было бы, скажем, поместить операции маппинга в конструктор. Но такое возможно не для любого маппинга; в частности, в конструкторе нельзя осуществить доступ к базе данных.
Таким образом, мы вынуждены оставить операции маппинга в бизнес-логике. И если они имеют компактный вид, то в этом ничего страшного нет. Если же маппинг занимает не пару строк, а больше, то лучше вынести его в так называемый маппер. Маппер — это класс, специально предназначенный для копирования данных. Это, в общем-то, допотопная вещь и бойлерплейт. Но зато за ними можно скрыть наши многочисленные присвоения — сделать код чище и стройнее.
Запомните: код, который слишком разросся, нужно выносить в отдельную структуру. В нашем случае операций маппинга было действительно многовато, поэтому мы вынесли их в отдельный класс — в маппер.
Разрешать ли мапперам доступ в базу данных? Можете по умолчанию разрешить — так часто поступают из соображений простоты и прагматики. Но это подвергает вас определённым рискам.
Проиллюстрирую на примере. Создадим на основе имеющегося DTO сущность
![](https://habrastorage.org/r/w1560/getpro/habr/post_images/ef3/9e8/b82/ef39e8b82a023a2fad6afe8e3c22b5ec.png)
Для маппинга нам необходимо добыть из базы данных ссылку на группу покупателя. Поэтому я запускаю метод
Но неприятность нас ожидает не здесь, а в методе, выполняющем обратную операцию — преобразование сущности в DTO.
![](https://habrastorage.org/r/w1560/getpro/habr/post_images/e6a/e27/7ff/e6ae277ff6b22588b5bc5aaffabf9d0e.png)
При помощи цикла мы проходим все адреса, ассоциированные с имеющимся Customer, и преобразуем их в адреса DTO. Если вы используете ORM, то, вероятно, при вызове метода
![](https://habrastorage.org/r/w780q1/getpro/habr/post_images/e54/001/2bb/e540012bbb4a242d84e55a2dffa5ca11.jpg)
У вас есть набор родителей, у каждого из которых есть дети. Для всего этого вам нужно создать свои аналоги внутри DTO. Вам понадобится выполнить один
Предположим, что, всё же, наш метод
![](https://habrastorage.org/r/w780q1/getpro/habr/post_images/58c/9c3/2b3/58c9c32b3e8a01a96d500920dfe5ef07.jpg)
У проблемы с N+1 запросами есть простые типовые решения: в JPQL для извлечения детей вы можете воспользоваться
Но я бы сделал по-другому. Можно узнать, какова максимальная длина списка детей (это можно сделать, например, на основе поиска с пагинацией). Если в списке всего 15 сущностей, то нам потребуется всего 16 запросов. Вместо 5мс мы потратим на всё, скажем, 15мс — пользователь не заметит разницы.
Я бы не советовал вам оглядываться на производительность системы уже на начальном этапе разработки. Как сказал Дональд Кнуд: «Преждевременная оптимизация — корень зла». Нельзя оптимизировать с самого начала. Это именно то, что нужно оставить на потом. И что особенно важно: никаких предположений — только измерения и оценка измерений!
Так ли вы уверены, что вы компетентны, что вы настоящий эксперт? Будьте скромны в оценке себя. Не думайте, что поняли работу JVM, пока не прочтёте хотя бы пару книг о JIT-компиляции. Бывает, лучшие программисты из нашей команды подходят ко мне и говорят, что, как им кажется, они нашли более эффективную реализацию. Оказывается же, что они снова изобрели что-то, что только усложняет код. Поэтому я раз за разом отвечаю: YAGNI. Нам это не понадобится.
Зачастую для корпоративных приложений вообще не требуется никакой оптимизации алгоритмов. Узким местом для них, как правило, является не компиляция и не то, что касается работы процессора, а всевозможные операции ввода-вывода. Например, считывание миллиона строк из базы данных, объёмные записи в файл, взаимодействие с сокетами.
Со временем начинаешь понимать, какие узкие места содержит система, и, подкрепив всё измерениями, начнешь понемногу оптимизировать. А до этого момента сохраняйте код как можно более чистым. Вы обнаружите, что такой код гораздо легче поддаётся дальнейшей оптимизации.
Вернёмся к нашим DTO. Предположим, мы определили такую DTO:
![](https://habrastorage.org/r/w1560/getpro/habr/post_images/180/39d/f91/18039df91fd1cc76a2413b7f373e7c0b.png)
Она может понадобиться нам во множестве рабочих потоков. Но эти потоки разные и, вероятней всего, каждый use-case будет предполагать различную степень заполненности полей. Например, создать DTO нам явно нужно будет раньше, чем когда у нас будет полная информация о пользователе. Можно временно оставлять поля пустыми. Но чем больше полей будете игнорироваться, тем больше вам будет хотеться создать новое более строгое DTO для данного use-case.
Как вариант, можно создать копии избыточно большого DTO (в количестве имеющихся use-case-ов) и далее убрать из каждой копии лишние для неё поля. Но многим программистам, в силу ума и грамотности, действительно больно нажимать Ctrl+V. Аксиома гласит, что копипастить — плохо.
Можно прибегнуть к известному в теории ООП принципу наследования: просто определим некий базовый DTO и для каждого use-case создадим наследника.
![](https://habrastorage.org/r/w1560/getpro/habr/post_images/6f7/069/77f/6f706977fec8625d720323b64fe87ad8.png)
Известный принцип гласит: «Предпочитайте композицию наследованию». Прочитайте, что там написано: «extends». Вроде бы мы должны были «расширить» исходный класс. Но если вдуматься, то, что мы сейчас понаделали — вовсе не «расширение». Это самое настоящее «повторение» — тот же копипаст, вид сбоку. Поэтому наследование мы использовать не будем.
Но как же тогда нам быть? Как перейти к композиции? Сделаем так: пропишем в CustomerView поле, которое будет указывать на объект базового DTO.
![](https://habrastorage.org/r/w780q1/getpro/habr/post_images/1f2/d49/831/1f2d49831b9b286b8bc069c824ac2871.jpg)
Таким образом наша базовая структура будет вложена внутрь. Вот так выйдет настоящая композиция.
Используем ли мы наследование или решаем вопрос композицией — это всё частности, тонкости, возникшие глубоко в ходе нашей реализации. Они очень хрупкие. Что значит хрупкие? Посмотрите внимательно на этот код:
![](https://habrastorage.org/r/w1560/getpro/habr/post_images/921/eb1/ef2/921eb1ef20effd43b129ad5d28b88253.png)
Большинство разработчиков, которым я это показал, сразу выпалили, что число «2» повторяется, поэтому его нужно вынести в виде константы. Они не обратили внимание, что двойка во всех трёх случаях имеет совершенно разный смысл (или «бизнес-значение») и что её повторение — не более чем совпадение. Вынести двойку в константу — правомерное решение, однако очень хрупкое. Старайтесь не допускать в домен хрупкую логику. Никогда не работайте из него со внешними структурами данных, в частности, с DTO.
Итак, почему же работа по ликвидации наследования и введению композиции оказывается бесполезной? Именно потому, что DTO мы создаем не для себя, а для внешнего клиента. А как клиентское приложение будет парсить полученные от вас DTO — вам остаётся только догадываться. Но очевидно, что это будет иметь мало общего с вашей реализацией. Разработчики с той стороны могут и не сделать различия для базовых и небазовых DTO, которые вы так старательно продумали; наверняка они используют наследование, а возможно и тупо копипастят вот это всё.
![](https://habrastorage.org/r/w780q1/getpro/habr/post_images/be7/31c/9e4/be731c9e4cfe002cb3b59c11a708f623.jpg)
Вернёмся к общей картине приложения. Я бы советовал вам реализовывать доменную логику через паттерн Facade, расширяя фасады доменными сервисами по необходимости. Доменный сервис создаётся тогда, когда в фасаде накапливается слишком большое количество логики, и удобнее её вынести в отдельный класс.
Ваши доменные сервисы должны обязательно говорить на языке вашей доменной модели (её сущностей и объектов-значений). Они ни в коем случае не должны работать с DTO, потому как DTO, как вы помните, — это структуры, постоянно изменяемые с клиентской стороны, слишком хрупкие для домена.
![](https://habrastorage.org/r/w780q1/getpro/habr/post_images/3ad/eb8/657/3adeb86570a92f804ba491836c99b9e8.jpg)
Каково назначение фасада?
![](https://habrastorage.org/r/w780q1/getpro/habr/post_images/604/a9c/8e8/604a9c8e859ff1dc3fb38208d6bf836d.jpg)
Ещё пара слов об этом принципе. Если класс достиг некоторого неудобного для меня размера (скажем, 200 строк), то я должен попробовать разбить его на части. Но выделить новый класс из существующего не всегда просто. Нужны придумать какие-то универсальные способы. Один из таких способов состоит в поиске имён: вы пробуете подобрать название для какого-нибудь подмножества методов вашего класса. Как только у вас получилось найти имя — смело создавайте новый класс. Но и это не так просто. В программировании, как известно, всего две сложные вещи: это инвалидация кэша и придумывание имен. В данном случае, придумывание названия сопряжено с выявлением подзадачи — скрывающейся и потому ранее никем не выявленной.
Пример:
![](https://habrastorage.org/r/w780q1/getpro/habr/post_images/81f/fe3/82b/81ffe382b74f6bbd4a91626d83249a97.jpg)
В исходном фасаде
Ещё один пример:
![](https://habrastorage.org/r/w780q1/getpro/habr/post_images/2ac/84e/788/2ac84e788af60fb038091aed273a390e.jpg)
Предположим, в нашей системе есть класс
Разделение по уровням абстракции всегда предполагает, что извлекаемый класс становится зависимостью, а извлечение осуществляется для переиспользования.
Задача извлечения не всегда даётся просто. Она также может повлечь за собой некоторые сложности и потребовать какого-нибудь рефакторинга unit-тестов. Тем не менее, по моим наблюдениям, искать какой бы то ни было функционал по огромной монолитной кодовой базе приложения разработчикам ещё тяжелее.
![](https://habrastorage.org/r/w780q1/getpro/habr/post_images/625/5b2/346/6255b23467294d6224107235443379f1.jpg)
Многие консультанты расскажут про парное программирование, про то, что это универсальное решение любых проблем IT-разработки на сегодняшний день. В ходе него программисты развивают свои технические навыки и функциональные знания. Кроме того, интересен сам процесс, он сплачивает команду.
Если говорить не как консультанты, а по-человечески, самое важное здесь вот что: парное программирование улучшает «фактор автобуса». Суть же «фактора автобуса» в том, что людей, обладающих знаниями об устройстве системы, должно быть как можно больше. Потерять этих людей означает потерять последние ключи к этим знаниям.
Рефакторинг в формате парного программирования — искусство, требующее опыта и тренировки. Здесь полезны, например, практики агрессивного рефакторинга, проведение хакатонов, катов, Coding Dojos и др.
Парное программирование хорошо работает в случаях, когда нужно решать задачи высокой сложности. Сам процесс работы вдвоём не всегда прост. Зато он гарантирует вам, что вы избежите «переинженерии» — напротив, получите реализацию, которая адресует поставленные требования с минимальной сложностью.
![](https://habrastorage.org/r/w780q1/getpro/habr/post_images/a5a/5aa/666/a5a5aa666cb29c58dc9bb5b7d5bc6651.jpg)
Организация удобного формата работы — одна из ваших главных обязанностей перед командой. Вы должны не переставая заботиться об условиях труда разработчика — обеспечивать им полный комфорт и свободу творчества, особенно, если от них требуется наращивать проектную архитектуру и её сложность.
Эту глупость периодически выражают гласно или негласно. В сегодняшней практике архитекторы как таковые встречаются всё меньше. С приходом Agile эта роль постепенно перешла к старшим разработчикам, потому как обычно вся работа, так или иначе, строится вокруг них. Размер реализации постепенно растёт, и вместе с этим появляется потребность в рефакторинге и разрабатывается новая функциональность.
«Луковица» — самая чистая архитектура философии Transaction Script. Строя её, мы руководствуемся целью обеспечить защиту того кода, который считаем критичным, и для этого перемещаем его в доменный модуль.
![](https://habrastorage.org/r/w780q1/getpro/habr/post_images/a45/bb4/94a/a45bb494a51b04ecfe92606b744c0857.jpg)
В нашем приложении самыми важными являются доменные сервисы: они реализуют самые критичные потоки. Переместим их в доменный модуль. Безусловно, сюда же стоит перенести все свои доменные объекты — сущности и объекты-значения. Всё остальное, что мы с вами сегодня накодили — DTO, мапперы, валидаторы и пр. — становится, так сказать, первой линией обороны от пользователя. Потому как пользователь, увы, нам не друг, и необходимо защищать от него систему.
Внимание вот на эту зависимость:
![](https://habrastorage.org/r/w780q1/getpro/habr/post_images/c7d/273/6bd/c7d2736bdc016b732c5c431299bf32ea.jpg)
Модуль приложения будет зависеть от доменного модуля — именно так, а не наоборот. Прописывая связь такой, мы гарантируем, что DTO никогда не ворвутся на святую территорию доменного модуля: они попросту не видны и недоступны из доменного модуля. Получается, что мы в некотором смысле огородили территорию домена — ограничили к ней доступ посторонних.
Тем не менее, домену может понадобиться взаимодействовать с каким-нибудь внешним сервисом. С внешним — значит, с недружественным, потому что он снабжён своими DTO. Какие наши варианты?
Первый: пропустить недруга внутрь модуля.
![](https://habrastorage.org/r/w780q1/webt/un/tn/hk/untnhkiksv5id_i-6c4qwdjxy1c.jpeg)
Очевидно, что это плохой вариант: не исключено, что завтра внешний сервис не выполнит апгрейд на версию 2.0, и нам придётся перекраивать наш домен. Нельзя пускать врага внутрь домена!
Предлагаю другой подход: для взаимодействия создадим специальный адаптер.
![](https://habrastorage.org/r/w780q1/getpro/habr/post_images/123/8b1/09c/1238b109c54ac8f19524e6d41b32e85e.jpg)
Адаптер будет получать данные от внешнего сервиса, извлекать те из них, которые нужны нашему домену, и преобразовывать их в требуемые виды структур. В таком случае всё, что от нас требуется при разработке — соотнести вызовы к внешней системе с требованиями домена. Думайте об этом как об огромном таком адаптере. Я называю такой слой «антикоррупционным».
Например, нам может понадобиться выполнять из домена запросы LDAP. Для этого мы реализуем «антикоррупционный модуль»
![](https://habrastorage.org/r/w780q1/getpro/habr/post_images/520/fdc/84a/520fdc84a53641a26b957ac010a19c42.jpg)
В адаптере мы можем:
Таково назначение адаптера. По хорошему, на стыке с каждой внешней системой, с которой вам необходимо взаимодействовать, должен быть заведён свой адаптер.
![](https://habrastorage.org/r/w780q1/getpro/habr/post_images/b1a/663/9c5/b1a6639c58457537fb6a4d9c15e09eae.jpg)
Таким образом, домен будет направлять вызов не к внешнему сервису, а к адаптеру. Для этого в домене должна быть прописана соответствующая зависимость (от адаптера либо от того инфраструктурного модуля, в котором он находится). Но безопасна ли такая зависимость? Если её установить вот так, к нам в домен могут попасть DTO внешнего сервиса. Такого мы допускать не должны. Поэтому предлагаю вам другой способ моделирования зависимостей.
![](https://habrastorage.org/r/w780q1/getpro/habr/post_images/058/4a1/f7c/0584a1f7c309505e4e5ea0c2b0b26c21.jpg)
Создадим интерфейс, пропишем в нём сигнатуру нужных методов и поместим его внутрь нашего домена. Задача адаптера — реализовать этот интерфейс. Получается, интерфейс находится внутри домена, а адаптер — снаружи, в инфраструктурном модуле, который импортирует интерфейс. Таким образом, мы развернули направление зависимости в обратную сторону. Во время исполнения доменная система будет вызывать любой класс через интерфейсы.
Как видите, всего-навсего введя в архитектуру интерфейсы, мы сумели развернуть зависимости и тем самым обезопасить наш домен от попадания в него чужеродных структур и API. Такой подход называется инверсией зависимостей.
![](https://habrastorage.org/r/w780q1/getpro/habr/post_images/c2c/cc2/cd0/c2ccc2cd0309a4c8f97b7521cfc63fb7.jpg)
В общем случае, инверсия зависимостей предполагает, что вы размещаете интересующие вас методы в интерфейсе внутри вашего высокоуровневого модуля (в домене), а реализуете этот интерфейс снаружи — в том или ином низкоуровневом (инфраструктурном) некрасивом модуле.
Интерфейс, реализуемый внутри доменного модуля, должен говорить на языке домена, то есть он будет оперировать его сущностями, его параметрами и возвращаемыми типами. Во время исполнения домен будет вызывать любой класс посредством полиморфного вызова к интерфейсу. Фреймворки, предназначенные для внедрения зависимостей (например, Spring и CDI), снабдят нас конкретным экземпляром класса прямо в рантайме.
Но главное то, что во время компиляции доменный модуль не будет видеть содержимого внешнего модуля. Именно это нам и нужно. Никакая внешняя сущность не должна попасть в домен.
Как считает Дядя Боб, принцип инверсии управления (или, как он её называет, «архитектура плагинов») — это, возможно, лучшее, что вообще предлагает парадигма ООП.
![](https://habrastorage.org/r/w780q1/getpro/habr/post_images/a96/3cf/e13/a963cfe13d030fb1d7ec2bf0f780fe43.jpg)
Данную стратегию можно использовать для интеграции с любыми системами, для синхронных и асинхронных вызовов и сообщений, для отсылки файлов и т. д.
![](https://habrastorage.org/r/w780q1/getpro/habr/post_images/dc3/64d/cdb/dc364dcdb589b35a0597b6e478aace46.jpg)
Итак, мы решили, что защищать будем доменный модуль. Внутри него размещается доменный сервис, сущности, объекты-значения, а теперь и интерфейсы для внешних сервисов, плюс интерфейсы для репозитория (для взаимодействия с базой данных).
Структура выглядит так:
![](https://habrastorage.org/r/w780q1/getpro/habr/post_images/07e/983/0c7/07e9830c76d3b263743f8514b13bf42a.jpg)
В качестве зависимостей для домена объявлены модуль приложения, инфраструктурный модуль (посредством инверсии зависимостей), модуль репозитория (базу данных мы тоже считаем внешней системой), batch-модуль и, возможно, некоторые другие модули. Такая архитектура носит название «луковицы»; также её называют «чистой», «шестиугольной» и «порты и адаптеры».
Вкратце расскажу о модуле репозитория. Выносить ли его из домена — это вопрос. Задача репозитория — сделать логику чище, скрыв от нас ужас работы с персистентными данными. Вариант для олдскульных ребят — использовать для взаимодействия с базой данных JDBC:
![](https://habrastorage.org/r/w780q1/getpro/habr/post_images/7ed/d70/44a/7edd7044a76373b2c513e246bf6eabaa.jpg)
Также можно использовать Spring и его JdbcTemplate:
![](https://habrastorage.org/r/w780q1/getpro/habr/post_images/64b/d68/749/64bd68749f5b3de16591a23a8acda193.jpg)
Или же MyBatis DataMapper:
![](https://habrastorage.org/r/w780q1/getpro/habr/post_images/423/31b/aa1/42331baa178ff3ad8f0ea8fb92704265.jpg)
Но это до того сложно и некрасиво, что отбивает всякое желание делать что-то дальше. Поэтому предлагаю использовать JPA/Hibernate либо Spring Data JPA. Они дадут нам возможность отправлять запросы, построенные не на схеме БД, а непосредственно на основе модели наших сущностей.
Реализация для JPA/Hibernate:
![](https://habrastorage.org/r/w780q1/getpro/habr/post_images/110/ebf/ef7/110ebfef7dad2a2bb278c50668eecda9.jpg)
В случае же со Spring Data JPA:
![](https://habrastorage.org/r/w780q1/getpro/habr/post_images/10b/e2a/0ff/10be2a0ff082a8c1ac962c4754ec55a8.jpg)
Spring Data JPA умеет автоматически генерировать методы во время исполнения, такие, как, например, getById(), getByName(). Также он позволяет при необходимости выполнять JPQL-запросы — и не к БД, а к вашей собственной модели сущностей.
Код Hibernate JPA и Spring Data JPA действительно выглядит очень неплохо. Нужно ли нам вообще извлекать его из домена? По моему мнению, это не так-то и обязательно. Скорей всего, код будет даже чище, если оставить данный фрагмент внутри домена. Так что действуйте по ситуации.
![](https://habrastorage.org/r/w780q1/getpro/habr/post_images/055/f04/37c/055f0437cf059c9f7001db6b3d231e70.jpg)
Если вы всё же создаёте модуль репозитория, то для организации зависимостей лучше точно так же воспользоваться принципом инверсии управления. Для этого разместите в домене интерфейс и реализуйте его в модуле репозитория. Что касается логики репозитория, то лучше перенести её в домен. Это делает удобным тестирование, так как в домене вы сможете использовать Mock-объекты. Они позволят протестировать логику быстро и многократно.
Традиционно для репозитория в домене создаётся всего одна сущность. Разбивают на части её только тогда, когда она становится чересчур объёмной. Не забывайте о том, что классы должны быть компактными.
![](https://habrastorage.org/r/w780q1/getpro/habr/post_images/1c8/9e1/7e7/1c89e17e784e4db6fe8d7c713bcf1ab6.jpg)
Можно создать отдельный модуль, поместить в него извлечённый из фасада интерфейс и полагающиеся к нему DTO, затем упаковать в JAR, и в таком виде передавать своим Java-клиентам. Имея этот файл, они получат возможность направлять запросы к фасадам.
Помимо тех наших «врагов», которым мы поставляем функциональность, то есть клиентов, у нас есть враги и с другой стороны — те модули, от которых зависим мы сами. От этих модулей нам тоже необходимо защищаться. И для этого предлагаю вам несколько модифицированную «луковицу» — в ней вся инфраструктура объединяется в один модуль.
![](https://habrastorage.org/r/w780q1/getpro/habr/post_images/265/1f1/481/2651f1481062cfe6c18974df9a6dde9b.jpg)
Я называю такую архитектуру «прагматичной луковицей». Здесь разделение компонент проходит по принципу «моё» и «интегрируемое»: отдельно хранится то, что относится к моему домену, и отдельно — то, что относится к интеграции с внешними коллаборационистами. Таким образом, получается всего два модуля: домен и приложение. Такая архитектура очень хороша, но только тогда, когда модуль приложений имеет небольшие размеры. В противном случае вам лучше вернуться к традиционной «луковице».
Как я уже говорил ранее, если вашего приложения все боятся, считайте, что оно пополнило ряды легаси.![](https://habrastorage.org/r/w780q1/getpro/habr/post_images/12a/508/d1b/12a508d1b9595239f6d9b9a8a72c9474.jpg)
А вот тесты — это хорошо. Они дают нам чувство уверенности, благодаря которому можно продолжать работы по рефакторингу. Но к сожалению, эта уверенность спокойно может оказаться неоправданной. Объясню почему. TDD (разработка через тестирование) предполагает, что вы одновременно выступаете и автором кода, и автором тест-кейсов: читаете спецификации, реализуете функциональность и сразу же пишете для неё набор тестов. Тесты, предположим, выполнятся успешно. Но что, если вы неверно поняли требования спецификаций? Тогда тесты проверят не то, что нужно. А значит, ваша уверенность ничего не стоит. И всё оттого, что вы писали и код, и тесты единолично.
Но попробуем закрыть на это глаза. Тесты всё же необходимы, и в любом случае они дают нам уверенность. Больше всего мы, конечно же, любим функциональные тесты: они не подразумевают никаких побочных эффектов, никаких зависимостей — только входные и выходные данные. Для тестирования домена вам нужно воспользоваться mock-объектами: они позволят тестировать классы изолированно.
Что касается запросов к БД, то тестировать их неприятно. Эти тесты хрупкие, они требуют, чтобы сначала вы добавили в базу тестовые данные — и только после этого вы сможете перейти к тестированию функциональности. Но как вы понимаете, эти тесты также необходимы, даже если вы используйте JPA.
![](https://habrastorage.org/r/w780q1/getpro/habr/post_images/8bd/332/6ec/8bd3326ec6ca9886c78b4239491dc936.jpg)
Я бы сказал, что сила unit-тестов не в возможности их запуска, а в том, что заключает в себе процесс их написания. Пока вы пишете тест, вы заново осмысливаете и прорабатываете код — уменьшаете связанность, разбиваете на классы — словом, осуществляете очередной рефакторинг. Тестируемый код — это чистый код; он проще, в нём уменьшена связанность; в общем-то, он ещё и задокументирован (хорошо написанный unit-тест прекрасно описывает то, как работает класс). Неудивительно, что писать unit-тесты — трудно, особенно первые несколько штук.
![](https://habrastorage.org/r/w1560/getpro/habr/post_images/116/be7/361/116be7361d4d6a2dc8cac91983808378.png)
На этапе первых unit-тестов многие действительно пугаются перспективы, что действительно придётся что-то протестировать. Почему же они даются так сложно?
Потому что эти тесты являются первой нагрузкой на ваш класс. Это первый удар по системе, который, возможно, покажет, что она хрупкая и хлипкая. Но надо понимать, что эти несколько тестов — самые важные для вашей разработки. Они, в сущности, ваши лучшие друзья, потому что скажут всё как есть о качестве вашего кода. Если бояться этого этапа, то далеко уйти не получится. Вы должны запустить для своей системы тестирование. После этого сложность спадёт, тесты будут писаться быстрее. Добавляя их один за другим, вы создадите для вашей системы надёжную базу регрессионного тестирования. А это невероятно важно для дальнейшей работы ваших разработчиков. Им будет проще заниматься рефакторингом; Они будут понимать, что система может быть в любой момент регрессионно протестирована, что поэтому работать с кодовой базой — безопасно. И, уверяю вас, они займутся рефакторингом гораздо охотнее.
![](https://habrastorage.org/r/w1560/getpro/habr/post_images/eb5/3d7/cbd/eb53d7cbdc3b190b2b597d6427088b1a.png)
Мой вам совет: если вы чувствуете, что сегодня у вас много сил и энергии — посвятите себя написанию unit-тестов. И сделайте так, чтобы каждый из них был чистым, быстрым, имел свой вес и не повторял другие.
Резюмируя всё сказанное сегодня, я бы хотел напутствовать вас следующими советами:
Делая эти вещи, вы поможете и своей команде, и себе. И тогда, когда наступит день поставки продукта, — вы будете к нему готовы.
![](https://habrastorage.org/r/w780q1/getpro/habr/post_images/852/a4c/37d/852a4c37d17677fa3fb1e7e021f29e4a.jpg)
Успокойтесь, начальство всё равно не даст ничего переписать. Остаётся рефакторить. На что лучше всего потратить свои невеликие ресурсы? Как именно рефакторить, где проводить чистки?
Название этой статьи — в том числе отсылка к книге Дяди Боба «Чистая Архитектура», а сделана она на основе замечательного доклада Victor Rentea (твиттер, сайт) на JPoint (под катом он начнёт говорить от первого лица, но пока дочитайте вводную). Чтения умных книжек эта статья не заменит, но для такого короткого описания изложено весьма хорошо.
Идея в том, что популярные в народе вещи вроде «Clean Architecture» действительно являются полезными. Сюрприз. Если нужно решить вполне конкретную задачу, простой изящный код не требует сверхусилий и оверинжиниринга. Чистая архитектура говорит, что нужно защищать свою доменную модель от внешних эффектов, и подсказывает, как именно это можно сделать. Эволюционный подход к наращиванию объема микросервисов. Тесты, которые делают рефакторинг менее страшным. Вы ведь уже знаете всё это? Или знаете, но боитесь даже подумать об этом, ведь это же ужас что тогда делать придётся?
Кто хочет получить волшебную анти-прокрастинационную таблетку, которая поможет перестать трястись и начать рефакторить — добро пожаловать на видеозапись доклада или под кат.
Меня зовут Виктор, я из Румынии. Формально я являюсь консультантом, техлидом и ведущим архитектором в Румынском IBM. Но если бы меня попросили самому дать определение своей деятельности, то я евангелист чистого кода. Обожаю создавать красивый, чистый, поддерживаемый код — об этом, как правило, и рассказываю на докладах. Даже больше, меня вдохновляет преподавание: обучение разработчиков в областях Java EE, Spring, Dojo, Test Driven Development, Java Performance, а также в области упомянутого евангелизма — принципов чистоты паттернов кода и их разработки.
Опыт, на котором строится моя теория — в основном разработка корпоративных приложений для крупнейшего клиента IBM в Румынии — банковского сектора.
План на эту статью таков:
- Моделирование данных: структуры данных не должны становиться нашими врагами;
- Организация логики: принцип «декомпозиции кода, которого слишком много»;
- «Onion» — самая чистая архитектура философии Transaction Script;
- Тестирование как способ борьбы со страхами разработчика.
Но сначала давайте вспомним те главные принципы, о которых мы, как разработчики, должны помнить всегда.
Принцип единственной ответственности
![](https://habrastorage.org/getpro/habr/post_images/353/dd0/cd5/353dd0cd58012749b70e379e37231265.jpg)
Иначе говоря, количество vs качество. Как правило, чем больше функциональности содержит ваш класс, тем хуже она оказывается в качественном отношении. Разрабатывая большие классы, программист начинает путаться, ошибаться в построении зависимостей, а большой код, помимо всего прочего, сложнее отладить. Лучше разбить такой класс на несколько более мелких, каждый из которых будет отвечать за некоторую подзадачу. Пусть лучше у вас будет несколько сильносвязанных модулей, чем один — крупный и неповоротливый. Модульность также даёт возможность повторного использования логики.
Слабое связывание модулей
![](https://habrastorage.org/getpro/habr/post_images/f72/1c8/0f7/f721c80f7b3184be2218a01b66f7440a.jpg)
Степень связывания — это характеристика того, насколько тесно взаимодействуют друг с другом ваши модули. Она показывает то, насколько широко способен распространиться эффект от изменений, вносимых вами в какой-то одной точке системы. Чем выше связывание, тем сложней осуществлять модификации: вы меняете что-то в одном модуле, а эффект распространяется далеко и не всегда ожидаемым образом. Поэтому показатель связывания должен быть как можно ниже — этим вы обеспечите больший контроль над системой, подвергающейся модификациям.
Не повторяйтесь
![](https://habrastorage.org/getpro/habr/post_images/2b9/9de/5c0/2b99de5c00257d6123553c7bfaed0519.jpg)
Ваши собственные реализации могут быть хороши сегодня, но уже не так хороши завтра. Не разрешайте себе копировать свои собственные наработки и таким образом распространять их по кодовой базе. Вы можете копировать со StackOverflow, из книг — с любых авторитетных источников, которые (как вы точно знаете) предлагают идеальную (или близкую к тому) реализацию. Дорабатывать же свою собственную реализацию, встречающуюся не один раз, а размноженную по всей кодовой базе, может быть очень утомительно.
Простота и лаконичность
![](https://habrastorage.org/getpro/habr/post_images/7d4/8d4/c85/7d48d4c85408c620ddcbe644c4acb6f9.jpg)
На мой взгляд, это главный принцип, который нужно соблюдать в инженерных и программных разработках. «Преждевременная инкапсуляция — корень зла», — говорил Адам Биен. Иначе говоря, корень зла заключается в «пере-инженерии». Автор цитаты Адам Биен одно время занимался тем, что брал легаси-приложения и, полностью переписывая их код, получал кодовую базу объемом в 2-3 раза меньше исходного. Откуда берётся столько лишнего кода? Он ведь возникает не просто так. Его порождают испытываемые нами страхи. Нам кажется, что, нагромождая в большом количестве паттерны, плодя косвенность и абстракции, мы обеспечиваем нашему коду защиту — защиту от неизвестностей завтрашнего дня и завтрашних требований. Ведь на самом-то деле сегодня ничего из этого нам не нужно, изобретаем мы всё это только ради каких-то «будущих нужд». И не исключено, что эти структуры данных впоследствии будут мешать. Скажу честно, когда ко мне подходит какой-нибудь мой разработчик и озвучивает, что он придумал кое-что интересное, что можно добавить в продакшн-код, я отвечаю всегда одинаково: «Парень, тебе это не пригодится».
Кода не должно быть много, а тот, что есть, должен быть простым — только так с ним можно будет нормально работать. Это забота о ваших разработчиках. Вы должны помнить, что именно они являются ключевыми фигурами для вашей системы. Постарайтесь снизить их энергозатраты, уменьшить те риски, с которыми им придётся работать. Это не значит, что вам придётся создать свой собственный фреймворк, более того, я бы не советовал вам это делать: в своём фреймворке всегда будут баги, всем нужно будет его изучать и т.д. Лучше пустите в ход существующие средства, коих сегодня имеется масса. Это должны быть простые решения. Пропишите глобальные обработчики ошибок, примените технологию аспектов, генераторы кода, расширения Spring или CDI, настройте области действия Request/Thread, используйте манипуляцию и генерацию на лету байткода и пр. Всё это будет вашим вкладом в поистине важнейшую вещь — в комфорт вашего разработчика.
В частности, я бы хотел продемонстрировать вам применение областей Request/Thread. Я не раз наблюдал, как эта вещь невероятным образом упрощала корпоративные приложения. Суть в том, что она даёт вам возможность, будучи залогиненным пользователем, сохранять данные RequestContext. Таким образом, RequestContext будет в компактном виде хранить данные о пользователе.
![](https://habrastorage.org/getpro/habr/post_images/341/6c5/474/3416c54740aca3b8f514696bf9cf7f3d.jpg)
Как видите, реализация занимает всего пару строк кода. Прописав запрос в нужную аннотацию (несложно делается, если вы используете Spring или CDI), вы навсегда освободите себя от необходимости передавать методам пользовательский логин и что бы то ни было ещё: хранимые внутри контекста метаданные запроса будут прозрачным образом перемещаться по приложению. Scoped proxy же позволит вам в любой момент получить доступ к метаданным текущего запроса.
Регрессионные тесты
![](https://habrastorage.org/getpro/habr/post_images/5c5/edf/d7a/5c5edfd7a14d760bc335d8e65f0d5666.jpg)
Обновляющихся требований разработчики боятся потому, что боятся процедуры рефакторинга (модификации кода). И самый простой способ помочь им — создать надёжный набор тестов для регрессионного тестирования. С помощью него разработчик будет иметь возможность в любой момент протестировать свою наработку — убедиться, что она не поломает систему.
Разработчик не должен бояться ничего сломать. Вы должны сделать всё, чтобы рефакторинг воспринимался как что-то хорошее.
Рефакторинг — важнейший аспект разработки. Помните, ровно в тот момент, когда ваши разработчики испугаются рефакторинга, приложение можно считать перешедшим в разряд «легаси».
Где реализовывать бизнес-логику?
![](https://habrastorage.org/getpro/habr/post_images/4fb/8a2/08d/4fb8a208dc6d89279a663747de74369f.jpg)
Начиная реализацию любой системы (или компоненты системы), мы задаём себе вопрос: где лучше реализовать логику предметной области, то есть функциональные аспекты нашего приложения? Есть два противоположных подхода.
Первый из них основывается на философии Transaction Script. Здесь логика реализуется в процедурах, работающих с анемичными сущностями (то есть со структурами данных). Такой подход хорош тем, что в ходе его реализации можно опираться на сформулированные бизнес-задачи. Работая над приложениями для банковского сектора, я не раз наблюдал перевод бизнес-процедур в софт. Могу сказать, что это действительно очень естественно — соотносить сценарии с софтом.
Альтернативный подход — использовать принципы Domain-Driven Design. Здесь вам потребуется соотнести спецификации и требования с объектно-ориентированной методологией. Важно и тщательно продумать объекты, и обеспечить хорошую вовлеченность со стороны бизнеса. Плюс спроектированных таким образом систем в том, что в дальнейшем они легко поддерживаются. Однако по моему опыту, осваивать данную методологию достаточно непросто: более-менее смело вы почувствуете себя не раньше, чем через полгода её изучения.
Для своих разработок я всегда выбирал первый подход. Могу заверить, что в моем случае он работал идеально.
Моделирование данных
Сущности
Как же нам моделировать данные? Как только приложение принимает более-менее приличные размеры, обязательно появляются персистентные данные. Это такие данные, которые вам требуется хранить дольше остальных — они являются доменными сущностями (domain entities) вашей системы. Где их хранить — в базе данных ли, в файле или напрямую управляя памятью — не имеет значения. Важно то, как вы будете их хранить — в каких структурах данных.
![](https://habrastorage.org/getpro/habr/post_images/0a8/56b/127/0a856b127ec938a8c17eb345610b0ee8.jpg)
Вам как разработчику дан этот выбор, и только от вас зависит, будут ли эти структуры данных работать на вас или против вас при реализации функциональных требований в будущем. Чтобы всё было хорошо, вы должны реализовывать сущности, закладывая в них крупицы переиспользуемой доменной логики. Как конкретно? Продемонстрирую несколько способов на примере.
![](https://habrastorage.org/getpro/habr/post_images/c73/e56/7da/c73e567da2fb69ceb3d627fb408530cd.jpg)
Давайте посмотрим, чем я снабдил сущность Customer. Во-первых, я реализовал синтетический геттер
getFullName()
, который будет возвращать мне конкатенацию firstName и lastName. Также я реализовал метод activate()
— для контроля состояния моей сущности, таким образом инкапсулируя его. В этот метод я поместил, во-первых, операцию по валидации, и, во-вторых, присвоение значений полям status и activatedBy, благодаря чему не нужно прописывать для них сеттеры. Также я добавил в сущность Customer методы isActive()
и canPlaceOrders()
, осуществляющие внутри себя лямбда-валидацию. Это так называемая инкапсуляция предикатов. Такие предикаты пригодятся, если вы используете фильтры Java 8: можно передавать их в качестве аргументов фильтрам. Этими хелперами советую пользоваться.Возможно, вы используете какой-нибудь ORM вроде Hibernate. Предположим, у вас имеется две сущности с двусторонней связью. Инициализацию необходимо выполнять с обеих сторон, иначе, как вы понимаете, у вас возникнут проблемы при обращении к этим данным в дальнейшем. Но разработчики зачастую забывают проинициализировать объект с какой-нибудь одной из сторон. Вы же, разрабатывая эти сущности, можете предусмотреть специальные методы, которые будут гарантировать двустороннюю инициализацию. Посмотрите на
addAddress()
.![](https://habrastorage.org/getpro/habr/post_images/c55/7fe/273/c557fe273cc1f2658573f9e78a70ca1d.png)
Как видите, это вполне обычная сущность. Но внутрь неё заложена доменная логика. Такие сущности не должны быть скудными и поверхностными, но и не должны быть переполненными логикой. Переполненность логикой возникает чаще: если уж вы решили реализовать всю логику в домене, то для каждого use-case возникнет соблазн реализовать какой-нибудь специфический метод. А use-case-ов, как правило, бывает много. Вы получите не сущность, а одну большую кучу всевозможной логики. Старайтесь соблюдать здесь меру: в домен помещается только переиспользуемая логика и только в небольшом количестве.
Объекты-значения
Помимо сущностей вам, скорее всего, также понадобятся объекты-значения (object values). Это не что иное, как способ сгруппировать доменные данные, чтобы потом вместе перемещать их по системе.
Объект-значение должен быть:
- Небольшим. Никаких
float
для денежных переменных! Крайне осторожно подходите к выбору типов данных. Чем компактнее ваш объект, тем легче в нём разберётся новый разработчик. Это основа основ для комфортной жизни.
- Неизменяемым. Если объект действительно иммутабельный, то разработчик может быть спокоен, что ваш объект не изменит своего значения и не сломается после создания. Это закладывает основу для спокойной, уверенной работы.
![](https://habrastorage.org/getpro/habr/post_images/a0e/765/284/a0e76528432de8c55dc146805a8b286d.jpg)
А если добавить в конструктор вызов метода
validate()
, то разработчик сможет быть спокоен за валидность созданной сущности (при передаче, скажем, несуществующей валюты или отрицательной денежной суммы конструктор не отработает).Отличие сущности от объекта-значения
Объекты-значения отличаются от сущностей тем, что у них нет постоянного ID. В сущностях всегда будут поля, связанные с внешним ключом какой-нибудь таблицы (или другого хранилища). У объектов-значений таких полей нет. Возникает вопрос: отличаются ли процедуры проверки на равенство двух объектов-значений и двух сущностей? Поскольку у объектов-значений нет поля ID, чтобы заключить, что два таких объекта равны, придётся попарно сравнить значения всех их полей (то есть осмотреть всё содержимое). При сравнении же сущностей достаточно провести одно единственное сравнение — по полю ID. Именно в процедуре сравнения заключается главное отличие сущностей от объектов-значений.
Data Transfer Objects (DTOs)
![](https://habrastorage.org/getpro/habr/post_images/72a/d31/681/72ad31681509cdef20509d4514b02139.jpg)
В чём заключается взаимодействие с пользовательским интерфейсом (UI)? Ему вы должны передать данные для отображения. Неужели понадобится ещё одна структура? Так и есть. А всё потому, что пользовательский интерфейс вам совсем не друг. У него свои запросы: ему нужно, чтобы данные хранились в соответствии с тем, как они должны отображаться. Это так чудно — что именно порой требуют от нас пользовательские интерфейсы и их разработчики. То им нужно достать данные для пяти строк; то им приходит в голову завести для объекта булево поле
isDeletable
(может ли в принципе у объекта быть такое поле?), чтобы знать, делать ли активной кнопку «Удалить» или нет. Но возмущаться нечего. У пользовательских интерфейсов попросту другие требования.Вопрос в том, можно ли вверить им в пользование наши сущности? Вероятней всего, они их изменят, причем самым нежелательным для нас образом. Поэтому предоставим им кое-что другое — Data Transfer Objects (DTO). Они будут приспособлены специально под внешние требования и под логику, отличную от нашей. Некоторые примеры структур DTO: Form/Request (поступают из UI), View/Response (отправляются в UI), SearchCriteria/SearchResult и пр. Можно в некотором смысле назвать это API-моделью.
Первый важный принцип: DTO должно содержать минимум логики.
Перед вами пример реализации
CustomerDto
.![](https://habrastorage.org/getpro/habr/post_images/4ec/3bb/716/4ec3bb716c630f3b39ca82f8fe4c5e33.jpg)
Содержимое: private-поля, public-геттеры и сеттеры для них. Вроде бы всё супер. ООП во всей красе. Но одно плохо: в виде геттеров и сеттеров я реализовал слишком большое количество методов. В DTO же логики должно быть как можно меньше. И тогда какой мой выход? Я делаю поля публичными! Вы скажете, что такое плохо работает с method references из Java 8, что возникнут ограничения и пр. Но верите или нет, все свои проекты (10-11 штук) я делал вот с такими DTO. Брат жив. Теперь, поскольку мои поля — публичные, я имею возможность запросто присваивать значение
dto.fullName
, просто ставя знак равно. Что может быть прекраснее и проще?Организация логики
Маппинг
Итак, у нас есть задача: нам нужно преобразовать наши сущности в DTO. Реализуем преобразование так:
![](https://habrastorage.org/getpro/habr/post_images/995/dfd/ca8/995dfdca8c1945fb92a64cee09851259.jpg)
Как видите, объявив DTO, мы переходим к операциям маппинга (присвоения значений). Нужно ли быть senior developer, чтобы писать в таком количестве обычные присвоения? Для некоторых это настолько непривычно, что они начинают переобуваться на ходу: например, копировать данные при помощи какого-нибудь фреймворка для маппинга, используя рефлекшн. Но они упускают главное — то, что рано или поздно произойдёт взаимодействие UI с DTO, в результате которого сущность и DTO разойдутся в своих значениях.
Можно было бы, скажем, поместить операции маппинга в конструктор. Но такое возможно не для любого маппинга; в частности, в конструкторе нельзя осуществить доступ к базе данных.
Таким образом, мы вынуждены оставить операции маппинга в бизнес-логике. И если они имеют компактный вид, то в этом ничего страшного нет. Если же маппинг занимает не пару строк, а больше, то лучше вынести его в так называемый маппер. Маппер — это класс, специально предназначенный для копирования данных. Это, в общем-то, допотопная вещь и бойлерплейт. Но зато за ними можно скрыть наши многочисленные присвоения — сделать код чище и стройнее.
Запомните: код, который слишком разросся, нужно выносить в отдельную структуру. В нашем случае операций маппинга было действительно многовато, поэтому мы вынесли их в отдельный класс — в маппер.
Разрешать ли мапперам доступ в базу данных? Можете по умолчанию разрешить — так часто поступают из соображений простоты и прагматики. Но это подвергает вас определённым рискам.
Проиллюстрирую на примере. Создадим на основе имеющегося DTO сущность
Customer
.![](https://habrastorage.org/getpro/habr/post_images/ef3/9e8/b82/ef39e8b82a023a2fad6afe8e3c22b5ec.png)
Для маппинга нам необходимо добыть из базы данных ссылку на группу покупателя. Поэтому я запускаю метод
getReference()
, и он возвращает мне некоторую сущность. Запрос, скорей всего, уйдёт в базу данных (в некоторых случаях этого не происходит, и отрабатывает функция-заглушка).Но неприятность нас ожидает не здесь, а в методе, выполняющем обратную операцию — преобразование сущности в DTO.
![](https://habrastorage.org/getpro/habr/post_images/e6a/e27/7ff/e6ae277ff6b22588b5bc5aaffabf9d0e.png)
При помощи цикла мы проходим все адреса, ассоциированные с имеющимся Customer, и преобразуем их в адреса DTO. Если вы используете ORM, то, вероятно, при вызове метода
getAddresses()
выполнится ленивая загрузка. Если вы не используете ORM, то это будет открытый запрос ко всем детям данного родителя. И здесь вы рискуете вляпаться в «проблему N+1». Почему?![](https://habrastorage.org/getpro/habr/post_images/e54/001/2bb/e540012bbb4a242d84e55a2dffa5ca11.jpg)
У вас есть набор родителей, у каждого из которых есть дети. Для всего этого вам нужно создать свои аналоги внутри DTO. Вам понадобится выполнить один
SELECT
-запрос для обхода N родительских сущностей и далее — ещё N SELECT
-запросов, чтобы обойти детей каждой из них. Итого N+1 запрос. Для 1000 родительских сущностей Customer
такая операция займёт 5-10 секунд, что, конечно, долго.Предположим, что, всё же, наш метод
CustomerDto()
вызывается внутри цикла, преобразуя список объектов Customer в список CustomerDto
.![](https://habrastorage.org/getpro/habr/post_images/58c/9c3/2b3/58c9c32b3e8a01a96d500920dfe5ef07.jpg)
У проблемы с N+1 запросами есть простые типовые решения: в JPQL для извлечения детей вы можете воспользоваться
FETCH
по customer.addresses и затем соединить их при помощи JOIN
, а в SQL вы можете применить обход IN
и оператор WHERE
.Но я бы сделал по-другому. Можно узнать, какова максимальная длина списка детей (это можно сделать, например, на основе поиска с пагинацией). Если в списке всего 15 сущностей, то нам потребуется всего 16 запросов. Вместо 5мс мы потратим на всё, скажем, 15мс — пользователь не заметит разницы.
Об оптимизации
Я бы не советовал вам оглядываться на производительность системы уже на начальном этапе разработки. Как сказал Дональд Кнуд: «Преждевременная оптимизация — корень зла». Нельзя оптимизировать с самого начала. Это именно то, что нужно оставить на потом. И что особенно важно: никаких предположений — только измерения и оценка измерений!
Так ли вы уверены, что вы компетентны, что вы настоящий эксперт? Будьте скромны в оценке себя. Не думайте, что поняли работу JVM, пока не прочтёте хотя бы пару книг о JIT-компиляции. Бывает, лучшие программисты из нашей команды подходят ко мне и говорят, что, как им кажется, они нашли более эффективную реализацию. Оказывается же, что они снова изобрели что-то, что только усложняет код. Поэтому я раз за разом отвечаю: YAGNI. Нам это не понадобится.
Зачастую для корпоративных приложений вообще не требуется никакой оптимизации алгоритмов. Узким местом для них, как правило, является не компиляция и не то, что касается работы процессора, а всевозможные операции ввода-вывода. Например, считывание миллиона строк из базы данных, объёмные записи в файл, взаимодействие с сокетами.
Со временем начинаешь понимать, какие узкие места содержит система, и, подкрепив всё измерениями, начнешь понемногу оптимизировать. А до этого момента сохраняйте код как можно более чистым. Вы обнаружите, что такой код гораздо легче поддаётся дальнейшей оптимизации.
Предпочитайте композицию наследованию
Вернёмся к нашим DTO. Предположим, мы определили такую DTO:
![](https://habrastorage.org/getpro/habr/post_images/180/39d/f91/18039df91fd1cc76a2413b7f373e7c0b.png)
Она может понадобиться нам во множестве рабочих потоков. Но эти потоки разные и, вероятней всего, каждый use-case будет предполагать различную степень заполненности полей. Например, создать DTO нам явно нужно будет раньше, чем когда у нас будет полная информация о пользователе. Можно временно оставлять поля пустыми. Но чем больше полей будете игнорироваться, тем больше вам будет хотеться создать новое более строгое DTO для данного use-case.
Как вариант, можно создать копии избыточно большого DTO (в количестве имеющихся use-case-ов) и далее убрать из каждой копии лишние для неё поля. Но многим программистам, в силу ума и грамотности, действительно больно нажимать Ctrl+V. Аксиома гласит, что копипастить — плохо.
Можно прибегнуть к известному в теории ООП принципу наследования: просто определим некий базовый DTO и для каждого use-case создадим наследника.
![](https://habrastorage.org/getpro/habr/post_images/6f7/069/77f/6f706977fec8625d720323b64fe87ad8.png)
Известный принцип гласит: «Предпочитайте композицию наследованию». Прочитайте, что там написано: «extends». Вроде бы мы должны были «расширить» исходный класс. Но если вдуматься, то, что мы сейчас понаделали — вовсе не «расширение». Это самое настоящее «повторение» — тот же копипаст, вид сбоку. Поэтому наследование мы использовать не будем.
Но как же тогда нам быть? Как перейти к композиции? Сделаем так: пропишем в CustomerView поле, которое будет указывать на объект базового DTO.
![](https://habrastorage.org/getpro/habr/post_images/1f2/d49/831/1f2d49831b9b286b8bc069c824ac2871.jpg)
Таким образом наша базовая структура будет вложена внутрь. Вот так выйдет настоящая композиция.
Используем ли мы наследование или решаем вопрос композицией — это всё частности, тонкости, возникшие глубоко в ходе нашей реализации. Они очень хрупкие. Что значит хрупкие? Посмотрите внимательно на этот код:
![](https://habrastorage.org/getpro/habr/post_images/921/eb1/ef2/921eb1ef20effd43b129ad5d28b88253.png)
Большинство разработчиков, которым я это показал, сразу выпалили, что число «2» повторяется, поэтому его нужно вынести в виде константы. Они не обратили внимание, что двойка во всех трёх случаях имеет совершенно разный смысл (или «бизнес-значение») и что её повторение — не более чем совпадение. Вынести двойку в константу — правомерное решение, однако очень хрупкое. Старайтесь не допускать в домен хрупкую логику. Никогда не работайте из него со внешними структурами данных, в частности, с DTO.
Итак, почему же работа по ликвидации наследования и введению композиции оказывается бесполезной? Именно потому, что DTO мы создаем не для себя, а для внешнего клиента. А как клиентское приложение будет парсить полученные от вас DTO — вам остаётся только догадываться. Но очевидно, что это будет иметь мало общего с вашей реализацией. Разработчики с той стороны могут и не сделать различия для базовых и небазовых DTO, которые вы так старательно продумали; наверняка они используют наследование, а возможно и тупо копипастят вот это всё.
Фасады
![](https://habrastorage.org/getpro/habr/post_images/be7/31c/9e4/be731c9e4cfe002cb3b59c11a708f623.jpg)
Вернёмся к общей картине приложения. Я бы советовал вам реализовывать доменную логику через паттерн Facade, расширяя фасады доменными сервисами по необходимости. Доменный сервис создаётся тогда, когда в фасаде накапливается слишком большое количество логики, и удобнее её вынести в отдельный класс.
Ваши доменные сервисы должны обязательно говорить на языке вашей доменной модели (её сущностей и объектов-значений). Они ни в коем случае не должны работать с DTO, потому как DTO, как вы помните, — это структуры, постоянно изменяемые с клиентской стороны, слишком хрупкие для домена.
![](https://habrastorage.org/getpro/habr/post_images/3ad/eb8/657/3adeb86570a92f804ba491836c99b9e8.jpg)
Каково назначение фасада?
- Преобразование данных. Если мы имеем сущности с одного конца и DTO с другого, необходимо проводить преобразования из одного в другое. И это первое, для чего нужны фасады. Если процедура преобразования разрослась в объёме — применяйте классы-мапперы.
- Реализация логики. В фасаде вы начнёте писать основную логику приложения. Как только её становится много — выносите части в доменный сервис.
- Валидация данных. Помните, что любые поступающие от пользователя данные по определению являются некорректными (содержащими ошибки). В фасаде есть возможность провести валидацию данных. Эти процедуры при превышении объёма принято выносить в валидаторы.
- Аспекты. Можно пойти дальше и сделать так, чтобы каждый use-case проходил через свой фасад. Тогда получится надстроить на методы фасада такие вещи, как транзакции, логирование, глобальные обработчики исключений и др. Отмечу, очень важно иметь в любом приложении глобальные обработчики исключений, которые бы ловили все ошибки, не пойманные другими обработчиками. Они очень помогут вашим программистам — дадут им спокойствие и свободу действий.
Декомпозиция кода, которого много
![](https://habrastorage.org/getpro/habr/post_images/604/a9c/8e8/604a9c8e859ff1dc3fb38208d6bf836d.jpg)
Ещё пара слов об этом принципе. Если класс достиг некоторого неудобного для меня размера (скажем, 200 строк), то я должен попробовать разбить его на части. Но выделить новый класс из существующего не всегда просто. Нужны придумать какие-то универсальные способы. Один из таких способов состоит в поиске имён: вы пробуете подобрать название для какого-нибудь подмножества методов вашего класса. Как только у вас получилось найти имя — смело создавайте новый класс. Но и это не так просто. В программировании, как известно, всего две сложные вещи: это инвалидация кэша и придумывание имен. В данном случае, придумывание названия сопряжено с выявлением подзадачи — скрывающейся и потому ранее никем не выявленной.
Пример:
![](https://habrastorage.org/getpro/habr/post_images/81f/fe3/82b/81ffe382b74f6bbd4a91626d83249a97.jpg)
В исходном фасаде
CustomerFacade
часть методов связана непосредственно с покупателем, некоторые же — с предпочтениями покупателя. На основе этого я смогу расколоть класс на две части, когда он достигнет критических размеров. Получу два фасада: CustomerFacade
и CustomerPreferencesFacade
. Плохо только то, что оба этих фасада принадлежат одному уровню абстракции. Разделение же по уровням абстракции предполагает несколько другое.Ещё один пример:
![](https://habrastorage.org/getpro/habr/post_images/2ac/84e/788/2ac84e788af60fb038091aed273a390e.jpg)
Предположим, в нашей системе есть класс
OrderService
, в котором мы реализовали механизм уведомлений по электронной почте. Теперь мы создаём DeliveryService
и хотели бы использовать здесь тот же самый механизм уведомлений. Копипаст — исключён. Сделаем так: извлечём функциональность уведомлений в новый класс AlertService
и пропишем его в качестве зависимости для классов DeliveryService
и OrderService
. Здесь, в отличие от предыдущего примера, разделение произошло именно по уровням абстракции. DeliveryService
более абстрактен, чем AlertService
, так как использует его как составляющую своего рабочего потока.Разделение по уровням абстракции всегда предполагает, что извлекаемый класс становится зависимостью, а извлечение осуществляется для переиспользования.
Задача извлечения не всегда даётся просто. Она также может повлечь за собой некоторые сложности и потребовать какого-нибудь рефакторинга unit-тестов. Тем не менее, по моим наблюдениям, искать какой бы то ни было функционал по огромной монолитной кодовой базе приложения разработчикам ещё тяжелее.
Парное программирование
![](https://habrastorage.org/getpro/habr/post_images/625/5b2/346/6255b23467294d6224107235443379f1.jpg)
Многие консультанты расскажут про парное программирование, про то, что это универсальное решение любых проблем IT-разработки на сегодняшний день. В ходе него программисты развивают свои технические навыки и функциональные знания. Кроме того, интересен сам процесс, он сплачивает команду.
Если говорить не как консультанты, а по-человечески, самое важное здесь вот что: парное программирование улучшает «фактор автобуса». Суть же «фактора автобуса» в том, что людей, обладающих знаниями об устройстве системы, должно быть как можно больше. Потерять этих людей означает потерять последние ключи к этим знаниям.
Рефакторинг в формате парного программирования — искусство, требующее опыта и тренировки. Здесь полезны, например, практики агрессивного рефакторинга, проведение хакатонов, катов, Coding Dojos и др.
Парное программирование хорошо работает в случаях, когда нужно решать задачи высокой сложности. Сам процесс работы вдвоём не всегда прост. Зато он гарантирует вам, что вы избежите «переинженерии» — напротив, получите реализацию, которая адресует поставленные требования с минимальной сложностью.
![](https://habrastorage.org/getpro/habr/post_images/a5a/5aa/666/a5a5aa666cb29c58dc9bb5b7d5bc6651.jpg)
Организация удобного формата работы — одна из ваших главных обязанностей перед командой. Вы должны не переставая заботиться об условиях труда разработчика — обеспечивать им полный комфорт и свободу творчества, особенно, если от них требуется наращивать проектную архитектуру и её сложность.
«Я архитектор. По определению, я всегда прав».
Эту глупость периодически выражают гласно или негласно. В сегодняшней практике архитекторы как таковые встречаются всё меньше. С приходом Agile эта роль постепенно перешла к старшим разработчикам, потому как обычно вся работа, так или иначе, строится вокруг них. Размер реализации постепенно растёт, и вместе с этим появляется потребность в рефакторинге и разрабатывается новая функциональность.
Архитектура «луковица»
«Луковица» — самая чистая архитектура философии Transaction Script. Строя её, мы руководствуемся целью обеспечить защиту того кода, который считаем критичным, и для этого перемещаем его в доменный модуль.
![](https://habrastorage.org/getpro/habr/post_images/a45/bb4/94a/a45bb494a51b04ecfe92606b744c0857.jpg)
В нашем приложении самыми важными являются доменные сервисы: они реализуют самые критичные потоки. Переместим их в доменный модуль. Безусловно, сюда же стоит перенести все свои доменные объекты — сущности и объекты-значения. Всё остальное, что мы с вами сегодня накодили — DTO, мапперы, валидаторы и пр. — становится, так сказать, первой линией обороны от пользователя. Потому как пользователь, увы, нам не друг, и необходимо защищать от него систему.
Внимание вот на эту зависимость:
![](https://habrastorage.org/getpro/habr/post_images/c7d/273/6bd/c7d2736bdc016b732c5c431299bf32ea.jpg)
Модуль приложения будет зависеть от доменного модуля — именно так, а не наоборот. Прописывая связь такой, мы гарантируем, что DTO никогда не ворвутся на святую территорию доменного модуля: они попросту не видны и недоступны из доменного модуля. Получается, что мы в некотором смысле огородили территорию домена — ограничили к ней доступ посторонних.
Тем не менее, домену может понадобиться взаимодействовать с каким-нибудь внешним сервисом. С внешним — значит, с недружественным, потому что он снабжён своими DTO. Какие наши варианты?
Первый: пропустить недруга внутрь модуля.
![](https://habrastorage.org/webt/un/tn/hk/untnhkiksv5id_i-6c4qwdjxy1c.jpeg)
Очевидно, что это плохой вариант: не исключено, что завтра внешний сервис не выполнит апгрейд на версию 2.0, и нам придётся перекраивать наш домен. Нельзя пускать врага внутрь домена!
Предлагаю другой подход: для взаимодействия создадим специальный адаптер.
![](https://habrastorage.org/getpro/habr/post_images/123/8b1/09c/1238b109c54ac8f19524e6d41b32e85e.jpg)
Адаптер будет получать данные от внешнего сервиса, извлекать те из них, которые нужны нашему домену, и преобразовывать их в требуемые виды структур. В таком случае всё, что от нас требуется при разработке — соотнести вызовы к внешней системе с требованиями домена. Думайте об этом как об огромном таком адаптере. Я называю такой слой «антикоррупционным».
Например, нам может понадобиться выполнять из домена запросы LDAP. Для этого мы реализуем «антикоррупционный модуль»
LDAPUserServiceAdapter
.![](https://habrastorage.org/getpro/habr/post_images/520/fdc/84a/520fdc84a53641a26b957ac010a19c42.jpg)
В адаптере мы можем:
- Скрыть некрасивые вызовы API (в нашем случае скрываем метод, принимающий массив Object);
- Упаковать исключения в наши собственные реализации;
- Преобразовать чужие структуры данных в свои собственные (в наши доменные объекты);
- Проверить валидность поступающих данных.
Таково назначение адаптера. По хорошему, на стыке с каждой внешней системой, с которой вам необходимо взаимодействовать, должен быть заведён свой адаптер.
![](https://habrastorage.org/getpro/habr/post_images/b1a/663/9c5/b1a6639c58457537fb6a4d9c15e09eae.jpg)
Таким образом, домен будет направлять вызов не к внешнему сервису, а к адаптеру. Для этого в домене должна быть прописана соответствующая зависимость (от адаптера либо от того инфраструктурного модуля, в котором он находится). Но безопасна ли такая зависимость? Если её установить вот так, к нам в домен могут попасть DTO внешнего сервиса. Такого мы допускать не должны. Поэтому предлагаю вам другой способ моделирования зависимостей.
Принцип инверсии зависимостей
![](https://habrastorage.org/getpro/habr/post_images/058/4a1/f7c/0584a1f7c309505e4e5ea0c2b0b26c21.jpg)
Создадим интерфейс, пропишем в нём сигнатуру нужных методов и поместим его внутрь нашего домена. Задача адаптера — реализовать этот интерфейс. Получается, интерфейс находится внутри домена, а адаптер — снаружи, в инфраструктурном модуле, который импортирует интерфейс. Таким образом, мы развернули направление зависимости в обратную сторону. Во время исполнения доменная система будет вызывать любой класс через интерфейсы.
Как видите, всего-навсего введя в архитектуру интерфейсы, мы сумели развернуть зависимости и тем самым обезопасить наш домен от попадания в него чужеродных структур и API. Такой подход называется инверсией зависимостей.
![](https://habrastorage.org/getpro/habr/post_images/c2c/cc2/cd0/c2ccc2cd0309a4c8f97b7521cfc63fb7.jpg)
В общем случае, инверсия зависимостей предполагает, что вы размещаете интересующие вас методы в интерфейсе внутри вашего высокоуровневого модуля (в домене), а реализуете этот интерфейс снаружи — в том или ином низкоуровневом (инфраструктурном) некрасивом модуле.
Интерфейс, реализуемый внутри доменного модуля, должен говорить на языке домена, то есть он будет оперировать его сущностями, его параметрами и возвращаемыми типами. Во время исполнения домен будет вызывать любой класс посредством полиморфного вызова к интерфейсу. Фреймворки, предназначенные для внедрения зависимостей (например, Spring и CDI), снабдят нас конкретным экземпляром класса прямо в рантайме.
Но главное то, что во время компиляции доменный модуль не будет видеть содержимого внешнего модуля. Именно это нам и нужно. Никакая внешняя сущность не должна попасть в домен.
Как считает Дядя Боб, принцип инверсии управления (или, как он её называет, «архитектура плагинов») — это, возможно, лучшее, что вообще предлагает парадигма ООП.
![](https://habrastorage.org/getpro/habr/post_images/a96/3cf/e13/a963cfe13d030fb1d7ec2bf0f780fe43.jpg)
Данную стратегию можно использовать для интеграции с любыми системами, для синхронных и асинхронных вызовов и сообщений, для отсылки файлов и т. д.
Обзор «луковицы»
![](https://habrastorage.org/getpro/habr/post_images/dc3/64d/cdb/dc364dcdb589b35a0597b6e478aace46.jpg)
Итак, мы решили, что защищать будем доменный модуль. Внутри него размещается доменный сервис, сущности, объекты-значения, а теперь и интерфейсы для внешних сервисов, плюс интерфейсы для репозитория (для взаимодействия с базой данных).
Структура выглядит так:
![](https://habrastorage.org/getpro/habr/post_images/07e/983/0c7/07e9830c76d3b263743f8514b13bf42a.jpg)
В качестве зависимостей для домена объявлены модуль приложения, инфраструктурный модуль (посредством инверсии зависимостей), модуль репозитория (базу данных мы тоже считаем внешней системой), batch-модуль и, возможно, некоторые другие модули. Такая архитектура носит название «луковицы»; также её называют «чистой», «шестиугольной» и «порты и адаптеры».
Модуль репозитория
Вкратце расскажу о модуле репозитория. Выносить ли его из домена — это вопрос. Задача репозитория — сделать логику чище, скрыв от нас ужас работы с персистентными данными. Вариант для олдскульных ребят — использовать для взаимодействия с базой данных JDBC:
![](https://habrastorage.org/getpro/habr/post_images/7ed/d70/44a/7edd7044a76373b2c513e246bf6eabaa.jpg)
Также можно использовать Spring и его JdbcTemplate:
![](https://habrastorage.org/getpro/habr/post_images/64b/d68/749/64bd68749f5b3de16591a23a8acda193.jpg)
Или же MyBatis DataMapper:
![](https://habrastorage.org/getpro/habr/post_images/423/31b/aa1/42331baa178ff3ad8f0ea8fb92704265.jpg)
Но это до того сложно и некрасиво, что отбивает всякое желание делать что-то дальше. Поэтому предлагаю использовать JPA/Hibernate либо Spring Data JPA. Они дадут нам возможность отправлять запросы, построенные не на схеме БД, а непосредственно на основе модели наших сущностей.
Реализация для JPA/Hibernate:
![](https://habrastorage.org/getpro/habr/post_images/110/ebf/ef7/110ebfef7dad2a2bb278c50668eecda9.jpg)
В случае же со Spring Data JPA:
![](https://habrastorage.org/getpro/habr/post_images/10b/e2a/0ff/10be2a0ff082a8c1ac962c4754ec55a8.jpg)
Spring Data JPA умеет автоматически генерировать методы во время исполнения, такие, как, например, getById(), getByName(). Также он позволяет при необходимости выполнять JPQL-запросы — и не к БД, а к вашей собственной модели сущностей.
Код Hibernate JPA и Spring Data JPA действительно выглядит очень неплохо. Нужно ли нам вообще извлекать его из домена? По моему мнению, это не так-то и обязательно. Скорей всего, код будет даже чище, если оставить данный фрагмент внутри домена. Так что действуйте по ситуации.
![](https://habrastorage.org/getpro/habr/post_images/055/f04/37c/055f0437cf059c9f7001db6b3d231e70.jpg)
Если вы всё же создаёте модуль репозитория, то для организации зависимостей лучше точно так же воспользоваться принципом инверсии управления. Для этого разместите в домене интерфейс и реализуйте его в модуле репозитория. Что касается логики репозитория, то лучше перенести её в домен. Это делает удобным тестирование, так как в домене вы сможете использовать Mock-объекты. Они позволят протестировать логику быстро и многократно.
Традиционно для репозитория в домене создаётся всего одна сущность. Разбивают на части её только тогда, когда она становится чересчур объёмной. Не забывайте о том, что классы должны быть компактными.
API
![](https://habrastorage.org/getpro/habr/post_images/1c8/9e1/7e7/1c89e17e784e4db6fe8d7c713bcf1ab6.jpg)
Можно создать отдельный модуль, поместить в него извлечённый из фасада интерфейс и полагающиеся к нему DTO, затем упаковать в JAR, и в таком виде передавать своим Java-клиентам. Имея этот файл, они получат возможность направлять запросы к фасадам.
«Прагматичная Луковица»
Помимо тех наших «врагов», которым мы поставляем функциональность, то есть клиентов, у нас есть враги и с другой стороны — те модули, от которых зависим мы сами. От этих модулей нам тоже необходимо защищаться. И для этого предлагаю вам несколько модифицированную «луковицу» — в ней вся инфраструктура объединяется в один модуль.
![](https://habrastorage.org/getpro/habr/post_images/265/1f1/481/2651f1481062cfe6c18974df9a6dde9b.jpg)
Я называю такую архитектуру «прагматичной луковицей». Здесь разделение компонент проходит по принципу «моё» и «интегрируемое»: отдельно хранится то, что относится к моему домену, и отдельно — то, что относится к интеграции с внешними коллаборационистами. Таким образом, получается всего два модуля: домен и приложение. Такая архитектура очень хороша, но только тогда, когда модуль приложений имеет небольшие размеры. В противном случае вам лучше вернуться к традиционной «луковице».
Тесты
Как я уже говорил ранее, если вашего приложения все боятся, считайте, что оно пополнило ряды легаси.
![](https://habrastorage.org/getpro/habr/post_images/12a/508/d1b/12a508d1b9595239f6d9b9a8a72c9474.jpg)
А вот тесты — это хорошо. Они дают нам чувство уверенности, благодаря которому можно продолжать работы по рефакторингу. Но к сожалению, эта уверенность спокойно может оказаться неоправданной. Объясню почему. TDD (разработка через тестирование) предполагает, что вы одновременно выступаете и автором кода, и автором тест-кейсов: читаете спецификации, реализуете функциональность и сразу же пишете для неё набор тестов. Тесты, предположим, выполнятся успешно. Но что, если вы неверно поняли требования спецификаций? Тогда тесты проверят не то, что нужно. А значит, ваша уверенность ничего не стоит. И всё оттого, что вы писали и код, и тесты единолично.
Но попробуем закрыть на это глаза. Тесты всё же необходимы, и в любом случае они дают нам уверенность. Больше всего мы, конечно же, любим функциональные тесты: они не подразумевают никаких побочных эффектов, никаких зависимостей — только входные и выходные данные. Для тестирования домена вам нужно воспользоваться mock-объектами: они позволят тестировать классы изолированно.
Что касается запросов к БД, то тестировать их неприятно. Эти тесты хрупкие, они требуют, чтобы сначала вы добавили в базу тестовые данные — и только после этого вы сможете перейти к тестированию функциональности. Но как вы понимаете, эти тесты также необходимы, даже если вы используйте JPA.
Unit-тесты
![](https://habrastorage.org/getpro/habr/post_images/8bd/332/6ec/8bd3326ec6ca9886c78b4239491dc936.jpg)
Я бы сказал, что сила unit-тестов не в возможности их запуска, а в том, что заключает в себе процесс их написания. Пока вы пишете тест, вы заново осмысливаете и прорабатываете код — уменьшаете связанность, разбиваете на классы — словом, осуществляете очередной рефакторинг. Тестируемый код — это чистый код; он проще, в нём уменьшена связанность; в общем-то, он ещё и задокументирован (хорошо написанный unit-тест прекрасно описывает то, как работает класс). Неудивительно, что писать unit-тесты — трудно, особенно первые несколько штук.
![](https://habrastorage.org/getpro/habr/post_images/116/be7/361/116be7361d4d6a2dc8cac91983808378.png)
На этапе первых unit-тестов многие действительно пугаются перспективы, что действительно придётся что-то протестировать. Почему же они даются так сложно?
Потому что эти тесты являются первой нагрузкой на ваш класс. Это первый удар по системе, который, возможно, покажет, что она хрупкая и хлипкая. Но надо понимать, что эти несколько тестов — самые важные для вашей разработки. Они, в сущности, ваши лучшие друзья, потому что скажут всё как есть о качестве вашего кода. Если бояться этого этапа, то далеко уйти не получится. Вы должны запустить для своей системы тестирование. После этого сложность спадёт, тесты будут писаться быстрее. Добавляя их один за другим, вы создадите для вашей системы надёжную базу регрессионного тестирования. А это невероятно важно для дальнейшей работы ваших разработчиков. Им будет проще заниматься рефакторингом; Они будут понимать, что система может быть в любой момент регрессионно протестирована, что поэтому работать с кодовой базой — безопасно. И, уверяю вас, они займутся рефакторингом гораздо охотнее.
![](https://habrastorage.org/getpro/habr/post_images/eb5/3d7/cbd/eb53d7cbdc3b190b2b597d6427088b1a.png)
Мой вам совет: если вы чувствуете, что сегодня у вас много сил и энергии — посвятите себя написанию unit-тестов. И сделайте так, чтобы каждый из них был чистым, быстрым, имел свой вес и не повторял другие.
Советы
Резюмируя всё сказанное сегодня, я бы хотел напутствовать вас следующими советами:
- Как можно дольше (и чего бы это ни стоило) соблюдайте простоту: избегайте «переинженерии» и запоздалой оптимизации, не перегружайте приложение;
- Заботьтесь о своих разработчиках, предпринимайте меры в защиту их самих и того, что они делают;
- Идентифицируйте «вражеские» структуры данных и держите их на безопасном расстоянии от домена — внешние структуры должны обязательно оставаться снаружи;
- Если вы считаете, что логика разрослась и занимает много места — декомпозируйте: формулируйте названия подзадач и реализуйте их в отдельном классе;
- Помните об архитектуре «луковицы», а точнее, об её главной идее — размещении в домене только критичного кода и недоступности для домена внешних структур;
- Не бойтесь тестов: дайте им возможность повалить вашу систему, ощутите всю их пользу — в конце концов, они потому ваши друзья, что способны честно указать на проблемы.
Делая эти вещи, вы поможете и своей команде, и себе. И тогда, когда наступит день поставки продукта, — вы будете к нему готовы.
Что почитать
![](https://habrastorage.org/getpro/habr/post_images/852/a4c/37d/852a4c37d17677fa3fb1e7e021f29e4a.jpg)
- 7 Virtues of a Good Object
- NULL is the worst mistake in Computer Science
- The Clean Architecture
- New Programming Jargon
- Code quality: WTFs/minute
- Why every single element of SOLID is wrong!
- Good software is written 3 times
Минутка рекламы. Если вам понравился этот доклад с конференции JPoint — обратите внимание, что 19-20 октября в Санкт-Петербурге пройдет Joker 2018 — крупнейшая в России Java-конференция. В его программе тоже будет много интересного. На сайте уже есть первые спикеры и доклады.