JSONArray cannot be converted to JSONObject exception

Question

I have a JSON String structured in the following way and it throws an exception passing it into JSONArray timeJSONArray = new JSONArray(time);

This is the error Value [{"daysByte":158,"from":1020,"to":1260},{"daysByte":96,"from":1020,"to":1320}] at 0 of type org.json.JSONArray cannot be converted to JSONObject This is how I receive the array and I can't change it, so I'm having trouble converting it to a JSON Object instead of a JSON String which is the format it's currently in. What am I doing wrong?

[   
   [  
      {  
         "daysByte":30,
         "from":660,
         "to":1290
      },
      {  
         "daysByte":96,
         "from":660,
         "to":1320
      },
      {  
         "daysByte":128,
         "from":1050,
         "to":1290
      }
   ],
   [  
      {  
         "daysByte":252,
         "from":690,
         "to":840
      },
      {  
         "daysByte":252,
         "from":1050,
         "to":1260
      }
   ]
]

This is the code I am working with. I'm getting the value passed in as a string

public ArrayList<String> getTimeList(String time){

        System.out.println("PLACES ACTIVITY " + time);
        ArrayList<String> times = new ArrayList<>();
        try{
            //JSONObject timeJSONObject = new JSONObject(time);
            JSONArray timeJSONArray = new JSONArray(time);
            ArrayList<LegacyTimeSpan> timeSpanList = new ArrayList<>();

            LegacyTimeSpanConverterImpl converter = new LegacyTimeSpanConverterImpl();


            for(int i = 0; i < timeJSONArray.length(); i++){

                int daysByte = timeJSONArray.getJSONObject(i).getInt("daysByte");
                int from = timeJSONArray.getJSONObject(i).getInt("from");
                int to = timeJSONArray.getJSONObject(i).getInt("to");

                System.out.println("TO " + to);

                LegacyTimeSpan timeSpan = new LegacyTimeSpan(daysByte, from, to);
                timeSpanList.add(timeSpan);

            }

            Log.d("Time span list", timeSpanList.toString());
            WeekSpan weekSpan = converter.convertToWeekSpan(timeSpanList);
            List<DayTimeSpanPair> dayTimeSpanPair = weekSpan.toDayTimeSpanPairs();
            for(int i = 0; i< dayTimeSpanPair.size(); i++){

                String timeRange = buildTimeString(dayTimeSpanPair.get(i));
                times.add(timeRange);


            }
        } catch(JSONException e){
            Log.d("PLACES EXCEPTION JSON",e.getMessage());
        }

        return times;
    }

Don't just show the JSON, show the code as well. Always show the code. — Doug Stevenson
– Doug Stevenson, Commented Feb 26, 2016 at 6:28
See my answer to the above problem , basically there two arrays one within other , hence your have to consider the arrays, Currently you are trying to put first array to an JSONObject which is not correct — Ashish Ani
– Ashish Ani, Commented Feb 26, 2016 at 6:32
timeJSONArray.getJSONObject(i)? You don't have a JsonObject at i, you have an array — OneCricketeer
– OneCricketeer, Commented Feb 26, 2016 at 6:42

Vikrant Kashyap · Accepted Answer · 2016-02-26 06:47:24Z

1

This Code should work i think as u declare the json Format.

             [
                [
                  {
                  } ,{},{}             // Json Object Structure as u defined in you Question
topArray =      ],
                [  
                  {
                  },{},{}
                ]
             ]


