Parece que no puedo hacer que mi RecyclerView se complete por completo. Solo crea la primera celda, y las otras están en blanco. Aunque el bucle for parece ejecutarse varias veces.

    JsonObjectRequest root = new JsonObjectRequest(Request.Method.GET, URL, null, new Response.Listener<JSONObject>() {
        @Override
        public void onResponse(JSONObject response) {
            try {
                JSONObject feed = response.getJSONObject("feed");
                JSONArray results = feed.getJSONArray("results");
                //Log.d("FEED", String.valueOf(feed));w
                for(int i = 0; i<results.length(); i++){
                    JSONObject resultsObj = results.getJSONObject(i);

                    Log.d(TAG, "inside" + String.valueOf(i));
                    mImageUrls.add(resultsObj.getString("artworkUrl100"));
                    mTitle.add(resultsObj.getString("name"));
                    getRecyclerView();
                }

                adapter.notifyDataSetChanged();

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

        }
    }, new Response.ErrorListener() {
        @Override
        public void onErrorResponse(VolleyError error) {

        }
    });
    mRequestQueue.add(root);

}

private void getRecyclerView(){
    Log.d(TAG, "getRecyclerView");
    recyclerView = findViewById(R.id.recyclerView);
    adapter = new RecyclerViewAdapter(mImageUrls, mTitle, this);
    recyclerView.setAdapter(adapter);
    recyclerView.setLayoutManager(new LinearLayoutManager(this));
    recyclerView.setHasFixedSize(true);
}
0
Jason Yu 10 sep. 2018 a las 12:51

3 respuestas

La mejor respuesta

Cada vez que inicializa el adaptador con un valor. Es por eso que la lista contiene un valor y otros estaban en blanco. Debe mover getRecyclerView(); fuera del bucle.

Se parece a esto:

try {
            JSONObject feed = response.getJSONObject("feed");
            JSONArray results = feed.getJSONArray("results");
            //Log.d("FEED", String.valueOf(feed));w
            for(int i = 0; i<results.length(); i++){
                JSONObject resultsObj = results.getJSONObject(i);

                Log.d(TAG, "inside" + String.valueOf(i));
                mImageUrls.add(resultsObj.getString("artworkUrl100"));
                mTitle.add(resultsObj.getString("name"));

            }
             getRecyclerView();
           // No need to notify the data set changed 
           // adapter.notifyDataSetChanged();

        } catch (JSONException e) {
            e.printStackTrace();
        }
0
Faysal Ahmed 10 sep. 2018 a las 10:11

Use esto puede ser de ayuda.

private void getRecyclerView(){
    Log.d(TAG, "getRecyclerView");
    recyclerView = findViewById(R.id.recyclerView);
    adapter = new RecyclerViewAdapter(mImageUrls, mTitle, this);
    recyclerView.setAdapter(adapter);
    recyclerView.setLayoutManager(new LinearLayoutManager(this, 
    LinearLayoutManager.VERTICAL, false));
    recyclerView.setHasFixedSize(true);
}
0
Parveen 10 sep. 2018 a las 11:09

Dime si esto funciona:

JsonObjectRequest root = new JsonObjectRequest(Request.Method.GET, URL, null, new Response.Listener<JSONObject>() {
        @Override
        public void onResponse(JSONObject response) {
            try {
                JSONObject feed = response.getJSONObject("feed");
                JSONArray results = feed.getJSONArray("results");
                //Log.d("FEED", String.valueOf(feed));w
                for(int i = 0; i<results.length(); i++){
                    JSONObject resultsObj = results.getJSONObject(i);

                    Log.d(TAG, "inside" + String.valueOf(i));
                    mImageUrls.add(resultsObj.getString("artworkUrl100"));
                    mTitle.add(resultsObj.getString("name"));
                }

                getRecyclerView();
                adapter.notifyDataSetChanged();

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

        }
    }, new Response.ErrorListener() {
        @Override
        public void onErrorResponse(VolleyError error) {

        }
    });
    mRequestQueue.add(root);

}

private void getRecyclerView(){
    Log.d(TAG, "getRecyclerView");
    recyclerView = findViewById(R.id.recyclerView);
    adapter = new RecyclerViewAdapter(mImageUrls, mTitle, this);
    recyclerView.setAdapter(adapter);
    recyclerView.setLayoutManager(new LinearLayoutManager(this));
    recyclerView.setHasFixedSize(true); }

Lo que hice fue mover getRecyclerView(); fuera del ciclo, porque solo necesita llamarlo una vez, no cada vez que desea configurar la vista de reciclador.

0
Jacob Celestine 10 sep. 2018 a las 10:04