Неинициализированный массив бинарной сериализации - PullRequest
1 голос
/ 05 марта 2011

Я предоставил минимальный код для имитации сценария. Вот код:

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


namespace Serialization
{
    class Program
    {
        static void Main(string[] args)
        {
            string[] annotates = { "1", "2"};
            Guides[] g1 = new Guides[2];
            g1[0].comments = (string[])annotates.Clone();
            g1[1].comments = (string[])annotates.Clone();

            Guides[] g2 = new Guides[2];
            g2[0].comments = (string[])annotates.Clone();
            g2[1].comments = (string[])annotates.Clone();//to be commented later

            arrayStruct arrStr1 = new arrayStruct();
            arrStr1.guides_array = g1;

            arrayStruct arrStr2 = new arrayStruct();
            arrStr2.guides_array = g2;

            using (MoveSaver objSaver = new MoveSaver(@"C:\1.bin"))
            {
                MoveAndTime mv1 = new MoveAndTime();
                MoveAndTime mv2 = new MoveAndTime();
                mv1.MoveStruc = "1";
                mv1.timeHLd = DateTime.Now;
                mv1.arr = arrStr1;
                objSaver.SaveToFile(mv1);
                mv2.MoveStruc = "2";
                mv2.timeHLd = DateTime.Now;
                mv2.arr = arrStr2;
                objSaver.SaveToFile(mv2);
            }

            using (MoveSaver svrObj = new MoveSaver())
            {
                List<MoveAndTime> MVTobjs = svrObj.DeSerializeObject(@"C:\1.bin");
                foreach (MoveAndTime item in MVTobjs)
                {
                    Console.WriteLine(item.arr.guides_array[0].comments[0]);
                }
            }
        }

    }


    public class MoveSaver : IDisposable
    {
        public void Dispose()
        {
            if (fs != null)
            {
                fs.Close();
            }
        }
        FileStream fs;
        StreamWriter sw;
        public string filename { get; set; }
        public MoveSaver(string FileName)
        {
            this.filename = FileName;
            fs = new FileStream(filename, FileMode.OpenOrCreate, FileAccess.ReadWrite);
        }

        public MoveSaver()
        {

        }

        ~MoveSaver()
        {
            if (fs != null)
            {
                fs.Close();
            }

        }

        public List<MoveAndTime> DeSerializeObject(string filename)
        {
            List<MoveAndTime> retList = new List<MoveAndTime>();
            MoveAndTime objectToSerialize;
            Stream stream = File.Open(filename, FileMode.Open);
            BinaryFormatter bFormatter = new BinaryFormatter();
            while (stream.Position != stream.Length)
            {
                objectToSerialize = (MoveAndTime)bFormatter.Deserialize(stream);
                retList.Add(objectToSerialize);
            }
            stream.Close();
            return retList;
        }


        public bool SaveToFile(MoveAndTime moveTime)
        {
            try
            {
                BinaryFormatter bformatter = new BinaryFormatter();
                bformatter.Serialize(fs, moveTime);
                return true;
            }
            catch (Exception)
            {
                return false;
            }
        }
    }

    [Serializable]
    public struct MoveAndTime
    {
        public string MoveStruc;
        public DateTime timeHLd;
        public arrayStruct arr;
    }

    [Serializable]
    public struct arrayStruct
    { 
        public Guides[] guides_array;
    }

    [Serializable]
    public struct Guides
    {
        public string[] comments;
        public string name;
    }
}

В этом коде структура содержит несколько структур, одна из которых содержит массив. Попробуйте код, и он прекрасно скомпилируется, но в реальном сценарии весь массив не заполнен, поэтому будут другие элементы массива, не указанные. Чтобы увидеть этот эффект (в действии!), Прокомментируйте строку g2[1].comments = (string[])annotates.Clone(); и попробуйте код сейчас. Вы столкнетесь с ошибкой при десериализации. Как я могу избежать этого? Должен ли я определить структуру, содержащую массив, как класс, и добавить их все (надеюсь, я ищу решение на основе структуры)?

Edit: Я изменил структуры на класс и обновляя каждый экземпляр, он работает нормально. Вот код:

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

namespace Serialization
{
    class Program
    {
        static void Main(string[] args)
        {
            string[] annotates = { "1", "2"};
            GuidesClass[] g1 = new GuidesClass[2];
            g1[0] = new GuidesClass();
            g1[0].comments = (string[])annotates.Clone();
            g1[1] = new GuidesClass();
            g1[1].comments = (string[])annotates.Clone();

            GuidesClass[] g2 = new GuidesClass[2];
            g2[0] = new GuidesClass();
            g2[0].comments = (string[])annotates.Clone();
            g2[1] = new GuidesClass();
            //g2[1].comments = (string[])annotates.Clone();

            array_cls  arrStr1 = new array_cls();
            arrStr1.guides_array = g1;

            array_cls arrStr2 = new array_cls();
            arrStr2.guides_array = g2;

            using (MoveSaver objSaver = new MoveSaver(@"C:\1.bin"))
            {
                M_T mv1 = new M_T();
                M_T mv2 = new M_T();
                mv1.MoveStruc = "1";
                mv1.timeHLd = DateTime.Now;
                mv1.arr = arrStr1;
                objSaver.SaveToFile(mv1);
                mv2.MoveStruc = "2";
                mv2.timeHLd = DateTime.Now;
                mv2.arr = arrStr2;
                objSaver.SaveToFile(mv2);
            }

            using (MoveSaver svrObj = new MoveSaver())
            {
                List<M_T> MVTobjs = svrObj.DeSerializeObject(@"C:\1.bin");
                foreach (M_T item in MVTobjs)
                {
                    Console.WriteLine(item.arr.guides_array[0].comments[0]);
                }
            }
        }

    }


