22

我有三个对象，ObjectA 有一个 ObjectB，ObjectB 有一个 ObjectC。当 ObjectC 触发一个事件时，我需要 ObjectA 知道它，所以这就是我所做的......

public delegate void EventFiredEventHandler();

public class ObjectA
{
    ObjectB objB;

    public ObjectA()
    {
        objB = new ObjectB();
        objB.EventFired += new EventFiredEventHandler(objB_EventFired);
    }

    private void objB_EventFired()
    {
        //Handle the event.
    }
}

public class ObjectB
{
    ObjectC objC;

    public ObjectB()
    {
        objC = new ObjectC();
        objC.EventFired += new EventFiredEventHandler(objC_EventFired);
        objC.FireEvent();
    }

    public event EventFiredEventHandler EventFired;
    protected void OnEventFired()
    {
        if(EventFired != null)
        {
            EventFired();
        }
    }

    private void objC_EventFired()
    {
            //objC fired an event, bubble it up.
        OnEventFired();
    }
}

public class ObjectC
{
    public ObjectC(){}

    public void FireEvent()
    {
        OnEventFired();
    }

    public event EventFiredEventHandler EventFired;
    protected void OnEventFired()
    {
        if(EventFired != null)
        {
            EventFired();
        }
    }
}

这是处理这个问题的正确方法，还是有更好的方法？我根本不希望 ObjectA 知道 ObjectC，只是它引发了一个事件。

4

3 回答 3

21

另一种方法是使用添加/删除来包装它：

public class ObjectB
{
    ObjectC objC;

    public ObjectB()
    {
        objC = new ObjectC();
    }

    public event EventFiredEventHandler EventFired
    {
        add { this.objC.EventFired += value; }
        remove { this.objC.EventFired -= value; }
    }
}
于 2010-12-03T19:54:33.343 回答
4

我就是这样做的。但是我建议将您的触发机制更改为此以使其线程安全

protected void OnEventFired()
{
    var tmpEvent = EventFired;
    if(tmpEvent != null)
    {
        tmpEvent();
    }
}

如果 EventFired 在 null 检查和触发之间变为 null，这可以防止它失败。

此外，为您的事件委托遵循EventHandler 模式在某种程度上是一种标准。

protected virtual void OnEventFired(EventArgs e)
{
    var tmpEvent = EventFired;
    if(tmpEvent != null)
    {
        tmpEvent(this, EventArgs.e);
    }
}

我错了线程安全模式，这是完整的线程安全事件模式

/// <summary>
/// Delegate backing the SomeEvent event.
/// </summary>
SomeEventHandler someEvent;

/// <summary>
/// Lock for SomeEvent delegate access.
/// </summary>
readonly object someEventLock = new object();

/// <summary>
/// Description for the event
/// </summary>
public event SomeEventHandler SomeEvent
{
    add
    {
        lock (someEventLock)
        {
            someEvent += value;
        }
    }
    remove
    {
        lock (someEventLock)
        {
            someEvent -= value;
        }
    }
}

/// <summary>
/// Raises the SomeEvent event
/// </summary>
protected virtual OnSomeEvent(EventArgs e)
{
    SomeEventHandler handler;
    lock (someEventLock)
    {
        handler = someEvent;
    }
    if (handler != null)
    {
        handler (this, e);
    }
}
于 2010-12-03T19:43:03.450 回答
1

正如其他答案所述，这是他们的方法。

但是你可以超越！！！我刚刚在它上面实现了一个很好的数据结构，它就像给你一个旋转。

有一个自动事件冒泡会很好吗？您可以使用反射来实现它。我的方法是定义一个接口/基类来声明一个事件（或一组事件）。然后，基类的无参数构造函数将迭代其其他属性/字段，并自动注册成员事件以进行事件传播。

设计上有一些限制，但如果您有一个深层结构和/或许多（结构化）事件，那么在没有任何额外代码行的情况下设置所有内容可能会很好。

初始基类可以是：

class BaseObject {
    public BaseObject() {
        FieldInfo[] fInfos = this.GetType().GetFields(...);

        foreach (FieldInfo fInfo in fInfos) {
            object fInfoValue = fInfo.GetValue(this, null);
            if (fInfoValue is BaseObject) {
                BaseObject bMemberObject = (BaseObject)fInfoValue;

                bMemberObject.MyEvent += new EventHandler(delegate() {
                    if (this.MyEvent != null)
                        MyEvent();
                });
            }
    }

    public event MyEvent = null;

}

当然，正如已经建议的那样，遵循事件委托委托（对象发送者，EventArgs args）（为了清楚起见，我使用了一个更简单的事件）。自然，您的类A、B和C直接从BaseObject派生是隐含的。

请注意，可以实现任何逻辑来绑定结构化事件（您可以是使用名称和/或其他反射属性的嵌套事件注册。

于 2010-12-03T20:14:40.807 回答