Замечания по MVVM

alexey-malov commented 5 years ago

[x] В текстовые поля добавляется 0, если стереть текст и начать вводить новый
[x] HarmonicsSum.GetPoints вычисляет значения, начиная от 0. Возврат списка - тоже сомнительное решение

alexey-malov commented 5 years ago

[x] Почему уведомление об изменении "StringRepresentation" шлётся только из сеттера HarmonicType, а через другие сеттерны нет?

alexey-malov commented 5 years ago

[x] Выставляя наружу ViewModel интерфейс BindingList вы обязаны поддержать все типы модификаций, которые с ним могут выполнять клиенты

alexey-malov commented 5 years ago

[x] Определени доступности гармоник для редактирования должно находиться внутри ViewModel

alexey-malov commented 5 years ago

[x] MainForm достаточно объемным получился. Подумайте, как разбить его на независимые части

egoshin-igor commented 5 years ago

alexey-malov commented 5 years ago

alexey-malov commented 5 years ago

[ ] IListBoxViewModel -> переименовать, чтобы название имело отношение к содержимому

alexey-malov commented 5 years ago

alexey-malov commented 5 years ago

alexey-malov commented 5 years ago

alexey-malov commented 5 years ago

[ ] Строка "StringRepresentation" и аналогичные привязки используются в куче мест. Легко совершить ошибку

alexey-malov commented 5 years ago

[ ] Сделать так, чтобы Chart, отображающий сумму гармоник отображался не только внизу, но еще и сбоку. Устранить дублирование кода

alexey-malov commented 5 years ago

egoshin-igor / OOD