23

当陷阱发生时，我想向任何订阅者发送警报消息。

我创建的代码使用委托方法可以正常工作myDelegate del。

我的问题是：

  1. 我想知道用它EventHandler代替委托是否更好？我不确定代表和EventHandler我的情况有什么区别。

  2. notify(trapinfo t)，这就是我在这里获取陷阱信息所做的。但这似乎不是一个好主意。我阅读了一些介绍传递委托对象的在线教程；我想知道这是否适合我的情况？我该怎么做？有什么建议么？

非常感谢 ：）

我的代码：

public class trapinfo
    {
        public string info;
        public string ip;
        public string cause;
    }

    public class trap
    {
        public delegate void myDelegate(trapinfo t);
        public myDelegate del;

        trapinfo info = new trapinfo();

        public void run()
        {
            //While(true) 
            // If a trap occurred, notify the subscriber
            for (; ; )
            {
                Thread.Sleep(500);
                foreach (myDelegate d in del.GetInvocationList())
                {
                    info.cause = "Shut Down";
                    info.ip = "192.168.0.1";
                    info.info = "Test";
                    d.Invoke(info);
                }
            }
        }
    }
    public class machine
    {
        private int _occuredtime=0;

        public trapinfo info = new trapinfo();
        public void notify(trapinfo t)
        {
            ++_occuredtime;
            info.cause = t.cause;
            info.info = t.info;
            info.ip = t.ip;
            getInfo();
        }
        public void subscribe(trap t)
        {
            t.del += new trap.myDelegate(notify);
        }
        public void getInfo()
        {
            Console.WriteLine("<Alert>: cauese/{0}, info/ {1}, ip/{2}, time/{3}",
                info.cause, info.info, info.ip,_occuredtime);
        }
    }
    class Program
    {
        static void Main(string[] args)
        {
            trap t = new trap();
            machine machineA = new machine();
            machineA.subscribe(t);
            t.run();
        }
    }

2013-08-12 更新

观察者/可观察的设计模式怎么样，在我的例子中看起来很棒（EventHandler）。

在我的例子中，一台机器订阅了一个陷阱信使。（将机器添加到调用列表）一旦发生陷阱，我会向所有订阅的机器发送消息。（打电话HandleEvent来处理）

好处：

  • 不再关心GetInvocationList()，只需使用(+=)和(-=)决定发送陷阱的人。

  • 更容易理解我的程序的逻辑。

我知道有几种方法可以做到这一点，但我希望我能分析一下它的优缺点。

并感谢您的意见和建议，这将非常有帮助！

我阅读了Matthew Watson 建议的MSDN EventArgs文章。

这是我的活动版本：

public class TrapInfoEventArgs : EventArgs
{
    public int info { get; set; }
    public string  ip { get; set; }
    public string cause { get; set; }
}
public class trap
{
    public event EventHandler<TrapInfoEventArgs> TrapOccurred;

    protected virtual void OnTrapOccurred(TrapInfoEventArgs e)
    {
        EventHandler<TrapInfoEventArgs> handler = TrapOccurred;
        if (handler != null)
        {
            handler(this, e);
        }
    }


    public void run()
    {
        //While(true) 
        // If a trap occurred, notify the subscriber
        for (; ; )
        {
            Thread.Sleep(500);
            TrapInfoEventArgs args = new TrapInfoEventArgs();
            args.cause = "Shut Down";
            OnTrapOccurred(args);
        }
    }
}
public class machine
{
    public void c_TrapOccurred(object sender, TrapInfoEventArgs e)
    {
        Console.WriteLine("<Alert>: cauese/{0}, info/ {1}, ip/{2}, time/{3}",
            e.cause, e.info, e.ip, DateTime.Now.ToString());
    }
}
class Program
{
    static void Main(string[] args)
    {
        trap t = new trap();
        machine machineA = new machine();
        t.TrapOccurred += machineA.c_TrapOccurred; //notify machine A
        t.run();
    }
}
4

2 回答 2

21

事件和委托的区别在于：

事件声明在委托实例上增加了一层保护。此保护可防止委托的客户端重置委托及其调用列表，并且仅允许在调用列表中添加或删除目标

请参阅委托和事件之间有什么区别？

2）在我看来，您的订阅者不应随意更改代表。一个订阅者可以分配=给它而不是添加+=. 这将分配一个新的委托，因此，先前的委托及其调用列表将丢失，并且不再调用先前的订阅者。所以你肯定应该使用 Event 。或者您可以更改您的代码以使您的委托私有并编写附加函数来操作它以定义您自己的事件行为。

 //preventing direct assignment
 private myDelegate del ;

    public void AddCallback(myDelegate m){
        del += m;
    }

    public void RemoveCallback(myDelegate m){
        del -= m;
    }

    //or
    public static trap operator +(trap x,myDelegate m){
        x.AddCallback(m);
        return x;
    }
    public static trap operator -(trap x, myDelegate m)
    {
        x.RemoveCallback(m);
        return x;
    }

//usage  

//t.AddCallback(new trap.myDelegate(notify));
  t+=new trap.myDelegate(notify);
于 2013-08-11T09:12:27.170 回答
8

最好使用 anevent作为您的示例。

  • Visual Studio Form 和 WPF 设计人员可以理解An event，因此您可以使用 IDE 订阅事件。

  • 在 raiseevents时，您无需编写自己的foreach处理来遍历它们。

  • events是大多数程序员期望访问此功能的方式。

  • 如果您使用委托，则使用代码可能会以您希望防止的方式（例如重置其调用列表）来弄乱它。events不允许这种情况发生。

至于您的第二个问题：使用 anevent您将创建一个派生自的类EventArgs来保存数据，并在您引发它时将其传递给事件。然后，消费者将可以访问它。

有关详细信息，请参见此处：http: //msdn.microsoft.com/en-us/library/system.eventargs.aspx

于 2013-08-11T09:11:20.227 回答