Question

As the title says really. I have two columns. I want to put them into textviews so I did it. However only the bottom two results, one from each column gets shown. Very odd. Here is my code: http://pastebin.com/qNgfHfT3

The parsing/onPostExecute is towards the bottom where the issue is.

One thing to note: The logs labeled “work” & “dontwork” show all my results, however the logs in the onPostExecute (Google & Google1) only show the last result so I presume the error is in the transfer from parsing to displaying.

Would really appreciate any help here. Thanks.

Answer 1

If you are receiving a JSON response I’d suggest you to parse it by using Gson. It’s strongly recommendable as long as you can parse the whole thing in a pair of lines.
Note that creating a proper object it is as easy as doing the following:

YourObject object = gson.fromJson(responseReader, YourObject.class);

or even if you are retrieving a list of items:

Type listType = new TypeToken<List<YourObject>>() {}.getType();
List<YourObject> objects = gson.fromJson(responseReader, listType);

Here’s an example that fits exactly your needs

After the process is done you’ll have your object (or list of objects) available in an accesible variable.

EDIT:
First your Asynctask should have the following params:

public class HttpTask extends AsyncTask<Void, Void, ArrayList<Driver>> {

and your doInBackground method will need to pass that array to your onPostExecute:

@Override
protected ArrayList<Driver> doInBackground(Void... params) {

For the rest, I take it when the JSon parsing starts.

//PARSING JSON DATA
try {
    JSONObject json_data;
    Driver d;

    jArray = new JSONArray(result);
    int l = jArray.length();

    if(l>0){
        ArrayList<Driver> drivers = newArrayListList<Driver>();

        for (int i = 0; i < l; i++) {

            json_data = jArray.getJSONObject(i);

            d = new Driver(json_data.optString("Driver_full_name"), json_data.optString("Drives_for"));
            drivers.add(d);

            Log.i("work", returnString);
            Log.i("dontwork", somethingelse);

        }

   } catch (JSONException e1) {
       Log.d("DB", "Error somewhere");
       CurrentSeasonDrivers_DriverName.this.runOnUiThread(new Runnable() {
           public void run() {
               Toast.makeText(CurrentSeasonDrivers_DriversName, "Could not parse data so shut up", Toast.LENGTH_LONG).show();
           }
       });
   }
   return drivers;
}

protected void onPostExecute(ArrayList<Drivers>... drivers) {

   Log.i("Google", returnString);
   Log.i("Google1", somethingelse);

   String firstDriverName = drivers.get(0).name;
   String firstDriverDrivesFor = drivers.get(0).drivesfor;

   String secondDriverName = drivers.get(1).name;

   TextView drivername = (TextView) findViewById(R.id.DriverName);
   drivername.setText(firstDriverName);

   TextView drivesfor = (TextView) findViewById(R.id.DrivesFor);
   drivesfor.setText(firstDriverDrivesFor);

}

With this and an object for your driver will complete the circle.

public class Driver{

    public String name;
    public String drivesfor;

    public Driver(String _name, String _drivesfor){
        name = _name;
        drivesfor = _drivesfor;
    }

}

I guess you can take over from here.
Let me know about your progress.