MVP模式、WinForms - EventHandlers或Delegates

Question

我在几个使用被动视图的WinForms应用程序中实现了MVP模式。我通过实现一个包含属性和委托（以Action< T >和Func< T >形式）的接口来连接具体视图中的UI事件并回调到Presenter。

我即将开始一个新项目，并在网上进行了一些关于该模式的研究，包括这里的许多模式示例，并注意到它们都使用EventHandler来通知Presenter。

我不太理解为什么会在这种情况下使用事件，因为我认为视图只有一个Presenter。

我的问题是，这是为了保持与.NET框架使用事件的一致性，还是出于其他我没有看到的原因？

以下是我使用的模式的一个微不足道的示例：

public interface IViewAbstraction
{
    public ModelData ModelData { set; }
    public Action<ModelData> ModelDataChangedCallback { set; }
}

public class SomeWinForm : Form, IViewAbstraction
{
    private Action<ModelData> modelDataChanged;
    private ModelData model;

    public ModelData ModelData
    {
        set { /* when this property changes, update UI */ }
    }

    public Action<ModelData> ModelDataChangedCallback
    {
        set { modelDataChanged = value; }
    }

    private void OnSomeWinformsEvent(EventArgs args)
    {
        if (modelDataChanged == null) throw new Exception();

        modelDataChanged(model);
    }
}

public class Presenter
{
    private readonly IViewAbstraction view;
    private readonly IDataLayerAbstraction dataLayer;

    public Presenter(IViewAbstraction view, IDataLayerAbstraction dataLayer)
    {
        this.dataLayer = dataLayer;
        this.view = view;
        this.view.ModelDataChangedCallback = OnModelChanged;
        this.view.ModelData = dataLayer.GetData();
    }

    private void OnModelChanged(ModelData data)
    {
        // validate and save data.
    }
}

- Neil M

Answer 1

您的模式基本上与使用事件相同，但有一个关键区别。事件不公开底层委托（在您的示例中为ModelDataChangedCallback）。公开这一点是不好的实践，因为其他代码可以清除调用列表。事件将具有底层委托，可以添加或删除，但从类的范围之外永远不会被清除。

我不明白您关于不拥有多个订阅者的观点 - 这并不是不使用事件的原因。事件只是一个类表达“嗨！这件事已经发生”的方式，与Presenter对象的1对1映射是完全合理和正常的。

此外，您也不会得到那个看起来很奇怪的只写属性。

- RichK

---

同意只写属性，当我看到它们时，它会让我感到困扰，并且为复杂视图添加了比我想要的更多的样板代码。 - Neil M

但是，当使用事件时，他必须创建一个特定的ModelDataChangedEventArgs类来包含Presenter需要处理的数据，对吗？当有很多事件时，模板代码类的数量将会爆炸。所有的Action<T> MyDelegate都必须更改为EventHandler<MyCallbackEventArgs> MyEvent...由于这是一种1:1的映射，清除调用列表的可能性应该非常有限... - Joan Charmant