Json: How to handle not existing member

[paule96]

Currently, I am trying to implement a maybe weird case in my internal project. The basic concept is that any JSON object can be part of a JSON array.

So something like that:

[
   {
      "animal": "cat"
   },
   {
      "color": "green"
   }
]

If I now try to run something like that:

JsonDocument data = null; // something sets the value of JsonDocument so it's not null
data.Any(config: System.Linq.Dynamic.Core.SystemTextJson.Config.SystemTextJsonParsingConfig.Default, "x => x.number == 5"));

I will now get an exception like that:

The binary operator Equal is not defined for the types 'System.Object' and 'System.Int32'.

My question is, is there some kind of expression that I can write, that makes it possible to make a member optional?

Or some setting that I can set?

[StefH]

@paule96
Maybe you can try this:
https://dynamic-linq.net/advanced-configuration#allowequalsandtostringmethodsonobject-since-v1.6.0

[paule96]

@StefH
We tried this but it doesn't help us.
But I created reproduction tests with the commit:

paule96@c0b8e02

The following tests are falling:

• compare a non exsisting value with a number value (notExisting > 1)
• compare a non exsisting value with a boolean value (notExisting == true)

Comparing to strings are just working

[StefH]

@paule96
OK, thanks for the unit tests.

However, I would expect that when you try to query an non-existing member, you would get error like:
Unhandled exception. Microsoft.CSharp.RuntimeBinder.RuntimeBinderException: 'xxx' does not contain a definition for 'number'

[paule96]

Yes I would expect the same for the default path.
But also when you work with json, it's kinda normal to miss members.

So it would be nice to have a syntax or a configuration that allows to compare to non existing members.

[StefH]

#954

[StefH]

I did fix the code so that your unit tests pass. However you will need to provide ConvertObjectToSupportComparison = true and missing members are not yet supported.

[paule96]

very cool thank you. We will try it out the next days