Khóa trên một đối tượng

Question

Tôi thường thấy mã như vậy là shown here, tức là nơi một đối tượng được phân bổ và sau đó được sử dụng làm "đối tượng khóa".

Dường như với tôi rằng bạn có thể sử dụng bất kỳ đối tượng nào cho điều này, bao gồm cả sự kiện chính nó làm đối tượng khóa. Tại sao phân bổ một đối tượng mới mà không có gì? Sự hiểu biết của tôi là gọi khóa() trên một đối tượng không thực sự thay đổi đối tượng, cũng không thực sự khóa nó khỏi sử dụng, nó đơn giản được sử dụng như một trình giữ chỗ cho nhiều câu lệnh khóa để neo.

public class Shape : IDrawingObject, IShape 
{ 
    // Create an event for each interface event 
    event EventHandler PreDrawEvent; 
    event EventHandler PostDrawEvent; 

    object objectLock = new Object(); 

    // Explicit interface implementation required. 
    // Associate IDrawingObject's event with 
    // PreDrawEvent 
    event EventHandler IDrawingObject.OnDraw 
    { 
     add 
     { 
      lock (objectLock) 
      { 
       PreDrawEvent += value; 
      } 
     } 
     remove 
     { 
      lock (objectLock) 
      { 
       PreDrawEvent -= value; 
      } 
     } 
    } 
} 

Vì vậy, câu hỏi của tôi là, đây thực sự là một điều tốt để làm?

Answer 1

bao gồm các sự kiện riêng của mình

Không, bạn không thể làm điều đó. Một "sự kiện" thực sự chỉ là một số phương thức truy cập. Giả sử bạn có nghĩa là đại biểu ủng hộ, điều đó sẽ rất tệ - các đại biểu không thay đổi được: mỗi khi bạn thêm/xóa người đăng ký, bạn sẽ nhận được các đại biểu khác nhau.

Thực ra, trình biên dịch 4.0 hiện thực hiện điều này với mã không khóa bằng cách sử dụng Interlocked - có thể đáng giá theo cách tiếp cận này thay thế.

Trong ví dụ của bạn, objectLock đảm bảo rằng tất cả người gọi (để dụ đó) là khóa chống cùng đối tượng, đó là quan trọng - nhưng không có sự xấu xí của khóa trên this (đó là cách biên dịch C# sử dụng làm việc).

-

Cập nhật: ví dụ của bạn hiển thị mã số đó là cần thiết trước khi C# 4.0, truy cập vào một lĩnh vực như sự kiện bên kiểu nói chuyện trực tiếp đến lĩnh vực: lĩnh vực như sự kiện bình thường khóa không được tôn trọng. Điều này đã được thay đổi trong C# 4.0; bạn có thể bây giờ (trong C# 4.0) an toàn lại viết những dòng này như:

public class Shape : IDrawingObject, IShape 
{ 
    // Create an event for each interface event 
    event EventHandler PreDrawEvent; 
    event EventHandler PostDrawEvent; 

    event EventHandler IDrawingObject.OnDraw 
    { 
     add { PreDrawEvent += value; } 
     remove { PreDrawEvent -= value; } 
    } 
} 

Tất cả các hành vi đúng là sau đó theo sau.

Answer 2

Bạn nên khóa trên một trường tĩnh riêng vì điều này đảm bảo rằng nhiều luồng cố gắng truy cập khóa song song sẽ bị chặn. Khóa trên thể hiện của chính lớp đó (lock(this)) hoặc một số trường thể hiện có thể có vấn đề bởi vì nếu hai luồng gọi phương thức trên hai trường hợp khác nhau của đối tượng, chúng sẽ có thể đồng thời nhập vào câu lệnh khóa.

Answer 3

Bất kỳ thành viên loại tham chiếu riêng tư nào cũng sẽ thực hiện công việc. Miễn là nó là riêng tư và không bao giờ được phân công lại. Mà gõ một đối tượng đại biểu ra khỏi chạy, bạn chắc chắn không muốn nhìn thấy một khóa không làm công việc của mình chỉ đơn giản bởi vì mã khách hàng mà bạn không kiểm soát gán một xử lý sự kiện. Vô cùng khó khăn để gỡ lỗi.

Sử dụng thành viên riêng tư thực hiện công việc khác không phải là thứ có quy mô tốt. Nếu bạn phát hiện ra bạn cần khóa một vùng mã khác trong khi tái cấu trúc hoặc gỡ lỗi, bạn sẽ cần tìm một thành viên riêng khác. Đó là nơi mọi thứ có thể biến đổi nhanh chóng: bạn có thể chọn lại cùng một thành viên riêng tư. Deadlock gõ cửa.

Điều này không xảy ra nếu bạn dành một đối tượng khóa cho một tập hợp các biến chia sẻ cụ thể cần được bảo vệ. Cho phép bạn cung cấp cho nó một cái tên tốt quá.