MVVM, ObservableCollection не обновляет View

@siriusED · Регистрация: 20.05.2010

Студворк — интернет-сервис помощи студентам

Привт, есть коллекция итемом меню

C#
1
        public ObservableCollection<MenuitemModel> MainMenuItems { get; set; }

В конструкторе заполняю ее меню итемами

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
MainMenuItems = new ObservableCollection<MenuitemModel>()
{
    new()
    {
        Header = "File", Command = MainMenuItemClickCommand, Code = MainMenuCode.File
    },
    new()
    {
        Header = "Edit", Command = MainMenuItemClickCommand, Code = MainMenuCode.Edit
    },
    new()
    {
        Header = "Language", Command = MainMenuItemClickCommand, Code = MainMenuCode.Language
    },
}

Ну и теперь если я в коде меняю например вот так:

C#
1
2
MainMenuItems[2].Header = "Changed"; // Меняю титул пункта меню
OnPropertyChanged(new PropertyChangedEventArgs(nameof(MainMenuItems))); // Оповещаю View что коллекция поменялась

То все итемы меню как были так и остались все хедеры не меняются во обще, почему так? Это проблема коллекции или в чем может быть проблема?

@wizard41 · 03.04.2024, 13:18

Элд Хасп, ну Ок. Для некоего "общего" случая не скажу, но в абсолютном большинстве проектов, которые делать приходилось именно мне, использование авто-свойств сводилось к минимуму, т.к., как правило, в своих моделях эти свойства устанавливаются только "внутри", т.е. private set. Плюс к этому, многие свойства в добавок обладают некоторой доп. логикой, например, установить другое свойство, зависящее от этого...
Может возникнуть ситуация, что внедрение в "это все" функционала Fody окажется уже не совсем "мало-кодовым" вариантом, и в таком случае все же уместнее будет самому реализовать INPC со всеми необходимыми перегрузками.

@Элд Хасп · 03.04.2024, 13:41

Сообщение от wizard41

т.к., как правило, в своих моделях эти свойства устанавливаются только "внутри", т.е. private set

Можно задать приватный сеттер автосвойству - это не является припятсвием.

Сообщение от wizard41

Плюс к этому, многие свойства в добавок обладают некоторой доп. логикой, например, установить другое свойство, зависящее от этого...

А вот это плохо.

Лучше вынести создание зависимостей в отдельный метод.
Например, в моей реализации Простые реализации для тем на форуме это метод OnPropertyChanged. Во всех фреймворках (в том числе в PropertyChanged.Fody) есть аналогичные методы. В Fody? вроде, ещё были какие-то атрибуты для методов зависящих от свойств. Можно их использовать. Но только не тело свойства вписывать создание зависимостей.

@wizard41 · 03.04.2024, 13:48

Элд Хасп, да, это просто "мысли в слух". Все еще очень зависит от самой задачи: если у тебя два-три класса и достаточно много собственной логики взаимодействия между ними, то встраивание Fody может оказаться каплей в море, т.е. особой выгоды не принесет. Пару десятков строк самописной реализации INPC погоды не сделают вообще.
Другое дело 50 классов и 2000 свойств, вот тут уже да, кое-что сократится.

@dev3214 · 17.04.2024, 12:05

Сообщение от Элд Хасп

Или чуть более сложные в реализации, но упрощающие применение отсюда: https://github.com/INexteR/NewTrade... ViewModels

Какие логины-пароли для разных ролей?

Какая степень правильности выполнения проекта по 5-ти бальной системе?
Если условно...
Я понимаю, что, если каждый будет смотреть проект, то у каждого будут свои замечания различной степени тяжести..
Интересует ваш взгляд...
Шкала:
- 1 - проект выполнен неправильно;
- 5 - проект выполнен идеально;

@Элд Хасп · 17.04.2024, 15:21

Сообщение от dev3214

Какая степень правильности выполнения проекта по 5-ти бальной системе?

Какого?
Того на который я ссылку GitHub дал?

@dev3214 · 17.04.2024, 15:37

