Question

当有人搜索某个单词时，我试图替换我的RecyclerView中的项目。 GIF列表被发送到我的回调方法，我循环遍历这些对象，并希望替换RecyclerView中的每个对象。但是，当我尝试这样做时，如果我在我的适配器中调用ArrayList.set()，我会得到一个IndexOutOfBoundsException。

我明白为什么它会发生......我的适配器中的ArrayList中没有任何项目。我的问题是为什么？我做错了什么？

我的活动代码：

package hidden.package;

// Imports hidden

public class GifSearchActivity extends AppCompatActivity {

    @BindView(R.id.gifSearchBar)
    FloatingSearchView searchGifs;

    @BindView(R.id.gifSearchResults)
    RecyclerView searchResults;

    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.activity_gifsearch);
        ButterKnife.bind(this);

        final GIFResultsAdapter adapter = new GIFResultsAdapter();

        final SharedPreferences prefs = PreferenceManager
                .getDefaultSharedPreferences(GifSearchActivity.this);

        final GIFInterface i = new GIFInterface();
        i.setSearchListener(new GIFInterface.GIFSearchListener() {
            @Override
            public void onSearchResultsRetrieved(final GIFSearchResults gifSearchResults) {
                runOnUiThread(new Runnable() {
                    @Override
                    public void run() {
                        Log.d("GAB", "Results Retrieved...");
                        int count = 0;
                        for (GIF gif : gifSearchResults.getGIFs()) {
                            adapter.replaceItem(gif, count);
                            count++;
                        }
                    }
                });
            }
        });

        searchResults.setLayoutManager(new GridLayoutManager(getBaseContext(), 1, GridLayoutManager.VERTICAL, false));
        searchResults.setItemAnimator(new DefaultItemAnimator());
        searchResults.setAdapter(adapter);

        searchGifs.setOnQueryChangeListener(new FloatingSearchView.OnQueryChangeListener() {
            @Override
            public void onSearchTextChanged(String oldQuery, String newQuery) {
                i.searchForGif(prefs.getString("token", ""), newQuery);
            }
        });
    }
}

我的适配器代码：

package hidden.package;

// Imports hidden

@SuppressWarnings("deprecation")
public class GifResultsAdapter extends RecyclerView.Adapter<GifResultsAdapter.GifsViewHolder> {

    private ArrayList<GIF> gifs = new ArrayList<>();
    private Context con;

    public class GifsViewHolder extends RecyclerView.ViewHolder {

        ImageView gif;
        ProgressBar gifProgressBar;

        public GifsViewHolder(View itemView) {
            super(itemView);
            gif = (ImageView) itemView.findViewById(R.id.gif);
            gifProgressBar = (ProgressBar) itemView.findViewById(R.id.gifProgressBar);
        }
    }

    public void replaceItem(GIF gif, int position) {
        gifs.set(position, gif);
        notifyItemChanged(position);
    }

    @Override
    public GifsViewHolder onCreateViewHolder(ViewGroup parent, int viewType) {
        Log.d("GAB", "ViewHolder Created");
        View itemView = LayoutInflater.from(parent.getContext())
                .inflate(R.layout.util_gif_row, parent, false);
        this.con = parent.getContext();
        return new GifsViewHolder(itemView);
    }

    @Override
    public void onBindViewHolder(final GifsViewHolder holder, int position) {
        Log.d("GAB", "ViewHolder bound");
        final GIF gif = gifs.get(position);
        Glide.with(con)
                .load(gif.getUrl())
                .override(700, 700)
                .fitCenter()
                .placeholder(new ColorDrawable(AndroidHelper.darkenColor(con.getResources().getColor(R.color.colorAccent), 0.95f)))
                .into(holder.gif);
    }

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

编辑：这是例外......

02-25 00:41:46.371 29205-29205/ai.gab.android E/UncaughtException: java.lang.IndexOutOfBoundsException: Index: 0, Size: 0
  at java.util.ArrayList.set(ArrayList.java:427)
  at ai.gab.android.ui.adapter.GIFResultsAdapter.replaceItem(GIFResultsAdapter.java:59)
  at ai.gab.android.ui.activity.GifSearchActivity$1$1.run(GifSearchActivity.java:68)
  at android.os.Handler.handleCallback(Handler.java:751)
  at android.os.Handler.dispatchMessage(Handler.java:95)
  at android.os.Looper.loop(Looper.java:154)
  at android.app.ActivityThread.main(ActivityThread.java:6119)
  at java.lang.reflect.Method.invoke(Native Method)
  at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:886)
  at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:776)

非常感谢一双新鲜的眼睛！

Answer 1

你正在调用的方法是replaceItem它用另一个元素替换一个元素，如果你在列表为空时调用它，你应该首先创建一个方法来在适配器中添加Gif < / p>

public void addItem(GIF gif) {
    gifs.add(gif);
    notifyItemChanged(position);
}

而不是第一次获得Gif时调用replaceItem，您应该致电addItem

之后，当您需要替换特定元素时，可以使用replaceItem

Answer 2

我会做一些稍微不同的事情。我会在每个按键时间等待一小段时间，大约50 / 100ms，并且需要新内容加载的afetr（可能使用加载器）。

通过这种方式，您可以防止触发大量可能过时和无用的搜索，然后替换适配器的所有内容，使用新的结果集合。

在下面的例子中，我将使用AsyncTask来快速

在你的GifSearchActivity中为下面定义的AsyncTask添加一个名为gifSearchTask的变量，并用这个替换你的查询更改监听器。

通过这种方式，您将为每个查询更改触发一个事件，取消之前的if正在运行或挂起。使用新查询启动新的查询，最后用新的数据集替换Recycler视图的整个数据集。

searchGifs.setOnQueryChangeListener(new FloatingSearchView.OnQueryChangeListener() {
        @Override
        public void onSearchTextChanged(String oldQuery, String newQuery) {
            if (gifSearchTask != null && AsyncTask.Status.FINISHED != gifSearchTask.getStatus()) {
                gifSearchTask.cancel(true);
            }
            gifSearchTask = new GifSearchAsyncTask();
            gifSearchTask.execute(newQuery);
        }
    });


class GifSearchAsyncTask extends AsyncTask<String, Void, List<Gif>> {

    @Override
    protected List<Gif> doInBackground(String... params) {
        List<Gif> results = null;
        android.os.SystemClock.sleep(100);
        // if not cancelled
        if (!isCancelled()) {
            results = performSearch(params);
        }
        return results;
    }

    @Override
    protected void onCancelled() {
        // do nothing
    }

    @Override
    protected void onCancelled(Result result) {
        // do nothing
    }

    @Override
    protected void onPostExecute(List<Gif> results) {
        adapter.setData(results);
        gifSearchTask = null;
    }

    private List<Gif> performSearch(String ... params) {
        // perform search and retur the results
    }
}

Answer 3

我提出了一个更好的解决方案：创建一个clear()方法，并在搜索查询更改时调用它。

特别感谢this SO answer。现在，每次更改搜索查询时，都会删除当前项目，并添加新项目。

替换RecyclerView项时获取IndexOutOfBoundsException

3 个答案: