• Почему в модели фигурируют какие-то окна

• Id и Index - это разные понятия

• Если в гармонику будут добавлены еще какие-то параметры (цвет, запах, вкус), то в IMainWindow (ну или как он будет называться) будут добавляться всё новые и новые методы. И реализация превратится в God Object. Исправить

• Что делает метод void GetHarmonicById(int id)?

• Как у модели узнать, какая из гармоник в данный момент выделена?

• Класс Harmonic (объект модели) не должен заниматься представлением в виде, понятном пользователю.

• get-Свойства ShapeArgumentsHandler не должны иметь побочных эффектов. Сделайте в виде метода

• Убрать сеттеры. Изменить поведение полёта утки должно быть возможно только у Model Duck

• Если отменить команду вставки картинки, а затем выполнить модификацию документа, то картинка должна удалиться, т.к. она удаляется как из документа, так и из истории отмены команд

• Эта ситуация должна быть проверена в тестах

• Не надо глушить исключения в адаптере

• Упростить метод RemoveObserver

• Устранить дублирование кода между тестами (строка состояния)

• Сейчас реализация вставки фигуры в листовой объект не позволяет вызывающей стороне узнать, произошла вставка или не произошла

Наличие метода IsComposite плохо тем, что если клиент забудет вызвать этот метод, то будет работать с простой фигурой как с составной, хотя некоторые методы нарушают контракт родителя. А раз проверять всё равно нужно (хотя бы для Add), то лучше сделать метод GetGroup, который возаращает null или ссылку на группу

• Отсутутсвует метод для получения фигуры по её индексу