Re: [問題] event 和 delegate

[QQ29 (我愛阿蓉)]

請教各位~

我用Reflector去看他反組譯後的程式

發現如下

private EventHandler OnEvent;

        public event EventHandler OnEvent
        {
            add
            {
                EventHandler handler2;
                EventHandler onEvent = this.OnEvent;
                do
                {
                    handler2 = onEvent;
                    EventHandler handler3 = (EventHandler)
                      Delegate.Combine(handler2, value);
                    onEvent = Interlocked.CompareExchange<EventHandler>(ref
                    this.OnEvent, handler3, handler2);
                }
                while (onEvent != handler2);
            }
大方向和 Cole 大說的一樣

不過他多了一個do while並且作了一個判斷

但我查了一下

CompareExchange  比較 1和3的參數instance是否指向同一個 若一樣就讓1指向2

他一開始handle2 就指向onEvent 所以 CompareExchang 乍看之下一定會讓

ref this.Onevent 變成指向handle3(新增後的結果)

但為啥要把這local的 onEvent最後再指向CompareExchange回傳的原先值呢(還原)

然後while因為 條件不成立 跳出

搞得看起來while沒意義  根本可以不用這樣翻阿?

是有什麼考量或是 我誤解的地方呢

還是剛好這個interlock可以剛好做到multi-thread的保護

可是仔細想想

如果改成這樣

              do
              {
                lock (this)
                {
                   this.OnEvent = (EventHandler)Delegate.Combine(this.OnEvent2,
                   value);
                }
              }while(false);

不就好了嗎?

也可以做到 multi thread的保護 乍看之下也沒有問題....

請問一下 他那種寫法 有什麼意義嗎~
謝謝


※ 引述《cole945 (躂躂..)》之銘言：
:    delegate 和 event 的差別,
:    其實就像 field 和 property 的差別,
:    property 提供 getter/setter 來操作 field, 來避免不正確的付值,
:    event 的目的也是一樣, 提供 adder/remover 來操作 delegate
:    以大家常見的 EventHandler delegate 來舉例
:    public event EventHandler OnEvent;
:    其實是像
:    private EventHandler _onEvent;     <-- implicitly generated delegate
:    public event EventHandler OnEvent
:    {
:         add
:         {
:             _onEvent = (EventHandler)Delegate.Combine(_onEvent, value);
:         }
:         remove
:         {
:             _onEvent = (EventHandler)Delegate.Combine(_onEvent, value);
:         }
:    }
:    這就像 default property
:    public int MyProperty
:    {
:       set; get;
:    }
:    其實是如下的code是一樣的意思..
:    private int _MyProperty;     <-- impilictly declared
:    public int MyProperty
:    {
:       set
:       {
:           _MyProperty = value;
:       }
:       get
:       {
:           return _MyProperty;
:       }
:    }
:    event出現的目的就跟property一樣, 是為了隱藏 delegate 這個 list 的
:    實作細節, 多一個可以在中間操作的機會..
:    例如, 加多 multithread 的保護 (lock), 或是可以從中攔截 delegate
:    橋接到適當的 object 手上..
:    語法上的差異當然有, 但其實不是重點

--
※ 發信站: 批踢踢實業坊(ptt.cc)
◆ From: 219.87.64.222
※ 編輯: QQ29            來自: 219.87.64.222        (11/30 22:34)
※ 編輯: QQ29            來自: 219.87.64.222        (11/30 23:02)
推 petrushka:應該只是在不使用lock的情況下,確保combine前後,串列沒
→ petrushka:有被更改!如果combine的過程中,delegate串列更動了,下一
→ petrushka:再嘗試add value到串列中!如果用lock,基本上while也不需
→ petrushka:要! 但是, lock一整個delgate串列, 好嗎?
→ QQ29:不曉得耶 這樣感覺也OK阿 跟他的比起來又簡單  效率會差嗎?
推 deuter:解答在這裡 http://goo.gl/COGTD
→ deuter:看來是 C# 4.0 才開始用這樣的code
→ deuter:這四篇blog有詳細的說明 等有空再仔細讀