NGUI List<EventDelegate> 小坑

NGUI 3.0 之后 采用了 一种 全新的 事件系统 List<EventDelegate> ，使用起来并不麻烦。

但是最近做项目碰到一个小问题，特此分享一下。

 

PS NGUI3.6.4版本

PS 有空还是看看 源码，有些 时候 光理解 还是没用的。

 

一、问题出现在UI 逻辑一块，在一个 UITween 调用之后 要回调一个 事件。

代码如下：

public class MyTest2 : MonoBehaviour
{
    public UITweener tween;


    List<EventDelegate> list;
    public virtual void Start () {
        list = tween.onFinished;

        EventDelegate.Add(tween.onFinished, Test);
    }

    List<string> list1 = new List<string>();
    void Test()
    {
        Debug.Log(list == tween.onFinished);

        EventDelegate.Remove(tween.onFinished, Test);

        Debug.Log(" Test ");
    }

    void OnGUI()
    {
        if (GUI.Button(new Rect(10, 10, 150, 100), "I am a button"))
            tween.PlayForward();
        if (GUI.Button(new Rect(10, 110, 150, 100), "I am a button2"))
            tween.PlayReverse();
    }


}

 

但是之前 我是 以为  EventDelegate.Remove 这样就能够把 Onfinished 委托给去除的。

结果我发现不是如此，之后 这个 Tween 每次 调用 结束 之后， Test() 都会执行。

 

二、源码分析

看了下源码。发现问题出在这里：

这段代码就是 tween 结束后 调用的。

这里把 mTemp =  OnFinish

再把 原来的OnFinish 指向了 一个 新 的 new 的 list

然后在 mTemp  execute 执行完毕 之后 ，再旧瓶装新酒，把mTemp重新装进 OnFinish 这个list.

 

 

 

来观察一下 Add 的 流程

我们可以看到 他 使用了 onFinished 的 list ， add 了 一个 包含了 test() Callback  的 EventDelegate 

 

接下里 执行 的的时候 使用了 同样的 list ，编辑得到 刚才那个  包含了 test() Callback  的 EventDelegate ，然后执行其中的 test()

 

同样的 Remove 操作也是如此。

 

但是 我们可以看到 tween Execute 的 时候 调用 原来 增加进来的 Test 函数,

可是在 Test 函数 中 这个时候引用 的 onFinished 已经 变过了。成为了一个新的 new list.并且还没有把 原来的 委托放进来。

所以这样一来 ， 即使你调用 EventDelegate.Remove ， 因为 onFinished  list.cout == 0 , 没有 remove  test()

也就意味着 tween 结束后 会永远调用 test() 。因为在 tween Execute 之后，onFinish 这个 list 才会加入 原来 所有的 委托，这个时候 list.cout 才是1. 重新 加入了 test() 这个时候 remove 才有意义。

 

 

三、解决方法

但是我感觉 治标 不 治本，如果 我 想 调用 好几次 再 Remove 呢 ？ 以后有空 在弄吧。

其实就是 在 执行的 时候 判断 一下，oneShot 标志位，执行完毕后mTemp 中 remove test()，再放到OnFisish 的时候 test() 就没有了。

public class MyTest2 : MonoBehaviour
{
    public UITweener tween;


    List<EventDelegate> list;
    public virtual void Start () {
        list = tween.onFinished;

        //EventDelegate.Add(tween.onFinished, Test);
        EventDelegate.Add(tween.onFinished, Test, true);
    }

    List<string> list1 = new List<string>();
    void Test()
    {
        Debug.Log(list == tween.onFinished);

        //EventDelegate.Remove(tween.onFinished, Test);

        Debug.Log(" Test ");
    }

    void OnGUI()
    {
        if (GUI.Button(new Rect(10, 10, 150, 100), "PlayForward"))
        {
            Debug.Log(" PlayForward");
            tween.PlayForward();
        }

        if (GUI.Button(new Rect(10, 110, 150, 100), "PlayReverse"))
        {
            Debug.Log(" PlayReverse ");
            tween.PlayReverse();
        }
            
    }
}

 

上结果：