I\'m writing a special data structure that will be available in a .NET library and one of the features of this data structure is that is will be thread safe provided that only o
Could you not require the Read and Write methods take a thread or thread ID? Then you can just compare against the one that called it first, and if it doesn't match, throw an exception or return an error code, or ignore the request.
Otherwise, what you propose should work also. You need only compare the thread IDs.
Rather than compare thread ID's you should store the ambient Thread in your class during construction.
class SingleThreadedClass
{
private Thread ownerThread;
public SingleThreadedClass()
{
this.ownerThread = Thread.CurrentThread;
}
public void Read(...)
{
if (Thread.CurrentThread != this.ownerThread)
throw new InvalidOperationException();
}
public void TakeOwnership()
{
this.ownerThread = Thread.CurrentThread;
}
}
When I run into this situation I use a class I wrote called ThreadAffinity. It's entire purpose is to record the current thread and throw on invalid accesses from a different thread. You have to manually do the check but it encapsulates the small amount of work for you.
class Foo {
ThreadAffinity affinity = new ThreadAffinity();
public string SomeProperty {
get { affinity.Check(); return "Somevalue"; }
}
}
Class
[Immutable]
public sealed class ThreadAffinity
{
private readonly int m_threadId;
public ThreadAffinity()
{
m_threadId = Thread.CurrentThread.ManagedThreadId;
}
public void Check()
{
if (Thread.CurrentThread.ManagedThreadId != m_threadId)
{
var msg = String.Format(
"Call to class with affinity to thread {0} detected from thread {1}.",
m_threadId,
Thread.CurrentThread.ManagedThreadId);
throw new InvalidOperationException(msg);
}
}
}
Blog post on the subject: