Приложение MVVM. VMs и навигация

Здравствуйте.
Имеются следующие вопросы.
Как при соблюдении mvvm передать в VM свойства view такие как SelectedItem? Знаю, что можно передавать через параметр команды, но в моем случае за раз в VM нужно отправить 2 объекта - логин и пароль (в модели у меня нет этих свойств, соответственно, я не должен определять их в VM. Хотя многие так делают, но это всё-таки нарушение). Нашёл решение с помощью применения MultiBinding для параметра, но мне кажется, что это слишком громоздко, так как при таком подходе каждый раз придется реализовывать конвертер. Вообще, говоря, очень трудно приходится, когда в VM нет нужных свойств, от этого обстоятельства не выходит отключать кнопку (альтернативой может служить триггер на свойство IsEnabled во view, но от этого снова приходится прикладывать больше усилий из-за описания стиля и, возможно, конвертера. Элд Хасп подсказывал, что для этих целей можно сделать прокси класс, но я недоумеваю как и где. Было бы удобно, если бы по правилам mvvm в VM допускалось хранить все необходимые свойства, а не только те, которые имеются в модели.

По той же причине не стал помещать в VM команду для кнопки, которая предназначена для входа в роли гостя. Пока что в коде view подписал хелпера на щелчок по ней. В результате просто должен произойти переход к другому view, это не логика модели. Сама навигация вызывает дополнительный вопрос, нужны рекомендации по её реализации.

Также можно отдельно рассмотреть модель. Правда там пока что немного кода. Важны все замечания.

Ещё имеется dll библиотека с вспомогательными классами, применимыми к mvvm. По ней тоже можно сделать замечания.

Вот ссылка на репозиторий https://github.com/INexteR/NewTrade/tree/Trial. В нем также содержится документ с заданием.

Связанные темы Приложение MVVM, VMs и навигация: Создание Модели Авторизация. Приложение MVVM. VMs и фильтрация по значению свойства (выбрать по производителю). Приложение MVVM. VMs и поиск (фильтрация по текстовому значению). Приложение MVVM. Ограничение пользовательского ввода для численных свойств. Приложение MVVM. Представление авторизации - Валидация. Приложение MVVM. Контекстное меню (удаление, редактирование) для элемента (Товара) в ListBox. Приложение MVVM. Диалог добавления/редактирования товара: получение данных, вызов, взаимодействие с VM и прочее. Приложение MVVM. Быстрая "Нестрогая MVVM" реализация

Техническое Задание Авторизация При запуске приложения окно входа – первое, что видит пользователь. На ней пользователю предлагается ввести свой логин и пароль или есть возможность перейти на экран просмотра товаров в роли гостя. Только после удачной авторизации пользователь получает доступ к остальным модулям системы: • авторизованный клиент может просмотреть товары; • менеджер может просматривать товары; • администратор может добавлять/редактировать/удалять товары. Реализуйте необходимые интерфейсы для всех пользователей системы. После входа в любую учетную запись должна быть реализована возможность выхода на главный экран – окно входа. При переходе в любую учетную запись в интерфейсе (правый верхний угол) должны отображаться ФИО пользователя. После первой попытки неуспешной авторизации система выдает сообщение о неуспешной авторизации, а затем помимо ввода логина и пароля просит ввести captcha, состоящую из 4 символов (цифры и буквы латинского алфавита) и графического шума. CAPTCHA - должна содержать минимум 4 символа (буква или цифра), которые выведены не в одной линии. Символы должны быть либо перечеркнуты либо наложены друг на друга. После попытки неудачной авторизации с вводом captcha, система блокирует возможность входа на 10 секунд. Список товаров Очень важно постоянно получать актуальную информацию об остатках товаров на складах, поэтому необходимо реализовать вывод товаров, которые хранятся в базе данных, согласно предоставленному макету. При отсутствии изображения необходимо вывести картинку-заглушку из ресурсов (picture.png) . Вывод должен осуществляться списком. Обратите внимание, должна выводится вся информация о товаре из базы данных. Товары, которых нет на складе, должны отображаться на сером фоне. Пользователь должен иметь возможность искать товары, используя поисковую строку. Поиск должен осуществляться по всем доступным данным, в том числе по нескольким атрибутам одновременно. Пользователь должен иметь возможность отсортировать товары (по возрастанию и убыванию) по стоимости. Кроме этого, пользователь должен иметь возможность отфильтровать данные по производителю. Выбор производителя должен быть из выпадающего списка. Первым элементом в выпадающем списке должен быть “Все производители”, при выборе которого настройки фильтра сбрасываются. Поиск, сортировка и фильтрация должны происходить в реальном времени, без необходимости нажатия кнопки “найти”/ ”отфильтровать” и т.п. Фильтрация и поиск должны применяться совместно. Параметры сортировки, выбранные ранее пользователем, должны сохраняться и во время фильтрации с поиском. В верхней части окна необходимо показывать количество выведенных данных и общее количество записей в базе. Например, 15 из 37 . В случае, если данные в таблицу выводятся после фильтрации или поиска, количество выведенных данных необходимо обновить исходя из размера выборки. Реализуйте удаление товара. Товар, который присутствует в заказе, удалить нельзя. Товар, у которого есть дополнительные товары, удаляется из базы данных вместе с информацией о дополнительных товарах, если ни один из товаров (даже дополнительный) не заказан. Реализуйте переход на окно добавления товара и окно редактирования выбранного товара. Добавление/ редактирование товаров Необходимо реализовать возможность добавления и редактирования исходных товаров (сырья) для всей продукции компании. Редактировать данные может только администратор. Необходимо добавить возможность редактирования данных существующего товара, а также добавление нового товара в новом окне - форме для добавления/ редактирования товара. Переходы на данное окно должны быть реализованы из формы списка: для редактирования - при нажатии на конкретный элемент, для добавления - при нажатии кнопки “Добавить товар”. На форме должны быть предусмотрены следующие поля: наименование, категория (выпадающий список), количество на складе, единица измерения, поставщик, стоимость за единицу, изображение и подробное описание (с возможностью многострочного ввода) . Стоимость товара может включать сотые части, а также не может быть отрицательной. Минимальное количество также не может принимать отрицательные значения. При открытии формы для редактирования все поля выбранного объекта должны быть подгружены в соответствующие поля из базы данных, а таблица заполнена актуальными значениями. ID товара при добавлении не отображается, автоматически вычисляется +1 к имеющемуся в БД, при редактировании ID доступно только для чтения. Пользователь может добавить/ заменить изображение у товара. Изображение, которое загружает пользователь при добавлении или редактировании, должно сохраняться в папку с приложением. Для оптимального объема реализуйте ограничение на размер фото: 300Х200 пикселей. В базе данных необходимо хранить путь к изображению. При замене изображения, старое фото из папки должно быть удалено. Для того, чтобы администратор случайно не изменял несколько товаров, предусмотрите невозможность открытия более одного окна редактирования. После редактирования/добавления/удаления товаров данные в окне списка товаров должны быть обновлены.

0

Сообщение от Элд Хасп А где мы будем эту БД хранить?

Ну так мы же тасуем её как отдельный файл sqliteTest.db наравне с папкой решения. Это ли не хранение?

0

Сообщение от INexteR Ну так мы же тасуем её как отдельный файл sqliteTest.db наравне с папкой решения.

Не путайте Решение и Сборку.
Файл sqliteTest.db не включён в проект и не является частью сборки.
Кроме того, константное имя БД - это временное решение.
Потом можно будет задавать его из App или, вообще, из внешних настроек.

Так же - да, можно SQLite включить в сборку и дублировать его.
Но если вы будете использовать серверную бд?
Тогда вместо названия файла будет строка подключения.
А Code First для любой БД правильно создаст начальную базу.

0

Сообщение от Элд Хасп Так же - да, можно SQLite включить в сборку и дублировать его.

Просто я думал что мы используем SQLite для того чтобы избавиться от классов с тестовыми данными

Добавлено через 36 секунд
Ну хорошо, пусть будут

0

Сообщение от INexteR мы используем SQLite для того чтобы избавиться от классов с тестовыми данными

Не...
Это временное решение, ввиду того что без сервера я не могу проверять работу контекста БД.
Потом вы замените SQLite на нужную вам БД, и текстовые данные будут так же использоваться для её инициализации.

Добавлено через 2 минуты
Вариант с парсингом коллекций был неудобен тем, что требовал разного кода для работы с коллекциями и работой с БД.
Поэтому "простого переключения" от одного к другому не получалось.
Я бы отлаживал один код, а вы запускали другой - и неизвестно было бы где и как искать ошибку, если бы она возникла.
С SQLite снимается эта неопределённость.

1

Сообщение от INexteR Microsoft.Data.Sqlite.SqliteException: "SQLite Error 19: 'FOREIGN KEY constraint failed'."

Не учёл, что может быть два разделителя подряд для пустого поля.
В результате пропускались поля инициализации.
Фиксацию отправил.

0

Сообщение от Элд Хасп Не учёл, что может быть два разделителя подряд для пустого поля. В результате пропускались поля инициализации. Фиксацию отправил.

Спасибо.

0

INexteR, я там пару фиксаций отправил.

0

Элд Хасп, отправил фиксацию! Добавил необходимые интерфейсы типа ISuppliersSource, ICategoriesSource, но они только с одним методом получения коллекции, без события, так как счел его ненужным. Они загружаются только один раз перед товарами, а когда дело доходит до товаров, то все эти ранее загруженные коллекции прикрепляются к контексту. То есть ещё прикрепление реализовано в тривиальной манере. Также добавил окно AddOrUpdateDialog для соответствующих действий, но пока пустое. И ещё теперь есть команда удаления товара.

Возникли такие вопросы. Почему Вы в методах Add/Update передаете в контекст копию товара? Почему не сам объект? И почему в ProductsView в ListBox является выбранным первый элемент? У выделенного ListBoxItem фон обычно голубой, а тут он светло-серый. Я опознал этот элемент выбранным по тому, что при переходе к каталогу кнопка удаления включена. В разметке выбранный элемент не задан

0

Сообщение от INexteR Почему Вы в методах Add/Update передаете в контекст копию товара?

Потому что у нас приходит IProduct, а для CatalogContext нужен internal partial class Product, который может создать только сама Модель.

Добавлено через 3 минуты
Я сейчас доделаю маппер чтобы облегчить копирование и создание копий.
Потом дальше продолжу. А то появляется много дублирующего кода.

0

Сообщение от Элд Хасп Потому что у нас приходит IProduct, а для CatalogContext нужен internal partial class Product, который может создать только сама Модель.

А нельзя выполнить downcast?

0

INexteR, я вроде закончил с маппером.
Теперь можно будет приступить к созданию страницы редактрования-добавления товара.
Посмотрите внесённые изменения. и обратите внимание на изменения в интерфейсе IProductsViewModel.
Его команды в ProductsViewModel нужно реализовать.
Так же в IProductsViewModel стоит добавить все интерфейсы источников.

Добавлено через 3 минуты

Сообщение от INexteR А нельзя выполнить downcast?

Каким образом?
Мы же в View или VM будем создавать НОВЫЕ экземпляры товаров, с новыми данными.
Их объединяет с Товаром репозитория, только общий интерфейс. Я для downcast нужно чтобы "Товар Представления" был производным от "Товара Репозитория."

Добавлено через 1 минуту
Посмотрите пример в Shop.cs:
Маппер может создать экземпляр даже из анонимного класса.

Добавлено через 4 минуты

Сообщение от INexteR но они только с одним методом получения коллекции, без события, так как счел его ненужным.

У нас же асинхронная загрузка всех коллекций.
Как VM узнает что она завершилась?
Если загрузка завершается для всех одновременно, то можно сделать один общий интерфейс (переделать IManufacturersSource) с одним событием и несколькими источниками. Событие будет означать что нужно обновить ВСЕ источники.

Добавлено через 1 час 4 минуты
INexteR, добавил диалоги редактирования-просмотра.
Посмотрите последнюю фиксацию.

0

Сообщение от Элд Хасп Каким образом?

Вот это
Заменить на

Сообщение от Элд Хасп Маппер может создать экземпляр даже из анонимного класса.

Здорово

Сообщение от Элд Хасп Как VM узнает что она завершилась?

В каталоге используются только коллекции производителей/товаров. Остальные коллекции будут нужны только в диалоге добавления и редактирования. А к тому времени они уже будут загружены, так как я написал так, что поставщики, категории и т. д. загружаются до товаров. И когда товары загружаются, они используют ранее загруженные коллекции, то есть они прикрепляются к контексту

Добавлено через 1 час 12 минут

Сообщение от INexteR И почему в ProductsView в ListBox является выбранным первый элемент? У выделенного ListBoxItem фон обычно голубой, а тут он светло-серый. Я опознал этот элемент выбранным по тому, что при переходе к каталогу кнопка удаления включена. В разметке выбранный элемент не задан

А вот это не знаете?

Добавлено через 2 часа 33 минуты
Элд Хасп, я посмотрел вашу фиксацию.
Хочу сказать, что давайте не будем добавлять режимы дизайнера. У нас и так много кода получается. До этого я задавал вопрос по этому поводу просто из интереса.
И вот ещё насчет диалога. Для чего Вы описываете базовый класс для разных типов? Требуется же только добавление и редактирование товара. И зачем в диалоге режим View? Я его убрал, он по заданию не нужен

0

Сообщение от INexteR Заменить на

Для этого нужно чтобы product был объектом производным от Product из Модели (Репозитория).
Для удаления, это ещё может быть так.
А для обновления и добавления это точно не будет так.

Сообщение от INexteR Остальные коллекции будут нужны только в диалоге добавления и редактирования. А к тому времени они уже будут загружены,

Нельзя рассчитывать на то что к тому времени....
Вот вам приспичит изменить представление и показать сразу список производителей и чего-то там ещё.
И теперь из-за изменений представления менять Модель? Такого быть не должно.
Если событие загрузки у них единое, то и нужно их всех объединить в одном интерфейсе с одним событием.

Сообщение от INexteR А вот это не знаете?

Пока не было времени посмотреть.

Добавлено через 2 минуты
INexteR, на данный момент у вас уже всё есть для создания формы редактирования-добаления.
В основное Окно нужно только добавить кликер или команду для вызова диалога редактирования.

Добавлено через 10 минут
INexteR, интерфейс IProductsViewModel это уровень VM и он может "легально" использовать библиотечные классы VM.
Поэтому лучше в нём использовать RelayCommand с нужным типом. ICommand только добавляет лишнего кода.

Добавлено через 3 минуты

Сообщение от Элд Хасп C# 65 66 67 68 69 70 71 72 73 74 75         private void RemoveProductExecute(IProduct product)         {             try             {                 _shop.Remove(product);             }             catch (InvalidOperationException e)             {                 MessageBox.Show(e.Message);             }         }

В VM не может быть вызовов методов UI. В том числе MessageBox или консоли.

0

Сообщение от Элд Хасп Поэтому лучше в нём использовать RelayCommand с нужным типом. ICommand только добавляет лишнего кода.

Я подсмотрел тип команды в IAuthorizationViewModel. Там указан ICommand. Поэтому и тут так сделал

0

Сообщение от INexteR Остальные коллекции будут нужны только в диалоге добавления и редактирования. C# 52         public ObservableCollection<IManufacturer> Manufacturers { get; } = new();

Тоже лишнее.
Вы же сами пишите что эти коллекции иммутабельные.
Достаточно IEnumerable<IManufacturer>.

0

Сообщение от Элд Хасп В VM не может быть вызовов методов UI. В том числе MessageBox или консоли.

Ну это просто пока так на время)))) Я там сильно не задумывался

