XDiaBLo,
1. Если кружить по моему примеру, то можно еще сказать:
Добавление нового типа файла состоит следующих действий
(1) добавление нового производного класса для новой стратегии (если нужна новая)
(2) корректировка функции Build
(3) если старая стратегия формирует новый тип файла и возможно нужно еще код ее поправить.
В принципе если видно две стратегии из 7, то нужно и делать две, вот пример Build:
1.
2.
3.
4.
5.
6.
7.
8.
9.
10.
11.
12.
class StrategyBuilder
{
DataTransfer Buld(filename)
{
...
// если новый тип файла, а стратегия уже написана, добавляем в условия
// для различения типов фалов, передаем строку ext в конструктор если надо.
if (ext == "txt" || ext = "dat" || ext == "bat") return new MyDataTransfer1(ext);
if (ext == "doc") return MyDataTransfer2();
return null ; // неизвестный тип файла
}
}
2. Меня не очень улыбнули еще такие вещи:
a) сам класс Data - вроде по названию он сущностный, а там сидит функция Data для загрузки данный, да еще по ветвящему алгоритму, т.е. два в одном...
Я считаю что классы должны выполнять свои обязанности: кто данные хранит, кто алгоритмы выполняет. Объединять стоит, если классы маленькие получаются и нет смысла в разбиении.
б) И сама фунция Data:
Это:
1.
2.
3.
private Vector columnNames;
private Vector hiddenColumnNames;
private Vector hiddenColumns;
+ это:
1.
public Data(RegistryFileType fileType, Vector columnNames, Vector hiddenColumnNames){}
Весьма странная комбинация, как-то не по ООП-ному. Попытка через функцию передать массивы.
Тут тоже нужно пошерстить. Похоже код написан вообще по процедурному мышлению.
То что я привел может и не подойти или подойти частично, а вообще можно много разных вариантов предложить, все упирается в конкретную задачу, которую автор сообщения описал в начале размыто и туманно, поэтому и ответов мало...
И еще добавлю: сам не владею сильно-то шаблонами, т.к. ООП нет практики сейчас, но скажу что прочитав недавно книгу не стоит сразу пытаться выбрать шаблоны для рефакторинга своих прог. Я тоже, изучив их, сразу ринулся применять шаблоны, как результат - переделывал по нескольку раз, и еще доводил до слез пользователей, т.к. после переделки не совсем хорошо тестировал. Тут лучше поднабраться опыта и применять их для новых программ.
Вообще еще помимо GoF, обязательно почитайте про шаблоны GRASP - очень важные принципы, которые помогут правилно мыслить при проектировании ООП систем.
