Штурмуем Legacy code

[singalen]

Готовлю тренинг по легаси коду. Собираю из черновиков, дёргаю куски из Майкла Физерса.

Объявил на внутренней рассылке доклад-дискуссию. Спасибо конторе, что прижала к стенке и поставила дедлайн. Теперь из этого что-нибудь, да выйдет :D

Кто готов поделиться хорошими примерами? Чтобы характерный пример - на страничку-две, и к ней ещё можно было нарисовать диаграммку классов штук на 5-10-100 :D
Нарисую сам, вы только дайте.

Сорсфорж — конечно, годная тема, но там мало близких нам предметных областей (тупого CRUD-а, то есть).

Comments

[catty-ua.livejournal.com]

Расскажи им про Alt+F7 в Решарпере :)
Это первое, что они должны усвоить )

[singalen]

Поиск уже в каталоге приёмов - как usages, так и полнотекстовый, по любому говну. И последующая медитация.
Это ж find usages, как и в Идее?
А дерево вызовов он таки не строит? Эта тулза ещё получше будет.

[catty-ua.livejournal.com]

Ага, поиск он самый
Примитивная вещь, но почему-то до сих пор часто приходится наглядно объяснять необходимость :)

Дерево - нет вроде...но группирует красиво )

У меня есть забавный пример из собственной практики (.NET). Можешь рассказать, пусть порадуются )

Когда была молодая и горячая, под горячую же руку попался унылый легаси код, где все контролы на кодбихайнде aspx.cs были какого-то хрена public. Возмутившись аццки (это оскорбляло самые прекрасный чувства об инкапсуляции), контекстной заменой сделала из них protected.

Ну вот скажи, что оно должно было сломать ? ;)

Оказалось, у нас был кусок функционала, который тестировался раз в полгода, он использовал рефлексию и вынимал только паблик поля.

Кретинизм, конечно, там можно было без этого обойтись, но не все индусы, которые написали легаси код, мыслят концептуально так же, как и ты...

В итоге неделя разбирательств, 2 выходных отладки - и вуаля - нахожу свои прекрасные изменения 3х месячной давности..

Тут не спас бы Альт + Ф7 никак.

Мораль ? Ну прежде всего - такая, что с легаси кодом 100% уверенности быть не можеит, даже когда всё якобы очевидно )

[beskov.livejournal.com]

а какой навык формирует данный тренинг?
тренинг по legacy code — это как имитировать унаследованный код? как его править, чтобы ничего не развалилось? как восстановить архитектуру? как переписать критичные куски?

[singalen]

Эта мораль малополезна, на самом деле. Волка бояться - в лес не ходить.
Можно было бы назвать полнотекстовый поиск, но он не спас бы и здесь.
Как подсказываает КО, юнит тесты.

Нормальный паттерн, кстати - работать с публичными полями. Coding by convention, у нас в системе сейчас вообще всё так делается - и получается отличный код.

[singalen]

Спасибо, правильный вопрос :)

Что должен уметь слушатель по окончании - у меня в плане записано. Примерно так:

Определять три-четыре вида легаси, исследовать его , безопасно менять и принимать решения, как его править для своей задачи. И нет, до архитектуры мы доходить не будем.

Не так, что "до" слушатель не умел перечисленного, а "после" вдруг стал - но дать несколько приёмов по каждому пункту.

И пункт "как писать легаси код" у меня тоже есть. Скорее как шутка, но это ж дело такое.

Нормальные цели?

[green-serpent.livejournal.com]

Softlab :-)

[singalen]

Ты знал!

[singalen]

Я, к счастью, не оставил у себя образцов.

[green-serpent.livejournal.com]

а то!
Я его даже суппортил, местами :-)

[catty-ua.livejournal.com]

Наверно.
Я всего лишь к тому, что надо перепроверять всё в 2 раза больше, чем с обычным кодом. А про паттерны не надо со мной спорить, я не в состоянии.

[singalen]

В смысле?

[beskov.livejournal.com]

ну мне лично — смутно )
что такое, например, «исследовать легаси» как навык? что является ценным результатом этой деятельности?

[singalen]

Владение конкретными приёмами, например:
* строить и читать дерево вызовов;
* считать coupling на пальцах - параметры конструктора и обращения методов к полям;
* вызывать в тестах методы с заведомо неправильными данными и смотреть, что сломается;
* "бинарный поиск" скрытого функционала через комментирование кусков кода;
* "бинарный поиск" ошибок по истории сорсконтрола;

Элементарные вещи, но люди о них не думают.

Ну и ещё один резулльтат тренинга - выплакаться о своём проекте, узнать, что ты не один... социализация!

[singalen]

Кстати, ещё одна цель - оформить у слушателя словарь, которым можно описать его легаси. Обычно его описывают экспрессивно, но неинформативно.

Я не припоминаю, чтобы кто-то осмысливал проблемы кода через:
* количество моков и partial-моков в каждом тесте;
* длину среднего сетапа теста;
* количество моков, нулов и полноценных объектов, необходимох, чтобы инстанцировать класс;
* банальный branch coverage.

А чтобы их кто-то мерял - вообще ни разу не слышал.