Сообщение от Элд Хасп

Какого?
Того на который я ссылку GitHub дал?

Да.

Из сообщения мне показалось, что вы хорошо знакомы с проектом..

Ваша ссылка: https://github.com/INexteR/New... ViewModels
Полный проект: https://github.com/INexteR/NewTrade

@Элд Хасп · 17.04.2024, 16:07

Сообщение от dev3214

Из сообщения мне показалось, что вы хорошо знакомы с проектом..

Да. Но это в прошлом.
Много времени утекло - уже много не помню.
Проект делался, насколько помню, с целью демонстрации разделения функционала по слоям, для чего использовалась реализация каждого слоя в отдельном проекте.
Что касается самой логики, то на неё особого внимания не обращалось. Поэтому огрехов будет много.

Само обсуждение Решения во время его создания, должно быть здесь в разделе в какой-то теме.
Надо поискать.

Добавлено через 3 минуты
Посмотрите среди тем созданных INexteR

@melisey · 13.05.2024, 14:15

Сообщение от Элд Хасп

Для извещения об изменении свойства должна быть реализация интерфейса INotifyPropertyChanged в элементе коллекции, в вашем случае

Что делать в случае, если элемент коллекции ObservableCollection реализует INPC, свойства этого элемента используют INPC, а обновления DataGrid во View всё равно не происходит?..

Вообще в принципе как организовать вывод данных в табличном виде с обновлением View (при изменении свойств элемента коллекции)?

@xellan24rus · 13.05.2024, 14:21

melisey, указать двухстороннюю привязку и UpdateSourceTrigger

XML
1
Text="{Binding Path= Data, Mode=TwoWay, UpdateSourceTrigger=PropertyChanged}"

А вообще покать хотя бы код для начала. Без примера кода на какой ответ вы рассчитываете?

@Элд Хасп · 13.05.2024, 14:21

Сообщение от melisey

а обновления DataGrid во View всё равно не происходит?..

Надо дебажить.
Где-то что-то напутано.

Закиньте решение (минимальный воспроизводимы пример) на GitHub и дайте ссылку.

По моему опыту разбора подобных ошибок их топ: неверная реализация INPC и путаница в экземплярах VM.

@Andrey-MSK · 13.05.2024, 14:22

Сообщение от xellan24rus

Text="{Binding Path= Data, Mode=TwoWay, UpdateSourceTrigger=PropertyChanged}"

Сообщение от melisey

а обновления DataGrid во View всё равно не происходит

???

@xellan24rus · 13.05.2024, 14:23

Сообщение от Andrey-MSK

???

Как я понял автор из кода пытается менять значение, а во view ничего не меняется. На основе этого варианта и прикрепил часть кода

@Элд Хасп · 13.05.2024, 14:25

Сообщение от xellan24rus

а во view ничего не меняется. На основе этого варианта и прикрепил часть кода

Mode=TwoWay было бы актуально, если наоборот из View в VM значения не передавались.
А здесь противоположная ситуация.

@Andrey-MSK · 13.05.2024, 14:28

Сообщение от xellan24rus

Mode=TwoWay

Это режим привязки по умолчанию для объектов похожих на TetxtBox.

Сообщение от Andrey-MSK

UpdateSourceTrigger=PropertyChanged

Это позволяет отображать изменённое значение во время ввода.
Не знаю как оно ТС поможет...

@xellan24rus · 13.05.2024, 14:43

Сообщение от Элд Хасп

А здесь противоположная ситуация.

Тогда

OneWay

должно быть актуально.

Сообщение от Andrey-MSK

Не знаю как оно ТС поможет...

Пока что даже кода нет, сейчас остается только гадать что поможет

@melisey · 13.05.2024, 14:52

Сообщение от xellan24rus

А вообще покать хотя бы код для начала. Без примера кода на какой ответ вы рассчитываете?

xellan24rus , Элд Хасп, Andrey-MSK в ближайшее время создам воспроизводимый пример и закину его на GitHub. Спасибо за отклики.

Не хотел плодить ещё одну тему, но похоже придётся

