parsing Multiple JSON Url

Question

I am trying to parse multiple JSON URl in my app and show them in one list view but only what i can do is showing one URl and i want to show Multiple URl,s in one list view and the URl have the same array names and attributes and here is my code

  @Override
public void onStart() {
    super.onStart();
    if(connectiondetector.isConnected()){
        new jsontask().execute(

                "https://newsapi.org/v1/articles?source=google-news&sortBy=top&apiKey=ade8f00a634b4825a028837ec107afae"

        );

    }else {
        Toast.makeText(page.this, "no internet connection",Toast.LENGTH_SHORT).show();

    }


    // ATTENTION: This was auto-generated to implement the App Indexing API.
    // See https://g.co/AppIndexing/AndroidStudio for more information.

}

@Override
public void onStop() {
    super.onStop();


}



public class jsontask extends AsyncTask<String, String, List<moviemodel>> {


    @Override
    protected List<moviemodel> doInBackground(String... params) {
        BufferedReader reader = null;
        HttpURLConnection connection = null;
        try {
            URL url = new URL(params[0]);
            connection = (HttpURLConnection) url.openConnection();
            connection.connect();
            InputStream stream = connection.getInputStream();
            reader = new BufferedReader(new InputStreamReader(stream));
            StringBuffer buffer = new StringBuffer();
            String line = "";
            while ((line = reader.readLine()) != null) {
                buffer.append(line);

            }


            String finaljson = buffer.toString();
            JSONObject parentobject = new JSONObject(finaljson);

           // JSONObject parentobject1 = parentobject.getJSONObject("");
            JSONArray parentarray = parentobject.getJSONArray("articles");
            List<moviemodel>moviemodelList =  new ArrayList<>();
            for (int i = 0; i < parentarray.length(); i++){
                JSONObject finalobject = parentarray.getJSONObject(i);
                moviemodel moviemodel = new moviemodel();
                moviemodel.setAuthor(finalobject.getString("author"));
                moviemodel.setDescription(finalobject.getString("description"));
                moviemodel.setTitle(finalobject.getString("title"));
                moviemodel.setImage(finalobject.getString("urlToImage"));
                moviemodel.setUrl(finalobject.getString("url"));
                moviemodel.setPublishedAt(finalobject.getString("publishedAt"));



                moviemodelList.add(moviemodel);


            }

            return moviemodelList;

        } catch (MalformedURLException e) {
            e.printStackTrace();
        } catch (IOException e) {
            e.printStackTrace();
        } catch (JSONException e) {
            e.printStackTrace();
        } finally {
            if (connection != null) {
                connection.disconnect();
            }
            try {
                if (reader != null) {
                    reader.close();
                }
            } catch (IOException e) {
                e.printStackTrace();
            }

        }


        return null;
    }



    @Override
    protected void onPostExecute(List<moviemodel> result) {


        super.onPostExecute(result);
        newsadapter adapter = new newsadapter(getApplicationContext(),R.layout.row, result);
        lvnews.setAdapter(adapter);
        lvnews.setOnScrollListener(new AbsListView.OnScrollListener() {
            @Override
            public void onScrollStateChanged(AbsListView view, int scrollState) {

            }

            @Override
            public void onScroll(AbsListView view, int firstVisibleItem, int visibleItemCount, int totalItemCount) {
                if(firstVisibleItem+visibleItemCount == totalItemCount && totalItemCount!=0)
                {
                    if(flag_loading == false)
                    {
                        flag_loading = true;
                        new jsontask().execute("https://newsapi.org/v1/articles?source=ars-technica&sortBy=top&apiKey=ade8f00a634b4825a028837ec107afae")

                        ;}
                }
            }
        });
    }

}

Sarbjyot · Accepted Answer · 2016-09-30 15:55:35Z

1

you just Declare your List<moviemodel>moviemodelList = new ArrayList<>(); out of the asynctask and used the same list in both asynctask.If you use declare the list inside asynctask it re-initialized and the previous data which store from first asynctask clear from list.

Just declare the list where you declare all variable above the create method.

answered Sep 30, 2016 at 15:55

Sarbjyot

1362 silver badges8 bronze badges

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

Collectives™ on Stack Overflow

parsing Multiple JSON Url

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