Подтвердить что ты не робот

FragmentPagerAdapter не удаляет элементы (фрагменты) правильно

Я реализовал FragmentPagerAdapter и с помощью List<Fragment> для хранения всех фрагментов для моего ViewPager для отображения. В addItem() я просто добавляю экземпляр Fragment, а затем вызываю notifyDataSetChanged(). Я не уверен, что это необходимо или нет.

Моя проблема просто... начать с фрагмента 1

[Fragment 1] 

добавить новый фрагмент 2

[Fragment 1] [Fragment 2]

удалить фрагмент 2

[Fragment 1]

добавить новый фрагмент 3

[Fragment 1] [Fragment 2]

При добавлении новых фрагментов все кажется отличным. Как только я удаляю фрагмент, а затем добавляю вновь созданный фрагмент, старый фрагмент все еще отображается. Когда я иду a .getClass.getName(), он дает мне имя Фрагмента 3, но я все еще вижу Фрагмент 2.

Я считаю, что это может быть проблемой с instantiateItem() или такой, но я думал, что адаптер должен был обработать это для нас. Любые предложения были бы замечательными.

код адаптера...

public class MyPagerAdapter extends FragmentPagerAdapter {
public final ArrayList<Fragment> screens2 = new ArrayList<Fragment>();


private Context context;

public MyPagerAdapter(FragmentManager fm, Context context) {
    super(fm);
    this.context = context;
}

public void removeType(String name){
    for(Fragment f: screens2){
        if(f.getClass().getName().contains(name)){ screens2.remove(f); return; }
    }
    this.notifyDataSetChanged();
}

public boolean addSt(String tag, Class<?> clss, Bundle args){
    if(clss==null) return false;
    if(!clss.getName().contains("St")) return false; 
    if(!args.containsKey("cId")) return false;
    boolean has = false;
    boolean hasAlready = false;
    for(Fragment tab: screens2){
        if(tab.getClass().getName().contains("St")){
            has = true;
            if(tab.getArguments().containsKey("cId"))
                if(tab.getArguments().getLong("cId") == args.getLong("cId")){
                    hasAlready = true;
                }
            if(!hasAlready){
                // exists but is different so replace
                screens2.remove(tab);
                this.addScreen(tag, clss, args, C.PAGE_ST);
                // if returned true then it notifies dataset
                return true;
            }
        }
        hasAlready = false;
    }

    if(!has){ 
        // no st yet exist in adapter
        this.addScreen(tag, clss, args, C.PAGE_ST);
        return true;
    }

    return false;
}

public boolean removeCCreate(){
    this.removeType("Create");  
    return false;
}

@Override
public int getItemPosition(Object object) {

   return POSITION_NONE; //To make notifyDataSetChanged() do something
  }

public void addCCreate(){
    this.removeCCreate();
    Log.w("addding c", " ");
    this.addScreen("Create C",  CreateCFragment.class, null, C.PAGE_CREATE_C);
}

public void addScreen(String tag, Class<?> clss, Bundle args, int type){
    if(clss!=null){
        screens2.add(Fragment.instantiate(context, clss.getName(), args));
    }
}

@Override
public int getCount() {
    return screens2.size();
}


@Override
public Fragment getItem(int position) {
    return screens2.get(position); 
}

}

Я понимаю, что в коде используется некоторое "гетто" для определения типа фрагмента, но я написал этот код строго для тестирования функциональности. Любая помощь или идеи были бы замечательными, поскольку кажется, что не многие люди отважились в мир FragmentPagerAdapter s.

4b9b3361

Ответ 1

У меня такая же проблема, и мое решение перегрузило метод "destroyItem" следующим образом.

@Override
public void destroyItem(ViewGroup container, int position, Object object) {
    FragmentManager manager = ((Fragment)object).getFragmentManager();
    FragmentTransaction trans = manager.beginTransaction();
    trans.remove((Fragment)object);
    trans.commit();
}

Это работает для меня, есть ли у кого-нибудь другие решения?

Обновлено:

