Фрагмент с рециркуляцией, обновление данных с помощью Отто

Question

Я пытаюсь использовать Отто для достижения следующего варианта использования.

У меня есть Viewpager с фрагментами, который содержит RecyclerView каждый. Представления реселлера показывают сообщения, которые пользователи публикуют в приложении. Новая кнопка записи прикреплена к каждому из этих фрагментов. После публикации Button пользователь нажимает на новую активность и публикует там контент. Когда он возвращается к ViewPagerснова фрагмент экрана, я хочу обновить экран фрагмента и показать обновленное содержимое. Я использую Отто для этого.

Я могу сделать эту работу, когда присутствует только один фрагмент.

public class PlaceholderFragment extends Fragment {

private static final String ARG_SECTION_NUMBER = "section_number";


public static PlaceholderFragment newInstance(int sectionNumber) {
    PlaceholderFragment fragment = new PlaceholderFragment();
    Bundle args = new Bundle();
    args.putInt(ARG_SECTION_NUMBER, sectionNumber);
    fragment.setArguments(args);
    return fragment;
}


public PlaceholderFragment() {
}

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

    BusProvider.getInstance().register(this);
}

У меня есть метод register, вызываемый в методе onCreate метода PlaceholderFrament. поэтому, очевидно, когда несколько фрагментов загружаются несколько раз, я получаю следующую ошибку. Как мне справиться с этим?

Process: com.four.xxxx.xx, PID: 9934
java.lang.IllegalArgumentException: Object already registered.
        at com.squareup.otto.Bus.register(Bus.java:222)
        at com.four.xxxx.xx.PlaceholderFragment.onCreateView(PlaceholderFragment.java:94)
        at android.app.Fragment.performCreateView(Fragment.java:2053)

Редактировать 1: Класс PagerAdapter согласно запросу:

    public class FourScreenActivity extends Activity {

    SectionsPagerAdapter mSectionsPagerAdapter;
    ViewPager mViewPager;
    static final int POST_DEPT_REQUEST = 1;
    static final int POST_COLLEGE_REQUEST = 2;


    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.activity_four_screen);

        mSectionsPagerAdapter = new SectionsPagerAdapter(getFragmentManager());
        mViewPager = (ViewPager) findViewById(R.id.pager);
        mViewPager.setAdapter(mSectionsPagerAdapter);
    }

    @Override
    public boolean onCreateOptionsMenu(Menu menu) {
        getMenuInflater().inflate(R.menu.menu_main, menu);
        return true;
    }

    @Override
    public boolean onOptionsItemSelected(MenuItem item) {
        int id = item.getItemId();

        if (id == R.id.action_settings) {
            return true;
        }

        return super.onOptionsItemSelected(item);
    }

    public void postInSection1(View view) {
        Intent intent = new Intent(this, PostInSection1.class);
        startActivity(intent);

    }

    public void postInSection2(View view) {
        Intent intent = new Intent(this, PostInSection2.class);
        startActivity(intent);
    }


    public class SectionsPagerAdapter extends FragmentPagerAdapter {

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

        @Override
        public Fragment getItem(int position) {

            return PlaceholderFragment.newInstance(position + 1);
        }

        @Override
        public int getCount() {
            // Show 3 total pages.
            return 3;
        }

        @Override
        public CharSequence getPageTitle(int position) {
            Locale l = Locale.getDefault();
            switch (position) {

                case 0:
                    return getString(R.string.title_section1).toUpperCase(l);
                case 1:
                    return getString(R.string.title_section2).toUpperCase(l);
                case 2:
                    return getString(R.string.title_section3).toUpperCase(l);
            }
            return null;
        }
    }

}

Answer 1

Решил это!

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

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

Это код, который я изменил в методе get Item класса FragmentPagerAdapter из фрагмента, приведенного в вопросе.

 @Override
    public Fragment getItem(int position) {

        switch (position){
            case 0:
                return new SectionOneFragment();
            case 1:
                return new SectionTwoFragment();
            case 2:
                return new SectionThreeFragment();
        }

        //return PlaceholderFragment.newInstance(position + 1);
        return null;
    }

И теперь я вызываю bus.register() в методах onCreate этого нового класса, подписанный метод также реализован в соответствующих классах.

Answer 2

Ваш Viewpager кажется в порядке. Положил otto зарегистрироваться / отменить регистрацию на каждом onResume а также onPause как показано ниже onCreateView,

@Override
public void onResume() {
    super.onResume();
    BusProvider.getInstance().register(this);
}

@Override
public void onPause() {
    super.onPause();
    BusProvider.getInstance().unregister(this);
}

А также ViewPager реализация как ниже.

public class SectionsPagerAdapter extends FragmentStatePagerAdapter {
    private SparseArrayCompat<SectionFragmentInterface> fragmentSparseArrayCompat;

    public SectionsPagerAdapter (FragmentManager fm) {
        super(fm);
        fragmentSparseArrayCompat = new SparseArrayCompat<>();
    }

    @Override
    public Fragment getItem(int position) {
        SectionFragmentInterface fragment = fragmentSparseArrayCompat.get(position);

        switch (position) {
            case 0: {
                if (fragment == null) {
                    fragment = SectionOneListFragment.newInstance(position);
                }
            }
            break;
            case 1: {
                if (fragment == null) {
                    fragment = SectionTwoListFragment.newInstance(position);
                }
            }
            break;
        }

        fragmentSparseArrayCompat.put(position, fragment);

        return fragment;
    }

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

    @Override
    public CharSequence getPageTitle(int position) {
        return "";
    }

}