Новые блоги и статьи Все статьи Все блоги /
Беседа с ИИ о программистах, недопускающих к созданию и правке кода генеративные ИИ и причины этого zorxor 21.09.2026 Раньше я радовался или получал некоторые эмоции, пусть небольшие, но всё же, от самого процесса написания кода, рекомпиляции и запуска, видя постепенное развитие программы и прочее. А теперь лень. . .	Мобильное приложение ColorStep pavlinmavlin 17.09.2026 Реализовал приложение Красный, Зеленый, Синий в Unity3d + c#. Название изменил на ColorStep. Приложение прошло модерацию и теперь доступно для скачивания. Делал его сам, шаг за шагом — и вот,. . .	Запрет дублирования строк в табличной части Maks 13.09.2026 Реализация из решения ниже выполнена на нетиповом справочнике "Нормы ТО" с табличной часть "Виды ТО", разработанного в КА2, со следующими реквизитами: - ВидТО (СправочникСсылка. ВидыТО); - ВидГСМ. . .	Скрипты Tampermonkey для CyberForum, ChatGPT, Claude и пр. Jin X 06.09.2026 Скрипты Tampermonkey для CyberForum, ChatGPT, Claude и пр. Работая с форумом и нейросетями в браузере часто хочется что-то подкорректировать или добавить какого-то функционала. Ниже прикреплён. . .
Программа опроса у.з. расходомера SLS-720F Argus19 02.09.2026 Программа опроса у. з. расходомера SLS-720F Программа опрашивает один раз в минуту три ультразвуковых расходомера SLS-720F через интерфейс RS-485 по протоколу Modbus RTU. Опрашиваются регистры. . .	Hyper-V: Компьютер должен поддерживать доверенный платформенный модуль 2.0. Maks 31.08.2026 При установке Windows 11 на виртуальную машину Hyper-V 2-го поколения вылезла такая ошибка: Решение: в параметрах виртуальной машины, в разделе "Безопасность" (Security) активировать флаг. . .	Архитектура биовида Стива в Майнкрафте: Зачем бонобо кубический каннибализм anaschu 30.08.2026 Кубический Вагинокапитализм в Minecraft: Математический инвариант ОДУ и рок Стивов-бонобо Главная задача разработанной «Модели Всего» — наглядно продемонстрировать наличие системной «судьбы». . .	Оттачиваю умение писать js программы. russiannick 30.08.2026 Проектом выходного дня стало написание Книги шифров Виженера. Итогом стала версия 200, синий туман. Синий туман назван так, потому что замораживает текст под собой. Нажатие синих кнопок управляют. . .

@wizard41 3463 / 2784 / 575 Регистрация: 04.09.2018 Сообщений: 8,757 Записей в блоге: 3
	03.04.2024, 13:18
	Элд Хасп, ну Ок. Для некоего "общего" случая не скажу, но в абсолютном большинстве проектов, которые делать приходилось именно мне, использование авто-свойств сводилось к минимуму, т.к., как правило, в своих моделях эти свойства устанавливаются только "внутри", т.е. `private set`. Плюс к этому, многие свойства в добавок обладают некоторой доп. логикой, например, установить другое свойство, зависящее от этого... Может возникнуть ситуация, что внедрение в "это все" функционала Fody окажется уже не совсем "мало-кодовым" вариантом, и в таком случае все же уместнее будет самому реализовать INPC со всеми необходимыми перегрузками. 1

@wizard41 3463 / 2784 / 575 Регистрация: 04.09.2018 Сообщений: 8,757 Записей в блоге: 3
	03.04.2024, 13:48
	Элд Хасп, да, это просто "мысли в слух". Все еще очень зависит от самой задачи: если у тебя два-три класса и достаточно много собственной логики взаимодействия между ними, то встраивание Fody может оказаться каплей в море, т.е. особой выгоды не принесет. Пару десятков строк самописной реализации INPC погоды не сделают вообще. Другое дело 50 классов и 2000 свойств, вот тут уже да, кое-что сократится. 1