[RFC] Implement custom iterator for JSONArray to ensure fetched values are JSON compliant

Offen

#673 erstellt am 21.3.2022 3 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit: 5/5
Geschätzter Aufwand: Über eine Woche
Anfängerfreundlichkeit: 25/100

Issue-Typ: Bug
Klarheit: Muss geklärt werden
Aktivitätsstatus: Veraltet
Tech-Stack: java
Bereich: backend

Rechercherichtung

Start with JSONArray.iterator and review the related behavior in JSONObject, JSONArray put* operations, and toString methods; issue #672 provides the referenced context. Decide between lazy iterator wrapping and proactive wrapping during insertion, then verify that code-built values exposed to JSON processors are compliant without changing parsed objects.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

Active discussion Fix before the next release

This has been a "known" issue for some time, but as mentioned in #672, the JSONArray.iterator method is exposing the raw values from the backing array instead of JSON-compliant values.

We should do 1 of 2 things:

Implement a private/internal JSONArrayIterator class that would ensure any fetched values are properly "wrapped" and thus suitable output to JSON processors.
Update all of our put* operations to ensure that all values are properly "wrapped" in both JSONObject and JSONArray. This would also allow us to simplify our toString methods and remove the wrapping from there as all values should be proper JSON.

Option 1 keeps the wrapping "lazy" and thus would only guarantee JSON compliance on values that are fetched. Fetching a non-wrapped value multiple times would cause wrapping to happen each time.

Option 2 makes the wrapping more proactive; guaranteeing that our JSON compliance happens up-front even if the value is not fetched later. Fetching an initially non-wrapped value multiple times would cause NO wrapping to happen at fetch as it would have been pre-wrapped by the put operation.

Both options only affect code-built JSONObjects/Arrays. Parsed objects would be unaffected as the parser is only placing JSON-compliant values in the backing collections.

Thoughts on whether we should implement either option, something else, or neither?

Vorherrschende Sprache: Java
Sterne: 4.7k
Forks: 2.6k
Ø Merge: 6 T. 20 Std.
Gemergte PRs (30 T.): 2

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
Forken Sie das Repository und arbeiten Sie in einem Branch.
Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus stleary/JSON-java

JSONPointer pre-existing bug in toURIFragment() Offen

Fix before the next release

Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100

stleary/JSON-java#1082 · Erstellt am 20.9.2026
New JSONPointer tests needed Offen

Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 76/100

stleary/JSON-java#1081 · Erstellt am 20.9.2026
JSONML checks childNode instead of the reserved childNodes attribute Offen

Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 88/100

stleary/JSON-java#1080 · Erstellt am 19.9.2026
JSONML decodes text entities twice with keepStrings enabled Offen

Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 82/100

stleary/JSON-java#1079 · Erstellt am 19.9.2026
JSONObject.toString does not use an Enum's `toString()` but its `name()` Offen

Fix before the next release

Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100

stleary/JSON-java#838 · 16 Kommentare · 3 Reaktionen · Erstellt am 11.12.2023

Alle Issues in stleary/JSON-java