Адаптер просмотра Recycler получает старые данные (адаптер), даже когда пейджер просмотра отображает другой фрагмент

У меня есть ViewPager с 3 вкладками, каждая вкладка загружает фрагмент, новый экземпляр того же класса фрагментов, но с дополнительным параметром.

Ниже вы можете найти класс фрагмента, у меня есть RecyclerView в этом фрагменте, я пытаюсь открыть контекстное меню, когда пользователь долго нажимает на элемент в RecyclerView.

Я могу отобразить контекстное меню и получить позицию выбранного элемента для одного экземпляра фрагмента, когда я перехожу к следующему фрагменту, я получаю ту же позицию индекса, когда я отлаживаю дальше, я обнаруживаю, что получаю тот же адаптер для RecyclerView в предыдущем фрагменте. Это можно увидеть, когда я регистрирую размер адаптера для этого RecyclerView.

Может ли кто-нибудь помочь мне понять, почему я получаю данные предыдущего адаптера, когда я перешел к совершенно другому экземпляру этого фрагмента в ViewPager, даже когда я вижу правильные данные во фрагменте и получаю правильные значения в методе onBindViewHolder.

Ниже вы можете найти упрощенный код класса фрагмента.

public class HistoryFragment extends Fragment{
    private static final String LOG_TAG = HistoryFragment.class.getSimpleName();

    private RecyclerView mRecyclerView;
    private HistoryAdapter mHistoryAdapter;
    private List<Group> mGroups;

    private String mGroupType;

    /**
     * tried using a static int value to track which context menu was opened and on which fragment
     * refer post https://caughtinthemobileweb.wordpress.com/2013/10/22/android-viewpager-holds-reference-to-old-fragments/
     * BUG LOGGED AT https://code.google.com/p/android/issues/detail?id=19211
     * does not work :(
     */
    /*private static int selectedContextMenu;*/ // returns valid index as int is static, but adapter still has old data

    /**
     * The fragment argument for this fragment.
     */
    private static final String ARG_GROUP_TYPE = "group_type";

    public HistoryFragment() { }

    /**
     * Returns a new instance of this fragment
     */
    public static HistoryFragment newInstance(String groupType) {
        HistoryFragment fragment = new HistoryFragment();
        Bundle args = new Bundle();
        args.putString(ARG_GROUP_TYPE, groupType);
        fragment.setArguments(args);
        return fragment;
    }

    @Override
    public View onCreateView(LayoutInflater inflater, ViewGroup container, Bundle savedInstanceState) {
        View rootView = inflater.inflate(R.layout.fragment_history, container, false);
        mRecyclerView = (RecyclerView) rootView.findViewById(R.id.recyclerViewHistoryItem);
        RecyclerView.LayoutManager layoutManager = new LinearLayoutManager(getActivity(), LinearLayoutManager.VERTICAL, false);
        mRecyclerView.setLayoutManager(layoutManager);
        mGroups = new ArrayList<>();
        mHistoryAdapter = new HistoryAdapter(getActivity(), mGroups);
        mRecyclerView.setAdapter(mHistoryAdapter);


        Bundle bundle = getArguments();
        mGroupType = bundle.getString(ARG_GROUP_TYPE);
        if(mGroupType != null && !mGroupType.equals("")){
            loadDataFromDb();
        } else {
            if(IS_DEBUG_ON) Log.d(LOG_TAG, "invalid fragment arguments");
            getActivity().finish();
        }
        return rootView;
    }

    @Override
    public boolean onContextItemSelected(MenuItem item){
        // int id = item.getItemId();
        Log.d(LOG_TAG, "position2: " + mHistoryAdapter.getSelectedItemPosition() + 
                " " + mGroups.get(mHistoryAdapter.getSelectedItemPosition()).getName()); // returns old data
        Log.d(LOG_TAG, "size2: " + mHistoryAdapter.getItemCount()); // returns old data
        // Log.d(LOG_TAG, "position3: " + selectedContextMenu); // returns correct index, but cant use as adapter still has old data
        Log.d(LOG_TAG, "position4: " + ((HistoryAdapter) mRecyclerView.getAdapter()).getSelectedItemPosition()); // returns old data
        return true;
    }

    private void loadDataFromDb(){
        new LoadGroups().execute();
    }

    class HistoryAdapter extends RecyclerView.Adapter<HistoryFragment.HistoryAdapter.ViewHolder> {
        private Context mContext;
        private List<Group> mList;
        private int mSelectedItemPosition;

        HistoryAdapter(Context mContext, List<Group> mList) {
            this.mContext = mContext;
            this.mList = mList;
        }

        class ViewHolder extends RecyclerView.ViewHolder implements View.OnCreateContextMenuListener {
            View mView;
            TextView mTextViewName;
            TextView mTextViewDescription;
            ImageView mImageViewProfile;
            ImageView mImageCheck;

            public ViewHolder(View itemView) {
                super(itemView);
                this.mView = itemView;
                this.mTextViewName = (TextView) itemView.findViewById(R.id.textViewGroupName);
                this.mTextViewDescription = (TextView) itemView.findViewById(R.id.textViewGroupDescription);
                this.mImageViewProfile = (ImageView) itemView.findViewById(R.id.groupImage);
                this.mImageCheck = (ImageView) itemView.findViewById(R.id.imageCheck);
                this.mView.setOnCreateContextMenuListener(this); // registering listener
            }

