处理KeyNotFoundException的最佳方法


68

我正在使用字典为正在处理的程序执行查找。我在字典中运行了一堆键,并且我期望某些键没有值。我抓住KeyNotFoundException它发生的地方,并吸收它。所有其他异常将传播到顶部。这是处理此问题的最佳方法吗?还是应该使用其他查询?字典使用int作为其键,并使用自定义类作为其值。

Answers:


124

Dictionary.TryGetValue改为使用:

Dictionary<int,string> dictionary = new Dictionary<int,string>();
int key = 0;
dictionary[key] = "Yes";

string value;
if (dictionary.TryGetValue(key, out value))
{
    Console.WriteLine("Fetched value: {0}", value);
}
else
{
    Console.WriteLine("No such key: {0}", key);
}

是否仍建议使用它而不是ContainsKey()?
Sven van den Boogaart '17

3
@SvenB:是的,我会这么说。为什么要同时进行两次查找,一次检查密钥是否存在,然后一次获取值呢?
乔恩·斯基特

@JonSkeet我不知道我的猜测是,如果containskey为false,将避免不必要的分配。
Sven van den Boogaart '17

2
@SvenB:如果ContainsKey为true,则将一个不必要的分配与再次执行整个查找的成本进行比较...我会惊讶地发现任何更有效的实际示例。
乔恩·斯基特

1
@JonSkeet现在已经发行了新版本的dot.net,请更新答案并将其string value作为TryGetValue的一部分:if (dictionary.TryGetValue(key, out string value))
再见StackExchange

36

尝试使用:Dict.ContainsKey

编辑:
性能方面,我认为Dictionary.TryGetValue比其他建议的要好,但是我不愿意使用Out时,我认为ContainsKey更具可读性,但是如果您还需要该值,则需要更多行代码。


1
为什么得到反对票?请解释,以便我可以改善答案。
彼得

1
您能解释一下为什么out除非需要时才不想使用吗?

2
@wilbishardis只是一个习惯,我认为很难忘记当有=符号时可以更清晰地修改方法参数。现在这只是我的意见,这并不意味着每个人都感觉一样,而且在某些情况下,它是最好的选择int.TryParse是一个例子..
彼得·

32

一线解决方案使用 TryGetValue

string value = dictionary.TryGetValue(key, out value) ? value : "No key!";

要知道,变量的类型必须是在这种情况下字典,则返回字符串。在这里,您不能使用var进行变量声明。

如果您正在使用C#7,在这种情况下,你的CAN包括var和内联定义它:

string value = dictionary.TryGetValue(key, out var tmp) ? tmp : "No key!";

这也是一个不错的扩展方法,它将完全实现您想要实现dict.GetOrDefault(“ Key”)或dict.GetOrDefault(“ Key”,“ No value”)的目的

public static TValue GetOrDefault<TKey, TValue>(this Dictionary<TKey, TValue> dictionary, TKey key, TValue defaultValue = default(TValue))
{
      if (dictionary != null && dictionary.ContainsKey(key))
      {
           return dictionary[key];
      }
      return defaultValue;
 }

当您需要进行大量查找时,效果会好得多,也不会那么冗长。
ZZZ

16

这是一个单行解决方案(请记住,这会使查询两次。请参阅下面的tryGetValue版本,该版本应在长时间运行的循环中使用。)

string value = dictionary.ContainsKey(key) ? dictionary[key] : "default";

但是,我发现自己每次访问字典时都必须这样做。我希望它返回null,所以我可以这样写:

string value = dictionary[key] ?? "default";//this doesn't work

1
避免使用此解决方案,因为它需要在字典上进行两次查找。一个查询dictionary.ContainsKey,另一个查询dictionary[key]。使用@JernejNovak的答案可获得更好的性能。
再见StackExchange

有时候,性能并不是重中之重,而可读性则更为重要。乔恩·斯基特(Jon Skeet)的答案不是您想要散布在整个代码中的东西。我会说避免在大循环中这样做。我会在回答中做笔记。不过,不值得这么做。
MondayPaper's

string value = dictionary.ContainsKey(key) ? dictionary[key] : "default";string value = dictionary.TryGetValue(key, out value) ? value : "No key!";
相比之下,

2
杜德(Dude)...当我写下此答案时,下面没有答案(一年多以前)。这就是为什么我将自己的答案与Jon Skeets的答案进行比较的原因。其次,输出值可能更难读取。用户可能不知道您不能使用var来初始化该值,并且尚不清楚在设置该值之前会立即对其进行初始化。这是一个很好的解决方案,我会继续使用它,但这很棘手(在已经设置好值之后,通过再次为其自身设置值来工作)……这可能就是为什么有人花了一年多的时间添加一个更好的解决方案。
星期一纸

5

您应该使用Dictionary的'ContainsKey(string key)'方法检查密钥是否存在。使用正常程序流的异常不是一个好习惯。


2
正是我问这个问题的原因,我觉得自己在做的事不是好习惯。
丹·麦克莱恩

5

我知道这是一个旧线程,但是如果有帮助的话,先前的回答会很好,但是可以解决复杂性的注释和乱扔代码的问题(对我来说都是有效的)。

我使用自定义扩展方法以更优雅的形式包装了上述答案的复杂性,以使它不会在整个代码中乱七八糟,从而为null合并运算符提供了强大的支持。。。同时还可以最大化性能(通过以上答案)。

namespace System.Collections.Generic.CustomExtensions
{
    public static class DictionaryCustomExtensions
    {
        public static TValue GetValueSafely<TKey, TValue>(this IDictionary<TKey, TValue> dictionary, TKey key)
        {
            TValue value = default(TValue);
            dictionary.TryGetValue(key, out value);
            return value;
        }
    }
}

然后,您可以简单地通过导入名称空间System.Collections.Generic.CustomExtensions来使用它。

string value = dictionary.GetValueSafely(key) ?? "default";
By using our site, you acknowledge that you have read and understand our Cookie Policy and Privacy Policy.
Licensed under cc by-sa 3.0 with attribution required.