具有SortedList的RecycylerView显示重复项

时间:2018-07-01 21:14:05

标签: android listview android-recyclerview recycler-adapter sortedlist

我在列表中使用了SortedListRecyclerViewAdapter。因此,我(通过后台线程从后端加载)的项目通过侦听器发送到我的GameListAdapter。根据我在SortedListAdapterCallback中发送给具有相同ID的适配器的项的实现逻辑,应该替换而不是多次插入。不幸的是,这主要是但并非总是如此。

这是我的GameListAdapter

的构造函数
public GameListAdapter(RecyclerView list) {

    this.list = list;

    gamelistItems = new SortedList<>(GameListItem.class, new SortedListAdapterCallback<GameListItem>(this) {

        @Override
        public int compare(GameListItem o1, GameListItem o2) {

            return o1.getDate().compareTo(o2.getDate());

        }

        @Override
        public boolean areContentsTheSame(GameListItem oldItem, GameListItem newItem) {

            if (oldItem instanceof GameListHeader && newItem instanceof GameListHeader) {

                return oldItem.getDate().equals(newItem.getDate());

            } else if (oldItem instanceof Game && newItem instanceof Game) {

                Game gameOld = (Game) oldItem;
                Game gameNew = (Game) newItem;

                if (gameOld.getId() != gameNew.getId()) {
                    return false;
                }
                if (!gameOld.getTeamHome().equals(gameNew.getTeamHome())) {
                    return false;
                }
                if (!gameOld.getTeamAway().equals(gameNew.getTeamAway())) {
                    return false;
                }
                if (gameOld.getScoreHome() != gameNew.getScoreHome()) {
                    return false;
                }
                if (gameOld.getScoreAway() != gameNew.getScoreAway()) {
                    return false;
                }
                if (!gameOld.getState().equals(gameNew.getState())) {
                    return false;
                }
                return true;
            }

            return false;

        }

        @Override
        public boolean areItemsTheSame(GameListItem item1, GameListItem item2) {

            if (item1 instanceof GameListHeader && item2 instanceof GameListHeader) {

                return item1.getDate() == item2.getDate();

            } else if (item1 instanceof Game && item2 instanceof Game) {

                return ((Game) item1).getId() == ((Game) item2).getId();

            }

            return false;
        }

    });

    DataStorage.getInstance().registerListener(this);
}

通过这种方法将项目添加到列表中

@Override
public void onAddedGame(Game game) {

    Handler handler = new Handler();
    handler.post(() -> gamelistItems.add(game));

}

即使这些项目具有相同的ID,我也会在列表中看到它:

https://i.stack.imgur.com/vWYml.png

1 个答案:

答案 0 :(得分:0)

您用两种不同的方式比较GameListHeader.getDate()的响应。在areContentsTheSame()中是

return oldItem.getDate().equals(newItem.getDate());

areItemsTheSame()中使用

return item1.getDate() == item2.getDate();

尽管从您的发文中尚不清楚getDate()的响应是如何产生的,还是数据类型是什么,但我确实认为其中之一是不正确的。如果getDate()返回一个String,那么您就需要String.equals(String),而==是不正确的。