Я обнаружил, что этот код не был удален, и я добавил условие, чтобы избежать его.

@Override
public void destroyItem(ViewGroup container, int position, Object object) {
    if (position >= getCount()) {
        FragmentManager manager = ((Fragment) object).getFragmentManager();
        FragmentTransaction trans = manager.beginTransaction();
        trans.remove((Fragment) object);
        trans.commit();
    }
}

Ответ 2

Обновил этот пост и включил мое решение (если кто-то может улучшить, сообщите мне)

Хорошо, теперь я решил свою проблему хакерским способом, но да, это работает;). Если кто-то может улучшить мое решение, пожалуйста, дайте мне знать. Для моего нового решения я теперь использую CustomFragmentStatePagerAdapter, но он не сохраняет состояние, как должно, и сохраняет все фрагменты в списке. Это может вызвать проблему с памятью, если пользователь имеет более 50 фрагментов, например, обычный FragmentPagerAdapter. Было бы здорово, если бы кто-то мог добавить состояние к моему решению, не удаляя мои исправления. Спасибо.

Итак, здесь CustomFragmentStatePagerAdapter.java

package com.tundem.webLab.Adapter;

import java.util.ArrayList;

import android.os.Bundle;
import android.os.Parcelable;
import android.support.v4.app.Fragment;
import android.support.v4.app.FragmentManager;
import android.support.v4.app.FragmentTransaction;
import android.support.v4.view.PagerAdapter;
import android.util.Log;
import android.view.View;
import android.view.ViewGroup;

public abstract class CustomFragmentStatePagerAdapter extends PagerAdapter {
    private static final String TAG = "FragmentStatePagerAdapter";
    private static final boolean DEBUG = false;

    private final FragmentManager mFragmentManager;
    private FragmentTransaction mCurTransaction = null;

    public ArrayList<Fragment.SavedState> mSavedState = new ArrayList<Fragment.SavedState>();
    public ArrayList<Fragment> mFragments = new ArrayList<Fragment>();
    private Fragment mCurrentPrimaryItem = null;

    public CustomFragmentStatePagerAdapter(FragmentManager fm) {
        mFragmentManager = fm;
    }

    /**
     * Return the Fragment associated with a specified position.
     */
    public abstract Fragment getItem(int position);

    @Override
    public void startUpdate(ViewGroup container) {}

    @Override
    public Object instantiateItem(ViewGroup container, int position) {
        // If we already have this item instantiated, there is nothing
        // to do. This can happen when we are restoring the entire pager
        // from its saved state, where the fragment manager has already
        // taken care of restoring the fragments we previously had instantiated.

        // DONE Remove of the add process of the old stuff
        /* if (mFragments.size() > position) { Fragment f = mFragments.get(position); if (f != null) { return f; } } */

        if (mCurTransaction == null) {
            mCurTransaction = mFragmentManager.beginTransaction();
        }

        Fragment fragment = getItem(position);
        if (DEBUG)
            Log.v(TAG, "Adding item #" + position + ": f=" + fragment);
        if (mSavedState.size() > position) {
            Fragment.SavedState fss = mSavedState.get(position);
            if (fss != null) {
                try // DONE: Try Catch
                {
                    fragment.setInitialSavedState(fss);
                } catch (Exception ex) {
                    // Schon aktiv (kA was das heißt xD)
                }
            }
        }
        while (mFragments.size() <= position) {
            mFragments.add(null);
        }
        fragment.setMenuVisibility(false);
        mFragments.set(position, fragment);
        mCurTransaction.add(container.getId(), fragment);

        return fragment;
    }

    @Override
    public void destroyItem(ViewGroup container, int position, Object object) {
        Fragment fragment = (Fragment) object;

        if (mCurTransaction == null) {
            mCurTransaction = mFragmentManager.beginTransaction();
        }
        mCurTransaction.remove(fragment);

        /*if (mCurTransaction == null) { mCurTransaction = mFragmentManager.beginTransaction(); } if (DEBUG) Log.v(TAG, "Removing item #" + position + ": f=" + object + " v=" + ((Fragment)
         * object).getView()); while (mSavedState.size() <= position) { mSavedState.add(null); } mSavedState.set(position, mFragmentManager.saveFragmentInstanceState(fragment));
         * mFragments.set(position, null); mCurTransaction.remove(fragment); */
    }

