Json.Net DeserializeObject与OData.Delta失败

这个问题正在影响我的ASP.Net WebApi Patch方法,它看起来很像这样:

public MyModel Patch(int id, [FromBody]Delta<MyModel> newRecord){/*stuff here*/}

但这不是WebApi,这是问题 - 失败在Json.Net和OData.Delta之间。

问题是JsonConvert.DeserializeObject没有看到OData.Delta对象的整数,我想知道是否有我可以应用的解决方法或修复。

更新:在Json.Net库中已经编写了代码(请参见下面的内容)来解决这个问题。 只需要它被包含在下一次更新中(如果詹姆斯牛顿 - 金允许的话)

更新2:经过进一步测试后,我决定采取行动的最佳方案是停止使用OData.Delta并编写我自己的(请参阅答案)

单元测试证明存在问题(使用下面的语句以清晰起见)

测试1:失败,出现int(Int32):

class TestObjWithInt
{
    public int Int { get; set; }
}
[TestMethod]
public void IsApplied_When_IntIsDeserializedToDelta()
{
    string testData = "{"Int":1}";
    var deserializedDelta = JsonConvert.DeserializeObject<Delta<TestObjWithInt>>(testData);
    var result = deserializedDelta.GetChangedPropertyNames().Contains("Int");
    Assert.IsTrue(result);
}

测试2:长时间成功(Int64)

class TestObjWithLong
{
    public long Long { get; set; }
}
[TestMethod]
public void IsApplied_When_LongIsDeserializedToDelta()
{
    string testData = "{"Long":1}";
    var deserializedDelta = JsonConvert.DeserializeObject<Delta<TestObjWithLong>>(testData);
    var result = deserializedDelta.GetChangedPropertyNames().Contains("Long");
    Assert.IsTrue(result);
}

为了确保反序列化的开始,这两个测试都通过了。

[TestMethod]
public void IsApplied_When_LongIsDeserializedToTestObject()
{
    string testData = "{"Long":1}";
    var deserializedObject = JsonConvert.DeserializeObject<TestObjWithLong>(testData);
    var result = deserializedObject.Long == 1;
    Assert.IsTrue(result);
}
[TestMethod]
public void IsApplied_When_IntIsDeserializedToTestObject()
{
    string testData = "{"Int":1}";
    var deserializedObject = JsonConvert.DeserializeObject<TestObjWithInt>(testData);
    var result = deserializedObject.Int == 1;
    Assert.IsTrue(result);
}

我发现这个OData的错误报告听起来像一个类似的问题,但它的旧和封闭,所以可能不会。

任何帮助都会很棒。

使用语句(从测试文件的顶部):

using System;
using System.Linq;
using System.Web.Http.OData;
using Microsoft.VisualStudio.TestTools.UnitTesting;
using Newtonsoft.Json;

如果James Newton-King接受了解决方案 - 更改为版本6.0.6。 替换JsonSerializerInternalReader.cs行1581:

contract.TrySetMember(newObject, memberName, value);

有:

bool done = false;
while (!(done = done || contract.TrySetMember(newObject, memberName, value)))
{
    switch (reader.TokenType)
    {
        case JsonToken.Integer:
            if (value is long && ((long)value) <= Int32.MaxValue && ((long)value) >= Int32.MinValue)
                value = Convert.ToInt32(value);
            //Add else if (...) to cast to other data types here (none additional required to date).
            else
                done = true;
            break;
        default:
            done = true;
            break;
    }
}

OData.Delta <T>不适用于除Int64以外的任何数字类型的Json.Net。 最简单的方法是为OData.Delta <T>(我在公司时间完成的,因此我不能完整发布它抱歉)编写替代方案,其中包含如下方法:

private bool TrySetInt32(object value, PropertyInfo propertyInfo, bool isNullable)
{
    var done = false;
    if (value is Int32)
    {
        propertyInfo.SetValue(_obj, value);
        done = true;
    }
    else if (value == null)
    {
        if (isNullable)
        {
            propertyInfo.SetValue(_obj, value);
            done = true;
        }
    }
    else if (value is Int64) //Json.Net - fallback for numbers is an Int64
    {
        var val = (Int64)value;
        if (val <= Int32.MaxValue && val >= Int32.MinValue)
        {
            done = true;
            propertyInfo.SetValue(_obj, Convert.ToInt32(val));
        }
    }
    else
    {
        Int32 val;
        done = Int32.TryParse(value.ToString(), out val);
        if (done)
            propertyInfo.SetValue(_obj, val);
    }
    return done;
}

这个类可以是这样一个动态泛型:

public sealed class Patchable<T> : DynamicObject where T : class, new()

有了这样一个工作变量:

T _obj = new T();

在重写的TrySetMember方法中,我们需要使用反射来检查属性的基础类型,并调用相应的TrySet ...方法,如下所示:

if (underlyingType == typeof(Int16))
    done = TrySetInt16(value, propertyInfo, isNullable);
else if (underlyingType == typeof(Int32))
    done = TrySetInt32(value, propertyInfo, isNullable);

如果值设置成功,我们可以将属性名称添加到列表中,然后我们可以使用它来修补原始记录,如下所示:

if (done)
    _changedPropertyNames.Add(propertyInfo.Name);

public void Patch(T objectToPatch)
{
    foreach (var propertyName in _changedPropertyNames)
    {
        var propertyInfo = _obj.GetType().GetProperty(propertyName);
        propertyInfo.SetValue(objectToPatch, propertyInfo.GetValue(_obj));
    }
}

之后的68个单元测试,它似乎工作得很好。 这是一个例子:

class TestObjWithInt32
{
    public Int32 Int32 { get; set; }
    public Int32? SetNullable { get; set; }
    public Int32? UnsetNullable { get; set; }
}
[TestMethod]
public void IsApplied_When_Int32IsDeserializedToPatchable()
{
    string testData = "{"Int32":1,"SetNullable":1}";
    var deserializedPatchable = JsonConvert.DeserializeObject<Patchable<TestObjWithInt32>>(testData);
    var result = deserializedPatchable.ChangedPropertyNames.Contains("Int32");
    Assert.IsTrue(result);
    var patchedObject = new TestObjWithInt32();
    Assert.AreEqual<Int32>(0, patchedObject.Int32);
    deserializedPatchable.Patch(patchedObject);
    Assert.AreEqual<Int32>(1, patchedObject.Int32);
    Assert.IsNull(patchedObject.UnsetNullable);
    Assert.IsNotNull(patchedObject.SetNullable);
}

这是基于Rob解决方案的这个问题的实现:

public sealed class Patchable<T> : DynamicObject where T : class {
    private readonly IDictionary<PropertyInfo, object> changedProperties = new Dictionary<PropertyInfo, object>();
    public override bool TrySetMember(SetMemberBinder binder, object value) {
        var pro = typeof (T).GetProperty(binder.Name);
        if (pro != null)
            changedProperties.Add(pro, value);
        return base.TrySetMember(binder, value);
    }
    public void Patch(T delta) {
        foreach (var t in changedProperties)
            t.Key.SetValue(
                delta,
                t.Key.PropertyType.IsEnum ? Enum.Parse(t.Key.PropertyType, t.Value.ToString()) : Convert.ChangeType(t.Value, t.Key.PropertyType));
    }
}

我使用字典而不是时间对象删除了泛型类型参数中必需的空构造函数。

感谢Rob;)

链接地址: http://www.djcxy.com/p/82637.html

上一篇: Json.Net DeserializeObject failing with OData.Delta

下一篇: Collapsing margin on Flexbox children