c# - 在 JsonConvert.DeserializeObject 中反序列化对象时出现意外标记

Question

我有以下测试代码:

[TestClass]
public class TestJsonDeserialize
{
    public class MyClass
    {
        [JsonProperty("myint")]
        public int MyInt { get; set; }
        [JsonProperty("Mybool")]
        public bool Mybool { get; set; }
    }

    [TestMethod]
    public void Test1()
    {
        var errors = new List<string>();
        var json1 = "{\"myint\":1554860000,\"Mybool\":false}";
        var json2 = "{\"myint\":3554860000,\"Mybool\":false}";
        var i = JsonConvert.DeserializeObject<MyClass>(json2, new JsonSerializerSettings
        {
            Error = delegate (object sender, Newtonsoft.Json.Serialization.ErrorEventArgs args)
            {
                Debug.WriteLine(args.ErrorContext.Error.Message);
                errors.Add(args.ErrorContext.Error.Message);
                args.ErrorContext.Handled = true;
            }
        });
        Assert.IsTrue(errors.Count <= 1);
    }
}

对 JsonConvert.DeserializeObject 的调用产生了 2 个错误。其中一个是预期的，但另一个不是。 错误是:

• JSON 整数 3554860000 对于 Int32 来说太大或太小。路径“myint”，第 1 行，位置 19。
• 反序列化对象时出现意外标记: bool 值。路径“Mybool”，第 1 行，位置 34。

为什么第一个错误标记为已处理，但出现第二个错误。 我已经从 Newtonsoft.Json 8.0.2 更新到 9.0.1，但它仍然存在。 当传递第一个字符串时(json1 而不是 json2)，则不会发生任何错误。

Answer 1

更新

报告为 Issue 1194: JsonTextReader.ParseNumber leads to error after ThrowReaderError并被 Newtonsoft 关闭，因为在当时的当前版本中不可复制，随后发布为 Json.NET 10.0.1 .

原始答案

这可能是 JsonTextReader 中的错误.

在JsonTextReader.ParseNumber(ReadType readType, char firstChar, int initialPosition)有以下逻辑，有些简化:

else if (readType == ReadType.ReadAsInt32)
{

// Snip

        int value;
        ParseResult parseResult = ConvertUtils.Int32TryParse(_stringReference.Chars, _stringReference.StartIndex, _stringReference.Length, out value);
        if (parseResult == ParseResult.Success)
        {
            numberValue = value;
        }
        else if (parseResult == ParseResult.Overflow)
        {
            throw ThrowReaderError("JSON integer {0} is too large or small for an Int32.".FormatWith(CultureInfo.InvariantCulture, _stringReference.ToString()));
        }
        else
        {
            throw ThrowReaderError("Input string '{0}' is not a valid integer.".FormatWith(CultureInfo.InvariantCulture, _stringReference.ToString()));
        }
    }

    numberType = JsonToken.Integer;
}

// Snip
// Finally, after successfully parsing the number

ClearRecentString();

// index has already been updated
SetToken(numberType, numberValue, false);

在 ThrowReadError() 抛出异常时，流位置已经超过了过大的整数。但是，JsonReader.TokenType 的值尚未更新，仍返回 JsonToken.PropertyName对于成功解析的最后一个标记，即 "myint" 名称。后来异常被吞噬忽略后，流位置与当前token值不一致导致"Mybool"属性名被跳过，导致第二个错误。

如果，在调试器中，抛出异常时我手动调用

SetToken(JsonToken.Undefined);
ClearRecentString();

然后文件的其余部分就可以成功解析了。 (我不确定 JsonToken.Undefined 是否是正确的选择。)

您可能想要 report an issue到 Newtonsoft。

由于 JsonReader 没有传递给错误处理程序，我能找到的唯一解决方法是子类化 JsonTextReader，如下所示:

public class FixedJsonTextReader : JsonTextReader
{
    public FixedJsonTextReader(TextReader reader) : base(reader) { }

    public override int? ReadAsInt32()
    {
        try
        {
            return base.ReadAsInt32();
        }
        catch (JsonReaderException)
        {
            if (TokenType == JsonToken.PropertyName)
                SetToken(JsonToken.None);
            throw;
        }
    }
}

然后做:

var errors = new List<string>();
var json2 = "{\"myint\":3554860000,\"Mybool\":false}";

using (var reader = new FixedJsonTextReader(new StringReader(json2)))
{
    var settings = new JsonSerializerSettings
    {
        Error = delegate(object sender, Newtonsoft.Json.Serialization.ErrorEventArgs args)
        {
            Debug.WriteLine(args.ErrorContext.Error.Message);
            errors.Add(args.ErrorContext.Error.Message);
            args.ErrorContext.Handled = true;
        }
    };
    var i = JsonSerializer.CreateDefault(settings).Deserialize<MyClass>(reader);
}
Assert.IsTrue(errors.Count <= 1); // Passes