XML読込で、要素を出力したい[C#]

前提・実現したいこと

XMLを読み込んで、以下のソースでいう、GyoshuBunruiのMeshoと、
その中の要素のMesho「,」区切りでを出力したいのですが、どこが間違っているのか分かりません。
想定結果の通りになりません。ご教授願います。

想定結果---
業種１,業種１_１

今の出力結果
業種１,

C#のソースコードはVisualStudioで作成しました。

該当のソースコード

下記は、XMLのマスタに関するクラスです。

using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Threading.Tasks;
using System.Xml;
using System.Xml.Serialization;

namespace ConsoleApp1
{
    public class XmlMaster
    {
        [XmlElement("GyoshuBunrui")]
        public List<GyoshuBunrui> GyoshuBunruiList = new List<GyoshuBunrui>();

        public static XmlMaster Deserialize()
        {
            string filename = @"c:\users\XmlMaster.xml";
            XmlMaster result = null;

            if (!System.IO.File.Exists(filename)) return new XmlMaster();

            //xmlファイルでストリームを作成            
            using (System.IO.FileStream fs = new System.IO.FileStream(filename, System.IO.FileMode.Open, System.IO.FileAccess.Read))
            {
                //指定した型に逆シリアル化できるようにするオブジェクトを作成
                XmlSerializer xmlSerializer = new XmlSerializer(typeof(XmlMaster));
                //逆シリアル化を実行し、オブジェクトに格納
                result = xmlSerializer.Deserialize(fs) as XmlMaster;
            }
            return result;
        }
        //結果を作成します
        public string CreateList(List<GyoshuBunrui> list)
        {
            string str1 = "";
            string str2 = "";
            foreach (var rec in list)
            {
                str1 += rec.Mesho;
                if (rec.SubBunruiList.Count > 0) str2 = CreateList(rec.SubBunruiList);
            }
            return str1 + "," + str2;
        }
    }

    [XmlType("GyoshuBunrui")]
    public class GyoshuBunrui
    {
        [XmlAttribute(AttributeName = "CD")]
        public int CD = 0;
        [XmlAttribute(AttributeName = "Mesho")]
        public string Mesho = "";
        [XmlElement(ElementName = "SubBunruiList")]
        public List<GyoshuBunrui> SubBunruiList = new List<GyoshuBunrui>();
    }
}

下記は、実際に実行するメソッドです。
実際にはmainメソッドの中で、Initメソッドを呼び出し、
コンソールに文字列として出力します。

using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Threadi

namespace ConsoleApp1
{
    class XmlOpe
    {
        public string Init()
        {
            XmlMaster xmlMaster = XmlMaster.Deserialize();
　　　　　　　//結果を返します
            return xmlMaster.CreateList(xmlMaster.GyoshuBunruiList);
        }
    }
}

下記は、使うXMLファイルの中身です。

<?xml version="1.0" encoding="utf-8"?>
<XmlMaster xmlns:xs="http://www.w3.org/2001/XMLSchema-instance">
  <GyoshuBunrui CD="1" Mesho="業種１" />
    <SubBunruiList>
      <GyoshuBunrui CD="11" Mesho="業種１_１" />
    </SubBunruiList>
</XmlMaster>

行動規範の内容に同意します

回答1件

ベストアンサー

そういう出力になるのは、SubBunruiクラスがうまくパースできていないからですが、下記の1と2または3の修正をする必要があります。

1 タグがおかしい

大きな誤りとして、"XmlMaster.Xml"の業種1が行内でタグが閉じられているのが絶対的におかしいです。

XML
1<?xml version="1.0" encoding="utf-8"?>
2<XmlMaster xmlns:xs="http://www.w3.org/2001/XMLSchema-instance">
3  <GyoshuBunrui CD="1" Mesho="業種１">
4    <SubBunruiList>
5      <GyoshuBunrui CD="11" Mesho="業種１_１" />
6    </SubBunruiList>
7  </GyoshuBunrui>
8</XmlMaster>

でなければ、階層構造になっていません。

2 GyoshuBunruiクラスのアノテーションに対してXMLのタグがおかしい

XMLとクラスがあってません。もし、クラス側が正しいのだとしたら

XML
1<?xml version="1.0" encoding="utf-8"?>
2<XmlMaster xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance">
3  <GyoshuBunrui CD="1" Mesho="業種１">
4    <SubBunruiList CD="11" Mesho="業種１_１" />
5  </GyoshuBunrui>
6</XmlMaster>

でないと、うまくデシリアライズできません。

3 Xmlのタグに対して、GyoshuBunruiクラスのアノテーションがおかしい。

XMLとクラスがあってません。もし、XML側が正しいのだとしたら

csharp
1    public class GyoshuBunrui
2    {
3        [XmlAttribute(AttributeName = "CD")]
4        public int CD = 0;
5        [XmlAttribute(AttributeName = "Mesho")]
6        public string Mesho = "";
7        [XmlArray("SubBunruiList")]
8        [XmlArrayItem("GyoshuBunrui")]
9        public List<GyoshuBunrui> SubBunruiList = new List<GyoshuBunrui>();
10    }

こういうアノテーションをしないと、うまくデシリアライズできません。

参考情報
0. 業種1,業種1_1,になる
末尾に,がつきます。string.IsNullOrEmpty(str2)? str1 : $"{str1},{str2}"みたいなことをしないと、末尾に無駄な,がつきます
0. こういう場合、シリアライズをかけると容易にデバッグできます。得たい結果をクラスとして作ってそれをファイルにシリアライズすると、タグとアノテーションがあってないことが一目瞭然となります。
0. その手法は本当に最適ですか？
何がしたいかによりますが、XDocumentでLinqしたほうがいいということはないですか。特にカンマ区切りで出すのに、","を使うというのはちょっと考えられないくらい悪い解決だと思っています(今回末尾に不要なカンマが出ていましたが、同じようなパターンで先頭に無意味なカンマが出るバグもあります。そのどちらのバグも実装していた製品をメンテしていたことがあり、基本的にstring.Join(",", IEnumerable<T>)を使えるような解決策を考えたほうがいいです)

投稿2018/08/04 15:46