FragmentPagerAdapterがアイテム（フラグメント）を正しく削除しない
FragmentPagerAdapterを実装し、ViewPagerのすべてのフラグメントを保持するために_List<Fragment>_を使用して表示しました。 addItem()で、インスタンス化されたFragmentを追加し、notifyDataSetChanged()を呼び出します。これが必要かどうかはわかりません。
私の問題は単純に...フラグメント1から始まります
_[Fragment 1]
_新しいフラグメント2を追加
_[Fragment 1] [Fragment 2]
_フラグメント2を削除
_[Fragment 1]
_新しいフラグメント3を追加
_[Fragment 1] [Fragment 2]
_新しいフラグメントを追加するとき、すべてが素晴らしいようです。フラグメントを削除してから、新しくインスタンス化されたフラグメントを追加すると、古いフラグメントが引き続き表示されます。 .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);
}
}
_コードがフラグメントタイプを決定するための「ゲットー」手段を使用していることを理解していますが、機能をテストするためにこのコードを厳密に記述しました。 FragmentPagerAdaptersの世界に進出した人はあまりいないように思えるので、どんな助けやアイデアも素晴らしいでしょう。
私は同じ問題を抱えており、私の解決策はメソッド「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();
}
それは私のために働いています、誰か他の解決策がありますか？
更新：
これらのコードによりFragmentが不要になりましたので、それを回避するための条件を追加しました。
@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();
}
}
この投稿を更新し、ソリューションを追加しました（改善できる場合はお知らせください）
わかりました、私は今ハックのように私の問題を解決しましたが、ええ、それは働いています;）。誰かが私のソリューションを改善できるなら、私に知らせてください。私の新しいソリューションでは、CustomFragmentStatePagerAdapterを使用するようになりましたが、必要な状態は保存されず、すべてのフラグメントがリストに保存されます。ユーザーが通常のFragmentPagerAdapterのように50個を超えるフラグメントを持っている場合、これによりメモリの問題が発生する可能性があります。誰かが私の修正を削除せずに、State-thingを自分のソリューションに追加できるといいですね。ありがとう。
だからここに私の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でこれを使用します（削除、追加などのすべてのタブ操作を処理するクラス）
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、...これらのfragmenttypesのいずれかを2回使用する場合ではありません。
私は今のところ擬似的に動作させました。それは本当に汚い解決策であり、私はまだより良い解決策を探しています。助けてください。
コードを変更し、タブをこのタブに削除しました：
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の評判を加えました。それを解決した人に寄付することもできます。
ありがとう
「両方の長所」（@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))はそのアイテムがもう必要ないかどうかを判断するのに役立ちます。
私はあなたの状況に似ていました。最近、ViewPagerにフラグメントを追加および削除する必要がありました。最初のモードでは、フラグメント0、1、および2があり、2番目のモードでは、フラグメント0および3があります。両方のモードでフラグメント0を同じにして、情報を保持します。
必要なのは、FragmentPagerAdapter.getItemIdをオーバーライドして、異なるフラグメントごとに一意の番号が返されるようにすることだけでした。デフォルトでは、「position」が返されます。また、ViewPagerでアダプターを再度設定する必要がありました。新しいインスタンスは機能しますが、同じインスタンスに戻します。アダプターを設定すると、ViewPagerがすべてのビューを削除し、tryingを再度追加します。
ただし、トリックは、フラグメントを表示するたびにではなく、フラグメントをインスタンス化するときにのみアダプタがgetItemを呼び出すことです。これは、それらがキャッシュされ、getItemIdによって返される「位置」によってそれらを検索するためです。
3つのフラグメント（0、1、および2）があり、「1」を削除するとします。 getItemIdに「位置」を返す場合、フラグメント1の削除は機能しません。フラグメント1を削除した後にフラグメント2を表示しようとすると、ポケットベル/アダプタはその「位置」のフラグメントを取得したと判断し、フラグメント1を表示し続けるためです。
参考：アダプタを設定する代わりに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);
}
うまくいかなかった。私のソリューションは、プロジェクトにFragmentStatePagerAdapter.Javaを配置し、FragmentStatePagerAdapter2.Javaに名前を変更しました。 destroyItem（）では、エラーログに基づいて少し変更しました。から
// mFragments.set(position, null);
に
if (position < mFragments.size())mFragments.remove(position);
たぶん、あなたは同じ問題を抱えていない、ただログをチェックしてください。
本当の問題は、FragmentPagerAdapterがリスト内のフラグメントの位置をIDとして使用することです。したがって、新しいリストを追加するか、単にアイテムを削除すると、「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;
}
私は自分が気づくまでこの同じ問題を抱えていましたが、メインのアクティビティではなく、別のフラグメント内からPagerViewを作成していました。
私の解決策は、ChildFragment ManagerをFragment（State）PagerAdapterのコンストラクターに渡すことであり、親FragmentのFragment Managerではありません。
ChildFragmentManagerを使用すると、ViewPagerAdapterによって作成されたすべてのFragmentsは、親Fragmentが破棄されると自動的にクリーンアップされます。
何度も試してみて、最後の位置で3番目のフラグメントを正しく削除またはアタッチするように動作するようにしました。
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を呼び出します。このコードでは、フラグメントがキャッシュから出るとdestroyItemも呼び出されるため、位置> = mItemsが不可欠です。その場合、それを削除したくありません。動作しない唯一のものは、スワイプアニメーションです。最後のページを削除した後、「左にスワイプできません」アニメーションが新しい最後のページで正しく復元されません。スワイプすると、空白のページが表示され、跳ね返ります。