ViewPager с фрагментами, жизненные циклы?

Question

[Артем]

Есть приложение, в нем ViewPager и два фрагмента. Реализуется это так:

public class MainActivity extends FragmentActivity {

    Vector<Fragment> fragments;
    ViewPager viewPager;

    @Override
    public void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.main);

        viewPager = (ViewPager) findViewById(R.id.pager);

        fragments = new Vector<Fragment>();
        fragments.add(Fragment.instantiate(this, Fragment1.class.getName()));
        fragments.add(Fragment.instantiate(this, Fragment2.class.getName()));

        viewPager.setOffscreenPageLimit(fragments.size());
        PagerAdapter pagerAdapter = new PagerAdapter(super.getSupportFragmentManager(), fragments);

        viewPager.setAdapter(pagerAdapter);

        viewPager.setCurrentItem(0);
    }

    @Override
    public void onActivityResult(int requestCode, int resultCode, Intent data) {
        if(requestCode==1) {
            ((Fragment1)fragments.get(0)).load();
        }
    }

    class PagerAdapter extends FragmentPagerAdapter implements IconPagerAdapter {

        private List<Fragment> fragments;

        public PagerAdapter(FragmentManager fm, List<Fragment> fragments) {
            super(fm);
            this.fragments = fragments;
        }

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

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

    }

}

В классе Fragment1 у меня есть переменные, которые объявлены как свойства класса и инициализируются внутри метода onCreateView():

public class Fragment1 extends Fragment {

    ArrayList<ListItemObject> listItems = null;
    ArrayAdapter<ListItemObject> mAdapter;

    @Override
    public View onCreateView(LayoutInflater inflater, ViewGroup container, Bundle savedInstanceState) {
        super.onCreateView(inflater, container, savedInstanceState);

        listItems = new ArrayList<ListItemObject>();
        mAdapter = new MyAdapter(getActivity(), listItems);
        listView.setAdapter(mAdapter);
    }

    void load() {
            mAdapter.clear();
            ...
    }

}

Проблема вот в чем. Во фрагментах иногда (не слишком часто) mAdapter бывает равным нулю. То есть получается, что фрагмент пересоздается, но без вызова onCreateView(). Мне же нужно, чтобы он вообще не пересоздавался пока приложение запущено. Кроме того, внутри фрагмента используется getActivity(), который иногда возвращает null. Возможно, эти две проблемы связаны. Возможно ли это как-то исправить?

Answer 1

[ara89]

Используйте код из сэмпла в доках. Не храните ссылки на фрагменты в адаптере.

Чтобы фрагмент не пересоздавался можно вызывать метод setRetainInstance(true)

Если вам нужно находить фрагмент по тэгу, то и создать его можно со своим тэгом с помощью класса FragmentTransaction.
В случае с ViewPager, FragmentPagerAdapter назначает фрагментам свои тэги, это можно увидеть в его исходниках.
Там метод makeFragmentName(int viewId, long id) назначает фрагментам тэги вида:

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

где viewId — id ViewPagera, id — позиция фрагмента.
Соответственно, по этим тэгам можно брать фрагменты из ViewPagera

Answer 2

[recompileme]

перенесите определение адаптера и активити в онкриэйт, а не в онкриэйтвью:
Override public void onCreate(Bundle savedInstanceState) { super.onCreate(savedInstanceState); Log.d(TAG, "init"); dlgs = new ArrayList<ClsVideos>(); adapter = new AdpVideos(getActivity(),dlgs); dlgsSize = 0; offset = 0; activity = getActivity(); }

Comments

[recompileme]

@Override
    public void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        Log.d(TAG, "init");
        dlgs = new ArrayList<ClsVideos>();
        adapter = new AdpVideos(getActivity(),dlgs);
        dlgsSize = 0;
        offset = 0;
        activity = getActivity();

    }

[Артем]

Но это поможет подавить вылеты, а не устранить их. То есть, фрагмент просто будет пересоздаваться в этот момент. А хотелось бы сделать так, чтобы он висел в памяти без лишних пересоздаваний. Я вроде находил в сети указания на то, что можно создавать фрагменты с указанием тега и потом, в onCreate(savedInstanceState) если savedInstanceState!=null, то находить фрагмент по этому тегу используя findFragmentByTag(). Но опять-таки, как назначить тег фрагменту?

Answer 3

[recompileme]

можно отключить пересоздание фрагмента при смене ориентации активити: android:configChanges=«keyboardHidden|orientation|screenSize»

