ロックを使用して、以前の値に基づいて2つの関連フィールドを変更する最も効率的な方法はありますか？

Question

私はInterlocked.CompareExchangeを使用する方が高速であることを知っていますが、ロックに制限されている場合は最良の方法を探しています。

Joe Albahari says the following:

それは安全に（例えば、それが「増加」にPERCENTCOMPLETE値が可能です）、その前回値に基づいて新しいProgressStatusオブジェクトを割り当てることも可能です - コードの複数行にわたってロックせず。

編集：@ drchの回答に基づいて更新されたコード。小さなより効率的なロックでこれを行う方法があるのを待っています。このロックは、読み込み、書き込み、および新しいコンストラクタをすべてロック内に持ちます。より小さな\より効率的なロックを望んでいた。ありがとう。

class ProgressStatus 
{ 
    public readonly int PercentComplete; 
    public readonly string StatusMessage; 
    public ProgressStatus(int percentComplete, string statusMessage) 
    { 
     PercentComplete = percentComplete; 
     StatusMessage = statusMessage; 
    } 
} 
class Test 
{ 
    readonly object _statusLocker = new object(); 
    ProgressStatus _status; 
    void IncreaseProgress() 
    { 
     lock (_statusLocker) 
      _status = new ProgressStatus(_status.PercentComplete + 10, _status.StatusMessage); 
    } 
} 

Answer 1

上記の記事では、1行でロックするだけで、後でPart 5という高度なテクニックが参照されていました。

上記のコードに関しては、2つのロックの間で競合状態が発生する可能性があります。 2つのスレッドは同じ値を読み取ってから、両方のインクリメントを必要とする場合に同じ値を書き込むことができます。

ロック全体の読み取りと書き込みオーバー：

lock (_statusLocker) { 
    statusCopy = _status; 
    var newStatus = new ProgressStatus(statusCopy.PercentComplete + 10, statusCopy.StatusMessage); 
    _status = newStatus; 
} 

それとも単に：

lock (_statusLocker) { 
    _status = new ProgressStatus(_status.PercentComplete + 10, _status.StatusMessage); 
} 

Answer 2

IMOでは、上記のコードを使用して、ロックなしで行うことができます。これは、不変クラスを使用し、データのコピーのみを使用しているためです。読み取りと書き込みの参照は、C＃仕様によってアトミックになることが保証されています。

編集：しかし、スレッドの同期がABは***ですので、私はいくつかのより多くの意見を聞きたいと思います...

編集：私はコメントで指摘したようにはまた、欠陥がありますコード内に複数のスレッドがメソッドを呼び出す場合この場合、更新が失われる可能性があります。