C# ReaderWriterLockSlim Best Practice zur Vermeidung von Rekursionen

Question

Ich habe eine Klasse mit ReaderWriterLockSlim mit einer Lese- und einer Schreibmethode, die die read-Methode verwendet, um das zu ändernde Element abzurufen. Ein kurzes Beispiel wäre:

class FooLocker 
{ 
    ReaderWriterLockSlim locker = new ReaderWriterLockSlim(); 
    List<Foo> fooList = new List<Foo>(); 

    public void ChangeFoo(int index, string bar) 
    { 
     locker.EnterWriteLock(); 

     try 
     { 
      Foo foo = GetFoo(index); 
      foo.Bar = bar; 
     } 
     finally 
     { 
      locker.ExitWriteLock(); 
     } 
    } 

    public Foo GetFoo(int index) 
    { 
     locker.EnterReadLock(); //throws System.Threading.LockRecursionException 

     try 
     { 
      return fooList[index]; 
     } 
     finally 
     { 
      locker.ExitReadLock(); 
     } 
    } 

    //snipped code for adding instances etc. 
} 

Wie oben, dieser Code ein LockRecursionException wirft, wenn ChangeFoo() Aufruf weil eine Schreibsperre bereits gehalten wird, wenn GetFoo() versucht eine Lesesperre einzugeben.

Ich habe die Dokumentation für ReaderWriterLockSlim überprüft und ich kann LockRecursionPolicy.SupportsRecursion verwenden, um das oben genannte zu ermöglichen. In der Dokumentation wird jedoch empfohlen, dass dies nicht für neue Entwicklungen verwendet wird und nur bei der Aktualisierung von vorhandenem Code verwendet werden sollte.

Gegeben, was ist die beste Vorgehensweise, um das gleiche Ergebnis zu erreichen, wo eine Schreibmethode eine schreibgeschützte Methode verwenden kann, um das zu ändern, was geändert werden muss?

Answer 1

Sie können Ihre Klasse in exponierte Methoden und private innere Methoden aufteilen. Die inneren Methoden erledigen die Logik wie das Holen und die öffentlichen Methoden führen das Sperren aus. Beispiel:

class FooLocker 
{ 
    ReaderWriterLockSlim locker = new ReaderWriterLockSlim(); 
    List<Foo> fooList = new List<Foo>(); 


    public void ChangeFoo(int index, string bar) 
    { 
     locker.EnterWriteLock(); 

     try 
     { 
      Foo foo = UnsafeGetFoo(index); 
      foo.Bar = bar; 
     } 
     finally 
     { 
      locker.ExitWriteLock(); 
     } 
    } 

    public Foo GetFoo(int index) 
    { 
     locker.EnterReadLock(); //throws System.Threading.LockRecursionException 

     try 
     { 
     return UnsafeGetFoo(index); 
     } 
     finally 
     { 
      locker.ExitReadLock(); 
     } 
    } 

    private Foo UnsafeGetFoo(int index) 
    { 
     return fooList[index]; 
    } 
}