Считается ли класс большим, если он реализует 10 интерфейсов? Но при этом, в итоге должна остаться вся функциональность. Кол-во строк порядка 1.5 тысячи. Данный класс является базовым для других.
2 ответа
Как я понимаю, ваш BaseGrid
— это View
с некоторыми лишними зависимостями, от которых неплохо бы избавиться. Если вы не против потенциально деструктивных изменений, то развязать это дело можно примерно следующим образом:
- Намного удобнее, если модель реализует
IObservable,
а все виды являются ее подписчиками. Таким образом модельpush'ит
эвенты, в ответ на которые происходит обновления вида. Делать какой-то видObservable
(да еще иObserver'ом!
), на мой взгляд, не очень классное решение.
Сюда же видимо идет и
IGridNotify,
от которого можно было бы избавиться, сделав модельObservable
.
-
ISupportNavigator
(с семантикойIViewWithPages
) кажется мне хорошим интерфейсом дляView.
-
ISupportLoadBehavior
на самом деле является деталью реализации. То есть, вы можете сказать вашемуView
"загрузи мне вот такую-то модель" и вам, в принципе, должно быть все равно, как этотView
осуществит ее загрузку. А дальше уже разные поведения вида можно моделировать, например, с помощью сабклассинга типаBulkLoadingView
иViewThatLoadsEverythingSequentially.
-
IAccessible
— это либо неудачный выбор имени для интерфейса, либо результат нехорошего дизайна:
Обычно примерное назначение интерфейса должно быть понятно из его названия. Круто, если для каких-то уже известных сущностей вы сможете с большой степенью точности сказать, попадают ли они под слово, описывающее интерфейс, или нет. Вот, например,
List<string>
— онIAccessible
или нет?Если
IAccessible
нужен только для того, чтобы сообщить вашему виду, что он нацепляется на такую-то модель, то выкиньте этот интерфейс, передавайте модель (IModel / Model / SpecificModel
) через конструктор и заинкапсулируйте логику работы с моделью внутри вашего класса.
- Задачи
IFilterWindow
иISubFilterContainer,
на мой взгляд, более изящно решаются с помощью прокси-моделей:
То есть, вместо того, чтобы делать логику фильтрации частью вида, вы можете на каждый запрос-фильтр создавать новую фильтрующую модель и нацеплять ее на ваш вид. Такой подход проще тестируется и обладает гораздо меньшей связностью.
IFilteringModel(IModel, FilterQuery, AdditionalFilterQueries) : IModel
Вот, скажем, сколько изменений в вашем текущем подходе придется внести, чтобы делать фильтрацию на уровне базы данных, а не на клиентской части? В варианте с
IFilteringModel
нужно будет всего лишь добавить еще одну прокси-модель.
-
Не смог сообразить, что такое
IWorspace
(IWorkspace?
), и, собственно, зачем он здесь нужен. -
Про
IGroupViewBehavior
тоже ничего не могу сказать, хотя из названия кажется, что можно его оставить. -
IIterableGrid
кажется не самым плохим интерфейсом, но, если подумать, то он него можно и даже нужно избавиться:
Вот, предположим, у вас есть модель и есть ее некоторый вид. Вы писали-писали какой-то код и в некоторый момент времени у вас появилось желание проитерироваться по вашему
Grid'у
и узнать, какие элементы в нем сейчас отображаются. Почему не спросить эти элементы напрямую у модели, а не у какого-то ее вида?Лично я вижу две возможные причины, почему вы могли захотеть спрашивать данные напрямую у
View
— либо между элементамиView
и элементами модели нет взаимно-однозначного соответствия, либоView
нацепил поверх модели какую-то дополнительную информацию, к которой хочется получить доступ, но которой нет в модели.Первый кейс решается с помощью прокси-моделей (см. выше). Второй кейс также легко решается переносом всей информации, которую нацеплял
View
, в модель — ту же самую или расширенную (включающую в себя все данные из старой модели+
что-то еще). Опять же, в этом варианте мы резко расширяем свое возможности в плане тестирования.
-
Если все вышенаписанное соединить вместе, то получится вот такой вот дизайн (комментарии и критика привествуются)
:
public class BaseGrid : BaseWindow, IModelObserver, ISupportNavigator [IViewWithPages], IGroupViewBehavior(???), IWorkspace(???) { public BaseGrid(IModel model, ...) { ... } } public interface IModel : IModelObservable { ... } public interface IFilteringModel: IModel { ... }
-
Отлично, отделение представления от модели должно [помочь] распутать клубок. Я сразу и не сообразил, что класс, имплементирующий интерфейс
ISupportLoadBehavior
, может относиться ко View.– VladD7 фев 2013 в 17:59 -
@VladD Ну, я тоже мог не угадать, хотя мне кажется, что здесь речь идет именно о всемогущем
View
в стилеWinForms.
7 фев 2013 в 18:02
Если классу действительно никак "не обойтись" без всех этих интерфейсов и их, возможно, множественных реализациях, то можно разбить класс на несколько файлов. Ключевое слово partial вам в помощь.
А вообще не хорошо так раздувать классы. Возможно, ваш класс представляет из себя "кучу хлама", в которую вы просто-напросто свалили все, что у есть в приложении. Проверьте это. Ведь если это действительно так, то было бы целесообразно навести порядок, реализовать несколько классов, предназначенных для разных целей.
God Object,
который логично будет распилить. То есть, применение фильтров и перебор значений, скажем, еще укладываются в одну сущность, а вот оповещение кого-то там -- это уже точно другая область ответственности.