2

UPD

Activity

@Override
protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.main_screen_activity);
        mainFragment = new MainScreenFragment();
        getSupportFragmentManager()
            .beginTransaction()
            .replace(R.id.main_frame, mainFragment)
            .commit();
    }

Фрагмент

    @Override
    public void onResume() {
        super.onResume();
1:      redraw();
2:      getAccidents();
    }

    private void redraw(){
        Accidents.getVisible()
                 .subscribe(a -> listContent.addView(
3:                     Rows.getAccidentRow(getActivity(), listContent, a)
                  ));
    }
    private void getAccidents() {
        startRefreshAnimation(); //ProgressBar on
        Accidents.updated //подписка на Subject<Boolean>
                .observeOn(AndroidSchedulers.mainThread()) //Для ProgressBar
                .subscribe(b -> {
                        stopRefreshAnimation(); //ProgressBar off
                        redraw();
                 });
        Accidents.update(); //запуск сетевого запроса, который по окончанию зашлет true в подписку строкой выше
    }

По шагам:

  1. Запуская приложение. Все отрабатывает без проблем.
  2. Закрываю приложение кнопкой back.
  3. Запускаю заново.
  4. Строка 1: отрабатывает.
  5. Строка 2: отрабатывает, но в строке 3: на этот раз getActivity() возвращает null

Собственно вопрос, почему при одном и том же коде косяк происходит именно при перезапуске приложения?

7
  • Возможно у вас первый фрагмент ещё в памяти остался и при следующем зупаске вы его меняете на второй. Попробуйте в onCreate активити при добавлении фрагмента проверять не добавлен ли он уже. Например так: getSupportFragmentManager().findFragmentById(R.id.main_frame) != null
    – ЮрийСПб
    9 мар 2017 в 10:15
  • На схемки не нужно пулювать слюнями, что бы ни стояло за этим действием, вызов колбэков жизненного цикла происходит строго последовательно, согласно схемки. Предположить, что они вызываются "как попало" (асинхронно) говорит лишь о недостаточном понимании происходящего.
    – pavlofff
    9 мар 2017 в 10:30
  • @pavlofff ну так объясните, почему после onCreateActivity и onAttach getActivity() возвращает null
    – rjhdby
    9 мар 2017 в 10:37
  • 2
    В том коде, что находится в вашем вопросе, нет причин для описанного поведения. Проблема по этому коду не воспроизводится. Очевидно проблема в другом месте.
    – pavlofff
    9 мар 2017 в 11:01
  • Предлагаю сначала перенести получение getActivity() в сам метод onResume() и присвоить там полю класса. В дальнейшем обращаться к этому полю, а не через метод. Если не поможет, предлагаю исключить обращения к RxJava константами и проверить состояние метода getActivity(), я практически уверен, что проблемы не будет. Если так, то проблему нужно искать в самом Rx
    – pavlofff
    10 мар 2017 в 7:41

2 ответа 2

0

Добавлять Fragment к Activity нужно только при первом запуске Activity. Проверить первый раз запускается Activity или нет можно через параметр savedInstanceState метода onCreate, если он null - значит можно добавлять фрагмент.

Измените onCreate метод вашего Activity:

@Override
protected void onCreate(Bundle savedInstanceState) {
    super.onCreate(savedInstanceState);
    setContentView(R.layout.main_screen_activity);
    if (savedInstanceState == null) {
         mainFragment = new MainScreenFragment();
         getSupportFragmentManager()
             .beginTransaction()
             .replace(R.id.main_frame, mainFragment)
             .commit();
    }
}
3
  • @rjhdby Нет, создан ранее фрагмент или нет не является проблемой поведения в вопросе. тем более, что при закрытии кнопкой "назад" активити (и созданные ей классы, включая фрагмент) уничтожается. Очевидно, проблема с RxJava, но я с ней не работаю и понять, почему так происходит не помогу.
    – pavlofff
    10 мар 2017 в 7:39
  • 1
    Метод getActivity может возвращать null только если Fragment не прикреплен ни к одной Activity. В каком случае такое может произойти я написал в ответе выше. Проверить или Fragment прикреплен к Activity можено через метод Fragment.isAdded().
    – couldDog
    10 мар 2017 в 7:59
  • Не помогло, та же ошибка
    – rjhdby
    10 мар 2017 в 8:10
0

Workaround

Работать заставил, но решение вызывает больше вопросов, чем дает ответов.

Если кратко, то организовал подписку на контекст из метода onAttach()

private        FragmentActivity mActivity;
public  static Subject<Context> isAttached = PublishSubject.create();

@Override
public void onAttach(Context context) {
    super.onAttach(context);
    mActivity = (FragmentActivity) context;
    isAttached.onNext(mActivity);
}

public void redraw() {
    isAttached.subscribe(context -> {
                             listContent.removeAllViews();
                             Log.e("FRAGMENT", String.valueOf(this.isAdded()));
                             Accidents.getVisible()
                                      .sorted((a, b) -> MyUtils.reverseComparator(a.id, b.id))
                                      .subscribe(a -> listContent.addView(Rows.getAccidentRow(context, listContent, a)));
                             map.placeAccidents(context);
                         }
                        );
    if (mActivity != null) isAttached.onNext(mActivity);
}

Причем Log.e("FRAGMENT", String.valueOf(this.isAdded())); возвращает false в то время, когда context уже есть. Подозреваю, что как-то вся эта мистика связана с восстановлением из savedInstanceState но доказать не могу

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.