Question

我最近取得了WCF Windows服务的所有权，该服务大量使用以下静态Utility类来检索查找数据：

   public static class Utility
    {

       //begin code that causes increased memory consumption
        private static Dictionary<string, ErrorData> _errorData;

        internal static Dictionary<string, ErrorData> ErrorData

        {
            get
            {
                if (_errorData == null)
                {
                    _errorData = GetErrorData();
                }
                return _errorData;
            }

        }
        //end code that causes increased memory consumption

        /// GetErrorData method to get error messages from error xml
        /// </summary>         
        /// <returns>Dictionary of Error messages value for different fields.</returns>           
        internal static Dictionary<string, ErrorData> GetErrorData()
        {
            Dictionary<string, ErrorData> data = null;


                XmlDocument doc = LoadXmlDocument(Constants.ErrorMessagesFileName);
                XmlNodeList errorNode = doc.SelectNodes("/ErrorMessages/Error");
                data = new Dictionary<string, ErrorData>();

                foreach (XmlNode node in errorNode)
                {
                    ErrorData errorValues = new ErrorData();
                    errorValues.FieldName = node.Attributes["FieldName"].Value;
                    errorValues.ErrorMessage = node.Attributes["ErrorMessage"].Value;
                    data.Add(node.Attributes["code"].Value, errorValues);
                }


            return data;
        }
        internal static XmlDocument LoadXmlDocument(string xmlFileName)
        {
            XmlDocument doc = null;
            try
            {
                if (HttpRuntime.Cache[xmlFileName] == null)
                {
                    doc = new XmlDocument();
                    doc.Load(Constants.Folderpath + "\\" + xmlFileName);
                    HttpRuntime.Cache.Insert(xmlFileName, doc);
                }
                else
                {
                    doc = (XmlDocument)HttpRuntime.Cache[xmlFileName];
                }
            }
            catch (Exception ex)
            {
               //log
            }
            return doc;
        }
    }

如您所见，静态ErrorData属性使用私有支持字段。 ErrorData是一个使用文件系统上的XML资源构造的Dictionary，这就是为什么在初始检索时文件的内容存储在HttpRuntime.Cache中。

在正常负载下，该服务消耗大约120 MB的RAM。

在某些时候，团队成员认为有必要通过创建由延迟加载的静态字段支持的静态属性来引入另一级别的优化。无论如何，在仅几次调用服务之后，所述静态字段的存在导致相当严重的内存泄漏（500MB +）。

当我删除静态字段和属性（客户端改为调用Utility.GetErrorData（））时，内存消耗将恢复到正常水平。

任何人都可以解释为什么这个静态字段的存在导致内存泄漏？ WCF服务正在运行InstanceContextMode.PerCall，如果这会产生影响。

非常感谢。

Answer 1

如果错误文件非常大，那么静态版本会将巨大的XML文档加载到内存中，并且永远不会释放它。以前，如果客户端调用了GetErrorData（），那么数据就会被加载到内存中并返回，从而清理内存。

此处没有同步，因此如果未加载静态变量，则几个同时发出的请求将分别开始加载错误文档。只有一个字典会获胜并保存到静态变量中。但是，如果错误文件很大，同时加载它的多个线程会增加内存压力。如果是这种情况，我希望下一个垃圾收集会回收额外的实例并释放大部分内存。

另请注意，静态实例版本会加载错误文件一次。因此，如果创建了其他错误，则永远不会将这些错误返回给客户端。

Answer 2

当你谈到改变时，我不完全确定你的意思是什么代码改变。但是，从阅读代码我的猜测是你最终多次调用GetErrorData，而字典只是填满了大量的重复条目。如果添加日志代码，显示哪些代码重复输入？

马丁

Answer 3

尝试转储w3p进程并检查大对象堆上的内容以及占用大块内存的对象。这将帮助您找到确切的内存泄漏原因。

您还应该检查加载到内存中的程序集。 XMLserializers就是其中一个例子。

请参阅Tess关于内存泄漏的博客。 http://blogs.msdn.com/b/tess/archive/2008/03/17/net-debugging-demos-lab-6-memory-leak.aspx

Answer 4

添加同步是否会修复“内存泄漏”？

也就是说，例如使LoadXmlDocument（）和GetErrorData（）都是私有的，并修改ErrorData属性，就像这样

    private static Dictionary<string, ErrorData> _errorData;
    private static object lockObject = new object();

    internal static Dictionary<string, ErrorData> ErrorData
    {
        get
        {
            lock (lockObject)
            {
                if (_errorData == null)
                {
                    _errorData = GetErrorData();
                }
                return _errorData;
            }
        }

    }

注意：通常，内存泄漏意味着应用程序随着时间的推移会慢慢消耗越来越多的内存（永远不会回收）。这是您正在观察的内容，或者当您更改实施时，您的内存消耗是否会变得更稳定？要真正验证您确实存在内存泄漏以及真正的原因（无法收集/最终确定哪些对象），您通常必须使用内存分析器。

为什么这段代码导致内存泄漏？

4 个答案: