2

Допустим у меня есть XML:

<root>
<node attr1="" attr2="" attr3="">
Text
</node> 
</root>

и мне нужно считать значение каждого атрибута в определенную переменную.

Сейчас я прохожу в цикле foreach по всем атрибутам, а потом с помощью трёх if я анализирую имя и присваиваю значение атрибута своей переменной.

Подскажите, есть ли более лаконичное решение этой задачи?

Ведь если мне потребуется считывать больше атрибутов, то мне придется плодить дополнительные if...

6
  • В нормальной ситуации вам не должно хотеться разложить атрибуты по отдельным переменным. Если вы расскажете, для чего это вам, возможно, подойдёт другая идея.
    – VladD
    Commented 19 авг. 2015 в 21:45
  • Файл XML у меня является неким скриптом. Каждый Node у меня является действием, а аттрибут описанием действия(Name,Type,Action). Я считываю данные в List<T>, а потом на основании этих действий, программа выполняет некоторую работу. Как-то так.
    – iluxa1810
    Commented 19 авг. 2015 в 21:52
  • Окей, понял. А не нужна ли вам случайно десериализация?
    – VladD
    Commented 19 авг. 2015 в 21:53
  • Возможно, это хороший вариант, но я с ней не разу не работал-(
    – iluxa1810
    Commented 19 авг. 2015 в 21:54
  • Добавил решение с десериализацией
    – VladD
    Commented 19 авг. 2015 в 22:18

2 ответа 2

7

Например, можно так:

var s = "<root><node attr1=\"\" attr2=\"\" attr3=\"\">Text</node></root>";
var doc = XDocument.Parse(s);
foreach (var node in doc.Descendants("node"))
{
    foreach (var attr in node.Attributes())
        Console.WriteLine("Name = {0}, Value = {1}", attr.Name, attr.Value);
}

Или если вам нужно собрать все атрибуты в один объект, то так:

var attrDict = node.Attributes().ToDictionary(attr => attr.Name, attr => attr.Value);
foreach (var kvp in attrDict)
    Console.WriteLine(kvp);

Если вы хотите проверить какой-то отдельный атрибут, это делается так:

var attr1 = node.Attribute("attr1");
if (attr1 != null)
{
    string attr1Value = attr1.Value;
    // ...
}

С учётом комментариев, возможно, вам подойдёт такое решение:

Описываем структуру данных как сериализируемую:

// так называется узел в XML
[XmlRoot("node")]
public class Animal
{
    // так называется атрибут в XML
    [XmlAttribute("length")]
    public int Length { get; set; }

    [XmlAttribute("name")]
    public string Name { get; set; }

    [XmlAttribute("color")]
    public string Color { get; set; }
}

Сама десериализация:

var s = "<root><node length=\"10\" name=\"Eeyore\" color=\"gray\">Text</node></root>";
var doc = XDocument.Parse(s);

var xmlSerializer = new XmlSerializer(typeof(Animal));
foreach (var node in doc.Descendants("node"))
{
    var animal = (Animal)xmlSerializer.Deserialize(node.CreateReader());
    // ...
}

Обратите внимание, что десериализатор сконвертировал для вас строку "10" в int.

Учтите, что десериализация сложна и налагает свои требования. Например, мне пришлось объявить класс Animal как public, и сделать свойства доступными для записи. Поэтому решение с десериализацией, возможно, не самое подходящее.

11
  • А для чего вы используете XDocument.Parse и XmlSerializer совместно? Вам это не кажется странным десериализовывать документ по частям?
    – pavelip
    Commented 20 авг. 2015 в 17:13
  • @pavelip: Нет, автор ведь не расписал структуру остальной части XML-документа. XDocument чтобы легко переходить между частями, XmlSerializer для собственно десериализации. Полностью отказаться от XDocument можно было бы, если бы было гарантировано, что весь документ представляет собой сериализированную структуру. Наоборот, если в нём есть «лишние» данные, пришлось бы писать ненужные пустые классы только для того, чтобы десериализировать их и выкинуть.
    – VladD
    Commented 20 авг. 2015 в 17:18
  • @pavelip: Вам, в свою очередь, не кажется странным, что у Deserialize есть перегрузка с XmlReader'ом?
    – VladD
    Commented 20 авг. 2015 в 17:20
  • С "лишними данными" XmlSerializer справляется отлично - он их спокойно пропускает. Про перегрузку не кажется странным :)
    – pavelip
    Commented 20 авг. 2015 в 17:29
  • @pavelip: Под «лишними» данными я имею в виду, например, узел <root/>, а не данные внутри <node/>. Как вы объясните XmlSerializer'у, что надо под root'ом найти все узлы node и десериализовать их? Может я чего-нибудь не знаю — покажите код лучше. (Код в вашем ответе создаёт лишний узел для root. Если в реальном xml такой лишней информации много, вам придётся создавать много фиктивных классов.)
    – VladD
    Commented 20 авг. 2015 в 18:07
1

Вариант 1

Сериализовать можно сразу весь xml документ:

const string XmlString = 
    "<root>" +
        "<node attr1=\"1\" attr2=\"mystr1\" attr3=\"true\">Text1</node>" +
        "<node attr1=\"2\" attr2=\"mystr2\" attr3=\"false\">Text2</node>" + 
    "</root>";

TaskRoot root;
var xmlSerializer = new XmlSerializer(typeof(TaskRoot));
using (TextReader reader = new StringReader(XmlString))
{
    root = (TaskRoot)xmlSerializer.Deserialize(reader);
}

foreach (var node in root.Nodes)
{
    Console.WriteLine("{0} {1} {2} {3}", node.Attr1, node.Attr2, node.Attr3, node.Text);
}

Для этого потребуется описать объектное представление xml документа:

[XmlRoot("root")]
public class TaskRoot
{
    [XmlElement("node")]
    public List<TaskNodeRoot> Nodes { get; set; }
}

public class TaskNodeRoot
{
    // так называется атрибут в XML
    [XmlAttribute("attr1")]
    public int Attr1 { get; set; }

    [XmlAttribute("attr2")]
    public string Attr2 { get; set; }

    [XmlAttribute("attr3")]
    public bool Attr3 { get; set; }

    [XmlText]
    public string Text { get; set; }
}

Вывод будет такой:

1 mystr1 True Text1
2 mystr2 False Text2

Вариант 2

Можно упростить код и написать отдельный helper-класс:

public static class XmlSerializerEx
{
    public static T Deserialize<T>(string xmlString)
    {
        var xmlSerializer = new XmlSerializer(typeof(T));
        using (TextReader reader = new StringReader(xmlString))
        {
            return (T)xmlSerializer.Deserialize(reader);
        }
    }
}

Тогда код будет выглядеть так:

var root = XmlSerializerEx.Deserialize<TaskRoot>(XmlString);

Ваш ответ

Нажимая «Отправить ответ», вы соглашаетесь с условиями пользования и подтверждаете, что прочитали политику конфиденциальности.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.