    public class MoveSaver : IDisposable
    {
        public void Dispose()
        {
            if (fs != null)
            {
                fs.Close();
            }
        }
        FileStream fs;
        StreamWriter sw;
        public string filename { get; set; }
        public MoveSaver(string FileName)
        {
            this.filename = FileName;
            fs = new FileStream(filename, FileMode.OpenOrCreate, FileAccess.ReadWrite);
        }

        public MoveSaver()
        {

        }

        ~MoveSaver()
        {
            if (fs != null)
            {
                fs.Close();
            }

        }

        public List<M_T> DeSerializeObject(string filename)
        {
            List<M_T> retList = new List<M_T>();
            M_T objectToSerialize;
            Stream stream = File.Open(filename, FileMode.Open);
            BinaryFormatter bFormatter = new BinaryFormatter();
            while (stream.Position != stream.Length)
            {
                objectToSerialize = (M_T)bFormatter.Deserialize(stream);
                retList.Add(objectToSerialize);
            }
            stream.Close();
            return retList;
        }


        public bool SaveToFile(M_T moveTime)
        {
            try
            {
                BinaryFormatter bformatter = new BinaryFormatter();
                bformatter.Serialize(fs, moveTime);
                return true;
            }
            catch (Exception)
            {
                return false;
            }
        }
    }


    [Serializable]
    public class M_T
    {
        public string MoveStruc;
        public DateTime timeHLd;
        public array_cls arr;
    }

    [Serializable]
    public class array_cls
    {
        public GuidesClass[] guides_array = new GuidesClass[10];
    }

    [Serializable]
    public class GuidesClass
    {
        public string[] comments;
        public string name;
    }
}

1 Ответ

1 голос
/ 05 марта 2011

Я поиграл с кодом, и я могу воспроизвести как struct, так и class;в конечном счете, я подозреваю, что проблема здесь заключается в том, что BinaryFormatter не предназначен для такой же возможности добавления, т. е. я подозреваю, что он неправильно интерпретирует данные из объекта next как часть текущего.* FWIW, эти действительно не должны быть структурами, и вы (IMO) ужасно перегружаете код сохранения / загрузки.Я изменил его так, чтобы метод сохранения занял List<MoveAndTime>, а код загрузки возвратил single List<MoveAndTime> (т. Е. В потоке только один крайний объект), и он работал нормально, поддерживая мойтеория.

Если вам нужно постепенно добавлять отдельные объекты, я бы предложил protobuf-net;вы можете использовать SerializeWithLengthPrefix для записи объекта способом, подходящим для добавления, и DeserializeWithLengthPrefix для чтения одного объекта из потока или DeserializeItems для чтения (как последовательность) всех элементовв потоке.


Например, используя protobuf-net v2 (в данный момент доступен только как код):

using System;
using System.Collections.Generic;
using System.IO;
using ProtoBuf;


namespace Serialization
{
    class Program
    {
        static void Main(string[] args)
        {
            string[] annotates = { "1", "2" };
            Guides[] g1 = new Guides[2];
            g1[0].comments = (string[])annotates.Clone();
            g1[1].comments = (string[])annotates.Clone();

            Guides[] g2 = new Guides[2];
            g2[0].comments = (string[])annotates.Clone();
            g2[1].comments = (string[])annotates.Clone();//to be commented later

            arrayStruct arrStr1 = new arrayStruct();
            arrStr1.guides_array = g1;

            arrayStruct arrStr2 = new arrayStruct();
            arrStr2.guides_array = g2;

            using (Stream file = File.Create(@"1.bin"))
            {
                MoveAndTime mv1 = new MoveAndTime();
                MoveAndTime mv2 = new MoveAndTime();
                mv1.MoveStruc = "1";
                mv1.timeHLd = DateTime.Now;
                mv1.arr = arrStr1;
                Serializer.SerializeWithLengthPrefix(file, mv1, PrefixStyle.Base128, Serializer.ListItemTag);
                mv2.MoveStruc = "2";
                mv2.timeHLd = DateTime.Now;
                mv2.arr = arrStr2;
                Serializer.SerializeWithLengthPrefix(file, mv2, PrefixStyle.Base128, Serializer.ListItemTag);
            }

            using (Stream file = File.OpenRead(@"1.bin"))
            {
                List<MoveAndTime> MVTobjs = Serializer.Deserialize<List<MoveAndTime>>(file);
                foreach (MoveAndTime item in MVTobjs)
                {
                    Console.WriteLine(item.arr.guides_array[0].comments[0]);
                }
            }
        }

    }

    [ProtoContract]
    public struct MoveAndTime
    {
        [ProtoMember(1)]
        public string MoveStruc;
        [ProtoMember(2)]
        public DateTime timeHLd;
        [ProtoMember(3)]
        public arrayStruct arr;
    }

    [ProtoContract]
    public struct arrayStruct
    {
        [ProtoMember(1)]
        public Guides[] guides_array;
    }

    [ProtoContract]
    public struct Guides
    {
        [ProtoMember(1)]
        public string[] comments;
        [ProtoMember(2)]
        public string name;
    }
}

v1 (доступно как dll; более стабильно)будет работать почти так же, но не поддерживает структуры - только классы.

Но следует подчеркнуть:

  • они должны быть классами
  • открытые поля плохиеидея
...