0

Сообщение от INexteR Я подсмотрел тип команды в IAuthorizationViewModel. Там указан ICommand.

Там у нас ещё не было разделения на библиотеки разных уровней.
И ссылка на RelayCommand была бы циклической, что создало бы ошибку во время сборки.
Сейчас мы ужу разделили интерфейсы, поэтому можно безопасно использовать RelayCommand.

1

Сообщение от Элд Хасп И ссылка на RelayCommand была бы циклической, что создало бы ошибку во время сборки.

Ах вот оно что

0

Сообщение от INexteR это просто пока так на время)))

Да.
Для нестрого MVVM или тестирования - чёрт с ним.
А так у нас задан отлов исключений в App.
Если всё правильно вызывается, то исключения будут отловлены там.

0

Сообщение от Элд Хасп Пока не было времени посмотреть.

Вроде это случается из-за того, что по умолчанию в представлении коллекции первый элемент установлен в качестве CurrentItem

Добавлено через 5 минут

Сообщение от Элд Хасп А так у нас задан отлов исключений в App.

Но что-то они как-то не ловятся. Быть может, Вы видели эти строки:
Добавлено через 4 минуты

Сообщение от Элд Хасп Нельзя рассчитывать на то что к тому времени.... Вот вам приспичит изменить представление и показать сразу список производителей и чего-то там ещё.

Да, согласен, требования могут варьироваться в течение времени. Но в этом задании, честно сказать, ничего не будет меняться

0