    @Override
    public void setPrimaryItem(ViewGroup container, int position, Object object) {
        Fragment fragment = (Fragment) object;
        if (fragment != mCurrentPrimaryItem) {
            if (mCurrentPrimaryItem != null) {
                mCurrentPrimaryItem.setMenuVisibility(false);
            }
            if (fragment != null) {
                fragment.setMenuVisibility(true);
            }
            mCurrentPrimaryItem = fragment;
        }
    }

    @Override
    public void finishUpdate(ViewGroup container) {
        if (mCurTransaction != null) {
            mCurTransaction.commitAllowingStateLoss();
            mCurTransaction = null;
            mFragmentManager.executePendingTransactions();
        }
    }

    @Override
    public boolean isViewFromObject(View view, Object object) {
        return ((Fragment) object).getView() == view;
    }

    @Override
    public Parcelable saveState() {
        Bundle state = null;
        if (mSavedState.size() > 0) {
            state = new Bundle();
            Fragment.SavedState[] fss = new Fragment.SavedState[mSavedState.size()];
            mSavedState.toArray(fss);
            state.putParcelableArray("states", fss);
        }
        for (int i = 0; i < mFragments.size(); i++) {
            Fragment f = mFragments.get(i);
            if (f != null) {
                if (state == null) {
                    state = new Bundle();
                }
                String key = "f" + i;
                mFragmentManager.putFragment(state, key, f);
            }
        }
        return state;
    }

    @Override
    public void restoreState(Parcelable state, ClassLoader loader) {
        if (state != null) {
            Bundle bundle = (Bundle) state;
            bundle.setClassLoader(loader);
            Parcelable[] fss = bundle.getParcelableArray("states");
            mSavedState.clear();
            mFragments.clear();
            if (fss != null) {
                for (int i = 0; i < fss.length; i++) {
                    mSavedState.add((Fragment.SavedState) fss[i]);
                }
            }
            Iterable<String> keys = bundle.keySet();
            for (String key : keys) {
                if (key.startsWith("f")) {
                    int index = Integer.parseInt(key.substring(1));
                    Fragment f = mFragmentManager.getFragment(bundle, key);
                    if (f != null) {
                        while (mFragments.size() <= index) {
                            mFragments.add(null);
                        }
                        f.setMenuVisibility(false);
                        mFragments.set(index, f);
                    } else {
                        Log.w(TAG, "Bad fragment at key " + key);
                    }
                }
            }
        }
    }
}

Здесь мой обычный FragmentAdapter.java

package com.tundem.webLab.Adapter;

import java.util.LinkedList;
import java.util.List;

import android.support.v4.app.FragmentManager;

import com.tundem.webLab.fragments.BaseFragment;
import com.viewpagerindicator.TitleProvider;

public class FragmentAdapter extends CustomFragmentStatePagerAdapter implements TitleProvider {
    public List<BaseFragment> fragments = new LinkedList<BaseFragment>();

    private int actPage;

    public FragmentAdapter(FragmentManager fm) {
        super(fm);
    }

    public void setActPage(int actPage) {
        this.actPage = actPage;
    }

    public void addItem(BaseFragment fragment) {
        // TODO if exists don't open / change to that tab
        fragments.add(fragment);
    }

    public BaseFragment getActFragment() {
        return getItem(getActPage());
    }

    public int getActPage() {
        return actPage;
    }

    @Override
    public BaseFragment getItem(int position) {
        if (position < getCount()) {
            return fragments.get(position);
        } else
            return null;
    }

    @Override
    public int getCount() {
        return fragments.size();
    }

    @Override
    public String getTitle(int position) {
        return fragments.get(position).getTitle();
    }

