rumi
rumi

Reputation: 3298

how to produce xml of list of objects using XElement in c#

I am trying to convert a list of objects into xml using XElement. My code is as below

 var employees = new List<Employee>();

 employees.Add(new Employee("1", " Ali","",1000));
 employees.Add(new Employee("2", "Billy","",1001));

 var xml = new XElement("root");
 xml.Add(new XElement("Employees"));

 foreach (var employee in employees)
  {
    xml.Add(new XElement("employee", employee.Name));
  };

I want an output like this

<root>
 <Employees>
   <employee>Ali</employee>
   <employee>Billy</employee>
 </Employees>
</root>

but the output I m getting is

 <root>
 <Employees/>
   <employee>Ali</employee>
   <employee>Billy</employee>
</root>

I tried moving the Employees object inside the loop like below but that does not help either

 foreach (var employee in employees)
  {
    xml.Add(new XElement("Employees", new XElement("employee", employee.Name)));

  };

Upvotes: 0

Views: 1238

Answers (2)

nb1987
nb1987

Reputation: 1410

Within your loop, you should call Add() on your employees element rather than on your root:

 var employees = new List<Employee>();

 employees.Add(new Employee("1", " Ali","",1000));
 employees.Add(new Employee("2", "Billy","",1001));

 var xml = new XElement("root");
 var employeesElement = new XElement("Employees");

 foreach (var employee in employees)
 {
    employeesElement.Add(new XElement("employee", employee.Name));
 }
 xml.Add(employeesElement);

Upvotes: 2

har07
har07

Reputation: 89285

XElement constructor can handle IEnumerable so you can add employee elements at once while creating the parent Employees. In fact, the entire XML can be created at once :

var xml = new XElement("root",
    new XElement("Employees",
        employees.Select(e => new XElement("employee", e.Name))
    )
);

Upvotes: 2

Related Questions