"Ітогі підведьом" - семинар по легаси

[singalen]

Пришло шестеро :) Отличное количество для поговорить, имнсхо %)

Я оцениваю свою работу на 3+/4-.

Подготовился неважно, но основное принёс и дал :) Основное - это:

• словари запахов кода,
• тестов,
• и паттерны страданий - типичные проблемы при работе с легаси (см. часть вторую содержания, раздел "CHANGING SOFTWARE").

У меня эти полезные списочки переведены на русский и распечатаны на раздатках. Не уверен, что это законно с т.з. авторских прав - надо курить лицензии.

1. (игра: 15 мин) колупаемся в грязи: различи запах кода.

25%. Попробовали распознавать только на трёх самых простых примерах.
Надо подобрать материалы получше.

2. Вопросы и ответы Майкла Физерса.

Тоже охватили где-то на 25%. Ну, может, на 40%.
Тему целой книги, конечно, наскоком не проработаешь, но обсудить обсудили. Надеюсь, что слушателей заинтересовал :)

4. Наши приёмы и инструменты.

0% - не дошли мы до толковой практики за отсутствием хороших образцов. Тренировать практические приёмы, хотя бы и call tree, надо на хороших примерах с чётко поставленной задачей.
Примеры я скачивал первые попавшиеся, так что увы мне. Над этим напралвением ещё будем работать.

6. Почему мы пишем легаси код.

70% - поговорили нормально.

Запахи тестов

50%. Нашёл самый удачный вариант - взять примеры из самой книги, и предложить слушателям определить, который относится к какому запаху. Беру на вооружение на потом.
Практика оппа - не оставалось ни времени, ни музыки, ни сил.

7. Предложи рефакторинг. Эстимируй его.

0%. Не было хороших примеров, и не дошли. Но все к тому моменту подустали, так что и нормально.

Спасибо тем, кто пришёл %))) Будем продолжать - пара тем осталась совершенно нераскрытыми.

Comments

principles

[arumad.livejournal.com]

завдання назвати тут запахи мене кидає в транc,
а знайти 3 порушені принципи - легко:
public bool LoginUser(string userName, string password) {
CentralizedLoginService service = new CentralizedLoginService();
return service.ValidateUser(userName, password);
}
потирено тут.
тому принципи мені цікавіші - важливіші та легші.
от і продовжив коротку тему з work-shop`а.
а про принцип Деметри написав, бо тоді намагався, та забув назву.

Re: principles

[singalen]

Транс - это дело привычки. Кто что первым усвоил, то ему и кажется более естественным.

Можно говорить и на том, и на том языке. Они частично пересекаются, но это неважно.

"Запахи" мне нравятся больше, потому что они собраны в компактный список, описаны кратко и чётко. Поэтому им проще быстро научить.

Принципы - это для тех, у кого есть мозг и время прочитать хотя бы десяток таких вот вумных авторов типа Мартина %)

А, к слову.
Мне не нравится open-closed, потому что он плохо работает внутри одного приложения. Когда ты в доме хозяин, лучше оставить модуль простым и текстуально изменяемым, чем имплементить разъёмы под наследников, лиснеры, адаптеры и прочие удлинители и тройники.

Re: principles

[arumad.livejournal.com]

загалом згоден. далі можна не читати (:
я мало читаю книжок по програмуванню, і віддаю перевагу тонким.
блін, спалився, у тебе ж написано "авторів".
тоді цілком згоден ):

принципи - ієрархічні (що теж принцип), домінативні та повні.
твій приклад заодно приклад повноти (:
проти одного OCP є два - YAGNI extends KISS.
і вибрати осмислено правда неможливо ):

"А, к слову."
про рекурсію проти ітерації сказав Ола Біні.
це із JRuby; автор Ioke; член-експерт по java-dyn та MOP.