Problem with C# XmlSerialization

Question

I have xml file:

<?xml version="1.0" encoding="utf-8"?>
<LabelTypesCollection xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance="xmlns:xsd="http://www.w3.org/2001/XMLSchema">
  <LabelTypes>    
    <LabelType>
      <Name>LabelTypeProduct</Name>
    </LabelType>
    <LabelType>
      <Name>LabelTypeClient</Name>
    </LabelType>    
  </LabelTypes>
</LabelTypesCollection>

And 2 c# classes:

[Serializable]
[XmlRoot("LabelTypesCollection")]
public class LabelTypesCollection
{
    private static string _labelTypesCollectionPath = Path.Combine(Environment.GetFolderPath(Environment.SpecialFolder.ApplicationData), Path.Combine(Program.ProgramName, "LabelTypesCollection.xml"));

    [XmlArray("LabelTypes", ElementName="LabelType")]
    public List<LabelType> LabelTypes { get; set; }

    public static LabelTypesCollection LoadAllLabelTypes()
    {
        FileInfo fi = new FileInfo(_labelTypesCollectionPath);
        if (!fi.Exists)
        {
            Logger.WriteLog("Could not find size_types_collection.xml file.", new Exception("Could not find size_types_collection.xml file."));
            return new LabelTypesCollection();
        }

        try
        {
            using (FileStream fs = fi.OpenRead())
            {
                XmlSerializer serializer = new XmlSerializer(typeof(LabelTypesCollection));
                LabelTypesCollection labelTypesCollection = (LabelTypesCollection)serializer.Deserialize(fs);
                return labelTypesCollection;
            }
        }
        catch (Exception ex)
        {
            Logger.WriteLog("Error during loading LabelTypesCollection", ex);
            return null;
        }
    }
}


[Serializable]    
public class LabelType
{
    [XmlElement("Name")]
    public string Name { get; set; }

    [XmlIgnore]
    public string TranslatedName
    {
        get
        {
            string translated = Common.Resources.GetValue(Name);
            return (translated == null) ? Name : translated;
        }
    }
}

And when I call:

LabelTypesCollection.LoadAllLabelTypes();

I get LabelTypeCollection object with empty LabelTypes list. There is no error or anything. Could anyone point me to the problem?

Remove your try-catch block and you might have a chance of working out what's going on. — batwad
– batwad, Commented Dec 6, 2010 at 11:19
Did you check the log. I am sure its unable to locate the file. What is the value of _labelTypesCollectionPath — Madhur Ahuja
– Madhur Ahuja, Commented Dec 6, 2010 at 11:20
Yes, I debugged it and file exists, it deserializes. I even go to console and create StreamReader on fs, call ReadToEnd() on it and file content appeared — Adrian Serafin
– Adrian Serafin, Commented Dec 6, 2010 at 11:32

AakashM · Accepted Answer · 2010-12-06 12:29:15Z

Change this

[XmlArray("LabelTypes", ElementName="LabelType")]

to this

[XmlArray]

The ElementName of an XmlArrayAttribute specifies the element name of the container, and is actually what you specify in the first parameter to the ctor! So the ctor you have says "this class serializes as a container named LabelTypes; no wait actually I want the container to be named LabelType". The named parameter is overwriting what the first unnamed parameter says.

And in fact, since you want the container element to be named LabelTypes, which is what the member is actually called, you don't need to specify it at all.

You may have been thinking of XmlArrayItemAttribute, which controls what the individual members of a serialized collection are named - but you don't need that here either.

My usual approach for working out xml serializer stuff is to build objects manually then look at the xml they serialize to. In this case, using the code you currently have produces xml like this:

<?xml version="1.0" encoding="utf-16"?>
<LabelTypesCollection xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xmlns:xsd="http://www.w3.org/2001/XMLSchema">
  <LabelType>
    <LabelType>
      <Name>one</Name>
    </LabelType>
    <LabelType>
      <Name>two</Name>
    </LabelType>
  </LabelType>
</LabelTypesCollection>

which is what tipped me off to the incorrect LabelType specifier.

Note that you also don't need the XmlRoot on LabelTypesCollection, or the XmlElement on Name, since you are just specifying what the xml serializer will come up with anyway.

Ran · Accepted Answer · 2010-12-06 12:23:27Z

2

Here's a suggestion.

Write a small test program that creates an instance of LabelTypesCollection, and adds some LabelType objects into it.

Then use an XmlSerializer to write the object to a file, and look at the Xml you get, to ensure that your input Xml is in the correct schema.

Maybe there's something wrong with one of your Xml elements.

answered Dec 6, 2010 at 12:23

Ran

6,1991 gold badge28 silver badges28 bronze badges

Comments

Pabuc · Accepted Answer · 2010-12-06 11:34:50Z

0

I really think you get an empty list because your code can't find the xml file. Also try instantiating your list. If you have the xml path correctly.

public List<LabelType> LabelTypes = new List<LabelType>();

answered Dec 6, 2010 at 11:34

Pabuc

5,6488 gold badges40 silver badges52 bronze badges

1 Comment

Adrian Serafin Over a year ago

I init LabelTypes in constructor and it didn't help. I'm 100% sure that xml file exists...

Collectives™ on Stack Overflow

Problem with C# XmlSerialization

3 Answers 3

Comments

Comments

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

Comments

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related