Linq-XML总是那么混乱吗？

var subset = from item in document.Descendants("Id") where item.Value == itemId.ToString() select new PurchaseItem() { Id = int.Parse(item.Parent.Element("Id").Value), Name = item.Parent.Element("Name").Value, Description = item.Parent.Element("Description").Value, Price = int.Parse(item.Parent.Element("Price").Value) }; 

XML的结构如下：

Id和价格都是整数值。 名称和描述是字符串。

我发现Linq to XML非常适合我用它，这只是一个片段。 但是，另一方面，我感觉它应该或可能更清洁。 在这个片段中，转换似乎是最明显的问题。

任何建议？

Answer 1

实际上，施放比调用int.Parse更好的IMO。 这是我写你的查询的方式：

 string id = itemId.ToString(); // We don't need to convert it each time! var subset = from item in document.Descendants("Id") where item.Value == id let parent = item.Parent select new PurchaseItem { Id = (int) parent.Element("Id"), Name = (string) parent.Element("Name"), Description = (string) parent.Element("Description"), Price = (int) parent.Element("Price") }; 

Answer 2

我假设你有一个“Items”节点？

您可以执行类似这样的操作，假设您使用XElement.Load（）加载文档

 var subset = from item in document.Elements("Item") where item.Element("Id").Value == itemId.ToString() select new PurchaseItem() { Id = int.Parse(item.Element("Id").Value), Name = item.Element("Name").Value, Description = item.Element("Description").Value, Price = int.Parse(item.Element("Price").Value) }; 

不是更好，但更容易阅读！

Answer 3

在您的示例中，您可以通过找到元素而不是来稍微整理一下，以避免每次都获取Parent ：

 var subset = from item in document.Descendants("Item") where item.Element("Id").Value == itemId.ToString() select new PurchaseItem() { Id = int.Parse(item.Element("Id").Value), Name = item.Element("Name").Value, Description = item.Element("Description").Value, Price = int.Parse(item.Element("Price").Value) }; 

Answer 4

考虑为PurchaseItem编写一个带有XML元素的新构造函数，这样你就可以编写：

 select new PurchaseItem(item.Parent);