Дублирование пунктов меню с помощью ViewPager и Fragments

Я создаю приложение для Android (минимум SDK уровня 10, Gingerbread 2.3.3) с некоторыми фрагментами в ViewPager. Я использую ActionBarSherlock для создания панели действий и android-viewpagertabs для добавления вкладок в ViewPager, как в клиенте Market.

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

Но тут происходят две странные вещи:

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

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

Код FragmentActivity, который отображает ViewPager и его вкладки:

public class MainActivity extends FragmentActivity {

    public static final String TAG = "MainActivity";

    private ActionBar actionBar;
    private Adapter adapter;
    private ViewPager viewPager;
    private ViewPagerTabs tabs;

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

        actionBar = getSupportActionBar();

        adapter = new Adapter(getSupportFragmentManager());
        adapter.addFragment(getString(R.string.title_tab_subscriptions),
                SubscriptionGridFragment.class);
        // adding more fragments here

        viewPager = (ViewPager) findViewById(R.id.viewpager);
        viewPager.setAdapter(adapter);

        tabs = (ViewPagerTabs) findViewById(R.id.tabs);
        tabs.setViewPager(viewPager);
    }

    public static class Adapter extends FragmentPagerAdapter implements
            ViewPagerTabProvider {

        private FragmentManager fragmentManager;
        private ArrayList<Class<? extends Fragment>> fragments;
        private ArrayList<String> titles;

        public Adapter(FragmentManager fm) {
            super(fm);
            fragmentManager = fm;
            fragments = new ArrayList<Class<? extends Fragment>>();
            titles = new ArrayList<String>();
        }

        public void addFragment(String title, Class<? extends Fragment> fragment) {
            titles.add(title);
            fragments.add(fragment);
        }

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

        public String getTitle(int position) {
            return titles.get(position);
        }

        @Override
        public Fragment getItem(int position) {
            try {
                return fragments.get(position).newInstance();
            } catch (InstantiationException e) {
                Log.wtf(TAG, e);
            } catch (IllegalAccessException e) {
                Log.wtf(TAG, e);
            }
            return null;
        }

        @Override
        public Object instantiateItem(View container, int position) {
            FragmentTransaction fragmentTransaction = fragmentManager
                    .beginTransaction();
            Fragment f = getItem(position);
            fragmentTransaction.add(container.getId(), f);
            fragmentTransaction.commit();
            return f;
        }
    }

    @Override
    public boolean onCreateOptionsMenu(Menu menu) {
        BaseActivity.addGlobalMenu(this, menu);
        return true;
    }

    @Override
    public boolean onOptionsItemSelected(MenuItem item) {
        return BaseActivity.handleGlobalMenu(this, item);
    }

}

Код фрагмента, который должен иметь свои пункты меню:

public class SubscriptionGridFragment extends Fragment {

    private GridView subscriptionList;
    private SubscriptionListAdapter adapter;

    @Override
    public void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setHasOptionsMenu(true);
    }

    // ...

    @Override
    public void onCreateOptionsMenu(Menu menu, MenuInflater inflater) {
        inflater.inflate(R.menu.subscription_list, menu);
    }

    @Override
    public boolean onOptionsItemSelected(MenuItem item) {
        // ...
    }
}

person Julian    schedule 20.11.2011    source источник


Ответы (3)


Нашел это в примере проекта Android. В действии фрагмента onCreate необходимо проверить, активен ли еще параметр saveInstanceState, например:

    setContentView(R.layout.custom_detail_fragment);

    if (savedInstanceState != null) {
        return;
    }

    CDFrag = new CustomDetailFragment();
    getSupportFragmentManager().beginTransaction().add(R.id.custom_detail_fragment_container, CDFrag).commit();

Ключевой является эта часть:

    if (savedInstanceState != null) {
        return;
    }

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

person dbDev    schedule 24.08.2012

Была такая же проблема.

Моя ошибка заключалась в том, что я создал новый Viewpager в oncreate и установил его как id

viewpager.setId(viewpager.hashcode());
setContentView(viewpager);

Но это было большой ошибкой, потому что после изменения ориентации FragmentPagerAdapter пытается повторно использовать старые фрагменты. Он ищет в FragmentManager фрагмент с идентификатором, сгенерированным из идентификатора просмотрщика и позиции фрагмента.

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);

контейнер — Viewpager.

Решение:

viewpager.setId(R.id.viewpager);
setContentView(viewpager);
person passsy    schedule 23.02.2013

Спасибо SimonVT из freenode/#android-dev: «Совет: посмотрите, что на самом деле делает fragmentpageradapter, прежде чем переопределять instanceItem».

Удаление этого метода решило всю проблему.

person Julian    schedule 20.11.2011