C# десериализация протобуфа

Question

Почему десериализация protobuf не работает для переменной-члена OffsetDictionary? Это прекрасно работает, если я не использую словарь в качестве вспомогательного поля. Кроме того, он отлично работает, если тип OffsetDictionary изменяется со сложного типа на простой SortedDictionary. Я что-то здесь упускаю?

[ProtoContract]
Public class Test
{
    [ProtoMember(1)]
    public DateTime BaseDate {get; set;};

    [ProtoMember(2)]
    public SortedDictionary<short, SortedDictionary<short, uint>> OffsetDictionary
    {
        get
        {
            var output = new SortedDictionary<short, SortedDictionary<short, uint>>();
            if (this.Dictionary != null)
            {
                foreach (var item in this.Dictionary)
                {
                    var timeSpan = item.Key - this.BaseDate;
                    short offset = Convert.ToInt16(timeSpan.TotalDays);
                    output.Add(offset, item.Value);
                }
            }

            return output;
        }

        set
        {
            if (this.Dictionary == null)
            {
                this.Dictionary = new SortedDictionary<DateTime, SortedDictionary<short, uint>>();
            }

            foreach (var item in value)
            {
                this.Dictionary.Add(this.BaseDate.AddDays(item.Key), item.Value);
            }
        }
    }

    public SortedDictionary<DateTime, SortedDictionary<short, uint>> Dictionary { get; set; }
}

Answer 1

Проблема здесь заключается в предположении, что десериализатор вызывает установщик - либо вообще, либо именно тогда, когда вы ожидаете; не требуется для . Сериализатор предполагает достаточно типичные реализации, в этом случае совершенно разумно следующее:

// when field 2
var val = obj.OffsetDictionary;
bool setValue = false;
if (val == null)
{
    val = new SortedDictionary<short, SortedDictionary<short, uint>>();
    setValue = true;
}
do {
    val.Add(/* parse this entry */);
} while (/* still field 2 */)
if (setValue) obj.OffsetDictionary = val;

Хотя обратите внимание, что присвоение в начале (где setValue назначено) также будет законным.

Как это может случиться, вы можете своего рода сделать эту работу, используя

[ProtoMember(2, OverwriteList = true)]

, но ... это работает неправильно Причины IMO, поскольку это также может быть реализовано с тем же псевдокодом, что и выше, но просто добавив .Clear(), который не изменит вывод.

Честно говоря, я не уверен, что я действительно нравится общий дизайн здесь; лично я бы сохранил данные в форме, которую вы собираетесь сериализовать , и добавил бы служебные методы для Test, которые выполняют переворот, т.е.

[ProtoContract]
public class Test
{
    [ProtoMember(1)]
    public DateTime BaseDate { get; set; }

    [ProtoMember(2)]
    public SortedDictionary<short, SortedDictionary<short, uint>> OffsetDictionary { get; }
        = new SortedDictionary<short, SortedDictionary<short, uint>>();

    private short ToInt16(DateTime value) => (short)(value - BaseDate).TotalDays;

    public void Add(DateTime key, SortedDictionary<short, uint> value)
        => OffsetDictionary.Add(ToInt16(key), value);
    public bool TryGetValue(DateTime key, out SortedDictionary<short, uint> value)
        => OffsetDictionary.TryGetValue(ToInt16(key), out value);
}

Однако вы можно также сделать это с помощью слоя-обертки - хотя это гораздо больше работы:

[ProtoContract]
public class Test
{
    [ProtoMember(1)]
    public DateTime BaseDate { get; set; }

    private DictionaryWrapper _offsetDictionary;

    [ProtoMember(2)]
    public IDictionary<short, SortedDictionary<short, uint>> OffsetDictionary
        => _offsetDictionary ?? (_offsetDictionary = new DictionaryWrapper(this));

    public SortedDictionary<DateTime, SortedDictionary<short, uint>> Dictionary { get; }
        = new SortedDictionary<DateTime, SortedDictionary<short, uint>>();

