Подтвердить что ты не робот

Проблема двустороннего связывания с WPF ComboBox с использованием MVVM

У меня есть объект Activity со многими свойствами. Один из них выглядит следующим образом:

public ActivityStatus Status
{
    get { return status; }
    set { status = value; NotifyPropertyChanged("Status"); }
}

Класс ActivityStatus имеет только два свойства:

public Guid Guid
{
    get { return guid; }
    set { guid = value; NotifyPropertyChanged("Guid"); }
}
public string Name
{
    get { return name; }
    set { name = value; NotifyPropertyChanged("Name"); }
}

и Equals:

public override bool Equals(object otherObject)
{
    if (!(otherObject is ActivityStatus)) return false;
    return Equals(otherObject as ActivityStatus);
}
public bool Equals(ActivityStatus otherStatus)
{
    if (!(otherStatus is ActivityStatus) || otherStatus == null) return false;
    return Guid == otherStatus.Guid && Name == otherStatus.Name;
}

У меня есть класс ActivityViewModel как DataContext класса ActivityView. ActivityViewModel имеет свойство Activity типа Activity и, среди прочих, свойство ActivityStatuses типа ObservableCollection<ActivityStatus>. В ActivityView у меня есть ComboBox, объявленный следующим образом:

<ComboBox ItemsSource="{Binding ActivityStatuses}" 
          SelectedItem="{Binding Activity.Status, Mode=TwoWay}"
          DisplayMemberPath="Name" />

Это позволяет мне выбрать ActivityStatus из ComboBox, и это правильно обновляет свойство Status объекта Activity в свойстве Activity модели viewmodel. Проблема заключается в двусторонней привязке... при загрузке нового Activity, ComboBox.SelectedItem не обновляется, чтобы показать значение свойства Activity.Status.

Используя это объявление ComboBox, SelectedItem привязано к объекту ActivityStatus в Activity, и это другой объект для объекта с теми же значениями в свойстве viewmodel ActivityStatuses. Поэтому WPF Framework не считает, что элементы одинаковы и не выбирает элемент в ComboBox.

Если я назначу элемент из коллекции с теми же значениями в свойство Activity.Status после загрузки каждого Activity, то ComboBox найдет совпадение в своей коллекции ItemsSource и правильно установит свойство SelectedItem отображая значение. Я действительно не хочу этого делать, хотя, поскольку у меня есть много других подобных свойств в классе Activity, и мне придется повторять этот код в любом месте, где бы я ни хотел, привязаны к ComboBox es.

Итак, я также попробовал привязку к свойству ActivityStatus.Guid следующим образом:

<ComboBox ItemsSource="{Binding ActivityStatuses}" 
          SelectedValue="{Binding Activity.Status.Guid, Mode=TwoWay}"
          SelectedValuePath="Guid" 
          DisplayMemberPath="Name" />

Это правильно выбрало объект с тем же Guid, что и в свойстве Activity.Status из коллекции ComboBox.ItemsSource при загрузке различных объектов Activity. Проблема с этим методом заключается в том, что SelectedValue привязан к свойству ActivityStatus.Guid в объекте ActivityStatus, и поэтому при изменении значений в пользовательском интерфейсе обновляется только свойство "Guid" объекта ActivityStatus оставив имя без изменений. Объект в свойстве Activity.Status не изменяется, кроме значения его свойства Guid.

Как вы можете видеть, я также попытался реализовать метод Equals, поскольку я предположил, что ComboBox будет использовать это для сравнения объектов, но это не имеет никакого значения. Итак, наконец, я в недоумении и очень хочу найти простой способ решить эту проблему... надеюсь, есть простое свойство, которое я пропустил на ComboBox.

Я просто хочу иметь возможность выбрать элемент в ComboBox и соответствующим образом изменить объект Activity.Status и изменить значение свойства Activity.Status из кода и соответствующим образом обновить ComboBox.SelectedItem. Я был бы признателен за любые советы.

ОБНОВЛЕНИЕ → >

