C#でBinaryformatterを使ってクラスを繰り返しSerializeした後、ファイルをDeserializeしたい

Question

前提・実現したいこと

クラスをSerializeした後、そのファイルからクラスを配列で格納したいのですが、うまくいかず。。。
大変恐縮ですが、ご教授いただければ幸いです。
よろしくお願いします。

該当のソースコード

//連続して書くクラス
[Serializable]
public class data
{
 uint A;
 uint B;
 string C;
 List<double> D;
 List<byte> E;
}
 

//write関数
public void Write(data X, string filepath)
{
 Binaryfommatter bf = new Binaryformatter();
 FileStream fs = new FileStream(filepath,FileMode.Open);
 bf.Serialize(fs,data);
}

//Read関数
public data read(string filepath)
{
 Binaryfommatter bf = new Binaryformatter(); 
 FileStream fs = new FileStream(filepath,FileMode.Open);
 data dt = (dt)bf.Deserialize(fs);
}

試したこと

クラス一発だけSerializeしたときはDesrializeできることは確認できているのですが、
連発してSerializeしたときには処理がエクセプションなく関数が終了してしまう。

Comments

[kikukiku]

どのように連発してSerializeしているのか？Exceptionなく処理が終了する最低限のソースの開示が一番の早道かと思います。

[YAmaGNZ]

ソースを提示するのであれば、コンパイルの通るソースを提示すべきかと思います。
現状のソースですと、コンパイルを通るように修正しても動作しません。

Answer 1

次のように試してみてください。

using System;
using System.Collections.Generic;
using System.IO;
using System.Linq;
using System.Runtime.Serialization.Formatters.Binary;

namespace ConsoleApp1
{
    class Program
	{
		static void Main(string[] args)
		{
            string desktopDir = Environment.GetFolderPath(Environment.SpecialFolder.Desktop);
            string fileName = Path.Combine(desktopDir, "data.bin");
            using (var stream = File.Create(fileName))
            {
                var bf = new BinaryFormatter();
                bf.Serialize(stream, new Person("Alice", 13));
                bf.Serialize(stream, new Person("Bob", 14));
                bf.Serialize(stream, new Person("Charley", 15));
            }
            var People = Deserialize(fileName).ToArray();
            foreach (var person in People)
            {
                Console.WriteLine(person);
            }
            Console.ReadKey();
		}

        static IEnumerable<Person> Deserialize(string fileName)
        {
            using (var stream = File.OpenRead(fileName))
            {
                var bf = new BinaryFormatter();
                while (stream.Position < stream.Length)
                {
                    yield return (Person)bf.Deserialize(stream);
                }
            }
        }
	}

    [Serializable]
    class Person
    {
        public Person(string name, int age)
        {
            (Name, Age) = (name, age);
        }
        public string Name { get; set; }
        public int Age { get; set; }
        public override string ToString()
        {
            return $"{Name}({Age})";
        }
    }
}

Comments

[Zuishin]

(Name, Age) = (name, age);

ここでエラーが出る場合にはつぎのように書き換えてください。

Name = name;
Age = age;

[Zuishin]

return $"{Name}({Age})";

ここでエラーが出る場合には次のように書き換えてください。

return string.Format("{0}({1})", Name, Age);

[hihijiji]

ファイルポインタを使わなくてもFileStreamが場所を覚えてるんですね。

[Zuishin]

そうですね。
開いている間は覚えています。
質問を見たところ、データを一つ読むたびにファイルを閉じているのではないかと思いました。

[hihijiji]

と言うかBinaryFormatterは与えられたストリームを最後まで読んじゃうと思い込んでました。
勉強になりました。

[Inary]

無事クラスごとdeserialize出来ました。とても親切にご教授いただきありがとうございました。

Answer 2

ひょっとして、一つのファイルに追記でシリアライズ出力した後、それらをまとめて取り出したいってことでしょうか？
BinaryFormatterはそうゆう風に作られてませんので、それは簡単には出来ません。
実現するには別途ファイルポインタなどをの仕組みを自前で用意する必要があります。
とても面倒でバグの温床になり易いので、普通はしません。
初めから配列でシリアライズするかRDBMSなどを使ったほうが簡単で確実です。

Comments

[Inary]

回答ありがとうございます。
質問が下手で申し訳ありません。
仰るとおりで一つのファイルに追記でシリアライズ出力した後、まとめて配列で取り出したいということです。
BinaryFormatterが使用できないということは理解できました。
何万ものデータをバッファにためておくのはメモリが足りないので配列に格納することができない状況なので。。
別の方法というとテキスト出力もしくはRDBMSやNoSQLのようなデータベースによる出力を行う方が良いということですね。
大変ありがとうございました。