Не срабатывают команды

@Chizel · Регистрация: 19.11.2014

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

Здравствуйте! Использую MVVMLight

XML

        <StackPanel Grid.Column="0"
                    Grid.Row="1"
                    Orientation="Horizontal"
                    HorizontalAlignment="Left">
            <Button Command="{Binding Path=AddMedicament}"
                    Content="Добавить"
                    Grid.Column="0"
                    Grid.Row="1"
                    Margin="2"
                    Width="100"/>
            <Button Command="{Binding Path=RemoveMedicament}"
                    Content="Удалить"
                    Grid.Column="0"
                    Grid.Row="1"
                    Margin="2"
                    Width="100"/>
        </StackPanel>

C#

        public RelayCommand AddCommand { get; private set; }
        public RelayCommand RemoveCommand { get; private set; }
 
        public MainViewModel()
        {
            bd = new WorkingWithData();
            AddCommand = new RelayCommand(AddMedicament);
            RemoveCommand = new RelayCommand(RemoveMedicament);
        }
 
        public void AddMedicament()
        {
            
        }
        public void RemoveMedicament()
        {
            bd.RemoveMedicamentFromBase(SelectedMedicament);
        }

Поставил брейкпоинты в методах AddMedicament() и RemoveMedicament() при нажатии на кнопку не срабатывает ни тот, ни другой. Подскажите, в чем проблема?

@subud · 15.07.2017, 13:20

Chizel, то что вы написали - все верно, а вот что с датаконтекстом/биндингом/хз. Копайте в сторону что куда биндится. создайте какое либо свойство, проверьте подватывается ли оно в данном месте и т.д.

@Chizel · 17.07.2017, 08:16 **[ТС]**

Почему то архив не прикрепляется к сообщению. Вот, по ссылке проект, может посмотрит кто-нибудь, что с ним не так? https://drive.google.com/open?... WFINGlEYms

@HF · 17.07.2017, 08:53

Первая ошибка. Привязка должна идти к командам, а не методам.

C#

1
2
3

            <Button Command="{Binding Path=AddCommand}"
...
            <Button Command="{Binding Path=RemoveCommand}"

@Chizel · 17.07.2017, 09:02 **[ТС]**

HF, так AddCommand и RemoveCommand это ведь команды

C#

1 2	public RelayCommand AddCommand { get; private set; } public RelayCommand RemoveCommand { get; private set; }

@HF · 17.07.2017, 09:53

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

так AddCommand и RemoveCommand это ведь команды

Верно, а биндинг был к методам. Поменяйте и увидите что команды в основном будут работать.

Я код сейчас смотрю, пока мне не понятно. Но проблема в том что AddView не видит контекст AddViewModel. Видимо я не совсем понимаю как MVVMLight работает или локатор надо правильно настроить.

Добавлено через 27 минут
Ошибка номер 2:
AddViewModel.cs. Это были поля, должны быть свойства. Теперь кнопки тоже работают.

C#

1 2	public RelayCommand AddMedicamentCommand { get; private set; } public RelayCommand CanselCommand { get; private set; }

Добавлено через 12 минут
А теперь ошибки связанные с проектом:

WorkingWithData.cs
1. CSV файл. Формат файла неправильный.
1) Разделитель должен быть ";". И потому что в CSV это принято, и потому что местоположение это наверняка улица, а там "," полно.
2) формат даты неопределён и нет ни парсинга по формату, ни try..catch. Падаем сразу же в любом случае.
3) "

C#

1	ObservableCollection<Medicament> ListMedicaments

" определён почему-то здесь. Вы сделали уже его в MainViewModel, оттуда и контролируйте. Данные когда читаются, они не обязательно должны быть того же типа. Считали, передали их назад во вьюмодель и сконвертировали. Или предайте ссылку на коллекцию в класс ридер/врайтера и достаточно. А сейчас сразу же у вас появилась проблема с обновлением коллекции после изменения данных.

MainViewModel.cs
1) окно то создали

C#

1	AddViewWindow = new AddView();

а толку пока в нём нет. Контекст или не будет связан с MainViewModel или возвращаемся к проблеме с коллекцией, которую надо тогда обновить и сообщить что она обновилась. Один из правильных вариантов такой. Добавить строку

C#

1	AddViewWindow.DataContext = new AddViewModel();

Тогда увидите что данные появились в полях ввода. А до этого всё было пусто.

AddViewModel.xaml
1) Не нужно поля ввода по умолчанию заполнять информацией, которая не должна сохраняться по умолчанию. Так можно насоздавать кучу препаратов с названием "Название лекарства". Лучше форму чуток поменять красиво. Добавить Label. Пример для первой строки:

C#

            <Label Target="{Binding ElementName=MedicalName}" >Название лекарства</Label>
            <TextBox Name="MedicalName" Text="{Binding Path=Name, Mode=TwoWay, UpdateSourceTrigger=PropertyChanged}"
                     Margin="5,0,5,10"
                     Height="30"/>