Прочитав ответ "Ответ", я попробовал его образец кода в новом решении и увидел, что он работает так, как ожидалось. Затем я внимательно просмотрел его код и увидел, что он такой же, как у меня, поэтому снова запустил мое собственное решение (впервые с этого поста). К моему большому удивлению, он работал так, как я ожидал, без меня меняя код!

Это сильно меня озадачило, и я потратил некоторое время, чтобы узнать, что произошло. Оказывается, проблема была в Visual Studio 2010! Я добавил методы Equals к моим типам данных в качестве последнего этапа. По какой-то причине Visual Studio не создала проект типа данных при запуске приложения.

Таким образом, приложение должно было использовать старый DLL файл, и мои изменения не использовались... Я действительно задавался вопросом, почему мои точки разрыва в методах Equals никогда не попадали. Это привело к моему предположению, что внедрение методов Equals не помогло. Сегодня Visual Studio имеет такое же поведение и что я узнал, что произошло.

Я проверил порядок сборки проекта в своем решении, но он перечисляет проект типа данных в правильном месте в порядке. Однако при запуске приложения окно вывода в Visual Studio показывает, что DLL проекта загружается в другом порядке. Я не уверен, почему запуск приложения больше не выполняет полную сборку, но, по крайней мере, я знаю, что мне нужно создать этот проект после внесения изменений в него перед запуском приложения.

ЗАКЛЮЧИТЕЛЬНОЕ ОБНОВЛЕНИЕ → >

Я только выяснил, почему мой проект типа данных не строился... Я посмотрел в окне Configuration Manager и увидел, что платформа некорректна для этого проекта, а флажок Build отключен! Я понятия не имею, как это произошло, но мне очень приятно, что я, наконец, дошел до сути проблемы.

4b9b3361

Ответ 1

У меня для тебя плохие новости. Он должен работать. В другом месте есть ошибка/неожиданный побочный эффект, который вызывает вашу проблему.

Я собрал быстрый проект, чтобы сделать то, что вы пытаетесь сделать. Мне нравится видеть это здесь.

Создайте новый проект WPF под названием NestedProperties. Добавьте новый класс в корневой каталог и вставьте следующий код (я удалил много вещей, чтобы он был немного уродлив):

public sealed class ViewModel : DependencyObject
{
    public ObservableCollection<Activity> Activities 
           { get; private set; }
    public ObservableCollection<ActivityStatus> Statuses 
           { get; private set; }

    public static readonly DependencyProperty 
        SelectedActivityProperty =
        DependencyProperty.Register(
            "SelectedActivity",
            typeof(Activity),
            typeof(ViewModel),
            new UIPropertyMetadata(null));
    public Activity SelectedActivity
    {
        get { return (Activity)GetValue(SelectedActivityProperty); }
        set { SetValue(SelectedActivityProperty, value); }
    }

    public ViewModel()
    {
        Activities = new ObservableCollection<Activity>();
        Statuses = new ObservableCollection<ActivityStatus>();

        // NOTE!  Each Activity has its own ActivityStatus instance.
        // They have the same Guid and name as the instances in
        // Statuses!!
        for (int i = 1; i <= 4; i++)
        {
            var id = Guid.NewGuid();
            var aname = "Activity " + i;
            var sname = "Status " + i;
            Activities.Add(new Activity
            {
                Name = aname,
                Status = new ActivityStatus
                {
                    Name = sname,
                    Id = id,
                    InstanceType = "Activity"
                }
            });
            Statuses.Add(new ActivityStatus
            {
                Name = sname,
                Id = id,
                InstanceType = "Collection"
            });
        }
    }
}

