Question

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

这是我的组合框：

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

这是我的文本框（DataContext是在父网格上设置的。如果在文本框本身上设置了DataContext，则会出现相同的行为）

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

启动撤消后我使用的代码（代码段）是：

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;
                           }));
}

我知道我可以更新目的＆＃39;我的DataContext上的属性。这确实有效，但如果我这样做的话它并没有集中精力和选择。另外，由于所有绑定都已经到位并且其中一些绑定到整数，我只是更新文本并允许“魔法”＃39;处理翻译。这样我可以为任何文本框提供通用解决方案。为了完整起见，这是我的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，但复选框尚未更新其选定值，因此尚未执行更改值。

最终，问题在于您将逻辑与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）。

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

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

2 个答案: