Linq-XML всегда такой грязный?

Question

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 выглядит следующим образом:

<Items>
    <Item>
        <Id></Id>
        <Name></Name>
        <Description></Description>
        <Price></Price>
    </Item>
</Items>

Id и цена являются целочисленными значениями. Имя и описание являются строками.

Я нашел Linq to XML отлично подходит для того, для чего я его использовал, это всего лишь фрагмент. Но, с другой стороны, я чувствую, что он должен или мог бы быть чище. Кастинг кажется наиболее очевидной проблемой в этом фрагменте.

Любой совет?

Answer 1

На самом деле было бы лучше разыграть ИМО, чем звонить int.Parse. Вот как бы я написал ваш запрос:

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

Попробуйте написать новый конструктор для PurchaseItem, который принимает элемент XML, поэтому вы можете написать:

select new PurchaseItem(item.Parent);

Answer 3

В вашем примере вы можете немного привести в порядок, найдя элемент <Item/> вместо <Id/>, чтобы избежать получения 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

Я предполагаю, что у вас есть узел "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)
             };

Не намного лучше, но намного легче читать!