这段代码会杀死我的服务器吗？

Question

我的网站一直存在问题，它基本上超时并消失。我现在已经到了必须将应用程序池设置为每 5 分钟自动回收一次的地步，但即使这样也失败了，因为我刚刚下班回来，我的电子邮件收件箱充满了 4000 封电子邮件，所有电子邮件都具有相同的内容错误。

System.Data.SqlClient.SqlException：超时已过期。操作完成之前超时时间已过，或者服务器未响应。

今天早上我尝试了一个测试，在连接字符串上禁用池化，这也不起作用。

现在我在想，也许这不是连接泄漏的问题，我以前经历过所有这些，我认为这可能与我网站的核心静态属性有关，

是其中之一

public static List<Member> AllMembers
{
    get
    {
        if (HttpRuntime.Cache["Members"] != null)
        {
            return (List<Member>)HttpRuntime.Cache["Members"];
        }
        else
        {
            GetAllMembers();
            return (List<Member>)HttpRuntime.Cache["Members"];
        }
    }
}

这 每当我想要成员列表时就会调用，您可以看到，如果它为空，它将填充缓存，这将使用数据库，如果它不为空，那么它将返回缓存对象。我也有 SQLCacheDependancy ，它将清除这些缓存对象，以便再次填充它们。所以这个属性被称为很多。

现在这是一个网络应用程序，并且随着我的流量一直在增加其染色，

我的属性可能是原因吗？

非常感谢

Truegilly的任何帮助

Answer 1

假设您正确处理了所有内容，我有一个替代解释：

如果缓存为空/过期并且多个页面尝试同时调用 AllMembers ，那么每个页面最终可能会调用 GetAllMembers() 同时进行，减慢数据库查询速度。如果调用开始超时，这可能会启动恶性循环。

您可以在代码周围放置一个锁，这样每个属性只能进行一次数据库查询。我可以这样设置：

private static object _allMembersLock = new object();
public static List<Member> AllMembers
{
    get
    {
        lock (_allMembersLock)
        {
            List<Member> members = (List<Member>)HttpRuntime.Cache["Members"];
            if (members == null)
            {
                members = GetAllMembers();
                HttpRuntime.Cache["Members"] = members;
            }
            return members;
        }
    }
}

Answer 2

只是一些常见的怀疑：

1. 您是否处置了所有 SqlCommand 和读取器？

2. 您是否正在处置/关闭您的 SqlConnection？

3. 您的数据库能够处理负载吗？您那里有性能问题吗？

您帖子中的代码看起来不像问题。

Answer 3

我在这里没有看到任何保护措施来防止对 getter 的多个并发调用触发 GetAllMembers。当然，您一次只需要一个线程来加载该属性。似乎需要某种lock()。

您应该确保刷新缓存的代码也使用相同的 lock()。

如果可以从多个服务器调用此代码，您还需要确保支持 GetAllMembers 的任何数据库端程序逻辑（存储过程或来自客户端的动态 SQL）有效且正确地处理多个并发读者。