Comments

[Артем]

Пробовал. Проблема не в смене ориентации экрана, в этом случае работает отлично. Тут как-то организовать иначе надо… А как именно — никак не соображу.

[recompileme]

FragmentStatePagerAdapter: When pages are not visible to the user, their entire fragment may be destroyed, only keeping the saved state of that fragment.

Answer 4

[Андрей М]

Попробуй использовать

PagerAdapter pagerAdapter = new PagerAdapter(getChildFragmentManager(), fragments);

вместо

PagerAdapter pagerAdapter = new PagerAdapter(super.getSupportFragmentManager(), fragments);

Comments

[IrixV]

Как я сама до этого не додумалась, весь день вчера потратила на решение этой проблемы, Вы меня спасли! Спасибо!

Answer 5

[Артем]

То есть можно на выбор — либо сохранять в MainActivity теги созданных фрагментов и в адаптере в getItem() делать findFragmentByTag(), либо в том же getItem() получать каждый раз фрагменты с помощью Fragment.instantiate() и устанавливать для них setRetainInstance(true)? Я правильно понимаю? Если так, то как лучше делать?

Comments

[ara89]

FragmentPagerAdapter обычно держит фрагменты в памяти и не уничтожает их. соответственно метод getItem() в нем обычно вызывается один раз для каждой страницы.

setRetainInstance(true) вызывайте внутри самого фрагмента в onCreate()

[Артем]

Спасибо за информацию. Сделал так:

public class MainActivity extends FragmentActivity {
	
	ArrayList<String> fragmentTags=new ArrayList<String>();

	@Override
    public void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.main);
        
        viewPager = (ViewPager) findViewById(R.id.pager);
        
        viewPager.setOffscreenPageLimit(2);
        PagerAdapter pagerAdapter = new PagerAdapter(super.getSupportFragmentManager());
        
        viewPager.setAdapter(pagerAdapter);

        viewPager.setCurrentItem(0);
    }
    
    Fragment getFragment(int pos) {
    	if(pos<fragmentTags.size()) {
    		return getSupportFragmentManager().findFragmentByTag(fragmentTags.get(pos));
    	}
    	return null;
	}

	class PagerAdapter extends FragmentPagerAdapter {

		public PagerAdapter(FragmentManager fm) {
			super(fm);
		}
		
		@Override
		public Object instantiateItem(ViewGroup container, int position) {
			Fragment fragment = (Fragment) super.instantiateItem(container, position);
			if(position<fragmentTags.size()) fragmentTags.set(position,fragment.getTag());
    		else fragmentTags.add(fragment.getTag());
			return fragment;
		}

		@Override
		public Fragment getItem(int position) {
			Fragment fragment = null;
			switch(position) {
			case 0:
				fragment=Fragment.instantiate(MainActivity.this, Fragment1.class.getName());
				break;
	    	case 1:
	    		fragment=Fragment.instantiate(MainActivity.this, Fragment2.class.getName());
	    		break;
	    	}
			return fragment;
		}

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

	}

}

Внутри фрагмента:

public class Fragment1 extends Fragment {

	@Override
    public View onCreateView(LayoutInflater inflater, ViewGroup container, Bundle savedInstanceState) {
    	super.onCreateView(inflater, container, savedInstanceState);
    	setRetainInstance(true);

		//Заполнение фрагмента
	}

}

И для обращения к методу фрагмента Fragment1 из активити использую

Fragment fragment;
if((fragment=getFragment(0))!=null) ((Fragment1)fragment).myMethod();

Работает. Но плохо, что глюк носил плавающий характер и если он все же есть, то возможно я его не скоро увижу… Буду тестить.

[ara89]

я бы вам еще рекомендовал не обращаться к фрагментам напрямую из активити путем приведения типов. это источник будущих ошибок и вообще не красиво.
Фрагменты должны абстрагироваться от Активити контейнера, потому используте более гибкий механизм коммуникации.
Вариантов много: Callback интерфейсы, BroadcastReceiverы, какой нибудь EventBus фреймверк типа Otto

[Benderlidze]

@bartwell Добрый день, а есть рабочий пример?

[Артем]

@Benderlidze, Здравствуйте. Посмотрите код под фразой "Сделал так", он рабочий. Единственное, если нужно, можете учесть поправку от ara89.

Answer 6

[dmitriy_deomin]

Можно попробовать изменить лимит страниц
setOffscreenPageLimit

viewPager.setOffscreenPageLimit(5);