for(JSONArray objArray : topArray){
    for(JSONObject eachObject : objArray){
   System.out.println(eachObject.get("daysByte"););
   System.out.println(eachObject.get("from");
   System.out.println(eachObject.get("to");
    }
}

answered Feb 26, 2016 at 6:47

Vikrant Kashyap

6,9243 gold badges34 silver badges53 bronze badges

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

Comments

Vishal Mokal · Accepted Answer · 2016-02-26 07:56:20Z

1

Hi following code is working for your json I have tried. It is specific for your json not generic. so if you want you can use it.

try{
    JSONArray jsonArray = new JSONArray(data); //insted "Data" pass your json Strint
       for(int i=0 ; i<jsonArray.length() ; i++){
           JSONArray internalArray = jsonArray.getJSONArray(i);
           for(int j = 0 ; j < internalArray.length() ; j++){
               JSONObject internalObject = internalArray.getJSONObject(j);
               Log.d("data" ,  internalObject.getString("daysByte"));
               Log.d("data" ,  internalObject.getString("from"));
               Log.d("data" ,  internalObject.getString("to"));
           }
       }

   }catch(Exception e){
       Log.d("data" ,"Error");
   }
}

edited Feb 26, 2016 at 7:56

answered Feb 26, 2016 at 6:47

Vishal Mokal

8001 gold badge5 silver badges22 bronze badges

5 Comments

Vishal Mokal Over a year ago

If you find it useful then please mark it as answer

Rafa Over a year ago

It won't get past the first line. That is where it throws the exception so no code executes after it.

Vishal Mokal Over a year ago

I didn't get your comment will you please explain your problem again

Rafa Over a year ago

The Exception gets thrown at the line JSONArray jsonArray = new JSONArray(data) so nothing below it will execute. It will just jump straight to the "catch"

Vishal Mokal Over a year ago

I have tested using same json given by you it is working there might be chances you are getting different json

Ashish Ani · Accepted Answer · 2016-02-26 06:30:09Z

0

You have two arrays, one array within other.

You have to do like this:

for(JSONArray temp: timeJsonArray)
{
   // try to convert to json object
}

answered Feb 26, 2016 at 6:30

Ashish Ani

3321 silver badge7 bronze badges

2 Comments

Rafa Over a year ago

If the error is thrown at this line JSONArray timeJSONArray = new JSONArray(time); then how will I be able to make a for loop after it?

Ashish Ani Over a year ago

stackoverflow.com/a/2414332/5188230 see this, its a two dimensional JSON array

Anil · Accepted Answer · 2016-02-26 07:00:46Z

It is a 2D Array.

System.out.println("days");
String content = new Scanner(new File("C:/day.txt")).useDelimiter("\\Z").next();
Day[][] customDayWrap = new Gson().fromJson(content, Day[][].class);
    for (Day[] days : customDayWrap) {
        for (Day day : days) {
            System.out.println(day.getDaysByte());
            System.out.println(day.getFrom());
            System.out.println(day.getTo());
        }
    }

And your Day Class will be something like this.

public class Day {

@SerializedName("daysByte")
@Expose
private Integer daysByte;
@SerializedName("from")
@Expose
private Integer from;
@SerializedName("to")
@Expose
private Integer to;

/**
 * 
 * @return
 *     The daysByte
 */
public Integer getDaysByte() {
    return daysByte;
}

/**
 * 
 * @param daysByte
 *     The daysByte
 */
public void setDaysByte(Integer daysByte) {
    this.daysByte = daysByte;
}

/**
 * 
 * @return
 *     The from
 */
public Integer getFrom() {
    return from;
}

/**
 * 
 * @param from
 *     The from
 */
public void setFrom(Integer from) {
    this.from = from;
}

/**
 * 
 * @return
 *     The to
 */
public Integer getTo() {
    return to;
}

/**
 * 
 * @param to
 *     The to
 */
public void setTo(Integer to) {
    this.to = to;
}

}

I tested this (I am using Google GSON library), and I was able to successfully read it.

Ameen Maheen · Accepted Answer · 2016-02-26 07:03:36Z

0

Basically, there are two JSON arrays but you are accessing only one arrays that is why that error is shown

   try {
        JSONArray jsonArray = new JSONArray(a);

        for (int j=0;j<jsonArray.length();j++) {

            JSONArray timeJSONArray = jsonArray.getJSONArray(j);

            for(int i = 0; i < timeJSONArray.length(); i++){

                int daysByte = timeJSONArray.getJSONObject(i).getInt("daysByte");
                int from = timeJSONArray.getJSONObject(i).getInt("from");
                int to = timeJSONArray.getJSONObject(i).getInt("to");

                System.out.println("TO " + to);


            }
        }
    } catch (JSONException e) {
        e.printStackTrace();
    }

edited Feb 26, 2016 at 7:03

answered Feb 26, 2016 at 6:37

Ameen Maheen

2,7571 gold badge31 silver badges30 bronze badges

4 Comments

Chandan kushwaha Over a year ago

It is a single array only. Please check it.

Ashish Ani Over a year ago

@chandankumarkushwaha its a two array see this : stackoverflow.com/a/2414332/5188230

Chandan kushwaha Over a year ago

But in overall it is added in a single array. Please check

Ameen Maheen Over a year ago

hey man goto this link and paste your json there understand the structure first then you will be able to solve it json.parser.online.fr

Shree Krishna · Accepted Answer · 2016-02-26 07:20:46Z

0

After 1 hour of debugging your Json array I finally managed to figure out your actual issue. Its not only a Json Array its array inside the array.

So loop like this,

 for (int i = 0; i < timeJSONArray.length(); i++) {

            for(int j= 0;j<i;j++) {
                int daysByte = timeJSONArray.getJSONArray(i).getJSONObject(j).getInt("daysByte");
                int from = timeJSONArray.getJSONArray(i).getJSONObject(j).getInt("from");
                int to = timeJSONArray.getJSONArray(i).getJSONObject(j).getInt("to");
                Log.d("dataRecieved", "daybyte " + daysByte + "from " + from + "to " + to);
            }
}

And do others as you need.

answered Feb 26, 2016 at 7:20

Shree Krishna

8,5626 gold badges44 silver badges70 bronze badges

Collectives™ on Stack Overflow

JSONArray cannot be converted to JSONObject exception

6 Answers 6

Comments

5 Comments

2 Comments

Comments

4 Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

6 Answers 6

Comments

5 Comments

2 Comments

Comments

4 Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related