如何提高这些 linq 语句的可读性？

Question

我有一个分层数据树，其中包含名为 DataNode 的类的对象。

每个DataNode 都包含一个Attribute 对象的集合。每个属性本质上是一个键/值对，附加了一些辅助方法。例如，有一个名为 EqualsCodeIndex(x) 的辅助方法，它将 int 值的一个小集合 x 与 this 相匹配code> attribute 并返回 true 或 false。所有键和值都是字符串，因为整个事物都基于文本文件中包含的键/值存储。

为了简化对特定 DataNode 的访问，DataTree 类中有一个字典，它将树中的所有节点映射到唯一的代码

Dictionary<string, DataNode> Codes;

：特定 Attribute 值如下所示：

string AttributeValue = dataTree
    .Codes[@"R-1\CHE"]
    .Attributes
    .Single(x => x.EqualsCodeIndex(parentAttribute.CodeIndex))
    .Value.Trim();

如果我只需通过代码和代码索引检索一两个属性，这还不错，但如果我必须检索，则不太好十个或更多。

尝试简化陈述，并且允许 EqualsCodeIndex 对于集合中的所有属性返回 false 的可能性，我添加了一个扩展方法：

public static string AttributeValueMatching
    (this KeyValuePair<string, DataNode> pair, List<int> codeIndex)
{
    var attribute = pair.Value.Attributes
        .Single(x => x.EqualsCodeIndex(codeIndex))

    return attribute == null ? string.Empty : value;
}

这将原始 linq 语句简化为：

string attributeValue
    = dataTree.Codes[@"R-1\CHE"].AttributeValueMatching(codeIndex);

...哪个更好，但我有一种感觉，我错过了一些东西。

---

这种方法有问题吗？有没有一种我没有想到的更好、更干净的方法，也许可以更好地利用索引器，或者也许是一个流畅的界面？

Answer 1

我没有对你的整个问题的直接答案，但对于“这种方法的问题”部分我有一个建议。

请小心 SingleOrDefault() 之后的链接语句，因为它可能会返回 null。如果您绝对确定它始终具有单个值，也许只需调用 Single() 并在发生时处理错过的期望，而不是更通用的 NullReferenceException。

编辑
在撰写上述帖子时，您进行了相同的更改。继续...

Answer 2

您是否考虑过构建一个扩展方法数据树？

这样

public static class DataTreeExtensions
   {
      public static string FetchByAttribute(this DataTree d, string Attribute)
      {
         string AttributeValue = d
                         .Codes[Attribute]
                         .Attributes
                         .Single(x => x.EqualsCodeIndex(parentAttribute.CodeIndex))
                         .Value.Trim();

            return AttributeValue

      }
   }

您就可以随意重用“FetchByAttribute”，如下所示：

string myValue = myTree.FetchByAttribute(@"R-1\CHE");

编辑：从 DataNode 更改为 DataTree...

Answer 3

我认为将其变成具有两个参数的一种方法看起来会稍微好一些：

Codes.AttributeValueMatching(@"R-1\CHE", codeIndex)

或者您可以创建一个带有索引器的包装器：

CodesWrapper[@"R-1\CHE", codeIndex]