【C#】XMLの空タグ、空属性を削除したい。

Question

前提・実現したいこと

こんちには。
現在、C#にてXMLファイルを出力するアプリケーションを作っています。
XML自体の出力はできるのですが、空の要素や属性が多数あり見た目が非常に悪いです。
そのため、空の要素や属性をすべて削除する機能を実装させたいのですが、
私の知識では冗長性のあるコードしか書けなかったため、
どうすればスマートな実装ができるか、皆さんの意見をお聞きしたく質問させていただきました。

このようなXmlを

<root>
    <Name>hogehoge</Name>
    <Desc>hoge is hoge</Desc>
    <Category></Category>
    <Test/>
    <Features>
        <Feature From="">
        ...
        </Feature>
    </Features>
</root>

このように出力したいです。

<root>
    <Name>hogehoge</Name>
    <Desc>hoge is hoge</Desc>
    <Features>
        <Feature>
        ...
        </Feature>
    </Features>
</root>

できればコードだけで行いたいのですが
「コードだけでは難しい、こういう方法ならもっと楽」
という方法があれば教えていただけると幸いです。

よろしくお願いします。

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

開発環境：
Visual Studio 2017
C# WPF アプリケーション
.NET FW 4.7

追記

このXmlファイルは、私ではない人が編集することも考えられるものなので見た目の悪いものはできるだけ作りたくありません。

Xmlを構成するタグは独自タグなので、<br/>や<img>のalt属性など、消すと問題があるかもしれないものは含まれていないと思っていただいて構いません。

また、このXmlファイルは別のアプリケーション（自作のものではない）で読み込むためのXmlです。
そのため、使わない属性等があると誤作動を起こす可能性があります。
なので、できるだけ使わない属性を排除したいというのが考えです。

この考え方そのものが間違っている可能性もありますが……。
Xmlについては殆ど知らないといっても過言ではないので、まずそもそもが間違っているという場合は指摘してください。

情報の不足、大変申し訳ありませんでした。

Comments

[gazette2]

XmlSerializerですか。

[Lazialize]

アプリケーションで読み込んだ際にXML宣言がエラーを吐く原因になるみたいなので、XmlSerializerではなくXDcumentで行うことを考えています。

[Lazialize]

XmlSerializerのxml宣言を消せる場合は、方法を教えていただけると嬉しいです・・・。

Answer 1

XmlSerializerだと仮定して書きます。

例えば次のコードは空の要素を作ります。

[XmlElement]
public string Test { get; set; } = "";

この場合、[property name]Specifiedというbool形変数を作って出力をコントロールすることができます。

[XmlIgnore]
public bool TestSpecified = false;　　// Testという変数を無視してほしい。

[XmlIgnore]はTestSpecifiedという変数自体はXML要素ではないって言うことです。

「追加」
xml宣言文の消し方

XmlSerializer ser = new XmlSerializer(typeof(Group));
var settings = new XmlWriterSettings
{
	Indent = true,
	OmitXmlDeclaration = true
};

// A FileStream is used to write the file.
using (var fs = new FileStream("test.xml", FileMode.Create))
using (var writer = XmlWriter.Create(fs, settings))
	ser.Serialize(writer, myGroup);

Comments

[Lazialize]

加筆修正依頼の方へのお返事が遅れましたこと大変申し訳ありませんでした。

加筆修正依頼の方へも書いたとおり、XmlSerializerの使用は考えておりません。
XmlSerializerで出力した場合のXML宣言が消す方法がある場合は、助言をいただけると幸いです。

[gazette2]

XmlWriterSettingsを使用して消すことができます。

[Lazialize]

お早いお返事ありがとうございます。

XmlSerializerで対応できそうです。
お忙しいところ、ご丁寧にお答えいただき、ありがとうございます。

Answer 2

データ交換には普通はスキーマが必要です。
XMLのスキーマとしては、今は XML Schema が一般的です。
簡単なXML Schemaから始めよう

もう少し鷹揚にデータ交換したい場合は、JSONを使いましょう。

Comments

[Lazialize]

やはりスキーマが一般的なのですね。

すこし、スキーマについて勉強したいと思います。

[hihijiji]

スキーマはデータ構造の事で一般名詞です。
「XML Schema」は W3Cが標準化した固有名詞です。
混同なきようお願いします。

[Lazialize]

今回はgazette2さんの方法が一番簡単にできそうでしたので、gazette2さんをベストアンサーに選ばせていただきました。
この度はありがとうございました。

Answer 3

そのため、空の要素や属性をすべて削除する機能を実装させたいのですが、

そもそも論ですが、本当に削除してしまって大丈夫なのでしょうか。