@Chizel · 17.07.2017, 10:19 **[ТС]**

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

Верно, а биндинг был к методам. Поменяйте и увидите что команды в основном будут работать.

Ага, действительно ступил я. Спасибо за помощь.

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

Я код сейчас смотрю, пока мне не понятно. Но проблема в том что AddView не видит контекст AddViewModel. Видимо я не совсем понимаю как MVVMLight работает или локатор надо правильно настроить.

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

О, тут вообще все разобрали по полочкам, замечательно!

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

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

1) Разделитель должен быть ";". И потому что в CSV это принято, и потому что местоположение это наверняка улица, а там "," полно.

Исправил. Но местоположение это не улица, а аптечка, либо холодильник, так что тут запятых не будет.

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

формат даты неопределён и нет ни парсинга по формату, ни try..catch. Падаем сразу же в любом случае.

try catch добавить нужно конечно, но зачем определять формат даты? Дату я буду получать с окна AddView, то есть от календаря, вручную дата нигде забиваться не будет. Поэтому и ошибок быть не должно, календарь то ведь всегда в одном виде дату передает.

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

3) "
C#Выделить код
1
ObservableCollection<Medicament> ListMedicaments
" определён почему-то здесь.

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

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

1) Не нужно поля ввода по умолчанию заполнять информацией, которая не должна сохраняться по умолчанию.

Учту.

@HF · 17.07.2017, 11:00

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

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

Да вы лучше меня всё знаете.

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

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

try catch добавить нужно конечно, но зачем определять формат даты? Дату я буду получать с окна AddView, то есть от календаря, вручную дата нигде забиваться не будет. Поэтому и ошибок быть не должно, календарь то ведь всегда в одном виде дату передает.

Так то оно так, но если на будущее закладываться? На разных компьютерах, может быть разный формат. Даже вы сами можете поработать на одном компьютере (с русским регионом), а потом перейти на другой (английский формат дат и цифр)... и все ваши файлы перестанут работать.
Ну и, этот метод может в дальнейшем использоваться и как импортирование сторонних данных. Аптеки или поставщики будут например присылать свои данные. И вам потом разные форматы разгребать придётся.
Я бы сразу формат какой-то определил (импорт через DateTime.Parse) или сделал парсер разных форматов дат (сначала регэксп определяет, потом парсит).

Добавлено через 58 секунд

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

Ага, действительно ступил я. Спасибо за помощь.

Поэтому лучше Action методы кнопок делать приватными и тогда легко будет увидеть ошибку.

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

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

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

Я имел ввиду что можно не коллекцию передавать, а Raw данные, например простой список типа List<Medical>. Вызывать этот класс как контроллер данных со статическими методами (читать, писать, удалять). А во вьюмодели результат уже импортировать в коллекцию.
И ваш и мой вариант могут быть работоспособные. Просто мысли вслух

@Chizel · 17.07.2017, 11:52 **[ТС]**

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

Так то оно так, но если на будущее закладываться? На разных компьютерах, может быть разный формат. Даже вы сами можете поработать на одном компьютере (с русским регионом), а потом перейти на другой (английский формат дат и цифр)... и все ваши файлы перестанут работать.
Ну и, этот метод может в дальнейшем использоваться и как импортирование сторонних данных. Аптеки или поставщики будут например присылать свои данные. И вам потом разные форматы разгребать придётся.
Я бы сразу формат какой-то определил (импорт через DateTime.Parse) или сделал парсер разных форматов дат (сначала регэксп определяет, потом парсит).

Дельно, переделаю, чтобы была проверка формата дат.

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

Поэтому лучше Action методы кнопок делать приватными и тогда легко будет увидеть ошибку.

Да, так действительно будет лучше, на будущее учту, ну и тут переделаю, конечно же.

@subud 3 / 3 / 4 Регистрация: 29.10.2016 Сообщений: 48
	15.07.2017, 13:20	2
	Chizel, то что вы написали - все верно, а вот что с датаконтекстом/биндингом/хз. Копайте в сторону что куда биндится. создайте какое либо свойство, проверьте подватывается ли оно в данном месте и т.д. 0

@Chizel 31 / 14 / 5 Регистрация: 19.11.2014 Сообщений: 617
	17.07.2017, 08:16 [ТС]	3
	Почему то архив не прикрепляется к сообщению. Вот, по ссылке проект, может посмотрит кто-нибудь, что с ним не так? https://drive.google.com/open?... WFINGlEYms 0