    @Override
    public int getItemPosition(Object object) {
        return POSITION_NONE;
    }
}

И так я удаляю фрагмент. (Я знаю это немного больше, чем только .remove()). Быть свободным улучшить свое решение, вы также можете добавить этот код где-нибудь в адаптер, так что да. Это зависит от пользователя, который пытается это реализовать. Я использую это в своем TabHelper.java (класс, который обрабатывает все операции табуляции, такие как delete, add,...)

    int act = Cfg.mPager.getCurrentItem();
    Cfg.mPager.removeAllViews();
    Cfg.mAdapter.mFragments.remove(act);
    try {
        Cfg.mAdapter.mSavedState.remove(act);
    } catch (Exception ex) {/* Already removed */}
    try {
        Cfg.mAdapter.fragments.remove(act);
    } catch (Exception ex) {/* Already removed */}

    Cfg.mAdapter.notifyDataSetChanged();
    Cfg.mIndicator.notifyDataSetChanged();

Описание Cfg. вещь. Я сохраняю ссылку на эти объекты в классе cfg, поэтому я всегда могу использовать их без необходимости специального Factory.java...

Да. Надеюсь, я смог помочь. Не стесняйтесь улучшать это, но дайте мне знать, чтобы я мог улучшить свой код.

Спасибо.

Если я пропустил какой-либо код, дайте мне знать.


Мой старый ответ также работает, но только если у вас разные фрагменты. FileFragment, WebFragment,... Нет, если вы дважды используете один из этих фрагментов.

Я получил псевдо-работу сейчас. Это действительно грязное решение, и я все еще ищу лучшего. Пожалуйста помоги.

Я изменил код, где я удаляю вкладку:

   public static void deleteActTab()
        {   
            //We set this on the indicator, NOT the pager
            int act = Cfg.mPager.getCurrentItem();
            Cfg.mAdapter.removeItem(act);
            List<BaseFragment> frags = new LinkedList<BaseFragment>();
            frags = Cfg.mAdapter.fragments;

            Cfg.mPager = (ViewPager)Cfg.act.findViewById(R.id.pager);
            Cfg.mPager.setAdapter(Cfg.mAdapter);
            Cfg.mIndicator.setViewPager(Cfg.mPager);

            Cfg.mAdapter.fragments = frags;

            if(act > 0)
            {
                Cfg.mPager.setCurrentItem(act-1);
                Cfg.mIndicator.setCurrentItem(act-1);
            }

            Cfg.mIndicator.notifyDataSetChanged();
        }

Если кто-то может улучшить этот код, дайте мне знать. Если кто-то может сказать нам реальный ответ на эту проблему. пожалуйста, добавьте его здесь. Многие люди сталкиваются с этой проблемой. Я добавил репутацию 50 для тех, кто ее решает. Я также могу дать пожертвование тому, кто его решает.

Спасибо

Ответ 3

Взяв "лучшее из обоих миров" (я имею в виду ответы @Tericky Shih и @mikepenz), мы делаем это коротко и просто:

public class MyPagerAdapter extends FragmentPagerAdapter {

    public ArrayList<Fragment> fragments = new ArrayList<Fragment>();    

    ...

    @Override
    public void destroyItem(ViewGroup container, int position, Object object) {
        super.destroyItem(container, position, object);
        if (position >= getCount()) fm.beginTransaction().remove((Fragment) object).commit();
    }

    @Override
    public int getItemPosition(Object object) {
        if (fragments.contains(object)) return fragments.indexOf(object);
        else return POSITION_NONE;
    }
}

Основное отличие состоит в том, что если какой-либо фрагмент не изменяется, вам не нужно уничтожать его представление и не нужно возвращать POSITION_NONE для него. В то же время я столкнулся с ситуацией, когда ViewPager держал ссылку на элемент, который был уже уничтожен, поэтому проверка if (fragments.contains(object)) помогает определить, нужен ли этот элемент больше.

Ответ 4

Возможно, это поможет answer.

