问题描述
|
我最近获得了WCF Windows服务的所有权，该服务大量使用以下静态实用程序类来检索查找数据：
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;
}
}
解决方法
如果错误文件很大，则静态版本会将巨大的XML文档加载到内存中，并且永远不会释放它。以前，如果客户端调用了GetErrorData（），则数据将被加载到内存中并返回，从而关闭内存。
这里没有同步，因此，如果未加载静态变量，则多个同时请求将开始分别加载错误文档。只有一个字典会获胜并保存到静态变量。但是，如果错误文件很大，则多个线程同时加载它会增加内存压力。如果是这种情况，我希望下一个垃圾回收将回收额外的实例并释放大量此内存。
另请注意，静态实例版本仅一次加载错误文件。因此，如果创建了其他错误，则这些错误将永远不会返回给客户端。
, 我不确定要说的是什么代码更改。但是，从阅读代码后，我的猜测是，您最终会多次调用GetErrorData，而字典中却充满了很多重复的条目。如果添加日志记录代码，则显示重复输入了哪个代码？
马丁
, 尝试转储w3p进程，并检查大对象堆上有什么以及哪些对象正在占用大块内存。这将帮助您找到导致内存泄漏的确切原因。
您还应该检查加载到内存中的程序集。 XMLserializers是示例之一。
请参阅Tess的有关内存泄漏的博客。
http://blogs.msdn.com/b/tess/archive/2008/03/17/net-debugging-demos-lab-6-memory-leak.aspx
, 添加同步是否可以解决您的“内存泄漏”？
也就是说，例如将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;
}
}
}