LINQ 查询 - 根据具有与字符串匹配的属性的值来选择键？

Question

我有一个 IDictionary

，我需要选择 Bar.Prop1 与字符串值匹配的第一个 Foo。

public class Foo { }

public class Bar
{
    public String Prop1 { get; set; }
}

现在我已经这样了...

foreach (var kvp in MyDictionary)
{
    if (kvp.Value.Prop1 == theString)
    {
        var key = kvp.Key;
        //Do something with it
        break;
    }
}

但这看起来不像 LINQ 查询那么干净。 ReSharper 将其变成：

foreach (var kvp in MyDictionary.Where(kvp => kvp.Value.Prop1 == theString))
{
    var key = kvp.Key;
    //Do something with it
    //break; is unnecessary because I only get one kvp back anyways.
}

我只想要第一个匹配的项目，因为我不希望返回多个 KVP。这违背了业务逻辑，因此单元测试会解决这个问题。

Answer 1

我只想要第一个项目
匹配，因为我从来没想过
拿回超过 1 个 KVP。那
违背业务逻辑，所以
单元测试负责解决这个问题。

如果是这种情况，我认为您需要使用更强有力的代码保证您的意图，这就是 Single （或 SingleOrDefault）方法。 First 将返回与给定谓词匹配的任意多个对象中的第一个对象。如果很多违背了您的期望和业务规则，这似乎是一个错误。就这样对待它。

var key = MyDictionary.Single(pair => pair.Value.Prop1 == someValue).Key;

使用 Single 时，如果序列中存在多个匹配项，则会导致异常。

SingleOrDefault 允许 0 或 1，但不能更多。如果使用这种方法，您需要捕获结果并在执行其他操作（触发方法、访问属性等）之前与 null 进行比较。

Answer 2

var key = MyDictionary.First(kvp => kvp.Value.Prop1 == theString).Key;

Answer 3

@Bala R 的答案是正确的，但 Anthony Pegram 在他对 OP 的评论中提出了非常好的观点。如果您使用不同的键多次执行此操作，那么您应该做的是反转字典，这样您就不必每次需要一个值时都遍历整个集合。

// Do this just once: it's expensive
var reverseDict = MyDictionary.ToDictionary(kvp => kvp.Value.Prop1, kvp => kvp.Key);

...
// Do this as many times as you need: it's cheap
var key = reverseDict[someValue];