XMLのシリアライズの際に構造がおかしくなってしまいます。

Question

前提・実現したいこと

C#にてＸＭＬファイルからデシリアライズした内容を、別ファイルに同内容、同構造でシリアライズしたいです。
ＸＭＬファイルは以下の同要素名の階層が、繰り返し入れ子となっています。

<main>
 <products>
   <product name=pan>
     <substance name=komugi>
     </substance>
     <substance name=sio>
     </substance>
   </product>
   <product name=gohan>
     <substance name=kome>
     </substance>
     <substance name=mizu>
     </substance>
   </product>
 </products>
</main>

発生している問題・エラーメッセージ

シリアライズして出力した際に階層がおかしくなります。現状の出力結果は以下の通りです。

<main>
 <products>
   <product name=pan>
     <substance name=komugi>
     </substance>
     <substance name=sio>
     </substance>
     <substance name=kome>
     </substance>
     <substance name=mizu>
     </substance>
   </product>
   <product name=gohan>
   </product>
 </products>
</main>

該当のソースコード

現状のソースコードになります。

var output_xml = XDocument.Load(output_xml_filename).Element("main");

var xml = XDocument.Load(xml_filename).Element("main");

foreach (var main in xml.Elements("main"))
{
   foreach(var products in main.Elements("products"))
   {
      output_xml.Add(new XElement("products"));
      var output_products = output_xml.Element("products");

      foreach(var product in products.Elements("product"))
      {
         product_name = product.Attribute("name").Value;  
         output_products.Add(new XElement("product"
                                     , new XAttribute("name", product_name)));
         var output_products_product = output_products.Element("product");

         foreach(var substance in product.Elements("substance"))
         {
            substance_name = substance.Attribute("name").Value;              
            output_products_product.Add(new XElement("substance"
                                        , new XAttribute("name", substance_name)));
         }
      }
   }
}

試したこと

すみません。他の方法が思いつかず手詰まりになってしまっています。何卒ご教授頂きたいです。

補足情報（FW/ツールのバージョンなど）

開発はVisualStadio2017です。

Answer 1

こんにちは。

同内容同構造と言ってる以上、Zuishin さんの回答が大正解です。開いた xml を中身を見ることもせず output_xml に書くだけで良いのですから。

必要あるかはわかりませんが、現状のコードにおいて問題が起きた原因を示すと、

         output_products.Add(new XElement("product"
                                     , new XAttribute("name", product_name)));
         var output_products_product = output_products.Element("product");

ここで output_products に product という同名の element を複数追加しているため、直後の output_products.Element("product") で element を取得したときに必ず「最初の一つ」が手に入るため、その後 output_products_product に要素を追加すると、1つ目の product に全ての substance が追加されるのです。

         var output_products_product = new XElement("product", new XAttribute("name", product_name)));
         output_products.Add(output_products_product);

こうするだけで問題は解決します。output_products_product は常に新しく作った element になるためです。

Comments

[t.sei]

ありがとうございます。そういうことだったのですね。
理解が進みました。今後に役立たせていただきます。

Answer 2

xml.Save(ファイル名)

追記

以下のコードは XMLFile1.xml を読み込み、pan の子要素として yeast を追加し、out.xml に書き込みます。

using System;
using System.IO;
using System.Linq;
using System.Xml.Linq;

namespace ConsoleApp1
{
	class Program
	{
		static void Main(string[] args)
		{
			string dir = AppDomain.CurrentDomain.BaseDirectory;
			string xml_filename = Path.Combine(dir, "XMLFile1.xml");
			string output_xml_filename = Path.Combine(dir, "out.xml");
			var xml = XDocument.Load(xml_filename);
			var output_xml = XDocument.Parse(xml.ToString());
			var pan = output_xml
				.Root
				.Element("products")
				.Elements("product")
				.Where(a => (string)a.Attribute("name") == "pan")
				.First();
			pan.Add(new XElement("substance", new XAttribute("name", "yeast")));
			output_xml.Save(output_xml_filename);
			Console.WriteLine(output_xml);
			Console.ReadKey();
		}
	}
}

Comments

[t.sei]

申し訳ありません。記述が抜けていました。
xml.SaveはForeachループをすべて抜けた後記述しています。

[Zuishin]

output_xml ではなく xml を保存すればいいではありませんか。

[Zuishin]

保存の話ではなく、オブジェクトをコピーしたいのなら、var output_xml = XDocument.Parse(xml.ToString()); が簡単です。

[t.sei]

ありがとうございます。単純にコピーするわけではなかった為に自分で記述の必要がありました。
質問内容が簡素すぎて誤解を生んでしまい申し訳ございませんでした。

[Zuishin]

単純にコピーした後で必要なところだけ変更すればいいではありませんか。

[t.sei]

ありがとうございます。処理内容は複数の同構造（階層数はバラバラ）をもつＸＭＬファイルを一つのＸＭＬファイルとする処理でして、そのなかでシリアライズがうまくできずにおりました。はじめてＸＭＬファイルを扱うコードを記述しなければならず、理解が進まずにいたところです。ひとつひとつ手探りだったため、最初につまずいた箇所を質問させて頂きました。他にもやり方がいろいろあるとは思うのですが、思いつかずにいたところであります。何度もコメント頂きましてありがたく思います。今後ともご指導のほどよろしくお願い申し上げます。

[Zuishin]

やり方の一つのように言わないでください。

シリアライズには Save メソッドが用意されています。
せっかく LINQ to XML を使っているんですから、その機能を活用してください。

Answer 3

もうベストアンサー出ているところに失礼しますが、
単にnewしたXElementをvarで受けてそれをAddすれば済む話に見えます。

※追記
申し訳ありません。質問者さんがよく分からないまま場当たり的な解決法を採ったように見えたのでスマホから慌てて書き込んだ結果こんな雑な回答になりました。。
tomato様が原因から適切な解決法までわかりやすく記述して下さっているのでそちらを参照してください。

Comments

[t.sei]

ありがとうございます。今後に役立たせていただきます。

Answer 4

もうすでに簡単な方法は出ていますが、もしも自分で処理を書く必要があるならば
var output_products_product = output_products.Element("product");
を
var output_products_product = output_products.Elements("product").Last();

Comments

[t.sei]

ご教示ありがとうございます。目的の処理ができました。