Фрагмент с recyclerview, обновление данных с помощью otto

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

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

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

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

У меня есть метод регистрации, вызываемый в методе 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;
        }
    }

}

person 55597    schedule 05.06.2015    source источник
comment
Пожалуйста, добавьте также класс PagerAdapter. Я думаю, вы могли бы использовать его неправильно. Он может быть связан одним объектом Fragment с несколькими страницами, а не с несколькими экземплярами Fragment на каждой странице.   -  person Youngjae    schedule 05.06.2015
comment
Я использую GreenRobot, но, возможно, Отто такой же. Вы отменяете регистрацию автобуса? Я обнаружил, что с фрагментами лучше всего регистрироваться в onResume и отменять регистрацию в onPause.   -  person Smashing    schedule 05.06.2015
comment
@Smashing Я не понимаю, как это будет работать, пожалуйста, объясните. Прочтите мои комментарии к ответу, приведенному ниже.   -  person 55597    schedule 05.06.2015
comment
@ Ёнджэ, посмотри на мой ответ, это то, что ты хотел, чтобы я проверил в своем первом вопросе?   -  person 55597    schedule 06.06.2015
comment
@ 55597 // Да. Я не знал компонентов вашей реализации SectionOneFragment и других. Я пересматриваю свою реализацию, чтобы показать ViewPagerAdapter. Ваша реализация при ответе всегда будет воссоздавать фрагменты раздела, но моя воссоздает или дает существующий. Так что, пожалуйста, рассмотрите мою.   -  person Youngjae    schedule 06.06.2015


Ответы (2)


Решил!

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

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

Это код, который я изменил в методе getItem класса 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 этих новых классов, подписанный метод также реализован в соответствующих классах.

person 55597    schedule 06.06.2015
comment
Хотя вы решили свою проблему, но для конкретного случая. Лучше предоставить решение, когда вы используете только один Fragment в вашем FragmentPagerAdapter - person blueware; 18.03.2019

Ваш Viewpager выглядит нормально. Поместите otto registrater/unregister в каждый 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 "";
    }

}
person Youngjae    schedule 05.06.2015
comment
BusProvider.getInstance().onPause(); ? - person 55597; 05.06.2015
comment
@ 55597 // извините. Отредактировано. - person Youngjae; 05.06.2015
comment
OnPause вызывается, когда я перехожу к следующему действию для публикации. Следовательно, весь смысл использования Отто теряется. - person 55597; 05.06.2015
comment
@ 55597 // верно. как я сказал в комментарии к вопросу, причина, по которой otto потрачена впустую, заключается в том, что, как вы знаете, ваша реализация всегда воссоздает фрагмент, поэтому всегда обновляйте контент. но рассмотрите мой, так как он даст больше производительности. - person Youngjae; 06.06.2015