我有一本字典,将字符串映射到这样的对象:
Dictionary<string, object> myDic;

我事先知道对象的类型是基于字符串的,但是我的问题是我应该使用TryGetValue,还是使用try,catch语句直接查找。

例:

//TryGetValueMethod
object myObject = null;
myDic.TryGetValue("test", out myObject);

MyCustomType t1 = (MyCustomType) myObject;

//Direct lookup method
try
{
     MyCustomType t2 = (MyCustomType) myDic["test"];
     //Do something here...
} catch {}


您认为首选哪种方法?第二个是更干净的编码,因为没有多余的强制转换,但是我认为它的效率比第一个低,因为它没有异常。

最佳答案

我不认为您应该使用try / catch形成这样的逻辑路径。 Exception被认为是例外情况,其中某些地方“出了错”。

就个人而言,我更喜欢ContainsKey

if (myDic.ContainsKey("test")) {
   MyCustomType value = myDic["test"];
   // do something with the value
}


如果您认为找不到密钥意味着某事“出错”,那么我将省略测试,如果找不到密钥,则抛出异常。

编辑:这些天我尝试使用TryGetValue代替。它有点笨拙,但是一旦您习惯了它,还算不错。

MyCustomType value;
if (myDic.TryGetValue("test", out value)) {
   // do something with value
}


EDIT2:现在使用out var我肯定会更多地使用TryGetValue。同样,您可以编写一个CantGetValue方法(布尔结果相反),因为在大多数情况下,您想要在没有值时(而不是在有值时)做一些额外的事情。

if (dict.TryGetValue("test", out var value)) {
   // do something with value
}

// or

if (cache.CantGetValue("test", out var cachedValue)) {
   // cache value
}
// make use of value

09-06 04:07