ASP MVC - Comet/反向 Ajax/PUSH - 此代码线程安全吗？

Question

我正在尝试通过轮询服务器以了解数据变化并保持连接打开直到有内容可以响应来实现彗星风格的功能。

首先，我的控制器上有一个静态变量，它存储数据上次更新的时间：

public static volatile DateTime lastUpdateTime = 0;

因此，每当我轮询的数据发生变化时，该变量就会发生变化。

然后我有一个 Action，它将上次检索数据的时间作为参数：

public ActionResult Push(DateTime lastViewTime)
{
    while (lastUpdateTime <= lastViewTime)
    { 
        System.Threading.Thread.Sleep(10000);
    }
    return Content("testing 1 2 3...");
}

因此，如果 lastUpdateTime 小于或等于 lastViewTime，我们就知道有没有新数据，我们只是将请求保留在循环中，保持连接打开，直到有新信息，然后我们可以将其发送回客户端，客户端将处理响应，然后发出新请求，所以连接基本上始终处于打开状态。

这似乎工作正常，但我担心线程安全，这样可以吗？是否需要将lastUpdateTime标记为易失性？有更好的办法吗？

谢谢

编辑：也许我应该在更新时间值时使用锁定对象

private static object lastUpdateTimeLock = new object();

..

lock (lastUpdateTimeLock)
{
    lastUpdateTime = DateTime.Now;
}

Answer 1

关于您最初的问题，您必须小心 DateTimes，因为它们是 .NET 运行时中的实际对象。只有少数数据类型可以在不加锁的情况下进行本机访问（例如整数、布尔值）（假设您没有使用 Interlocked）。如果您想避免日期时间的任何问题，您可以将刻度作为长整数并使用 Interlocked 类来管理它们。

也就是说，如果您正在 .NET 应用程序中寻找彗星功能，不幸的是，您将不得不比您这里所得到的走得更远。 IIS/ASP.NET 无法按照您现在采用的方法进行扩展；在达到 100 个用户之前，您就会达到限制。除此之外，您必须切换到使用异步处理程序，并为传入请求实现自定义有界线程池。

如果您确实想要一个经过测试的 ASP.NET/IIS 解决方案，请查看 WebSync，它是一个完整的彗星专门为此目的而设计的服务器。

Answer 2

老实说，我担心的是保持打开的连接数量和空的 while 循环。您的连接可能没问题，但我肯定想做一些负载测试来确定。

while (lastUpdateTime <= lastViewTime) {} 似乎应该有一个 Thread.Sleep(100) 或其中的内容。否则我认为它会不必要地消耗大量的CPU周期。

对我来说，在 lastUpdateTime = DateTime.Now 周围锁似乎没有必要，因为之前的值并不重要。如果是 lastUpdateTime = lastUpdateTime + 1 或其他什么，那么也许就是这样。