Deconstruct JsonLogic for conversion into another format about json-everything HOT 8 CLOSED

@gregsdennis yes, I could really use that! I already found a solution to my problem but this would allow me to make the code a lot more readable!

from json-everything.

@gregsdennis thanks for your quick response and changes!

from json-everything.

Hey there. I think deserializing in the JsonLogic model might be overkill for your need. Have you considered just parsing into JsonNode and browsing the JSON data that way?

var rule = JsonNode.Parse(jsonLogicString).AsObject();
if (rule.ContainsKey("and"))
{
  foreach (var item in rule["and"].AsArray())
  // do stuff with items
}
else if (rule.ContainsKey("or"))
{
  // do stuff with orRule.Items
}

You may also want to have a read through #368 as that's the issue where the protected internal entered the picture.

I could make the properties public, but they're supposed to be immutible, so I'd have to change the property types as well. Since they're accessible via subclasses, it'll be a breaking change.

from json-everything.

I just realized you could do this as well because the .Net team made "decisions" with the JsonNode API:

var rule = JsonNode.Parse(jsonLogicString);
if (rule["and"] is not null)
{
  foreach (var item in rule["and"].AsArray())
  // do stuff with items
}
else if (rule.ContainsKey("or"))
{
  // do stuff with orRule.Items
}

from json-everything.

I did go that way, but I was hoping that instead of string checking I could check types. That should be more accurate and allow for stricter typed code. But I followed your suggestion and that also works.
What you maybe could do instead of exposing the Item property is exposing it as an enumerator. That seems to be what the .net team did for JsonDocument as well.

from json-everything.

Thanks for the recommendation. I'll continue to play around with things to see what I can come up with. Maybe extension methods would do?

from json-everything.

I think this could work:

public static IEnumerable<Rule> GetItems(this AndRule rule)
{
    return rule.Items.AsReadOnly();
}

Some of them are a bit redundant, but if you don't see the internal properties, I guess you wouldn't notice.

public static Rule GetInput(this AllRule rule)
{
	return rule.Input;
}

public static Rule GetRule(this AllRule rule)
{
	return rule.Rule;
}

from json-everything.

@birgerj would something like ☝️ work for you? I can push this out soon.

from json-everything.