Главная страница
Top.Mail.Ru    Яндекс.Метрика
Текущий архив: 2008.02.17;
Скачать: CL | DM;

Вниз

Вопрос к Мастерам: как рефакторить такой код.   Найти похожие ветки 

 
Черный Шаман   (2008-01-15 21:27) [0]

1. Программа представляет собой одну форму TMainForm с глобальным объектом MainForm.
2. Создано множество модулей с классами типа TControlCenter в которых в методах явно вызывается MainForm.объекты
3. Модули ссылаются друг на друга, причем все завязано на визуальные элементы MainForm.

При этом задание стоит полностью изменить визуальный дизайн приложения(на внутреннюю технологию скинов), но логика программы слишком завязана на конкретные визуальные элементы.

Пока есть такой вариант - создать в SVN новую ветку и создавая с нуля приложение потихоньку переносить в него старый код интегрируя с новый и убирая "лапшу".

Может есть способ получше? Все таки архитектором крупных проектов я не был, так что здесь есть люди поопытнее меня.


 
ketmar ©   (2008-01-15 21:38) [1]

создать — оно, конечно, хорошо. но я бы предложил сначала сесть, и нарисовать на листочке модель. а то чего там рефакторить, если не ясно, кто к кому цепляться должен, чтобы потом и с новым такая же фигня не получилась.


 
ketmar ©   (2008-01-15 21:39) [2]

а, да. извиняюсь за вмешательство, если что — у меня ж штаны не того цвета.


 
Alien1769 ©   (2008-01-15 21:42) [3]

ба, кетмар анкету поправил

зы сорри, за оффтоп


 
www   (2008-01-15 21:46) [4]


>  у меня ж штаны не того цвета.

это ты про

> кто к кому цепляться должен
?


 
Черный Шаман   (2008-01-15 21:55) [5]


> www   (15.01.08 21:46) [4]
>
>
> >  у меня ж штаны не того цвета.
>
> это ты про


Думаю, про значек :)


 
ketmar ©   (2008-01-15 22:12) [6]

>[4] www (15.01.08 21:46)
это я про вопрос к «Мастерам», коим я не являюсь за неимением верных штанов.


 
Alien1769 ©   (2008-01-15 22:18) [7]

дык ветка была про значки :))


 
ketmar ©   (2008-01-15 22:27) [8]

>[7] Alien1769 ©(15.01.08 22:18)
вообще-то я всего лишь усиленно намекаю на криво сформулированый сабж.


 
Kolan ©   (2008-01-15 22:32) [9]

Рефакторятт всегда одинакого — почуть чуть&#133

А какие методы использовать? Первое надо разорвать двунаправленые связи.
Есть такой рефакторинг Change bidirectional relationship to unidirectional(примерно так, могу точнее написать)&#133 Вот с его и начни.


 
Kolan ©   (2008-01-15 22:32) [10]

Change Bidirectional Association to Unidirectional



Страницы: 1 вся ветка

Текущий архив: 2008.02.17;
Скачать: CL | DM;

Наверх




Память: 0.49 MB
Время: 0.017 c
1-1194779709
mufan
2007-11-11 14:15
2008.02.17
TreeView.Items.Clear не работает???


2-1201260171
Леонид
2008-01-25 14:22
2008.02.17
Как блокировать сообщение об ошибке


2-1201455070
Бэтман
2008-01-27 20:31
2008.02.17
ComboBox


2-1200991020
MNNN
2008-01-22 11:37
2008.02.17
Скопировать с языком


6-1180410284
Dmitry_177
2007-05-29 07:44
2008.02.17
Перехват трафика