18

这是一个有趣的图书馆作家的困境。在我的库中(在我的情况下为 EasyNetQ),我正在分配线程本地资源。因此,当客户端创建一个新线程然后在我的库上调用某些方法时,就会创建新资源。在 EasyNetQ 的情况下,当客户端在新线程上调用“发布”时,会创建到 RabbitMQ 服务器的新通道。我希望能够检测到客户端线程何时退出,以便我可以清理资源(通道)。

我想出的唯一方法是创建一个新的“观察者”线程,它只会阻塞对客户端线程的 Join 调用。这里做一个简单的演示:

首先是我的“图书馆”。它抓取客户端线程,然后创建一个阻塞“加入”的新线程:

public class Library
{
    public void StartSomething()
    {
        Console.WriteLine("Library says: StartSomething called");

        var clientThread = Thread.CurrentThread;
        var exitMonitorThread = new Thread(() =>
        {
            clientThread.Join();
            Console.WriteLine("Libaray says: Client thread existed");
        });

        exitMonitorThread.Start();
    }
}

这是一个使用我的图书馆的客户。它创建一个新线程,然后调用我的库的 StartSomething 方法:

public class Client
{
    private readonly Library library;

    public Client(Library library)
    {
        this.library = library;
    }

    public void DoWorkInAThread()
    {
        var thread = new Thread(() =>
        {
            library.StartSomething();
            Thread.Sleep(10);
            Console.WriteLine("Client thread says: I'm done");
        });
        thread.Start();
    }
}

当我这样运行客户端时:

var client = new Client(new Library());

client.DoWorkInAThread();

// give the client thread time to complete
Thread.Sleep(100);

我得到这个输出:

Library says: StartSomething called
Client thread says: I'm done
Libaray says: Client thread existed

所以它有效,但它很丑。我真的不喜欢所有这些被阻塞的观察者线程挂在周围的想法。有没有更好的方法来做到这一点?

第一种选择。

提供一个返回实现 IDisposable 的 worker 的方法,并在文档中明确说明您不应在线程之间共享 worker。这是修改后的库:

public class Library
{
    public LibraryWorker GetLibraryWorker()
    {
        return new LibraryWorker();
    }
}

public class LibraryWorker : IDisposable
{
    public void StartSomething()
    {
        Console.WriteLine("Library says: StartSomething called");
    }

    public void Dispose()
    {
        Console.WriteLine("Library says: I can clean up");
    }
}

客户端现在有点复杂:

public class Client
{
    private readonly Library library;

    public Client(Library library)
    {
        this.library = library;
    }

    public void DoWorkInAThread()
    {
        var thread = new Thread(() =>
        {
            using(var worker = library.GetLibraryWorker())
            {
                worker.StartSomething();
                Console.WriteLine("Client thread says: I'm done");
            }
        });
        thread.Start();
    }
}

此更改的主要问题是它是 API 的重大更改。现有客户将不得不重新编写。现在这不是一件坏事,这意味着重新审视它们并确保它们正确清理。

非破坏性第二种选择。API 为客户端提供了一种声明“工作范围”的方法。范围完成后,库可以清理。该库提供了一个实现 IDisposable 的 WorkScope,但与上面的第一个替代方案不同,StartSomething 方法保留在 Library 类中:

public class Library
{
    public WorkScope GetWorkScope()
    {
        return new WorkScope();
    }

    public void StartSomething()
    {
        Console.WriteLine("Library says: StartSomething called");
    }
}

public class WorkScope : IDisposable
{
    public void Dispose()
    {
        Console.WriteLine("Library says: I can clean up");
    }
}

客户端只需将 StartSomething 调用放在 WorkScope 中......

public class Client
{
    private readonly Library library;

    public Client(Library library)
    {
        this.library = library;
    }

    public void DoWorkInAThread()
    {
        var thread = new Thread(() =>
        {
            using(library.GetWorkScope())
            {
                library.StartSomething();
                Console.WriteLine("Client thread says: I'm done");
            }
        });
        thread.Start();
    }
}