たとえば、HTMLの<br />は属性も中身も指定しない空要素ですが意味を持ちますし、<img>のalt=''は「画像は飾りなので省略して問題ない」、一方altなしでは「画像がコンテンツの最重要ポイントで、文字表現ができない」と正反対の意味となります（MDN）。

「見た目が非常に悪い」ものでも、「意味を破壊してしまったもの」よりはずっとマシですし、通常の用法では「XML文の見た目」を気にすることもないかと思います。

Comments

[Lazialize]

<br />は絶対に入っていないと言えるので問題ありません。
タグもすべて独自のタグでして、「無いと困る」という属性はないので大丈夫です。

XMLは私ではない第三者が編集する可能性もあり、見た目の悪いものはできるだけ作りたくありません。

書き忘れてしまったのですが、このXMLファイルは別のアプリケーション(自作のものではありません)で読み込むものでして、無駄な属性などがあると逆にそれが原因で誤作動、あるいはエラーを吐く可能性があるため、できるだけ使わない属性はなくしたいのです。

情報の不足、大変申し訳ありませんでした。
追記しておきます。

Answer 4

空要素と言っても、例えば要素の中に空要素があり、それを削除することで新たに空要素が生まれることがあります。
これに対応するには深さ優先探索を行って再帰的に削除していくのが一番簡単ではないでしょうか。

void RemoveEmpty(XElement element)
{
	foreach (var a in element.Attributes().ToArray())
	{
		if (string.IsNullOrEmpty(a.Value))
		{
			a.Remove();
		}
	}
	foreach (var e in element.Elements().ToArray())
	{
		RemoveEmpty(e);
	}
	if (string.IsNullOrEmpty(element.Value) && !element.HasAttributes)
	{
		element.Remove();
	}
}

string source = @"<root>
    <Name>hogehoge</Name>
    <Desc>hoge is hoge</Desc>
    <Category></Category>
    <Test/>
    <Features>
        <Feature From="""">Foo</Feature>
    </Features>
</root>";
var xml = XElement.Parse(source);
RemoveEmpty(xml);
string seperator = "--------------";
Console.WriteLine(source);
Console.WriteLine(seperator);
Console.WriteLine(xml.ToString());

出力

<root>
    <Name>hogehoge</Name>
    <Desc>hoge is hoge</Desc>
    <Category></Category>
    <Test/>
    <Features>
        <Feature From="">Foo</Feature>
    </Features>
</root>
--------------
<root>
  <Name>hogehoge</Name>
  <Desc>hoge is hoge</Desc>
  <Features>
    <Feature>Foo</Feature>
  </Features>
</root>

Comments

[Lazialize]

すいません。
どうやら、入れ違いになったようです。

今確認しました。
わざわざ出力結果まで出していただき、ありがとうございます。
XDocumentではこのように削除できるのですね。勉強になりました。

今回は既に解決済みにしてしまったので、ベストアンサーにはできませんが、
大変ためになる知識です。
ありがとうございました。

Answer 5

こんな感じでしょうか（入力がファイルとなっていますが、そこは適宜……）。

using System;
using System.Text;
using System.Xml;
using System.Xml.Xsl;
namespace ConsoleApp1
{
    class Program
    {
        static void Main(string[] args)
        {
            var xslt = new XslCompiledTransform();
            xslt.Load(@"../../XSLTFile1.xslt");

            var output = new StringBuilder();
            var settings = new XmlWriterSettings();
            settings.Indent = true;
            settings.OmitXmlDeclaration = false;

            using (var writer = XmlWriter.Create(output, settings))
            {
                xslt.Transform(@"../../XMLFile1.xml", writer);
            }

            Console.WriteLine(output.ToString());
        }
    }
}

XSLTFile1.xslt

<?xml version="1.0" encoding="utf-8"?>
<xsl:stylesheet version="1.0" xmlns:xsl="http://www.w3.org/1999/XSL/Transform"
    xmlns:msxsl="urn:schemas-microsoft-com:xslt" exclude-result-prefixes="msxsl"
>
  <xsl:output method="xml" indent="yes"/>
  <xsl:strip-space elements="*"/>
  
  <xsl:template match="@* | node()">
    <xsl:if test=".!=''">
      <xsl:copy>
        <xsl:apply-templates select="@* | node()"/>
      </xsl:copy>
    </xsl:if>
  </xsl:template>
</xsl:stylesheet>

Comments

[Lazialize]

回答ありがとうございました。
わざわざコードまで書いていただき恐縮です。

今回はgazette2さんの方法が一番簡単にできそうでしたので、gazette2さんをベストアンサーに選ばせていただきました。
この度はありがとうございました。