给定的密​​钥不在字典中。甚至当dictionary.ContainsKey(“given_key”)== true时

时间:2013-05-15 00:35:28

标签: c# .net exception dictionary

我有一个课程,收集来自无线电通信局或气象局的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;
}

1 个答案:

答案 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();