0

自分でC#でListやT[]をXmlシリアライズ・デシリアライズする拡張メソッドを書いたのですが、例外処理はどうすればいいのか(使う側で定義するのか使われる側で定義するのか等)と、あと書き方について指摘を頂きたいです。よろしくお願いします。

   ///<summary> Xmlをジェネリックな配列に読み込むメソッド </summary> 
    public static void LoadXml<T>(this T[] list, string path)
    {
        T[] xmldata = null;

        XmlDocument xdoc = new System.Xml.XmlDocument();
        XmlSerializer xml = new System.Xml.Serialization.XmlSerializer(typeof(T[]));

        xdoc.PreserveWhitespace = true;
        xdoc.Load(path);

        XmlNodeReader xnr = new System.Xml.XmlNodeReader(xdoc.DocumentElement);

        xmldata = (T[])xml.Deserialize(xnr);

        for (int i = 0; i < xmldata.Length; i++)
            list[i] = xmldata[i];
    }

    ///<summary> XmlをList<T>に読み込むメソッド </summary>
    public static void LoadXml<T>(this List<T> list, string path)
    {
        list.Clear();

        T[] xmldata = null;

        XmlDocument xdoc = new System.Xml.XmlDocument();
        XmlSerializer xml = new System.Xml.Serialization.XmlSerializer(typeof(T[]));

        xdoc.PreserveWhitespace = true;
        xdoc.Load(path);

        XmlNodeReader xnr = new System.Xml.XmlNodeReader(xdoc.DocumentElement);

        xmldata = (T[])xml.Deserialize(xnr);

        for (int i = 0; i < xmldata.Length; i++)
            list.Add(xmldata[i]);
    }

    /// <summary> ジェネリックな配列をXmlにセーブするメソッド </summary>
    public static void SaveXml<T>(this T[] list, string path)
    {
        T[] xmldata = null;
        xmldata = list.ToArray();

        using (FileStream fs = new FileStream(path, FileMode.Create))
        {
            XmlSerializer xml = new XmlSerializer(typeof(T[]));

            xml.Serialize(fs, xmldata);
        }
    }

    /// <summary> List<T>オブジェクトをXmlにセーブするメソッド </summary>
    public static void SaveXml<T>(this List<T> list, string path)
    {
        T[] xmldata = null;
        xmldata = list.ToArray();

        using (FileStream fs = new FileStream(path, FileMode.Create))
        {
            XmlSerializer xml = new XmlSerializer(typeof(T[]));

            xml.Serialize(fs, xmldata);
        }
    }
}

2 件の回答 2

1

まずSaveXmlですが、T[]List<T>の2バージョン記述するのは冗長で不毛です。両クラスが共通して実装しているインターフェースに対する拡張メソッドにすることでコードを統一できます。
同様にLoadXmlも2バージョン記述されていて冗長ですが、それ以前にT[]List<T>を第1引数に要求する意義がなく、新規のオブジェクトを作成しそれを返すべきです。つまり拡張メソッドであること自体が不適切です。
これらを踏まえると次のように記述できるでしょうか。

public static class XmlUtil {
    public static T[] LoadFrom<T>(string path) {
        var serializer = new XmlSerializer(typeof(T[]));
        using (var stream = File.OpenRead(path))
            return (T[])serializer.Deserialize(stream);
    }

    public static void SaveTo<T>(this IEnumerable<T> source, string path) {
        var serializer = new XmlSerializer(typeof(T[]));
        using (var stream = File.OpenWrite(path))
            serializer.Serialize(stream, source as T[] ?? source.ToArray());
    }
}

例外処理については何を懸念されているのかわかりませんでした。
気になる点として、XmlSerializerの他にもDataContractSerializerがありシリアル化方法なども異なりますが、利用者には選択権がありません。これに限らずどのようにシリアル化されるかの責務はメソッド側となってしまい、利用者は制御できないわけですが、それでいいのでしょうか。

1

XmlSerializerXmlReader、もしくはTextReaderStreamなどの一方向アクセスを行う型を引数に取ります。これは内部的にXMLノードを先頭から順に処理しており、XMLドキュメント全体へアクセスは不要であることを表しています。

一方のXmlDocumentはXMLドキュメント全体に対して自由な操作を提供するクラスです。ですので一般論としてはXmlReaderよりメモリ使用量や初期化にかかる時間が大きくなります。

ですので、不必要にXmlDocumentをインスタンス化することは避けた方がよいです。XmlDocument.PreserveWhitespaceに対応するXmlReaderSettings.IgnoreWhitespaceは既定でfalseですので、特にオプションを指定せずに

XmlReader xnr = XmlReader.Create(path);

と置き換えることができます。前述のとおりXmlSerializer.DeserializeStreamおよびTextWriterを受け付けますので、引数とする型はXmlReaderではなくFileStreamStreamReaderとすることもできます。

また例外処理についてはエラー発生時の復帰処理を含めて考える必要があります。例えば「保存失敗時は1秒後に再度書き込みを行い、10回失敗したらエラーとする」場合などこのメソッド専用の処理が必要な場合はSaveXml内で例外を処理する必然性がありますが、特にないのでしたらアプリケーションの呼び出し階層最上位に近い場所で汎用的な処理をすべきだと思います。

この質問に回答するには、ログインする必要があります。

求めていた回答ではありませんか? のタグが付いた他の質問を参照する。