XMLファイルのデシリアライズ

Question

「階層化された繰返し要素のXMLをデシリアライズする」
http://zero-config.com/dotnet/xmlserializer002.html

この入門サイト(?)の内容を改造します。
<user>と同じ階層に要素を追加してみます("admin"と名付けました)。

できたXML
<?xml version="1.0" encoding="utf-8"?>
<system-config>
  <system-name>XXXXX</system-name>
  <version>1.1</version>
  <users>
    <user id="0001">
      <email>[e-mail]</email>
      <expired>2015/12/31</expired>
    </user>
    <user id="0002">
      <email>[e-mail]</email>
      <expired>2015/09/30</expired>
    </user>
    <admin id="000a">
      <email>[e-mail]</email>
      <expired>2015/12/31</expired>
    </admin>
    <admin id="000b">
      <email>[e-mail]</email>
      <expired>2015/09/30</expired>
    </admin>
  </users>
</system-config>

こうした場合でのクラス設計がわかりません。
"public class SystemConfig"の書き方がミソだとは思うのですが、色々試しても処理が通りません。

通らない例
/// <summary>
    /// システム設定
    /// </summary>
    [System.Xml.Serialization.XmlRoot("system-config")]
    public class SystemConfig
    {
        /// <summary>
        /// システム名
        /// </summary>
        [System.Xml.Serialization.XmlElement("system-name")]
        public string SystemName {get; set;}
        /// <summary>
        /// バージョン
        /// </summary>
        [System.Xml.Serialization.XmlElement("version")]
        public string Version { get; set; }
        /// <summary>
        /// ユーザ情報
        /// </summary>
        [System.Xml.Serialization.XmlArray("users")]
        [System.Xml.Serialization.XmlArrayItem("user")]
        public List<Sample.Model.User> Users { get; set; }
    ///アドミニ情報
    [System.Xml.Serialization.XmlArray("users")]
        [System.Xml.Serialization.XmlArrayItem("admin")]
        public List<Sample.Model.Admin> Admins{ get; set; }
    }

・
・
・

UserクラスをコピペしたAdminクラスを宣言

丸投げ気味な質問になってしまってますが、よろしくお願いします。

Answer 1

既にに解決しているようですが、XMLについて他のやり方も書き残します。
※質問時点のXMLの構造は適切ではないので、ishi9様の回答のようにroleを付け加えるのが一番いいです。

• Classを自動で生成する方法

]

• Classを作成せずに動的にXMLを扱う方法

var str = @"<xml> ... </xml>"
var xml = XDocument.Parse(str);
var version = xml.Element("system-config").Element("version").Value;
foreach (var x in xml.Element("system-config").Element("users").Elements("user"))
{
  Console.WriteLine(x.Element("email").Value);
}

foreach (var x in xml.Element("system-config").Element("users").Elements("admin"))
{
  Console.WriteLine(x.Element("email").Value);
}

Comments

[nyahonyaho]

わざわざありがとうございます。しかしちんぷんかんぷんなので、「ホニャララでググれ」的なことを教えていただけないでしょうか

[aglkjggg]

Classを自動で生成する方法 については、
手動でクラスを書いているのかと思ったので、XMLをコピーした状態で「XMLをクラスとして貼り付ける」を選択すると自動でクラスが生成されるという意味でした。
何も考えずにXMLクラスを自動で生成してくれるので楽ですし、巨大かつ構造が複雑なXMLの場合にヒューマンエラーを防ぐことができます。

Classを自動で生成する方法 については、
System.Xml.Serialization.XmlSerializer を使うのではなく、
System.Xml.Linq.XDocument を使う方法もあるという紹介でした。
System.Xml.Linq.XDocument を使うと、classの生成が不要です。

[nyahonyaho]

重ねてわざわざありがとうございます。

Answer 2

usersに入れられるクラスは一種類だけです。他のクラスは入りません。(userとadminの共有はできない)
ですので、以下の様にUserクラスを拡張してみるのはいかがでしょうか？

    /// <summary>
    /// ユーザ情報
    /// </summary>
    [Serializable]
    public class User
    {
        /// <summary>
        /// ID
        /// </summary>
        [System.Xml.Serialization.XmlAttribute("id")]
        public string Id { get; set; }
        /// <summary>
        /// メールアドレス
        /// </summary>
        [System.Xml.Serialization.XmlElement("email")]
        public string MailAddress {get; set;}
        /// <summary>
        /// 有効期限
        /// </summary>
        [System.Xml.Serialization.XmlElement("expired")]
        public string Expired { get; set; }

        // 以下を追加

        /// <summary>
        /// 役割
        /// </summary>
        [System.Xml.Serialization.XmlElement("role")]
        public string Role { get; set; }
    }

<?xml version="1.0" encoding="utf-8"?>
<system-config>
  <system-name>XXXXX</system-name>
  <version>1.1</version>
  <users>
    <user id="0001">
      <email>[e-mail]</email>
      <expired>2015/12/31</expired>
      <role>user</role>
    </user>
    <user id="0002">
      <email>[e-mail]</email>
      <expired>2015/09/30</expired>
      <role>user</role>
    </user>
    <userid="000a">
      <email>[e-mail]</email>
      <expired>2015/12/31</expired>
      <role>admin</role>
    </user>
    <userid="000b">
      <email>[e-mail]</email>
      <expired>2015/09/30</expired>
      <role>admin</role>
    </user>
  </users>
</system-config>

Comments

[ishi9]

もうひとつ考えられるのはUsersとは別にAdminsを作ってしまう事です。
以下サンプル

c#
///アドミニ情報
[System.Xml.Serialization.XmlArray("admins")] // ←ここをadminsにする
[System.Xml.Serialization.XmlArrayItem("admin")]
public List<Sample.Model.Admin> Admins{ get; set; }

xml
<?xml version="1.0" encoding="utf-8"?>
<system-config>
<system-name>XXXXX</system-name>
<version>1.1</version>
<users>
<user id="0001">
<email>[e-mail]</email>
<expired>2015/12/31</expired>
</user>
<user id="0002">
<email>[e-mail]</email>
<expired>2015/09/30</expired>
</user>
</users> ←userはusersの中に入れる
<admins> ←adminはadminsの中に入れる
<admin id="000a">
<email>[e-mail]</email>
<expired>2015/12/31</expired>
</admin>
<admin id="000b">
<email>[e-mail]</email>
<expired>2015/09/30</expired>
</admin>
</admins>
</system-config>

[nyahonyaho]

で、できた。素直にXmlElementで作っていけばよかったのですね。ありがとうございます。

Answer 3

C#については、ほとんど、門外漢なのですが、 users 要素に対する宣言が2つあり、余計なように見えます。

    /// <summary>
    /// ユーザ情報
    /// </summary>
    [System.Xml.Serialization.XmlArray("users")]
    [System.Xml.Serialization.XmlArrayItem("user")]
    public List<Sample.Model.User> Users { get; set; }
    ///アドミニ情報
    [System.Xml.Serialization.XmlArrayItem("admin")]
    public List<Sample.Model.Admin> Admins{ get; set; }

ではだめでしょうか？