wpf mvvm проверка ошибок

Question

У меня есть простое приложение wpf, и я пытаюсь отключить кнопку сохранения, если в форме есть ошибки.

Проблема в том, что, хотя проверка, кажется, работает отлично, я не знаю почему, но я все время получаю ложные данные от метода, который отвечает за проверку ошибок.

Позвольте мне сделать это более понятным, предоставив код.

Это код из MainWindow.Xaml.cs

private readonly HashSet<ValidationError> errors = new HashSet<ValidationError>();
    private Lazy<MainWindowViewModel> viewModel;

    public MainWindow() {
        InitializeComponent();
        InitializeValidaton();
    }

    void InitializeValidaton() {
        viewModel = new Lazy<MainWindowViewModel>();
        Validation.AddErrorHandler(this, ErrorChangedHandler);
    }

    private void ErrorChangedHandler(object sender, ValidationErrorEventArgs e) {
        if (e.Action == ValidationErrorEventAction.Added) {
            errors.Add(e.Error);
        } else {
            errors.Remove(e.Error);
        }
        //I set a breakpoint here and it returns the correct value. False if it has errors  and True if not
        viewModel.Value.IsValid = !errors.Any();

    }

Это команда для кнопки

 public ICommand SaveItem {
        get { return new RelayCommand(SaveItemExecute,CanSaveItem); }
    }
 private bool CanSaveItem() {
        return IsValid;
    }

  //I set up here a breakpoint and it returns the correct value just once.
  //The application looked up on CanSaveItem all the time and except the first time, it returns wrong value
  private bool _isValid;
    public bool IsValid {
        get { return _isValid; }
        set {
            _isValid = value;
            RaisePropertyChanged("IsValid");
  }
    }

Правила валидации

[Required(ErrorMessage = "Please enter Title")]
    [StringLength(100, ErrorMessage = "The maximum length is 100")]
    string Name { get; set; }

Я не знаю, имеет ли это смысл, но кнопка, которую я хочу деактивировать, находится в пользовательском контроле.

Я не могу понять, почему метод canExecute, который находится в userControl, срабатывает более одного раза. Какой бы метод ни использовался, у меня такая же реакция. Я упоминаю userControl, потому что, если я использую тот же метод (который в части ICommand) в mainWindow, он срабатывает только один раз

Я буду признателен, если кто-нибудь может мне помочь с этим.

Спасибо

Answer 1

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

MainWindow.xaml

<StackPanel>
    <TextBox x:Name="ValidatedTextBox" Width="200">
        <TextBox.Text>
            <Binding Path="EnteredText" UpdateSourceTrigger="PropertyChanged" NotifyOnValidationError="True">
                <Binding.ValidationRules>
                    <local:NotEmptyInputRule ValidatesOnTargetUpdated="True" />
                </Binding.ValidationRules>
            </Binding>
        </TextBox.Text>
    </TextBox>
    <Button Content="Save" Width="60" IsEnabled="{Binding IsValid}" />
</StackPanel>

Имущество EnteredText должен существовать в ViewModel:

class MainWindowViewModel : INotifyPropertyChanged
{
    public ICommand SaveItem
    {
        get { return new SimpleCommand(SaveItemExecute, CanSaveItem); }
    }

    public void SaveItemExecute()
    {
        //save
    }

    private bool CanSaveItem()
    {
        return IsValid;
    }

    //I set up here a breakpoint and it returns the correct value just once.
    //The application looked up on CanSaveItem all the time and except the first time, it returns wrong value
    private bool _isValid;
    public bool IsValid
    {
        get { return _isValid; }
        set
        {
            _isValid = value;
            OnPropertyChanged("IsValid");
        }
    }

    public string EnteredText { get; set; }

    public event PropertyChangedEventHandler PropertyChanged;

    protected virtual void OnPropertyChanged(string propertyName)
    {
        if (this.PropertyChanged != null)
        {
            this.PropertyChanged(this, new PropertyChangedEventArgs(propertyName));
        }
    }

}

И не забудьте установить DataContext в главном окне.

    public MainWindow()
    {
        InitializeComponent();
        InitializeValidaton();
        this.DataContext = viewModel.Value;
    }

Есть также класс Command и правило проверки.

public class SimpleCommand : ICommand
{
    /// <summary>
    /// Gets or sets the Predicate to execute when the CanExecute of the command gets called
    /// </summary>
    public Predicate<object> CanExecuteDelegate { get; set; }

    /// <summary>
    /// Gets or sets the action to be called when the Execute method of the command gets called
    /// </summary>
    public Action<object> ExecuteDelegate { get; set; }

    public SimpleCommand(Action execute, Func<bool> canExecute)
    {
        this.ExecuteDelegate = _ => execute();
        this.CanExecuteDelegate = _ => canExecute();
    }

    #region ICommand Members

    /// <summary>
    /// Checks if the command Execute method can run
    /// </summary>
    /// <param name="parameter">THe command parameter to be passed</param>
    /// <returns>Returns true if the command can execute. By default true is returned so that if the user of SimpleCommand does not specify a CanExecuteCommand delegate the command still executes.</returns>
    public bool CanExecute(object parameter)
    {
        if (CanExecuteDelegate != null)
            return CanExecuteDelegate(parameter);
        return true;// if there is no can execute default to true
    }

    public event EventHandler CanExecuteChanged
    {
        add { CommandManager.RequerySuggested += value; }
        remove { CommandManager.RequerySuggested -= value; }
    }

    /// <summary>
    /// Executes the actual command
    /// </summary>
    /// <param name="parameter">THe command parameter to be passed</param>
    public void Execute(object parameter)
    {
        if (ExecuteDelegate != null)
            ExecuteDelegate(parameter);
    }

    #endregion

}

class NotEmptyInputRule : ValidationRule
{
    public override ValidationResult Validate(object value, CultureInfo cultureInfo)
    {
        if (value != null)
        {
            string input = value as string;

            if (input.Length > 0)
                return new ValidationResult(true, null);
        }

        return new ValidationResult(false, "Validation error. Field input required.");
    }
}