我不喜欢第一种选择,因为它不会强迫图书馆用户考虑范围。

4

5 回答 5

9

您可以创建具有终结器的线程静态监视器。当线程处于活动状态时,它将持有监视器对象。当thead死亡时,它将停止持有它。稍后,当 GC 启动时,它将最终确定您的监视器。在终结器中,您可以引发一个事件,该事件将通知您的框架有关(观察到的)客户端线程的死亡。

可以在此要点中找到示例代码:https ://gist.github.com/2587063

这是它的副本:

public class ThreadMonitor
{
    public static event Action<int> Finalized = delegate { };
    private readonly int m_threadId = Thread.CurrentThread.ManagedThreadId;

    ~ThreadMonitor()
    {
        Finalized(ThreadId);
    }

    public int ThreadId
    {
        get { return m_threadId; }
    }
}

public static class Test
{
    private readonly static ThreadLocal<ThreadMonitor> s_threadMonitor = 
        new ThreadLocal<ThreadMonitor>(() => new ThreadMonitor());

    public static void Main()
    {
        ThreadMonitor.Finalized += i => Console.WriteLine("thread {0} closed", i);
        var thread = new Thread(() =>
        {
            var threadMonitor = s_threadMonitor.Value;
            Console.WriteLine("start work on thread {0}", threadMonitor.ThreadId);
            Thread.Sleep(1000);
            Console.WriteLine("end work on thread {0}", threadMonitor.ThreadId);
        });
        thread.Start();
        thread.Join();

        // wait for GC to collect and finalize everything
        GC.GetTotalMemory(forceFullCollection: true);

        Console.ReadLine();
    }
}

我希望它有所帮助。我认为它比你额外的等待线程更优雅。

于 2012-05-03T16:41:07.930 回答
4

由于您不直接控制线程创建,因此您很难知道线程何时完成其工作。另一种方法可能是让您强制客户在完成后通知您:

public interface IThreadCompletedNotifier
{
   event Action ThreadCompleted;
}

public class Library
{
    public void StartSomething(IThreadCompletedNotifier notifier)
    {
        Console.WriteLine("Library says: StartSomething called");
        notifier.ThreadCompleted += () => Console.WriteLine("Libaray says: Client thread existed");
        var clientThread = Thread.CurrentThread;
        exitMonitorThread.Start();
    }
}

这样,任何呼叫您的客户端都将被迫传递某种通知机制,该机制将告诉您何时完成它的工作:

public class Client : IThreadCompletedNotifier
{
    private readonly Library library;

    public event Action ThreadCompleted;

    public Client(Library library)
    {
        this.library = library;
    }

    public void DoWorkInAThread()
    {
        var thread = new Thread(() =>
        {
            library.StartSomething();
            Thread.Sleep(10);
            Console.WriteLine("Client thread says: I'm done");
            if(ThreadCompleted != null)
            {
               ThreadCompleted();
            }
        });
        thread.Start();
    }
}
于 2012-05-03T13:49:02.553 回答
1

除了做任何异步花哨的东西来完全避免线程之外,我会尝试将所有监视合并到一个线程中,该线程会轮询所有已访问您的库的线程的 .ThreadState 属性,例如,每 100 毫秒(我不确定您需要多快清理资源...)

于 2012-05-03T13:57:14.630 回答
1

如果客户端线程调用您的库并在内部分配一些资源,则客户端应该“打开”您的库并取回一个令牌以进行所有进一步的操作。该标记可以是库内部向量的 int 索引,也可以是指向内部对象/结构的 void 指针。坚持客户必须在终止前关闭令牌。

这就是所有此类 lib 调用中 99% 的工作原理,其中必须在客户端调用中保留状态,例如。套接字句柄,文件句柄。

于 2012-05-03T13:48:20.517 回答
0

你的.Join解决方案对我来说看起来很优雅。阻塞的观察者线程并不是一件可怕的事情。

于 2012-05-03T13:48:53.627 回答