Comments (8)
@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.
Related Issues (20)
- Update Nuget package meta-data HOT 6
- Error "unknown context value" Raised After Checking for Existence HOT 2
- Online schema validator: Is successful validation of illegal json just a quirk or intentional HOT 5
- System.MissingMethodException for IReadOnlyCollection<JsonNode> after updating from JsonSchema.Net.Generation 4.3.0 to 4.3.0.2 HOT 5
- Can't get DiscriminatorKeyword working HOT 2
- Schema Evaluation fails with "Cannot resolve schema at {uri}" HOT 5
- Could not load file or assembly 'JsonPointer.Net, Version=5.0.0.0, Culture=neutral, PublicKeyToken=17ed63f672b0e278'. The system cannot find the file specified. HOT 10
- IndexOutOfRangeException occurred when evaluating schema with parallel execution. HOT 1
- jsonnet as sister of json-e HOT 3
- Escaping single quote in json path string HOT 8
- Extract data annotation attributes from JsonSchema.Net.Generation into separate project HOT 2
- Is there a way to do JSON validation that does case insensitive matching on property names? HOT 8
- Support language packs in schema playground
- sileo-featured.json
- How to export or save JsonSchema to plaintext? HOT 4
- How to write a JsonSchemaBuilder for a class that has a property of a class? HOT 4
- Schema generation doesn't work on site HOT 1
- maximum/minimum use decimal which do not parse certain many double "number" strings HOT 3
- `Json.Schema.JsonSchema.Evaluate` throws "`System.OverflowException`: Value was either too large or too small for a Decimal." HOT 4
- Get json data from only required properties based on the attributes set HOT 4
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from json-everything.