CanExecute на RelayCommand <T>не работает - PullRequest
Новая
       33

CanExecute на RelayCommand <T>не работает

11 голосов
/ 21 февраля 2010

Я пишу приложение WPF 4 (с VS2010 RC) с использованием MVVM Light V3 alpha 3 и сталкиваюсь со странным поведением ...

У меня есть команда, которая открывает Window, и это Окно создает ViewModel и так далее - ничего странного там нет.

В этом Window у меня есть RelayCommand с, например: 

CategoryBeenSelected = new RelayCommand(() => OnCategoryUpdate = true);

Снова ничего странного - все работает так, как я ожидал.

Проблема в том, что у меня не может быть метода / лямбда-выражения CanExecute с универсальным RelayCommand.

Это работает: 

DeleteCategoryCommand = new RelayCommand<int>(DeleteCategory);

Но это не так: 

DeleteCategoryCommand = new RelayCommand<int>(DeleteCategory, CanDeleteCategory);

Окно не появляется. Я имею в виду, я нажимаю кнопку, которая открывает окно, и приложение просто блокируется, и через несколько секунд метод InitializeComponent в Window выдает NullReferenceException (ссылка на объект не установлена ​​для экземпляра объекта)

Короче говоря, если я добавлю CanExecute метод к RelayCommand<T>, Window, которому принадлежит , то ViewModel (с RelayCommand<T>) создать невозможно. Если я удалю CanExecute, появится Window.

Где здесь проблема? Я в замешательстве.

Спасибо.

РЕДАКТИРОВАТЬ: По запросу, вот трассировка стека: 

A first chance exception of type 'System.NullReferenceException' occurred in PresentationFramework.dll
   at GalaSoft.MvvmLight.Command.RelayCommand`1.CanExecute(Object parameter)
   at System.Windows.Controls.Primitives.ButtonBase.UpdateCanExecute()
   at System.Windows.Controls.Primitives.ButtonBase.OnCommandChanged(DependencyObject d, DependencyPropertyChangedEventArgs e)
   at System.Windows.DependencyObject.OnPropertyChanged(DependencyPropertyChangedEventArgs e)
   at System.Windows.FrameworkElement.OnPropertyChanged(DependencyPropertyChangedEventArgs e)
   at System.Windows.DependencyObject.NotifyPropertyChange(DependencyPropertyChangedEventArgs args)
   at System.Windows.DependencyObject.UpdateEffectiveValue(EntryIndex entryIndex, DependencyProperty dp, PropertyMetadata metadata, EffectiveValueEntry oldEntry, EffectiveValueEntry& newEntry, Boolean coerceWithDeferredReference, Boolean coerceWithCurrentValue, OperationType operationType)
   at System.Windows.DependencyObject.SetValueCommon(DependencyProperty dp, Object value, PropertyMetadata metadata, Boolean coerceWithDeferredReference, Boolean coerceWithCurrentValue, OperationType operationType, Boolean isInternal)
   at System.Windows.DependencyObject.SetValue(DependencyProperty dp, Object value)
   at MS.Internal.Xaml.Runtime.ClrObjectRuntime.SetValue(Object inst, XamlMember property, Object value)
   at MS.Internal.Xaml.Runtime.PartialTrustTolerantRuntime.SetValue(Object obj, XamlMember property, Object value)
   at System.Xaml.XamlObjectWriter.Logic_ApplyPropertyValue(ObjectWriterContext ctx, XamlMember prop, Object value, Boolean onParent)
   at System.Xaml.XamlObjectWriter.Logic_DoAssignmentToParentProperty(ObjectWriterContext ctx)
   at System.Xaml.XamlObjectWriter.WriteEndObject()
   at System.Windows.Markup.WpfXamlLoader.TransformNodes(XamlReader xamlReader, XamlObjectWriter xamlWriter, Boolean onlyLoadOneNode, Boolean skipJournaledProperties, Boolean shouldPassLineNumberInfo, IXamlLineInfo xamlLineInfo, IXamlLineInfoConsumer xamlLineInfoConsumer, XamlContextStack`1 stack, IStyleConnector styleConnector)
   at System.Windows.Markup.WpfXamlLoader.Load(XamlReader xamlReader, IXamlObjectWriterFactory writerFactory, Boolean skipJournaledProperties, Object rootObject, XamlObjectWriterSettings settings, Uri baseUri)
   at System.Windows.Markup.WpfXamlLoader.LoadBaml(XamlReader xamlReader, Boolean skipJournaledProperties, Object rootObject, XamlAccessLevel accessLevel, Uri baseUri)
   at System.Windows.Markup.XamlReader.LoadBaml(Stream stream, ParserContext parserContext, Object parent, Boolean closeStream)
   at System.Windows.Application.LoadComponent(Object component, Uri resourceLocator)
   at ApuntaNotas.Views.CategoryEditorView.InitializeComponent() in c:\Users\Jesus\Documents\Visual Studio 2010\Projects\ApuntaNotas\ApuntaNotas\Views\CategoryEditorView.xaml:line 1
   at ApuntaNotas.Views.CategoryEditorView..ctor() in C:\Users\Jesus\Documents\Visual Studio 2010\Projects\ApuntaNotas\ApuntaNotas\Views\CategoryEditorView.xaml.cs:line 18
A first chance exception of type 'System.NullReferenceException' occurred in PresentationFramework.dll

Ответы [ 3 ]

7 голосов
/ 23 февраля 2010

Похоже, что RelayCommand приведёт значение параметра к универсальному T.

Но вы не можете привести ноль к структуре, как говорит исключение!

Если вы инициализируете RelayCommand с обнуляемой структурой, он будет работать как положено! 

RelayCommand<int?> or RelayCommand<Nullable<int>>

НТН

2 голосов
/ 23 февраля 2011

Арктур ​​был прав в определении проблемы, однако мне не понравилось решение использовать обнуляемые примитивы. Лично мне не нравятся обнуляемые примитивы, если у меня нет веских причин использовать их.

Вместо этого я изменил реализацию RelayCommand следующим образом: 

    bool ICommand.CanExecute(object parameter)
    {
        if (parameter == null && typeof(T).IsValueType)
        {
            return CanExecute(default(T));
        }
        return CanExecute((T)parameter);
    }

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

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

        <Button Content="Fit To Width" Command="{Binding Path=FitToWidthCommand}" CommandParameter="{Binding ElementName=imageScrollViewer, Path=ActualWidth}" />
        <Button Content="Fit To Height" Command="{Binding Path=FitToHeightCommand}" CommandParameter="{Binding ElementName=imageScrollViewer, Path=ActualHeight}" />

В приведенном выше XAML вы заметили, что параметр команды привязан к фактической ширине элемента управления. Однако WPF вызовет CanExecute по команде кнопки до того, как элемент управления «imageScrollViewer» будет обязательно размечен / отрисован - поэтому фактической ширины / высоты нет. К тому времени, когда пользователь нажимает кнопку и вызывается «Выполнить», элемент управления размечается, поэтому значения отправляются команде. Если нет - я думаю, что неудача - это то, чего следует ожидать - но только тогда, когда пользователь действительно нажимает кнопку.

Конечно, мне не нравится различное поведение CanExecute и Execute, но сейчас оно, кажется, соответствует ограничениям, представленным платформой. Я могу найти сценарий, в котором это вызывает у меня горе, но мне пока нравятся перемены.

0 голосов
/ 22 февраля 2010

Может быть, в это время параметр null?

...