Синяя Борода. История одного проекта
Андрей МайоровBYTE-force
twitter.com/xorets
Agenda
• Детали проекта• В чем сложности?• Почему так получилось?• Как исправлять ситуацию?
http://www.flickr.com/photos/lofink/4501610335/
Использование и развитие
• Используется в наших веб-проектах• Развивается по остаточному
принципу
История развития
XML + XSLT + Transformation script, Classic ASP, JS Scriptlets, MS SQL, Stored Procedures, XSLT, Pre-AJAX, ASP.NET, custom HttpHandler, custom XML-based page renderer, Business Logic in Stored Procs, XML from SQL Server, XSLT with extensions, Utility Classes in C#, NHibernate, ObjectQuery functionality, Automatic mapping Object – XML, AJAX Backoffice, JS-web services, CRUD, Custom Event Bus, DI Container (Unity), ASP.NET MVC…
Крупные модернизации
• Внедрение ORM• Преобразование объектов в XML• DI container• Переходим на ASP.NET MVC
You we are here
Сборок совсем мало
http://www.flickr.com/photos/lofink/4501610335/
Сегодняшние проблемы проекта
Сложен reuse частей проекта
http://hq-wallpapers.info/tag/Карточный%20домик
Unit tests не работают
https://picasaweb.google.com/lh/photo/bdJ2w6x9L1H7kUzDsDSyIg
Сложно развивать систему
http://sqlblog.com/blogs/merrill_aldrich/archive/2011/08/10/ouch-chasing-the-isv-or-that-code-makes-my-teeth-hurt-t-sql-tuesday-ish-21.aspx
Неясно, куда класть код
http://www.flickr.com/photos/lofink/4501610335/
Проблемы – у разработчиков
• Люди боятся• Люди не понимают• Людям сложно
У кода нет проблем: работает и не падает.
Основная беда – монолитность кода
God-классы
• Куча функционала, тысячи строк• Со временем только растут
Зло!
Публичные статические поля
• Singleton.Instance• Context.Current
Зло!
«Магическое» конфигурирование
• Неявное, по первому обращению• В конструкторе через глобальный
сервис• В статическом конструкторе• Жесть! Зло!
Слишком большие проекты
• 1000 классов на 9 проектов• 600 классов – в 2 проектах• Проекты на ~50 классов живут явно
лучше других
Зло!
«Борода» детектед
http://www.kiter.by/2010/07/blog-post.html
«Очень синяя борода» - Творческое Объединение «Экран».
Прости, любимая, так получилось...
Надо «расчесать» код
http://www.nrk.no/kultur-og-underholdning/1.7321778
http://www.flickr.com/photos/lofink/4501610335/
Призывы не работают
Конкретные цели: тестопригодный код маленькие пэкеджи
Наши текущие проблемы
Слишком много NHibernate
• За 5 лет проник повсеместно• Конфигурируется «магически»• Текущая сессия – в контексте запроса • Все классы лезут туда напрямую• Вспомогательный статический
класс зависит от HttpRuntime
Ближайшие цели
• Не лазать в базу через контекст запроса• Только по прямой ссылке• Не лазать из методов доменных объектов
• Подготовиться к другому ORM
Решение
• Домен – в отдельный пакет• Убрать логику из доменных объектов *• Сделать обертку для сессии• Передавать обертку в сервисы явно
* Уточнение будет позже
«Анемичная» доменная модель
• Модель без поведений• Фаулер не одобряе:• Нужен ОРМ• Нет инкапсуляции и полиморфизма• Скатываемся в процедурный стиль
Поведение в методе
obj.Publish() – инкапсуляцияPublish() везде разный – полиморфизм
Настройка под заказчика
• У разных клиентов – разные требования к Article.Publish().
Command pattern
В среднесрочной перспективе:
• Убираем логику из доменных объектов• Реализуем её командами• Делаем контейнер домена
Запись данных гораздо сложнее чтения
CRUD головного мозга
http://cthulhucrochet.blogspot.com/2010/09/time-to-switch-to-garlic-shampoo.html
CQRS
• Command-Query Responsibility Segregation• Грег Янг• Начнем с разделения чтения и
записи
Стратегия разделения на куски
???
Паттерн vs. Антипаттерн
Попробуй решить
вот так
Никогда такне делай!
addconf.ru
Пожалуйста, поставьте оценку моему докладу.
Ваше мнение очень важно.
Спасибо!