parseJSONObject() and parseJSONArray() do not return null when parsing fails

[processing-bot]

Created by: denezth

Description

Both parseJSONObject() and parseJSONArray() throw a RuntimeException if the provided JSON String is not valid.

Expected Behavior

The documentation states:

If the String does not contain JSONObject data or cannot be parsed, a null value is returned.

Both functions should therefore return null in the event of a parsing error. This would not prevent the stackTrace from being displayed in console so that the user can understand the reason.

Steps to Reproduce

1. Pass an invalid JSON String to either parseJSONObject() or parseJSONArray()

JSONObject object = parseJSONObject("invalidJSONString");

println(object);

2. Run the sketch. A RuntimeException is thrown, stopping execution unless we use a try..catch.

My Environment

• Processing version: 4.0a3
• Operating System and OS version: MacOS 10.14.6

Possible Solution

In PApplet.java, we could catch the RuntimeException inside the parseJSONObject/Array(). Something like this:

public JSONObject parseJSONObject(String input) {
  try {
    return new JSONObject(new StringReader(input));
  } catch (RuntimeException e) {
    e.printStackTrace();
    return null;
  }
}

I will open a pull request.

[processing-bot]

Created by: benfry

Resolved with 53246fb

Thanks again for your help!

[processing-bot]

Created by: github-actions[bot]

This issue has been automatically locked. To avoid confusion with reports that have already been resolved, closed issues are automatically locked 30 days after the last comment. Please open a new issue for related bugs.