notifydatasetchanged（）不更新回收站视图

Question

i am trying to update recyclerview with images from a API using ASYNC task but its not working properly . The problem is with the notifyDataSetChanged() method in the onPostExecute() of the ASYNC task. i have checked the json response and the result of the onPostExecute() the value are correct but the notifydatasetchanged() is not updating the adapter with the new data.

MainActivity :

public class MainActivity extends AppCompatActivity {

    public static final String LOG_TAG = MainActivity.class.getName();
    public MovieUtils utils = new MovieUtils();


    private RecyclerView mRecyclerView;
    private MovieAdapter mAdapter;
    private RecyclerView.LayoutManager mLayoutManager;
    public ArrayList<String> movieData=new ArrayList<String>();

    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.activity_main);
        mRecyclerView = (RecyclerView) findViewById(R.id.my_recycler_view);
        mRecyclerView.setHasFixedSize(true);
        //Using a grid layout manager to display the movies in a grid view.
        mLayoutManager = new GridLayoutManager(getApplicationContext(), 2);
        mRecyclerView.setLayoutManager(mLayoutManager);
        //using the jsonParser to parse the String response and create a new data set.
        // passing the data to the Adapter.
        mAdapter = new MovieAdapter(movieData, getApplicationContext());
        mRecyclerView.setAdapter(mAdapter);
        DownloadMovieDetail downloadMovieDetail= new DownloadMovieDetail();
        downloadMovieDetail.execute();

    }

    private class DownloadMovieDetail extends AsyncTask<String, Void, ArrayList<String>> {

        @Override
        protected ArrayList<String> doInBackground(String... params) {
            ArrayList<String> movieDb = new ArrayList<String>();
            try {
                movieDb = utils.downloadMovie(responseString);
            } catch (IOException e) {
                e.printStackTrace();
            }
            return movieDb;
        }

        @Override
        protected void onPostExecute(ArrayList<String> strings) {
            movieData.addAll(strings);
            Log.v(LOG_TAG,"the movie data"+movieData.toString());
            mAdapter.notifyDataSetChanged();
        }
    }
}

Movie Adapter:

public class MovieAdapter extends       RecyclerView.Adapter<MovieAdapter.ViewHolder> {
    public static final String LOG_TAG = MovieAdapter.class.getName();

    public ArrayList<String> dummyData;
    public Context context;

    public static class ViewHolder extends RecyclerView.ViewHolder {
        public ImageView MovieImageView;
        public ViewHolder(View itemView) {
            super(itemView);
            MovieImageView=(ImageView) itemView.findViewById(R.id.movie_image);
        }
    }

    //Constructor for the adapter.
    public MovieAdapter(ArrayList<String> data, Context c){
        this.context=c;
        dummyData=data;
    }


    @Override
    public MovieAdapter.ViewHolder onCreateViewHolder(ViewGroup parent, int viewType) {
        View v = LayoutInflater.from(parent.getContext())
                .inflate(R.layout.movie_view, parent, false);
        return new ViewHolder(v);
    }

    @Override
    public void onBindViewHolder(MovieAdapter.ViewHolder holder, int position) {
        Picasso.with(context).load(dummyData.toString()).into(holder.MovieImageView);

    }

    @Override
    public int getItemCount()
    {
        return dummyData.size();
    }
}

Answer 1

Define a setter for dummyData ArrayList in your adapter like this

public void setDummyData(ArrayList<String> data){
     this.dummyData = data;
     notifyDataSetChanged();
}

and then onPostExecute() replace

mAdapter.notifyDataSetChanged();

with

mAdapter.setDummyData(movieData);

Answer 2

You did't change the data on the adapter class. Create a setter for the ArrayList and then call notifyDataSetChanged()

public void setList(ArrayList<String> dummyData){
   this.dummyData = dummyData;
   notifyDataSetChanged();
}

And on onPostExecute just call

adapter.setList(string);