Используйте FragmentStatePagerAdapter вместо FragmentPagerAdapter.

Потому что FragmentPagerAdapter не разрушает представления. Для получения дополнительной информации прочтите answer.

Ответ 5

У меня была ситуация, похожая на вашу. Недавно мне нужно было добавить и удалить фрагменты из ViewPager. В первом режиме у меня есть Фрагменты 0, 1 и 2, а во втором режиме у меня есть Фрагменты 0 и 3. Я хочу, чтобы Фрагмент 0 был одинаковым для обоих режимов и сохранял информацию.

Все, что мне нужно было сделать, это переопределить FragmentPagerAdapter.getItemId, чтобы убедиться, что я вернул уникальный номер для каждого другого фрагмента - по умолчанию возвращается "позиция". Мне также пришлось снова установить адаптер в ViewPager - новый экземпляр будет работать, но я верну его обратно к тому же экземпляру. Настройка адаптера приводит к тому, что ViewPager удаляет все представления и пытается их добавить снова.

Однако, трюк заключается в том, что адаптер только вызывает getItem, когда он хочет создать экземпляр фрагмента - не каждый раз, когда он показывает это. Это происходит потому, что они кэшируются и просматривают их по "позиции", возвращенной getItemId.

Представьте, что у вас есть три фрагмента (0, 1 и 2), и вы хотите удалить "1". Если вы вернете "позицию" для getItemId, то удаление фрагмента 1 не будет работать, потому что когда вы попытаетесь показать фрагмент 2 после удаления фрагмента 1, пейджер/адаптер подумает, что он уже получил фрагмент для этой "позиции" и продолжит отображение фрагмента 1.

FYI: я попробовал notifyDataSetChanged вместо установки адаптера, но это не сработало для меня.

Сначала, пример getItemId переопределить и что я сделал для моего getItem:

public class SectionsPagerAdapter extends FragmentPagerAdapter
{
    ...

    @Override
    public long getItemId(int position)
    {
        // Mode 1 uses Fragments 0, 1 and 2. Mode 2 uses Fragments 0 and 3
        if ( mode == 2 && position == 1 )
            return 3;
        return position;
    }

    @Override
    public Fragment getItem(int position)
    {
        if ( mode == 1 )
        {
            switch (position)
            {
                case 0:
                    return <<fragment 0>>;
                case 1:
                    return <<fragment 1>>;
                case 2:
                    return <<fragment 2>>;
            }
        }
        else    // Mode 2
        {
            switch (position)
            {
                case 0:
                    return <<fragment 0>>;
                case 1:
                    return <<fragment 3>>;
            }
        }
        return null;
    }
}

Теперь изменение режима:

private void modeChanged(int newMode)
{
    if ( newMode == mode )
        return;

    mode = newMode;

    // Calling mSectionsPagerAdapter.notifyDataSetChanged() is not enough here
    mViewPager.setAdapter(mSectionsPagerAdapter);
}

Ответ 6

Не получилось. Моему решению был добавлен FragmentStatePagerAdapter.java в мой проект, переименованный в FragmentStatePagerAdapter2.java. В destroyItem() я немного изменился на основе журналов ошибок. Из

// mFragments.set(position, null);

to

if (position < mFragments.size())mFragments.remove(position);

Возможно, у вас нет такой же проблемы, просто проверьте журнал. Упование это помогает кому-то!

Ответ 7

После многих попыток я заработал, чтобы он правильно удалял или прикреплял третий фрагмент в конечной позиции.

Object fragments[] = new Object[3];
int mItems = 2;
MyAdapter mAdapter;
ViewPager mPager;


public void addFragment(boolean bool) {

    mAdapter.startUpdate(mPager);

    if (!bool) {
        mAdapter.destroyItem(mPager, 2, fragments[2]);
        mItems = 2;
        fNach = false;
    }
    else if (bool && !fNach){
        mItems = 3;
        mAdapter.instantiateItem(mPager,2);
        fNach = true;
    }
    mAdapter.finishUpdate(mPager);
    mAdapter.notifyDataSetChanged();

}