public sealed class Activity : DependencyObject
{
    public static readonly DependencyProperty NameProperty =
        DependencyProperty.Register(
            "Name",
            typeof(string),
            typeof(Activity),
            new UIPropertyMetadata(null));
    public string Name
    {
        get { return (string)GetValue(NameProperty); }
        set { SetValue(NameProperty, value); }
    }
    public static readonly DependencyProperty StatusProperty =
        DependencyProperty.Register(
            "Status",
            typeof(ActivityStatus),
            typeof(Activity),
            new UIPropertyMetadata(null));
    public ActivityStatus Status
    {
        get { return (ActivityStatus)GetValue(StatusProperty); }
        set { SetValue(StatusProperty, value); }
    }
}
public sealed class ActivityStatus
{
    public Guid Id { get; set; }
    public string Name { get; set; }
    /// <summary>
    /// indicates if this instance came from 
    /// the ComboBox or from the Activity
    /// </summary>
    public string InstanceType { get; set; }
    public ActivityStatus()
    {
        Id = Guid.NewGuid();
    }
    public override bool Equals(object otherObject)
    {
        if (!(otherObject is ActivityStatus)) return false;
        return Equals(otherObject as ActivityStatus);
    }
    public bool Equals(ActivityStatus otherStatus)
    {
        if (!(otherStatus is ActivityStatus) ||
            otherStatus == null) return false;
        return Id == otherStatus.Id &&
            Name == otherStatus.Name;
    }
}

Теперь откройте MainWindow и вставьте это в:

<Window
    x:Class="NestedProperties.MainWindow"
    xmlns="http://schemas.microsoft.com/winfx/2006/xaml/presentation"
    xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml"
    Title="MainWindow"
    xmlns:t="clr-namespace:NestedProperties"
    SizeToContent="Height"
    MaxHeight="350"
    Width="525">
    <Window.DataContext>
        <t:ViewModel />
    </Window.DataContext>
    <Grid>
        <Grid.RowDefinitions>
            <RowDefinition
                Height="auto" />
            <RowDefinition
                Height="auto" />
            <RowDefinition />
        </Grid.RowDefinitions>
        <Grid.ColumnDefinitions>
            <ColumnDefinition />
            <ColumnDefinition />
        </Grid.ColumnDefinitions>
        <Label>Select an Activity:</Label>
        <ComboBox
            Grid.Row="1"
            ItemsSource="{Binding Activities}"
            SelectedItem="{Binding SelectedActivity}"
            DisplayMemberPath="Name" />
        <Label
            Grid.Column="1">Select a Status</Label>
        <ComboBox
            Grid.Row="1"
            Grid.Column="1"
            ItemsSource="{Binding Statuses}"
            SelectedItem="{Binding SelectedActivity.Status}"
            DisplayMemberPath="Name" />
        <ContentControl
            Grid.Row="2"
            Grid.ColumnSpan="2"
            Content="{Binding SelectedActivity}">
            <ContentControl.ContentTemplate>
                <DataTemplate>
                    <StackPanel>
                        <Label>Selected Activity:</Label>
                        <TextBlock
                            Text="{Binding Name}" />
                        <Label>Activity Status</Label>
                        <TextBlock
                            Text="{Binding Status.Name}" />
                        <Label>Status Id</Label>
                        <TextBlock
                            Text="{Binding Status.Id}" />
                        <Label>Status came from</Label>
                        <TextBlock
                            Text="{Binding Status.InstanceType}" />
                    </StackPanel>
                </DataTemplate>
            </ContentControl.ContentTemplate>
        </ContentControl>
    </Grid>
</Window>

Когда вы запустите это, вы обнаружите, что у вас есть четыре действия и четыре состояния. Если вы перейдете в комбинацию "Действия", вы увидите, что каждый статус отмечен как Активность, то есть это экземпляр, заданный для Activity в конструкторе ViewModel. Вы также увидите, что поле со списком статуса изменяется с изменением активности, что означает, что работает метод Equals.

Затем измените статус для каждого действия. Вы увидите тип изменений статуса Коллекция, что означает, что этот экземпляр был создан и добавлен в коллекцию Statuses в конструкторе.

Так почему это работает, но ваш код не является? Я не уверен. Ваша проблема кроется в другом месте вашего кода.

Ответ 2

Человек, я не знаю, правильно ли я выполнил ваш вопрос, но когда вы скажете

при загрузке новой операции,

Вы добавляете этот новый Activity в свою коллекцию ActivityStatuses? Потому что, если вы этого не сделаете, я уверен, что привязка не будет работать, потому что SelectedItem должен быть в ItemsSource.

Просто мысль.