Tôi có nên khóa 'sự kiện' không?

Question

Tôi có nên khóa sự kiện trong trường hợp sau:

sự kiện foo;

chủ đề A: sẽ gọi foo + = handler;

chủ đề B: sẽ gọi foo - = handler;

Tôi có nên khóa foo không?

Answer 1

Khóa trên foo là một ý tưởng tồi, bởi vì giá trị sẽ thay đổi mỗi lần. Bạn nên khóa trên một biến mà không thay đổi:

private readonly object eventLock = new object(); 
private EventHandler fooHandler; 

public event EventHandler Foo 
{ 
    add 
    { 
     lock (eventLock) 
     { 
      fooHandler += value; 
     } 
    } 
    remove 
    { 
     lock (eventLock) 
     { 
      fooHandler -= value; 
     } 
    } 
} 

private void OnFoo(EventArgs e) 
{ 
    EventHandler handler; 
    lock (eventLock) 
    { 
     handler = fooHandler; 
    } 
    if (handler != null) 
    { 
     handler(this, e); 
    } 
} 

Lưu ý rằng nếu bạn sử dụng một sự kiện trường như thế nào, như thế này:

public event EventHandler Foo; 

sau đó bạn sẽ tự động nhận được một " khóa (này) "trên thêm/loại bỏ, mặc dù bạn sẽ phải tự thêm nó khi tìm nạp trình xử lý trước khi gọi nó (giả sử bạn muốn chắc chắn rằng bạn đọc giá trị được ghi gần đây nhất). Cá nhân tôi không phải là một fan hâm mộ của khóa trên "này", nhưng bạn có thể không nhớ - và nó chắc chắn làm cho mã đơn giản hơn.