1

У меня возникла такая проблема: мне нужно чтобы если в TextBox ошибка, то кнопка становилась неактивной. Проблема в том что TextBox-ов таких несколько. Я пробовал делать обычную валидацию на исключениях и несколько флагов для каждого из полей обозначающих их валидность и биндить кнопку. Но это очень муторно. А если еще добавлять поля? Все новые флаги создавать? Ужасно неудобно и коряво. Подскажите пожалуйста какой то более красивый способ.

6
  • Посмотрите на этот вопрос.
    – Vlad
    Commented 18 апр. 2016 в 18:05
  • @Vlad я прекрасно представляю себе механизмы валидации. Не надо мне вот эти очевидные вещи совать. И конкретно ту тему в частности я уже видел раньше. Она не решает мою проблему.
    – Developer
    Commented 18 апр. 2016 в 18:07
  • Тогда непонятно, в чем проблема? Засовываете в VM все свойства, которые нужно валидировать, вешаете соответствующие атрибуты и привязываете их к TextBox. Доступность команды связываете с наличием ошибок валидации. При добавлении новых полей никаких дополнительных телодвижений не потребуется.
    – Vlad
    Commented 18 апр. 2016 в 18:15
  • @Vlad в кнопке проблема. Исключительно в ней. Она должна быть не активна если есть ошибки ввода в TextBox-ах.
    – Developer
    Commented 18 апр. 2016 в 18:17
  • Так привяжите CanExecute команды, которая привязывается к кнопке, к наличию ошибок валидации.
    – Vlad
    Commented 18 апр. 2016 в 18:20

2 ответа 2

3

Если вы используете MVVM, то все довольно просто. Делаете валидацию на источнике данных. Свойства с ошибками валидации собираете в список. Если список пустой, то ошибок нет, а значит можно нажимать на кнопку. Вот пример VM:

public sealed class MainVm : INotifyPropertyChanged, IDataErrorInfo
{
    // перечень свойств с ошибками
    private readonly HashSet<string> propertiesWithErrors = new HashSet<string>();

    // метод валидации свойства
    private string Validate(string propertyName)
    {
        var value = GetType()
            .GetProperty(propertyName)
            .GetValue(this, null);

        var results = new List<ValidationResult>();

        var context = new ValidationContext(this, null, null) { MemberName = propertyName };

        if (!Validator.TryValidateProperty(value, context, results))
        {
            // добавляем информацию о том, что это свойство содержит ошибку
            propertiesWithErrors.Add(propertyName);
            return results.First().ErrorMessage;
        }

        // удаляем информацию об ошибке в свойстве
        propertiesWithErrors.Remove(propertyName);
        return string.Empty;
    }
    // фамилия (бросает PropertyChanged)
    [Required(AllowEmptyStrings = false)]
    public string Surname { get; set; }

    // имя (бросает PropertyChanged)
    [Required(AllowEmptyStrings = false)]
    public string Name { get; set; }

    // команда, которая привязывется к кнопке. Допустим, применить какие-либо изменения
    public ICommand ApplyCommand { get; }

    // реализация IDataErrorInfo
    public string Error
    {
        get { throw new NotImplementedException(); }
    }
    string IDataErrorInfo.this[string propertyName]
    {
        get { return Validate(propertyName); }
    }
    // ------

    public MainVm()
    {
        // создаем команду
        // первый аргумент Execute (ничего не делаем)
        // второй аргумент CanExecute (true, если нет свойств с ошибками)
        ApplyCommand = new RelayCommand(o => { }, o => !propertiesWithErrors.Any());
    }

RelayCommand можно стащить отсюда. В представлении все еще проще. Привязываем свойства к TextBox, а кнопку к команде.

<Window ...>
    <Window.DataContext>
        <vm:MainVm/>
    </Window.DataContext>
    <StackPanel>
        <TextBox Text="{Binding Name, ValidatesOnDataErrors=True, UpdateSourceTrigger=PropertyChanged}"/>
        <TextBox Text="{Binding Surname, ValidatesOnDataErrors=True, UpdateSourceTrigger=PropertyChanged}"/>
        <Button Content="Button" Command="{Binding ApplyCommand}"/>
    </StackPanel>
</Window>

UPD

Нет, атрибут Required никакого отношения к INotifyPropertyChanged не имеет. Свойтсво, на самом деле, должно выглядеть вот так:

public string Name 
{ 
    get { return name; } 
    set
    {
        if (name != value)
        {
            name = value;
            OnPropertyChanged(nameof(Name));
        }
    }
} 
private string name;

private void OnPropertyChanged(string propertyName)
{
    var tmp = PropertyChanged;
    if (tmp != null)
    {
        tmp(this, new PropertyChangedEventArgs(propertyName));
    }
}

Атрибут Required говорит о том, что у данного свойства должно быть установлено значение, отличное от null и string.Empty.

13
  • че за атрибут Required?
    – Developer
    Commented 18 апр. 2016 в 18:56
  • Из System.ComponentModel.DataAnnotations
    – Vlad
    Commented 18 апр. 2016 в 18:58
  • я не понял, оно что само вызывает PropertyChanged??? Т.е. это как бы аналог Fody.PropertyChanged от .NET?
    – Developer
    Commented 18 апр. 2016 в 19:05
  • @DarkByte, что оно?
    – Vlad
    Commented 18 апр. 2016 в 19:07
  • ну то что этот атрибут сам вызывает PropertyChanged при изменении свойства?
    – Developer
    Commented 18 апр. 2016 в 19:09
1

Можно для кнопки добавить поле Count, если textbox поменял свой статус на неправильное, то +1 к count, если он поменял с неправильного на правильное, то -1 count. (Здесь надо аккуратно сделать, чтобы поле добавляло\отменяло только при смене статуса на противоположное, т.е. чтобы интервал значения count был от 0 до количества textbox.) Соответственно, как только count обратился в 0 - кнопка работает.

8
  • вот я уже и сам до этого додумался. Ну, если более элегантного решения никто не предложит, наверное так и сделаю.
    – Developer
    Commented 18 апр. 2016 в 17:30
  • можно с событиями сделать
    – R0manych
    Commented 18 апр. 2016 в 17:34
  • в смысле с событиями?
    – Developer
    Commented 18 апр. 2016 в 17:44
  • Тут еще проблема в том что приходится объявлять биндинг всегда к текстовому полю. Потому что инкрементировать счетчик надо и в случае неверной конвертации. Ужас конечно как неудобно все равно...
    – Developer
    Commented 18 апр. 2016 в 17:46
  • Например если у меня поле double, но все равно придется текстовое для биндинга объявлять, чтобы в сеттере проверить еще и конвертацию в double. Это ведь тоже ошибка...
    – Developer
    Commented 18 апр. 2016 в 17:47

Ваш ответ

Нажимая «Отправить ответ», вы соглашаетесь с условиями пользования и подтверждаете, что прочитали политику конфиденциальности.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.