Question

我可能担心错误的优化，但我有这种唠叨的想法，它一遍又一遍地解析xml树，也许我在某处读它。不记得了。

无论如何，这就是我正在做的事情：

using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Xml.Linq;
using System.Net;

namespace LinqTestingGrounds
{
    class Program
    {
        static void Main(string[] args)
        {
            WebClient webClient = new WebClient();
            webClient.DownloadStringCompleted += new DownloadStringCompletedEventHandler(webClient_DownloadStringCompleted);
            webClient.DownloadStringAsync(new Uri("http://www.dreamincode.net/forums/xml.php?showuser=335389"));
            Console.ReadLine();
        }

        static void webClient_DownloadStringCompleted(object sender, DownloadStringCompletedEventArgs e)
        {
            if (e.Error != null)
            {
                return;
            }

            XDocument xml = XDocument.Parse(e.Result);

            User user = new User();
            user.ID = xml.Element("ipb").Element("profile").Element("id").Value;
            user.Name = xml.Element("ipb").Element("profile").Element("name").Value;
            user.Rating = xml.Element("ipb").Element("profile").Element("rating").Value;
            user.Photo = xml.Element("ipb").Element("profile").Element("photo").Value;
            user.Reputation = xml.Element("ipb").Element("profile").Element("reputation").Value;
            user.Group = xml.Element("ipb").Element("profile").Element("group").Element("span").Value;
            user.Posts = xml.Element("ipb").Element("profile").Element("posts").Value;
            user.PostsPerDay = xml.Element("ipb").Element("profile").Element("postsperday").Value;
            user.JoinDate = xml.Element("ipb").Element("profile").Element("joined").Value;
            user.ProfileViews = xml.Element("ipb").Element("profile").Element("views").Value;
            user.LastActive = xml.Element("ipb").Element("profile").Element("lastactive").Value;
            user.Location = xml.Element("ipb").Element("profile").Element("location").Value;
            user.Title = xml.Element("ipb").Element("profile").Element("title").Value;
            user.Age = xml.Element("ipb").Element("profile").Element("age").Value;
            user.Birthday= xml.Element("ipb").Element("profile").Element("birthday").Value;
            user.Gender = xml.Element("ipb").Element("profile").Element("gender").Element("gender").Element("value").Value;

            Console.WriteLine(user.ID);
            Console.WriteLine(user.Name);
            Console.WriteLine(user.Rating);
            Console.WriteLine(user.Photo);
            Console.WriteLine(user.Reputation);
            Console.WriteLine(user.Group);
            Console.WriteLine(user.Posts);
            Console.WriteLine(user.PostsPerDay);
            Console.WriteLine(user.JoinDate);
            Console.WriteLine(user.ProfileViews);
            Console.WriteLine(user.LastActive);
            Console.WriteLine(user.Location);
            Console.WriteLine(user.Title);
            Console.WriteLine(user.Age);
            Console.WriteLine(user.Birthday);
            Console.WriteLine(user.Gender);

            //Console.WriteLine(xml);            
        }
    }
}

这是Good Enough™还是有更快的方法来解析我需要的东西？

PS。我正在DownloadStringCompleted事件中执行大部分操作，我不应该这样做吗？第一次使用这种方法。谢谢！

Answer 1

不了解效率，但为了便于阅读，请使用profile变量，而不是一遍又一遍地遍历整个事物：

 User user = new User();
 var profile = xml.Element("ipb").Element("profile");
 user.ID = profile.Element("id").Value;

Answer 2

我相信xml序列化是解决此类问题的方法。只要您的属性与xml元素匹配，这将是微不足道的。否则，您只需使用XmlElement和XmlAttribute属性类映射它们。下面是一些简单的代码，用于将xml反序列化到类中：

public T Deserialise(string someXml)
    {   
        XmlSerializer reader = new XmlSerializer(typeof (T));
        StringReader stringReader = new StringReader(someXml);
        XmlTextReader xmlReader = new XmlTextReader(stringReader);
        return (T) reader.Deserialize(xmlReader);
    }

Answer 3

我除了Oded的回答：提高可读性的另一种方法是使用XPathSelectElement扩展方法。

所以你的代码看起来像是：

user.ID = xml.XPathSelectElement("ipb/profile/id").Value;

Answer 4

不，它不是一遍又一遍地解析XML;只有一次，当你打电话

XDocument.Parse(e.Result);

之后的调用只是访问xml对象中的树结构。

“解析”意味着分析非结构化文本字符串（例如来自文件）并从中创建数据结构（例如树）。您的... .Element("foo")调用不会解析，而是访问XDocument.Parse()调用构建的数据结构的部分内容。

如果您想知道您的代码是否重复某些步骤并且可以进行优化，那么是的，您将冗余地遍历ipb/profile。这不是解析，但是对Element（“foo”）的调用必须做一些比较字符串参数和子元素名称的工作。 @ Oded的建议出于可读性原因修复了这个问题，但也有助于提高效率。

这是解析XML的低效方法吗？

4 个答案: