事件循环

时间:2016-07-25 10:11:42

标签: c# winforms user-interface circular-reference

在以下情况下,我经常发现自己:

我有一个绑定到某些数据的用户控件。每当更新控件时,都会更新基础数据。每当更新基础数据时,控件都会更新。因此,很容易陷入永无止境的更新循环(控制更新数据,数据更新控制,控制更新数据等)。

通常我通过bool(例如updatedByUser)解决这个问题所以我知道控件是以编程方式还是由用户更新,然后我可以决定是否触发事件以更新基础数据。这似乎不是很整洁。

是否有一些处理此类情景的最佳做法?

编辑:我已添加以下代码示例,但我想我已经回答了我自己的问题......?

public partial class View : UserControl
{
    private Model model = new Model();

    public View()
    {
        InitializeComponent();
    }

    public event EventHandler<Model> DataUpdated;

    public Model Model
    {
        get
        {
            return model;
        }
        set
        {
            if (value != null)
            {
                model = value;
                UpdateTextBoxes();
            }
        }
    }

    private void UpdateTextBoxes()
    {
        if (InvokeRequired)
        {
            Invoke(new Action(() => UpdateTextBoxes()));
        }
        else
        {
            textBox1.Text = model.Text1;
            textBox2.Text = model.Text2;
        }
    }

    private void textBox1_TextChanged(object sender, EventArgs e)
    {
        model.Text1 = ((TextBox)sender).Text;
        OnModelUpdated();
    }

    private void textBox2_TextChanged(object sender, EventArgs e)
    {
        model.Text2 = ((TextBox)sender).Text;
        OnModelUpdated();
    }

    private void OnModelUpdated()
    {
        DataUpdated?.Invoke(this, model);
    }
}

public class Model
{
    public string Text1 { get; set; }
    public string Text2 { get; set; }
}

public class Presenter
{
    private Model model;
    private View view;

    public Presenter(Model model, View view)
    {
        this.model = model;
        this.view = view;

        view.DataUpdated += View_DataUpdated;
    }

    public Model Model
    {
        get
        {
            return model;
        }
        set
        {
            model = value;
            view.Model = model;
        }
    }

    private void View_DataUpdated(object sender, Model e)
    {
        //This is fine.
        model = e;

        //This causes the circular dependency.
        Model = e;
    }
}

3 个答案:

答案 0 :(得分:2)

一种选择是在数据自上次以来没有改变的情况下停止更新。例如,如果数据是类的形式,您可以检查数据是否与上次触发事件时的实例相同,如果是这种情况,请停止传播。

这是许多MVVM框架在事件未实际更改时阻止引发PropertyChanged事件的行为:

private string _someProperty = "";
public string SomeProperty
{
    get
    {
        return _someProperty;
    }
    set
    {
        if ( _someProperty != value )
        {
           _someProperty = value;
           RaisePropertyChanged();
        }
    }
}

您可以类似地为Windows窗体实现此概念。

答案 1 :(得分:1)

您要找的是Data Binding。它允许您连接两个或多个属性,以便当一个属性更改时,其他属性将自动神奇地更新。

在WinForms中它有点难看,但在你的情况下就像魅力一样。首先,您需要一个代表您的数据的类,并实现INotifyPropertyChanged以在数据更改时通知控件。

public class ViewModel : INotifyPropertyChanged
{

    private string _textFieldValue;
    public string TextFieldValue {
        get
        {
            return _textFieldValue;
        }

        set
        {
            _textFieldValue = value;
            NotifyChanged();
        }
    }

    public void NotifyChanged()
    {
        if (PropertyChanged != null) PropertyChanged(this, new PropertyChangedEventArgs(null));
    }

    public event PropertyChangedEventHandler PropertyChanged;
}

Form/Control中,您将ViewModel.TextFieldValue的值绑定到textBox.Text。这意味着只要TextFieldValue的值发生更改,Text属性就会更新,并且每次Text属性更改TextFieldValue都会更新。换句话说,这两个属性的值将是相同的。这解决了你遇到的循环循环问题。

public partial class Form1 : Form
{
    public ViewModel ViewModel = new ViewModel();

    public Form1()
    {
        InitializeComponent();
        // Connect:  textBox1.Text <-> viewModel.TextFieldValue
        textBox1.DataBindings.Add("Text", ViewModel , "TextFieldValue");
    }
}

如果您需要修改表单/控件外部的值,只需设置ViewModel

的值
form.ViewModel.TextFieldValue = "new value";

控件将自动更新。

答案 2 :(得分:0)

你应该研究MVP - 它是Winforms UI的首选设计模式。 http://www.codeproject.com/Articles/14660/WinForms-Model-View-Presenter

除了允许您避免循环事件之外,

使用该设计模式为您提供了更易读的代码。

为了实际避免循环事件,您的视图应该只导出一个属性,一旦设置它将确保不会调用txtChanged_Event。

类似的东西:

public string UserName
    {
        get
        {
            return txtUserName.Text;
        }
        set
        {
            txtUserName.TextChanged -= txtUserName_TextChanged;
            txtUserName.Text = value;
            txtUserName.TextChanged += txtUserName_TextChanged;
        }
    }

或者您可以使用私人财产的MZetko's答案