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

[singalen]

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

Comments

[profuel.livejournal.com]

звучит неплохо, особенно с учетом множества мыслей в голове — как минимум тебе море пищи для измышлений по своему проекту будет :)

[singalen]

А чё по своему проекту. В своём, не поверишь, у нас легаси практически нет ;D
Как-то так получилось, что мы его хорошо написали :D :D :D

copyrigth

[arumad.livejournal.com]

ти переклав і роздав вільно-доступні матеріали.
а приклади кода ти друкував для власного використання,
якщо конкретно - щоби під піцу підкладати.
книжку саму ти теж не давав.
ще й радив ўсім її почитати.
все законно. без сарказму.

Re: copyrigth

[singalen]

ОК, что, если я теперь хочу сделать сайт с упражнениями? :)
Тут всё становится более туманным.

Re: copyrigth

[arumad.livejournal.com]

як ти відповів раніше ніж я пошту дочитав??
згоден. без підтримки його відвідувачів читачів заважке діло.
1) якщо нема книжного коду - нема троблем.
1.1) твій початковий підхід із open source legacy code із надихає (:
подвійний виграш, потрійна робота.
2) стандартні cop e wright кажуть "не дозволено (пере-)друкувати цілком або частинами".
2.1) зі часів atari відомо, що jawbreaker не pacman, бо зображення інші.
хоча це й гиденько.
2.2) там купа виключень (найбільше - ґуґл), включно з ознаёмленням та популяризацією.
дурнувато виглядає, всі цим користуються.
постав банер - питання закрите.

про зв’язок із програмерськими принципами.
очевидно, що я занадно лінивий, аби цим зайнятися.
очевидно, що хтось ні, хоча й зовсім не націлено (:
огляд тої ж убер книги
і той принцип, що назву не згадав - Деметри (LoD),
ў статті відсутній теж.
в принципі він є випадком інкапсуляції...
загальна діаграма принципів та шаблонів -
взагалі мабуть непідйомна задача.

Re: copyrigth

[singalen]

Я немного потерялся в твоих идеях :)

Ответил я быстро, потому что мне ЖЖ кидает нотификации в джаббер.

Без книжного кода обойдёмся. Даже если делать свой не через замену картинок, а полностью составлять от руки.

А дальше недопонимаю, к чему это ты.

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.