生产就绪，多线程 c# http 服务器

Question

我在 c# .NET 中实现了一个 HTTP 服务器：

public class HttpServer
{
    private HttpListener listener;

    public HttpServer()
    {
        listener = new HttpListener();
        listener.Prefixes.Add("http://localhost:8080/");
    }

    public void Start()
    {
        lock(this) {
            listener.Start();
            AsyncProcessing(listener);
        }
    }

    public void Stop()
    {
        lock (this) {
            listener.Stop();
        }
    }

    private void AsyncProcessing(HttpListener listener)
    {
        if (listener == null)
            return;

        listener.BeginGetContext(new AsyncCallback(Callback), listener);
    }

    private void Callback(IAsyncResult result)
    {
        HttpListenerContext context = null;

        lock(this) {
            HttpListener listener = (HttpListener)result.AsyncState;
            if (!listener.IsListening)
                return;

            context = listener.EndGetContext(result);

            AsyncProcessing(listener);
        }

        /* handle request */
    }
}

我对此实现有一些疑问：

• 我在这里和那里添加了一些锁来防止竞争条件，但我很困惑：真的需要这些吗？我在文档中读到所有公共非静态方法都不是线程安全的。为什么我没有看到考虑到这一事实的代码？
• HttpListenerContext 的行为如何？他是否与 HttpListener 有某种联系？或者我可以同时使用多个 HttpListenerContexts 吗？
• 我听说 HttpListener 不会为生产系统做好准备，但我从未见过支持这种说法的论据。是真是假？
• 还有其他我没有提到的事情需要考虑吗？

感谢您的想法

Answer 1

请记住，我不是多线程方面的专家，因此您应该注意尽可能地验证我所说的任何内容。

如果其他人知道，并且想窃取我的整个答案并编辑或更正详细信息，请随时这样做。

让我们一一处理您的问题：

我在这里和那里添加了一些锁来防止竞争条件，但我很困惑：这些真的需要吗？我在文档中读到所有公共非静态方法都不是线程安全的。为什么我没有看到考虑到这一事实的代码？

嗯，是的，也不是。锁通常用于防止多个线程同时访问同一个数据结构，因为它会破坏数据结构。考虑在一个线程上对数组进行排序，并在另一个线程的中间插入一个元素，这两个线程的正确时间会破坏数组的内容。

现在，在您的代码中，您锁定了this，这绝不是一个好主意。外部代码可能也会锁定同一个对象，这是你无法控制的，所以为了创建生产就绪代码，我不会那样做。

如果您的代码中需要锁，我会构造特定的锁对象并使用它们。

换句话说，添加这个：

private readonly object _Lock = new object();

然后在您拥有lock(this) 的任何地方将其替换为lock(_Lock)。这样，如果需要，您还可以拥有多个锁。

至于实际需要锁，我对此不是 100% 确定的。我不确定的事情是你在调用 Stop 之前锁定，并且你锁定在回调和锁定中，你检查侦听器是否仍在运行。

这将阻止您在接受请求后但在您实际处理请求之前停止侦听器。换句话说，听起来您会阻止关闭服务器，而打开的请求仍在处理中。

但是，不，您不会阻止这种情况，因为您可能会在将锁定部分留在回调中之后停止服务器，但在注释代码完全执行期间或之前，您仍然会遇到这个问题。

然而这也意味着您已经有效地序列化了一些回调方法，即调用 EndGetContext 并重新启动 BeginGetContext 循环的部分。这是否是一个好的模式，我不知道。

HttpListenerContext 的行为如何？他是否与 HttpListener 有某种联系？或者我可以同时使用多个 HttpListenerContexts 吗？

在这里，我将做一个猜测。该类没有对侦听器类的引用，或，它具有与之合作的线程安全方式。

如果对请求/响应数据的每次访问都必须序列化，那么基于线程的 http 侦听器系统就不会太多了。

无论如何，如果有疑问，请检查您正在访问的上下文类的方法和/或属性的文档，如果您需要采取措施确保线程安全，文档会说明。

我听说 HttpListener 不会为生产系统做好准备，但我从未见过支持这种说法的论据。是真是假？

（见问题评论）

还有其他我没有提到的事情我应该考虑吗？

多线程代码很难编写。从你的问题来看，我冒昧地猜测你并没有做很多事情，老实说，虽然我做了很多事情，但我仍然觉得自己如履薄冰。

我的建议如下：

• 您真的需要多线程吗？
• 您有其他了解更多这方面的同事可以帮助您吗？