Xử lý các điều kiện Race trong C#

Question

Tôi đang viết một ứng dụng có giao diện truyền thông lớp.
Điều này được thực hiện để trừu tượng hóa các giao tiếp từ phần giao diện người dùng của ứng dụng và cũng để làm cho nó có khả năng mở rộng/duy trì hơn.
Ví dụ:

xem xét mỗi hộp trong hình trên như là một lớp riêng biệt.
Giao diện chung của giao diện phổ biến các biến chuỗi mô tả dữ liệu được giao dịch và liên kết "sức khỏe", lần lượt được sao chép lên Ứng dụng thông qua một loạt các cuộc gọi chức năng công khai. Ví dụ, ứng dụng sẽ thực hiện cuộc gọi đến App-Sub-System:

class Application 
{ 
    private void SomeUpdateFunction() 
    { 
     this.textBox1.AppendText(this.AppSubSystem.GetText()); 
    } 
} 

class AppSubSystem 
{ 
    public string GetText() 
    { 
     return this.GenericCommsInterface.GetText(); 
    } 
} 

class GenericCommsInterface 
{ 
    public string GetText() 
    { 
     string sRetVal = this.sText; // sText is populated by other functions in the class. 
     this.sText = null; // Suspected race condition is here. 
     return sRetVal; 
    } 
} 

sText là dân cư không đồng bộ bởi các chức năng khác trong lớp.
Tôi tin rằng tình trạng cuộc đua đang diễn ra giữa string sRetVal = this.sText; và dòng sau this.sText = null;.
Ai đó có thể đề xuất một cách để tránh hoặc ngăn chặn tình trạng cuộc đua này? Sẽ sử dụng trợ giúp StringBuilder hoặc có cách nào khác mà tôi nên thực hiện việc này không?

Answer 1

Có thể bạn nên mua khóa bất cứ lúc nào bạn muốn chạm vào this.sText - trong các chức năng cập nhật nó, cũng như chức năng GetText của bạn. Điều này sẽ đảm bảo rằng chỉ có một chủ đề tại một thời gian là rối tung với nó, như (giả sử thread của bạn có khóa) chủ đề khác sẽ ngồi và chờ đến khi thread hiện tại được thực hiện.

Tôi khuyên bạn nên sử dụng StringBuilder, một phần để đơn giản hóa khóa, như khóa chuỗi đã xảy ra hoặc được chuyển đổi ở giữa hoạt động bị khóa (và do đó được mở khóa, từ góc nhìn của người ngoài) gây ra mojo xấu. Something như thế này sẽ giúp:

lock (this.sbText) 
{ 
    sRetVal = this.sbText.ToString(); 
    this.sbText.Length = 0; 
} 

Ngoài ra, bạn có thể khóa trên this, nhưng đó là xấu xí - ổ khóa của bạn nên được bên trong, như tin càng tốt, để tránh tác dụng phụ kỳ lạ (như nếu một số khác đối tượng đã cố gắng để có được một khóa trên đối tượng này - nó không thể làm như vậy trong khi sbText đã được thay đổi).

Answer 2

public string GetText() 
{ 
    lock(someObject) 
    { 
     string sRetVal = this.sText; // sText is populated by other functions in the class. 
     this.sText = null; // Suspected race condition is here. 
     return sRetVal; 
    } 
}  

trong thiết lập của bạn

lock(someObject) 
{ 
    //... 
    this.sText = value; 
} 

Answer 3

Mã này chắc chắn sẽ không làm việc trong một môi trường ren là bạn không bảo vệ stab. Bạn cần khóa tất cả mọi người truy cập vào nó.