UIButton.OnClick() error handling

[mulova]

All UIButtons are not responsive after the UIButton.onClick EventDelegates throws error.
It's because UIButton.current is not null afterward.

1.         protected virtual void OnClick ()
2.         {
3.                 if (current == null && isEnabled)
4.                 {
5.                         current = this;
6.                         try {
7.                                 EventDelegate.Execute(onClick);
8.                         } finally {
9.                                 current = null;
10.                         }
11.                 }
12.         }
13.  

[ArenMook]

Ideally the try/catch block should be inside EventDelegate.Execute to cover all cases. Still, you shouldn't have exceptions in your code to begin with, they can lead to all kinds of problems.

[mulova]

You are absolutely right. But there may be exceptional cases hardly found.
And Programmers like me always make mistakes. 
If all the buttons doesn't work in real situation it would be like hell.
I think exception handling makes NGUI robust.

[ArenMook]

Yup, which is why I already added the try/catch block to the EventDelegate.Execute back when I wrote that last reply.