1. Синяя Борода.
История одного проекта
Андрей Майоров
BYTE-force
twitter.com/xorets

2. Agenda
• Детали проекта
• В чем сложности?
• Почему так получилось?
• Как исправлять ситуацию?

3. http://www.flickr.com/photos/lofink/4501610335/

4. Использование и развитие
• Используется в наших веб-проектах
• Развивается по остаточному
принципу

5. История развития
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…

6. Крупные модернизации
• Внедрение ORM
• Преобразование объектов в XML
• DI container
• Переходим на ASP.NET MVC

7. You we are here

8. Сборок совсем мало
http://www.flickr.com/photos/lofink/4501610335/

9. Сегодняшние
проблемы проекта

10. Сложен reuse частей проекта
http://hq-wallpapers.info/tag/Карточный%20домик

11. Unit tests не работают
https://picasaweb.google.com/lh/photo/bdJ2w6x9L1H7kUzDsDSyIg

12. Сложно развивать систему
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

13. Неясно, куда класть код
http://www.flickr.com/photos/lofink/4501610335/

14. Проблемы – у разработчиков
• Люди боятся
• Люди не понимают
• Людям сложно
У кода нет проблем:
работает и не падает.

15. Основная беда –
монолитность кода

16. God-классы
• Куча функционала, тысячи строк
• Со временем только растут
Зло!

17. Публичные статические поля
• Singleton.Instance
• Context.Current
Зло!

18. «Магическое» конфигурирование
• Неявное, по первому обращению
• В конструкторе через глобальный
сервис
• В статическом конструкторе
• Жесть!
Зло!

19. Слишком большие проекты
• 1000 классов на 9 проектов
• 600 классов – в 2 проектах
• Проекты на ~50 классов живут явно
лучше других
Зло!

20. «Борода» детектед
http://www.kiter.by/2010/07/blog-post.html

21. Прости, любимая,
так получилось...
«Очень синяя борода» - Творческое Объединение «Экран».

22. Надо «расчесать» код
http://www.nrk.no/kultur-og-underholdning/1.7321778

23. Призывы не работают
http://www.flickr.com/photos/lofink/4501610335/

24. Конкретные цели:
тестопригодный код
маленькие пэкеджи

25. Наши текущие проблемы

26. Слишком много NHibernate
• За 5 лет проник повсеместно
• Конфигурируется «магически»
• Текущая сессия – в контексте запроса
• Все классы лезут туда напрямую
• Вспомогательный статический
класс зависит от HttpRuntime

27. Ближайшие цели
• Не лазать в базу через контекст запроса
• Только по прямой ссылке
• Не лазать из методов доменных объектов
• Подготовиться к другому ORM

28. Решение
• Домен – в отдельный пакет
• Убрать логику из доменных объектов *
• Сделать обертку для сессии
• Передавать обертку в сервисы явно
* Уточнение будет позже

29. «Анемичная» доменная модель
• Модель без поведений
• Фаулер не одобряе:
• Нужен ОРМ
• Нет инкапсуляции и полиморфизма
• Скатываемся в процедурный стиль

30. Поведение в методе
obj.Publish() –
инкапсуляция
Publish() везде разный –
полиморфизм

31. Настройка под заказчика
• У разных клиентов – разные требования
к Article.Publish().

32. Command pattern

33. В среднесрочной перспективе:
• Убираем логику из доменных объектов
• Реализуем её командами
• Делаем контейнер домена

34. Запись данных гораздо
сложнее чтения

35. CRUD головного мозга
http://cthulhucrochet.blogspot.com/2010/09/time-to-switch-to-garlic-shampoo.html

36. CQRS
• Command-Query Responsibility
Segregation
• Грег Янг
• Начнем с разделения чтения и
записи

37. Стратегия разделения на куски
? ??

38. Паттерн vs. Антипаттерн
Попробуй Никогда
решить так
вот так не делай!

39. Вопросы, пожалуйста
Андрей Майоров
BYTE-force
xor@byte-force.com
twitter.com/xorets

40. addconf.ru

41. Пожалуйста, поставьте
оценку моему докладу.
Ваше мнение очень важно.
Спасибо!