Is calling an extension method on a “null” reference (i.e. event with no subscribers) evil?

Question

Evil or not evil?

public static void Raise(this EventHandler handler, object sender, EventArgs args)
{
   if (handler != null)
   {
      handler(sender, arg         


        

Answer 1

You can always declare your events like this (not that i recommend it):

public event EventHandler OnClicked = delegate { };

That way they have something assigned to them when you call them, so they don't throw a null pointer exception.

You can probably get rid of the delegate keyword in C# 3.0...