如何重构重复的事件处理代码

Question

我有以下类，它允许某些对象订阅更改事件。问题是我还有类 B 和 C 需要此功能，允许对象订阅相同类型的事物。我们当然不想复制和粘贴这种行为。

我们考虑过从公共基类继承，但是我们所有的类（包括 A、B 和 C）都已经从公共基类继承。我们不想将此行为添加到 BaseClass 中，因为从 BaseClass 继承的其他类 E、F、G 不需要此行为。

有更好的解决方案吗？

public class A : BaseClass
{

    /*other properties and code */

    public event EventHandler OnChange;
    private bool _hasChanged;
    public bool HasChanged
    {
        get { return _hasChanged; }
        set
        {
            _hasChanged = value;
            //only need to notify when we've changed.
            if (value)
            {
                if (OnChange != null)
                    OnChange(this, EventArgs.Empty);
            }
        }
    }
}

Answer 1

考虑一种面向方面的编程方法，例如 PostSharp 示例 中使用的方法。它允许您使用属性注入此类样板代码。

如果您创建了适当的方面，则可以使用如下代码：

public class A : BaseClass
{
    public event EventHandler OnChanged;

    [ChangedNotify("OnChanged")]
    public bool HasChanged { get; set; }
}

或者，如果想法是为多个属性使用单个 OnChange 事件，您可以将其硬编码到方面中，从而减少代码到

public class A : BaseClass
{
    [NotifyOnChanged]
    public bool HasChanged { get; set; }
}

Answer 2

如果我们暂时不使用继承怎么办？

1-假设，不是从公共基类继承，而是用实现事件机制的对象来编写需要事件机制的客户端类。

假设我们的类是

 public class EventNotifier
{
    public event EventHandler OnChange;
    private bool _hasChanged;
    public bool HasChanged
    {
        get { return _hasChanged; }
        set
        {
            _hasChanged = value;
            //only need to notify when we've changed. 
            if (value)
            {
                if (OnChange != null)
                    OnChange(this, EventArgs.Empty);
            }
        }
    }
}

2-

 public class A
{
    private EventNotifier eventNotifier;
    public EventNotifier MyEventNotifier { get { return eventNotifier; } }

    public A()
    {
        eventNotifier = new EventNotifier();
    }


}

3- 现在你的类 A 的用户（继承/组合类 A 的类）

这是用于如果 B 包含 A

 public class b
{
    A obj ;
    public b()
    {
        obj = new A();
        obj.MyEventNotifier.OnChange += new EventHandler(delegate { Console.WriteLine("Hi"); });
        obj. MyEventNotifier.HasChanged = true;
    }
}

Answer 3

您可以考虑在 BaseClass 和 A、B、C 之间引入一个包含常见行为的中间类。这样你就不会污染不需要该行为的 E、F、G。

             BaseClass
  -----------------------------
  |                           |
-----                   NotifyBaseClass
E,F,G                         |
                            -----
                            A,B,C

注意 虽然 AOP 看起来很不错，但我在尝试让 Postsharp 与其他技术（例如 MS Code Analysis 和 MSBuild）一起使用时遇到了重大问题。

Answer 4

为可通知对象创建一个子类可能是一种可行的方法，但如果将方面以这种方式乘以多种不同的类，则可能会很棘手。另一种方法是将其包含在您的基类中并为其定义一个接口，然后您可以简单地添加相关类的接口。

运行时，只需检查它是否是 IChangeable （或其他内容），然后仅连接到事件