我试图重构以下内容 - 这有效,但如果我开始在XML中获得更多元素,它将无法管理:
HttpResponseMessage response = await httpClient.GetAsync("https://uri/products.xml");
string responseAsString = await response.Content.ReadAsStringAsync();
List<Product> productList = new List<Product>();
XDocument xdocument = XDocument.Parse(responseAsString);
var products = xdocument.Descendants().Where(p => p.Name.LocalName == "item");
foreach(var product in products)
{
var thisProduct = new Product();
foreach (XElement el in product.Nodes())
{
if(el.Name.LocalName == "id")
{
thisProduct.SKU = el.Value.Replace("-master", "");
}
if (el.Name.LocalName == "availability")
{
thisProduct.Availability = el.Value == "in stock";
}
}
productList.Add(thisProduct);
}
给出以下XML URL
<rss xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
xmlns:xsd="http://www.w3.org/2001/XMLSchema"
xmlns="http://base.google.com/ns/1.0" version="0">
<channel>
<title>Product Feed</title>
<link></link>
<description>Products</description>
<item>
<availability>in stock</availability>
<id>01234-master</id>
...
</item>
<item>
<availability>in stock</availability>
<id>abcde-master</id>
...
</item>
</channel>
</rss>
理想情况下,我想删除循环和if语句,并且有一个LINQ查询,它以简洁的方式从XML返回我需要的字段(id,availability等...)并填充一个简单的类有了这些数据。
有人可以帮忙吗?
答案 0 :(得分:2)
有时你必须为你所编写的代码感到高兴。有时没有更聪明的&#34;写它的方式......你只能写一点&#34;更好&#34;:
List<Product> productList = new List<Product>();
XDocument xdocument = XDocument.Parse(responseAsString);
XNamespace ns = "http://base.google.com/ns/1.0";
var products = from x in xdocument.Elements(ns + "rss")
from y in x.Elements(ns + "channel")
from z in y.Elements(ns + "item")
select z;
foreach (var product in products)
{
var prod = new Product();
productList.Add(prod);
foreach (XElement el in product.Elements())
{
if (el.Name == ns + "id")
{
prod.SKU = el.Value.Replace("-master", string.Empty);
}
else if (el.Name == ns + "availability")
{
prod.Availability = el.Value == "in stock";
}
}
}
注意:
Descendants()
在道德上是错误的。有一个固定的位置,item
将是/rss/channel/item
,你完全了解它。它不是//item
。因为明天可能会有rss/foo/item
今天不存在。您尝试编写代码,以便与可以添加到xml的其他信息向前兼容。我讨厌xml名称空间......还有xml有多个嵌套名称空间。我多么讨厌那些。但是比我更聪明的人认为他们存在。我接受。我用它们编码。在LINQ-to-XML中,它非常简单。有一个XNamespace
甚至有一个重载+
运算符。
请注意,如果您是微型优化器(我尽量不这样做,但我必须承认,但我的手有点痒),您可以预先计算内部使用的各种ns + "xxx"
for
周期,因为它不是从这里清除,但它们都是在每个周期重建。一个XName
如何构建在里面......哦......这是一件令人着迷的事情,相信我。
private static readonly XNamespace googleNs = "http://base.google.com/ns/1.0";
private static readonly XName idName = googleNs + "id";
private static readonly XName availabilityName = googleNs + "availability";
然后
if (el.Name == idName)
{
prod.SKU = el.Value.Replace("-master", string.Empty);
}
else if (el.Name == availabilityName)
{
prod.Availability = el.Value == "in stock";
}
答案 1 :(得分:0)
请尝试以下操作:
using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Xml;
using System.Xml.Linq;
using System.IO;
namespace ConsoleApplication1
{
class Program
{
const string FILENAME = @"c:\temp\test.xml";
static void Main(string[] args)
{
new Item(FILENAME);
}
}
public class Item
{
public static List<Item> items { get; set; }
public string availability { get; set; }
public string id { get; set; }
public Item() { }
public Item(string filename)
{
string xml = File.ReadAllText(filename);
XDocument doc = XDocument.Parse(xml);
XElement root = doc.Root;
XNamespace ns = root.GetDefaultNamespace();
Item.items = doc.Descendants(ns + "item").Select(x => new Item() {
availability = (string)x.Element(ns + "availability"),
id = (string)x.Element(ns + "id")
}).ToList();
}
}
}