            @Override
            public void onCreateContextMenu(ContextMenu menu, View v, ContextMenu.ContextMenuInfo menuInfo) {
                if(mGroupType.equals(GroupContract.GroupEntry.GROUP_TYPE_HISTORY_GROUP)){
                    menu.add(Menu.NONE, R.id.action_resubscribed_group, Menu.NONE, R.string.action_resubscribed_group);
                } else if(mGroupType.equals(GroupContract.GroupEntry.GROUP_TYPE_SUBSCRIBED_GROUP)){
                    if(!mList.get(getAdapterPosition()).isDefault()){
                        menu.add(Menu.NONE, R.id.action_unsubscribe_group, Menu.NONE, R.string.action_unsubscribe_group);
                    }
                    menu.add(Menu.NONE, R.id.action_mute, Menu.NONE, R.string.action_mute);
                    menu.add(Menu.NONE, R.id.action_view_group_info, Menu.NONE, R.string.action_view_group_info);

                }
            }
        }

        @Override
        public HistoryAdapter.ViewHolder onCreateViewHolder(ViewGroup parent, int viewType) {
            View view = LayoutInflater.from(mContext).inflate(R.layout.recycler_view_history, parent, false);
            return new HistoryAdapter.ViewHolder(view);
        }

        @Override
        public void onBindViewHolder(final HistoryAdapter.ViewHolder holder, final int position) {
            if(mGroupType.equals(GroupContract.GroupEntry.GROUP_TYPE_SUBSCRIBED_GROUP)){
                holder.mImageCheck.setVisibility(View.VISIBLE);
            } else {
                holder.mImageCheck.setVisibility(View.GONE);
            }
            holder.mTextViewName.setText(mList.get(position).getName());
            holder.mTextViewDescription.setText(mList.get(position).getDescription());

            holder.mView.setOnLongClickListener(new View.OnLongClickListener() {
                @Override
                public boolean onLongClick(View v) {
                    // mSelectedItemPosition = position; // tried this, returns correct 
                    // selectedContextMenu = position; // also tried this, returns correct
                    mSelectedItemPosition = holder.getAdapterPosition(); // this also returns correct 
                    Log.d(LOG_TAG, "position1 - onBind: " + mSelectedItemPosition + " "+ mList.get(mSelectedItemPosition).getName()); // returns correct data
                    return false;
                }
            });

        }

        @Override
        public void onViewRecycled(ViewHolder holder) {
            holder.mView.setOnLongClickListener(null); // clearing listener to avoid collision between list items
            super.onViewRecycled(holder);
        }

        @Override
        public int getItemViewType(int position) {
            return 0;
        }

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

        int getSelectedItemPosition() {
            Log.d(LOG_TAG, "getter: " + mSelectedItemPosition); // this returns wrong data, shows that fragment is referring old adapter data
            return mSelectedItemPosition;
        }
    }

    // get data from db
    class LoadGroups extends AsyncTask<Void, Void, Void>{

        @Override
        protected Void doInBackground(Void... voids) {
            mGroups = GroupContract.getGroupEntriesFromDB(getActivity(), mGroupType);
            return null;
        }

        @Override
        protected void onPostExecute(Void aVoid) {
            super.onPostExecute(aVoid);

            mHistoryAdapter = new HistoryAdapter(getActivity(), mGroups);
            mRecyclerView.setAdapter(mHistoryAdapter);
        }
    }
}

Ниже вы можете найти класс адаптера ViewPager.

public class HomeTabsPagerAdapter extends FragmentPagerAdapter {
    private Context mContext;

    HomeTabsPagerAdapter(Context context, FragmentManager fm) {
        super(fm);
        mContext = context;
    }

    @Override
    public Fragment getItem(int position) {
        switch (position) {
            case 0:
                return HistoryFragment.newInstance("TYPE_1");
            case 1:
                return HistoryFragment.newInstance("TYPE_2");
            case 2:
                return HistoryFragment.newInstance("TYPE_3");
        }
        return null;
    }

    @Override
    public int getCount() {
        return 3;
    }

    @Override
    public CharSequence getPageTitle(int position) {
        switch (position) {
            case 0:
                return mContext.getString(R.string.text_type_1);
            case 1:
                return mContext.getString(R.string.text_type_2);
            case 2:
                return mContext.getString(R.string.text_type_3);
        }
        return null;
    }

    // Doesn't work even when I use this or not
    /*@Override
    public int getItemPosition(Object object) {
        // try http://stackoverflow.com/a/8024557
        // return super.getItemPosition(object);
        return POSITION_NONE;
    }*/
}

person John    schedule 12.05.2017    source источник


Ответы (2)


Я подозреваю, что использование RecyclerView.LayoutManager может быть проблемой, потому что у вас есть статический класс, и первые два созданных фрагмента будут иметь одинаковое содержимое (второго). То же самое и для следующих фрагментов. Кроме того, getItemPosition(Object object) -> return POSITION_NONE можно использовать только с FragmentStatePagerAdapter. FragmentAdapter, который у вас есть, никогда не изменяет фрагменты после создания. Итак, я бы предложил вместо:

RecyclerView.LayoutManager layoutManager = new LinearLayoutManager (getActivity(), LinearLayoutManager.VERTICAL, false); mRecyclerView.setLayoutManager(layoutManager);

использовать

mRecyclerView.setLayoutManager (новый LinearLayoutManager (getActivity()));

//или вместо этого установите GridLayoutManager

используйте FragmentStatePagerAdapter, если вы планируете обновлять содержимое фрагментов.

person Helmwag    schedule 31.05.2017

У меня была аналогичная проблема, и я исправил ее, поместив это в свой фрагмент:

@Override
public void onDetach() {
    super.onDetach();
    recyclerView.getRecycledViewPool().clear();
}

В вашем случае вы можете вызвать getRecycledViewPool при смене вкладки.

person Emerson Nascimento    schedule 21.04.2021