@Chizel 31 / 14 / 5 Регистрация: 19.11.2014 Сообщений: 617
	17.07.2017, 10:19 [ТС]	7
	Сообщение от HF Верно, а биндинг был к методам. Поменяйте и увидите что команды в основном будут работать. Ага, действительно ступил я. Спасибо за помощь. Сообщение от HF Я код сейчас смотрю, пока мне не понятно. Но проблема в том что AddView не видит контекст AddViewModel. Видимо я не совсем понимаю как MVVMLight работает или локатор надо правильно настроить. Приложение еще не дописано, поэтому контекст я не настроил пока. То есть изначально я накидал одно главное окно, начал проверять, а не работают команды. Писал дальше, параллельно пытаясь разобрать в чем причина. За подсказку на счет свойств тоже спасибо, упустил этот момент. О, тут вообще все разобрали по полочкам, замечательно! Добавлено через 20 минут Сообщение от HF 1) Разделитель должен быть ";". И потому что в CSV это принято, и потому что местоположение это наверняка улица, а там "," полно. Исправил. Но местоположение это не улица, а аптечка, либо холодильник, так что тут запятых не будет. Сообщение от HF формат даты неопределён и нет ни парсинга по формату, ни try..catch. Падаем сразу же в любом случае. try catch добавить нужно конечно, но зачем определять формат даты? Дату я буду получать с окна AddView, то есть от календаря, вручную дата нигде забиваться не будет. Поэтому и ошибок быть не должно, календарь то ведь всегда в одном виде дату передает. Сообщение от HF 3) " C#Выделить код 1 ObservableCollection<Medicament> ListMedicaments " определён почему-то здесь. Я не могу коллекцию передать по ссылке в модель. Концепция MVVM это запрещает, поэтому коллекция есть в модели, а во вью модели обертка над ней. Сообщение от HF 1) Не нужно поля ввода по умолчанию заполнять информацией, которая не должна сохраняться по умолчанию. Учту. 0

@HF 1163 / 749 / 181 Регистрация: 09.09.2011 Сообщений: 2,314 Записей в блоге: 2
	17.07.2017, 11:00	8
	Сообщение от Chizel Я не могу коллекцию передать по ссылке в модель. Концепция MVVM это запрещает, поэтому коллекция есть в модели, а во вью модели обертка над ней. Да вы лучше меня всё знаете. Конечно делайте как задумали. Это я просто столкнулся и показалось чем-то неправильным. Но наверное просто вы ещё не до конца всё согласовали в коде. Сообщение от Chizel try catch добавить нужно конечно, но зачем определять формат даты? Дату я буду получать с окна AddView, то есть от календаря, вручную дата нигде забиваться не будет. Поэтому и ошибок быть не должно, календарь то ведь всегда в одном виде дату передает. Так то оно так, но если на будущее закладываться? На разных компьютерах, может быть разный формат. Даже вы сами можете поработать на одном компьютере (с русским регионом), а потом перейти на другой (английский формат дат и цифр)... и все ваши файлы перестанут работать. Ну и, этот метод может в дальнейшем использоваться и как импортирование сторонних данных. Аптеки или поставщики будут например присылать свои данные. И вам потом разные форматы разгребать придётся. Я бы сразу формат какой-то определил (импорт через DateTime.Parse) или сделал парсер разных форматов дат (сначала регэксп определяет, потом парсит). Добавлено через 58 секунд Сообщение от Chizel Ага, действительно ступил я. Спасибо за помощь. Поэтому лучше Action методы кнопок делать приватными и тогда легко будет увидеть ошибку. Добавлено через 5 минут Сообщение от Chizel Я не могу коллекцию передать по ссылке в модель. Концепция MVVM это запрещает, поэтому коллекция есть в модели, а во вью модели обертка над ней. Я имел ввиду что можно не коллекцию передавать, а Raw данные, например простой список типа List<Medical>. Вызывать этот класс как контроллер данных со статическими методами (читать, писать, удалять). А во вьюмодели результат уже импортировать в коллекцию. И ваш и мой вариант могут быть работоспособные. Просто мысли вслух 1

@Chizel 31 / 14 / 5 Регистрация: 19.11.2014 Сообщений: 617
	17.07.2017, 11:52 [ТС]	9
	Сообщение от HF Так то оно так, но если на будущее закладываться? На разных компьютерах, может быть разный формат. Даже вы сами можете поработать на одном компьютере (с русским регионом), а потом перейти на другой (английский формат дат и цифр)... и все ваши файлы перестанут работать. Ну и, этот метод может в дальнейшем использоваться и как импортирование сторонних данных. Аптеки или поставщики будут например присылать свои данные. И вам потом разные форматы разгребать придётся. Я бы сразу формат какой-то определил (импорт через DateTime.Parse) или сделал парсер разных форматов дат (сначала регэксп определяет, потом парсит). Дельно, переделаю, чтобы была проверка формата дат. Сообщение от HF Поэтому лучше Action методы кнопок делать приватными и тогда легко будет увидеть ошибку. Да, так действительно будет лучше, на будущее учту, ну и тут переделаю, конечно же. 0

Не срабатывают команды

Решение