我有一个课程,收集来自无线电通信局或气象局的7天预报数据,并将其显示在网页上。脚本每30分钟运行一次,以便从无线电通信局获取更新数据。
该局以制表符分隔的格式提供带有标题行的数据。提取完所有字段后,我将值放入Dictionary<string,string>
进行解析。出于显而易见的原因,组织数据的重要领域是“forecast_date”。所以在开始解析之前,我确保我的字典实际上包含了这个密钥。
这是我正在做的一个非常简化的例子:
static object ForecastLockingObj= new object();
private void UpdateWeather()
{
if(isTimeForUpdate())
{
lock(ForecastLockingObj)
{
if(isTimeForUpdate())
{
Dictionary<string, string> ForecastData = Get7DayForecast();
int forecastDate = int.MinValue;
if (ForecastData.ContainsKey("forecast_date") && int.TryParse(ForecastData["forecast_date"], out forecastDate))
{
//Parse the data
SetNextUpdateTime();
}
}
}
}
}
这实际上适用于大多数情况。但偶尔我会得到以下例外:
[KeyNotFoundException: The given key was not present in the dictionary.]
System.ThrowHelper.ThrowKeyNotFoundException() +28
System.Collections.Generic.Dictionary`2.get_Item(TKey key) +7457036
CoA.WebUI.Controls.WeatherWidget.UpdateWeather() in C:\dev\WeatherWidget.cs:231
其中第231行是检查“forecast_date”是否存在的if语句,然后尝试将其解析为整数。注意;该服务可靠地将日期呈现为整数(例如20130515),因此这更像是一种健全性检查。
ContainsKey
不应该抛出此异常,所以我觉得它必须是我在TryParse中引用ForecastData["forecast_date"]
的地方。
我的问题是这个;当然,如果ContainsKey返回false,则TryParse不应该运行。那么为什么它会在一个语句中报告一个键的存在,然后在下一个语句中否认它的存在...当我们在一个锁中时,我们正在处理的词典是非静态的和本地的?
撇开;这通常发生在下午,当时无线电通信局发布下一个长期预报。例外情况发生在几个页面加载,然后是权限本身。
这是完整的Get7DayForecast方法
private Dictionary<string, string> Get7DayForecast()
{
int linenumber = 0;
int locationNameKey = 0;
List<string> keys = new List<string>();
Dictionary<string, string> myLocationData = new Dictionary<string, string>();
FtpWebRequest ftp = (FtpWebRequest)WebRequest.Create(ForecastURL);
ftp.Method = WebRequestMethods.Ftp.DownloadFile;
ftp.Credentials = new NetworkCredential();
FtpWebResponse ftp_response = (FtpWebResponse)ftp.GetResponse();
if (ftp_response.WelcomeMessage.StartsWith("230") && ftp_response.StatusDescription.StartsWith("150"))
{
Stream ftp_responseStream = ftp_response.GetResponseStream();
StreamReader ftp_reader = new StreamReader(ftp_responseStream);
while (ftp_reader.Peek() >= 0)
{
linenumber++;
string line = ftp_reader.ReadLine();
List<string> temp = (List<string>)line.Split(ForecastDelimiter).ToList<string>();
if (linenumber == 1)
{
//Break if the header line does not contain the fields we require
if (!ForecastRequiredFields.All(line.Contains)) { break; }
keys = temp;
locationNameKey = keys.IndexOf(ForecastLocationFieldName);
}
else if (temp.Count == keys.Count && temp[locationNameKey] == ForecastLocationName)
{
for (int i = 0; i < keys.Count; i++)
{
myLocationData.Add(keys[i], temp[i]);
}
//Break if we've just parsed the data we were looking for
break;
}
}
ftp_reader.Close();
}
ftp_response.Close();
return myLocationData;
}
答案 0 :(得分:2)
老实说,我不明白为什么你的代码会失败,但是你应该考虑利用Trace
看看是什么。而且使用TryGetValue
代替它也不会有害。
var map = Get7DayForecast();
string forecastDateString;
if (!map.TryGetValue("forecast_date", out forecastDateString))
{
Trace.WriteLine("forecast_date entry was not found.");
return;
}
int foreCastDate;
if (!int.TryParse(forecastDateString, out foreCastDate)
{
Trace.WriteLine("Value was not a valid integer: " + forecastDateString);
return;
}
SetNextUpdateTime();