public class MyAdapter extends FragmentPagerAdapter {
    MyAdapter(FragmentManager fm) {
        super(fm);
    }

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

    @Override
    public CharSequence getPageTitle(int position) {
    ... (code for the PagerTitleStrip)
    }

    @Override
    public Fragment getItem(int position) {
        Fragment f = null;
        switch (position) {
            case 0:
                f = new Fragment1();
                break;
            case 1:
                f = new Fragment2();
                break;
            case 2:
                f = new Fragment3();
                break;
        }
        return f;
    }

    @Override
    public Object instantiateItem(ViewGroup container, int position) {
        Object o = super.instantiateItem(container,position);
        fragments[position] = o;
        return o;
    }

    @Override
    public void destroyItem(ViewGroup container, int position, Object object) {
        super.destroyItem(container, position, object);
        System.out.println("Destroy item " + position);
        if (position >= getCount()) {
                FragmentManager manager = ((Fragment) object).getFragmentManager();
                FragmentTransaction ft = manager.beginTransaction();
                ft.remove((Fragment) object);
                ft.commit();
        }

    }
}

Некоторое уточнение: чтобы получить ссылку на объект для вызова destroyItem, я сохранил объекты, возвращенные из объекта instantiateItem в массиве. Когда вы добавляете или удаляете фрагменты, вы должны объявить об этом с помощью startUpdate, finishUpdate и notifyDataSetChanged. Количество элементов должно быть изменено вручную, для добавления вы увеличиваете его и создаете экземпляр, тогда getItem создает его. Для удаления вы вызываете destroyItem, и в этом коде важно положение >= mItems, потому что destroyItem также вызывается, если фрагмент выходит из кеша. Вы не хотите его удалять. Единственное, что не работает, - это анимация прокрутки. После удаления последней страницы анимация "не может прокрутить влево" на новой последней странице не восстановлена ​​правильно. Если вы проведете его, появится пустая страница, и она отскакивает назад.

Ответ 8

Реальная проблема заключается в том, что FragmentPagerAdapter использует позицию фрагмента в вашем списке как идентификатор. Поэтому, если вы добавляете новый список или просто удаляете элементы, элемент "instantiateItem" найдет разные фрагменты для новых элементов в списке...

@Override
public Object instantiateItem(ViewGroup container, int position) {
    if (mCurTransaction == null) {
        mCurTransaction = mFragmentManager.beginTransaction();
    }

    final long itemId = getItemId(position);

    // Do we already have this fragment?
    String name = makeFragmentName(container.getId(), itemId);
    Fragment fragment = mFragmentManager.findFragmentByTag(name);
    if (fragment != null) {
        if (DEBUG) Log.v(TAG, "Attaching item #" + itemId + ": f=" + fragment);
        mCurTransaction.attach(fragment);
    } else {
        fragment = getItem(position);
        if (DEBUG) Log.v(TAG, "Adding item #" + itemId + ": f=" + fragment);
        mCurTransaction.add(container.getId(), fragment,
                makeFragmentName(container.getId(), itemId));
    }
    if (fragment != mCurrentPrimaryItem) {
        fragment.setMenuVisibility(false);
        fragment.setUserVisibleHint(false);
    }

    return fragment;
}

и

  private static String makeFragmentName(int viewId, long id) {
    return "android:switcher:" + viewId + ":" + id;
}

и

     * Return a unique identifier for the item at the given position.
 * <p>
 * <p>The default implementation returns the given position.
 * Subclasses should override this method if the positions of items can change.</p>
 *
 * @param position Position within this adapter
 * @return Unique identifier for the item at position
 */
public long getItemId(int position) {
    return position;
}

Ответ 9

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

Мое решение состояло в том, чтобы передать ChildFragment Manager в конструктор PagerAdapter Fragment (State), а не в Fragment Manager родительского Fragment.

Используя ChildFragmentManager, все фрагменты, созданные ViewPagerAdapter, очищаются автоматически при уничтожении родительского фрагмента.