    class DictionaryWrapper : IDictionary<short, SortedDictionary<short, uint>>
    {
        public DictionaryWrapper(Test parent)
        {
            _parent = parent;
        }
        private readonly Test _parent;
        private DateTime ToDateTime(short value) => _parent.BaseDate.AddDays(value);
        private short ToInt16(DateTime value) => (short)(value - _parent.BaseDate).TotalDays;

        SortedDictionary<short, uint> IDictionary<short, SortedDictionary<short, uint>>.this[short key]
        {
            get => _parent.Dictionary[ToDateTime(key)];
            set => _parent.Dictionary[ToDateTime(key)] = value;
        }

        int ICollection<KeyValuePair<short, SortedDictionary<short, uint>>>.Count => _parent.Dictionary.Count;

        bool ICollection<KeyValuePair<short, SortedDictionary<short, uint>>>.IsReadOnly => false;

        void IDictionary<short, SortedDictionary<short, uint>>.Add(short key, SortedDictionary<short, uint> value)
            => _parent.Dictionary.Add(ToDateTime(key), value);

        void ICollection<KeyValuePair<short, SortedDictionary<short, uint>>>.Add(KeyValuePair<short, SortedDictionary<short, uint>> item)
            => _parent.Dictionary.Add(ToDateTime(item.Key), item.Value);

        void ICollection<KeyValuePair<short, SortedDictionary<short, uint>>>.Clear()
            => _parent.Dictionary.Clear();

        private ICollection<KeyValuePair<DateTime, SortedDictionary<short, uint>>> AsCollection => _parent.Dictionary;
        bool ICollection<KeyValuePair<short, SortedDictionary<short, uint>>>.Contains(KeyValuePair<short, SortedDictionary<short, uint>> item)
            => AsCollection.Contains(new KeyValuePair<DateTime, SortedDictionary<short, uint>>(ToDateTime(item.Key), item.Value));

        bool IDictionary<short, SortedDictionary<short, uint>>.ContainsKey(short key)
            => _parent.Dictionary.ContainsKey(ToDateTime(key));

        private IEnumerator<KeyValuePair<short, SortedDictionary<short, uint>>> GetEnumerator()
        {
            foreach (var item in _parent.Dictionary)
                yield return new KeyValuePair<short, SortedDictionary<short, uint>>(ToInt16(item.Key), item.Value);
        }
        IEnumerator<KeyValuePair<short, SortedDictionary<short, uint>>> IEnumerable<KeyValuePair<short, SortedDictionary<short, uint>>>.GetEnumerator()
            => GetEnumerator();

        IEnumerator IEnumerable.GetEnumerator()
            => GetEnumerator();

        bool IDictionary<short, SortedDictionary<short, uint>>.Remove(short key)
            => _parent.Dictionary.Remove(ToDateTime(key));

        bool ICollection<KeyValuePair<short, SortedDictionary<short, uint>>>.Remove(KeyValuePair<short, SortedDictionary<short, uint>> item)
            => AsCollection.Remove(new KeyValuePair<DateTime, SortedDictionary<short, uint>>(ToDateTime(item.Key), item.Value));

        bool IDictionary<short, SortedDictionary<short, uint>>.TryGetValue(short key, out SortedDictionary<short, uint> value)
            => _parent.Dictionary.TryGetValue(ToDateTime(key), out value);

        // these are kinda awkward to implement
        ICollection<short> IDictionary<short, SortedDictionary<short, uint>>.Keys
            => throw new NotSupportedException();

        ICollection<SortedDictionary<short, uint>> IDictionary<short, SortedDictionary<short, uint>>.Values
            => throw new NotSupportedException();

        void ICollection<KeyValuePair<short, SortedDictionary<short, uint>>>.CopyTo(KeyValuePair<short, SortedDictionary<short, uint>>[] array, int arrayIndex)
            => throw new NotSupportedException();
    }
}