CODEWITHSUNDEEP
c#xmlserializerixmlserializable
Daniele Mereu
Daniele Mereu

Reputation: 115

C# XML serialization type of IXmlSerializable

I want to serialize and deserialize an object like this:

public class Root
{
    public string title;
    [XmlArray("lista")]
    [XmlArrayItem("Parameter")]
    public List<Parameter> lista;
}

public class Parameter : IXmlSerializable
{
    public string name;
    public int val;

    public void WriteXml(XmlWriter writer)
    {
        writer.WriteElementString("name",name);
        writer.WriteElementString("val",val.ToString());
    }

    public void ReadXml(XmlReader reader)
    {           
        reader.ReadToDescendant("name");
        name = reader.ReadElementContentAsString();
        val = reader.ReadElementContentAsInt();
    }

    public XmlSchema GetSchema()
    {
        return (null);
    }
}

The problem is that when I deserialize this object, it is added only the first element of the list. Example:

Serialized:

<Root>
<title>title</title>
<lista>
<Parameter>
  <name>pa</name>
  <val>1</val>
</Parameter>
<Parameter>
  <name>pb</name>
  <val>2</val>
</Parameter>
</lista>
</Root>

Deserialized:

<Root>
<title>title</title>
<lista>
<Parameter>
  <name>pa</name>
  <val>1</val>
</Parameter>
</lista>
</Root>

I've realized an Fiddle example. Could someone help me?

Upvotes: 3

Views: 2141

Answers (2)

esiprogrammer
esiprogrammer

Reputation: 1438

You are just reading the first element you shoud add reader.Read(); at the end of RealXml method to create a loop:

public void ReadXml(XmlReader reader)
{
  reader.ReadToDescendant("name");
  name = reader.ReadElementContentAsString();
  val = reader.ReadElementContentAsInt();
  reader.Read();
}

but you don't need to implement IXmlSerializable if you don't have a particular reason for it. you could simply use XmlSerializer as below:

 XmlSerializer serializer = new XmlSerializer(typeof(Root));
 StringBuilder result = new StringBuilder();
 using (var writer = XmlWriter.Create(result))
 {
     serializer.Serialize(writer, root);  /// serialize              
 }

 using (var reader = new StringReader(result.ToString()))
 {
    var deserialized = (Root)serializer.Deserialize(reader); //// deserialize from string
 }

Upvotes: 4

Tim Rutter
Tim Rutter

Reputation: 4679

If you don't have to use a custom serializer changing Parameter class to the following solves the problem (tested in your fiddle example):

public class Parameter
{
    public string name {get;set;}
    public int val {get;set;}

}

Upvotes: 0

Related Questions