我可以强制更新绑定到 ComboBox.SelectedItem 的 Textbox DataContext 吗？

Question

我正在为我的应用程序实现撤消/重做功能，但遇到了问题。基本上我有一个绑定到项目列表的组合框。然后我有几个文本框，它们在组合框中具有所选项目的 DataContext，并且文本绑定到该项目的属性。当用户发出撤消命令并且要撤消的项目是文本框的文本更改时，我希望首先选择组合框中的关联项目。然后，我想立即用原始文本更新文本框。理论上，这应该更新所选项目的数据绑定属性。然而，发生的事情是选择发生了变化，但在我更改文本框的文本属性之前，数据绑定文本框没有更新，因此它正在更改组合框中先前选择的项目的属性值。

这是我的组合框：

<ComboBox x:Name="myComboBox" 
          ItemsSource="{Binding MyItems, UpdateSourceTrigger=PropertyChanged}"
          DisplayMemberPath="Name"
          SelectionChanged="myComboBox_SelectionChanged" />

这是我的文本框（DataContext 设置在父 Grid 上。如果 DataContext 设置在文本框本身上，也会发生相同的行为）

<TextBox x:Name="purposeTxtBox" 
         Text="{Binding Purpose, Mode=TwoWay, UpdateSourceTrigger=PropertyChanged}" />

我在撤消启动后使用的代码（sn-p）是：

case UndoAction.PURPOSE_CHANGE:
    SelectComboBoxItem(myComboBox, itemToSelect);
    purposeTxtBox.Text = itemValue;
    FocusAndSelect(purposeTxtBox);

还有 SelectComboBoxItem 方法：

private void SelectComboBoxItem(ComboBox box, object item)
{
    Dispatcher.BeginInvoke(DispatcherPriority.Input,
                           new Action(delegate()
                           {
                               box.SelectedItem = item;
                           }));
}

我知道我可以更新 DataContext 上的“Purpose”属性。这确实有效，但如果我这样做，它不会专注和选择。另外，由于所有绑定都已经到位，其中一些绑定到整数，我打算只更新文本并允许“魔术”处理翻译。这样我就可以为任何文本框提供通用解决方案。为了完整起见，这是我的 FocusAndSelect 函数：

private void FocusAndSelect(TextBox box)
{
    box.SelectAll();
    Dispatcher.BeginInvoke(DispatcherPriority.Input,
                           new Action(delegate()
                           {
                               box.Focus();         // Set Logical Focus
                               Keyboard.Focus(box); // Set Keyboard Focus
                           }));
}

Answer 1

我认为问题在于：

Dispatcher.BeginInvoke(DispatcherPriority.Input,
                       new Action(delegate()
                       {
                           box.SelectedItem = item;
                       }));

您告诉系统更新组合框的值，但您使用的是异步方法（我认为）。这意味着它会在 UI 线程上执行此操作（尽管我认为您是从 UI 线程调用它）但无论如何 - 现在不会发生。

然后在下一行，你直接设置了 purposeTxtBox.Text = itemValue ，但是checkbox还没有更新它的选中值，所以还没有执行value-changed。

最终，问题在于您将逻辑与 UI 混合在一起。您不应该通过进入 UI 并更改某些内容来更改所选值，以便更新您的数据。您应该直接更改数据，并让 UI 在需要时自行更新。

举个简单的例子，复选框显示布尔值，你不想在 UI 中创建一个复选框并在业务逻辑或数据逻辑中选中它并取消选中它，复选框不应该持有该值- 你的程序应该，并且复选框应该只是绑定到它。您需要将该值更改为 false，将布尔值设置为 false，而不是转到复选框，将其设置为未选中并让绑定更新布尔值。这有点倒退了！

你应该做的是：

    private Item selectedItem;
    public Item SelectedItem
    {
        get { return selectedItem; }
        set
        {
            if (selectedItem != value)
            {
                selectedItem = value;
                //perform your ItemChange events here, if you have any
                OnPropertyChanged("SelectedItem");
            }
        }
    }

您的组合框应将其 SelectedItem 绑定到该属性。您的文本框应将其文本属性绑定到网格的数据上下文 SelectedItem.SomeText 属性。

当您更改选择时，您不会转到 UI、更改组件、等待 UI 线程处理、返回、触发事件等等等

您只需更改对象 - UI 会做出反应。

这是更好的做法。这也是为什么我们有像 MVVM 和 MVC 这样的模型来强制在 UI 和逻辑之间进行这种划分的原因。

Answer 2

与您的问题没有直接关系（我希望它有用），但您可以使用委托组织撤消：

readonly Stack<Action> _undo = new Stack<Action>();

string _someProperty;
// property to bind
public string SomeProperty
{
    get { return _someProperty; }
    set
    {
        var old = _someProperty; // capture old value
        _undo.Push(() =>
        {
            _someProperty = old;
            OnPropertyChanged();
        });
        _someProperty = value;
        OnPropertyChanged();
    }
}

void Undo()
{
    if (_undo.Count > 0)
        _undo.Pop()();
}

更改属性值（通过绑定）将记录委托设置以前的值。调用 Undo() 将简单地播放代表从最后一个到第一个 (LIFO)。

您也可以将选择和焦点操作添加到委托中（不确定这是否是个好主意）。