Iterating the JSON object using Java not working exactly

Question

I'm trying to get one JSON object and iterate it. In this case it should iterate only once because I'm having only one complete JSON object, if it has multiple, it should iterate multiple times. But rather it shows the system.out.println string twice. I don't understand whats wrong in this code?

String str = "{\"userId\":\"1234\",\"businessPrimaryInfo\":{\"53bd2a4\":{\"businessSpecificInfo\":{\"businessType\":\"Manufacturing\",\"tradingPlace\":\"Hyderabad\"}},\"53bd2a4e\":{\"businessSpecificInfo\":{\"businessType\":\"Milling\",\"tradingPlace\":\"Mumbai\"}}}}";
JSONObject jsonObj = new JSONObject(str);
Iterator<String> keys = jsonObj.keys();
// Iterate all the users.
while (keys.hasNext()) {
    String key = keys.next();
    try {
    // If the user has Business primary info as JSON object.
    if (jsonObj.has("businessPrimaryInfo")) {
        JSONObject jsonBizPrimaryInfo = jsonObj
            .getJSONObject("businessPrimaryInfo");
        System.out.println("Object is:" + jsonBizPrimaryInfo);

    }
    } finally {

    }

}

Please have a look and let me know where I'm doing wrong.

As you don't use key I assume keys has size 2 and the if-check directly accesses the root object without any iteration logic present. — Smutje
– Smutje, Commented Jul 21, 2014 at 6:41

Jason · Accepted Answer · 2014-07-21 06:44:46Z

2

The two keys that are causing your loop to iterate twice are: userId and businessPrimaryInfo. But you ask the main JSON object if it has a businessPrimaryInfo key. Which it has for every iteration.

What you need to do is check that the current key is businessPrimaryInfo.

Or, you can just ask the primary object if it has a businessPrimaryInfo key, and if so, ask for that child object.

answered Jul 21, 2014 at 6:44

Jason

11.8k3 gold badges45 silver badges48 bronze badges

Sign up to request clarification or add additional context in comments.

Collectives™ on Stack Overflow

Iterating the JSON object using Java not working exactly

1 Answer 1

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

Comments

Your Answer

Sign up or log in

Post as a guest

Related