Как удалить наблюдателя из liveata, чтобы он не показывался дважды при переходе назад к фрагменту - PullRequest
Новая
       89

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

3 голосов
/ 12 марта 2020

У меня есть фрагмент, который отображает всплывающее окно, когда пользователь успешно вошел в систему. Если я перехожу к новому фрагменту и возвращаюсь, всплывающее окно с предыдущим именем пользователя отображается снова. Я исправил эту проблему с помощью SingleLiveEvent, но теперь мне нужно провести рефакторинг своего кода, чтобы использовать MediatorLiveData, поскольку мои данные могут поступать из 2 источников (удаленного и базы данных), и он не совместим с SingleLiveEvent.

Я пытался использовать Обертка событий и удаление наблюдателей в onDestroyView (), но пока ничего не работает, функция liveata onChanged продолжает вызываться, когда я возвращаюсь к фрагменту. Вот часть моего фрагмента: 

@Override
public View onCreateView(LayoutInflater inflater, ViewGroup container,
                         Bundle savedInstanceState) {
    // Inflate the layout for this fragment
    binding = FragmentDashboardBinding.inflate(inflater, container, false);
    binding.setLifecycleOwner(getActivity());

    //Get the attendanceViewModel for registering attendance
    attendanceViewModel = ViewModelProviders.of(this).get(AttendanceViewModel.class);
    attendanceViewModel.getAttendance().observe(getViewLifecycleOwner(), attendanceAndMember -> {
        if (attendanceAndMember != null && attendanceAndMember instanceof AttendanceMemberModel) {
            clokedInOutMember = attendanceAndMember.member;
        }
        showResultClockInOutPopup();
    });

    return binding.getRoot();
}

private void showResultClockInOutPopup() {

    clockInBuilder = new AlertDialog.Builder(getActivity());

    View view = getLayoutInflater().inflate(R.layout.status_clock_in_out_popup, null);
    TextView responseClockInOut = view.findViewById(R.id.responseClockInOut);
    Button dismissButton = view.findViewById(R.id.dismissButton);

    //Setup Popup Text
    if (clokedInOutMember != null) {
        if (StringToBool(clokedInOutMember.is_clocked_in_temp)) {
            responseClockInOut.setText("Bienvenue " + clokedInOutMember.name + ", tu es bien enregistré(e).");
        } else {
            responseClockInOut.setText("Désolé de te voir partir " + clokedInOutMember.name + ", à bientôt!");
        }
    } else {
        responseClockInOut.setText("Oups, il semblerait qu'il y ait une erreur...\n Essaye à nouveau.");
    }

    [..SETUP ALERTDIALOG...]

        //Dismiss popup
        dismissButton.setOnClickListener(v -> {
            clockInResultDialog.dismiss();
            clockInResultPopupShowed = false;
            clokedInOutMember = null;
        });

        clockInResultDialog.show();
        clockInResultPopupShowed = true;
    }

}

@Override
public void onDestroyView() {
    attendanceViewModel.getAttendance().removeObservers(this);
    super.onDestroyView();
}

А вот мой ViewModel, я должен использовать преобразования, так как я получаю userId из фрагмента, передавая Viewmodel, который передает его в хранилище для запроса ( может есть лучший способ?): 

public class AttendanceViewModel extends AndroidViewModel {

private AttendanceRepository repository = AttendanceRepository.getInstance();
public LiveData<AttendanceMemberModel> mAttendanceAndMember;
private MutableLiveData<String> mId =  new MutableLiveData<>();

private MediatorLiveData<AttendanceMemberModel> mObservableAttendance = new MediatorLiveData<AttendanceMemberModel>();
{
    mObservableAttendance.setValue(null);

    mAttendanceAndMember = Transformations.switchMap(mId, id -> {
        return repository.saveAttendance(id);
    });

    mObservableAttendance.addSource(mAttendanceAndMember, mObservableAttendance::setValue);
}


public AttendanceViewModel(@NonNull Application application) {
    super(application);
}

public LiveData<AttendanceMemberModel> getAttendance() {
    return mObservableAttendance;

}

public void setMemberId(String id) {
    mId.setValue(id);
}


@Override
protected void onCleared() {
    mObservableAttendance.setValue(null);
    super.onCleared();
}
}

Ответы [ 3 ]

0 голосов
/ 12 марта 2020

Используйте любой из них, который работает и ведет себя в соответствии с вашими потребностями. 

@Override
public void onPause() {
    attendanceViewModel.getAttendance().removeObservers(this);
    super.onPause();
}

@Override
public void onStop() {
    attendanceViewModel.getAttendance().removeObservers(this);
    super.onStop();
}

Жизненный цикл фрагмента

Посмотрите на жизненный цикл фрагмента , это даст вам немного больше идеи. Дайте мне знать, если это работает или нет.

0 голосов
/ 12 марта 2020

Я могу предложить вам два пути. Сначала создайте переменную boolean независимо от того, отображается ли диалоговое окно в Fragment, и после показа диалогового окна установите его на true и перед отображением диалогового окна проверьте, отображается ли диалоговое окно. Второй способ - после показа диалогового окна установить значение livedata в ноль и проверить, является ли значение наблюдателя нулевым, прежде чем показывать диалог. Я предпочитаю второй способ.

0 голосов
/ 12 марта 2020

Лучший способ сделать то же самое - это связать вашу модель представления с помощью OnViewCreated meathod. 

    @Override
    public void onActivityCreated(@NonNull View view, @Nullable Bundle savedInstanceState) {
        super.onViewCreated(view, savedInstanceState);
        attendanceViewModel = ViewModelProviders.of(this).get(AttendanceViewModel.class);
        setUpObservers();
      }



    private void setUpObservers() {
        attendanceViewModel.getAttendance().observe(getViewLifecycleOwner(), attendanceAndMember -> {
        if (attendanceAndMember != null && attendanceAndMember instanceof AttendanceMemberModel) {
            clokedInOutMember = attendanceAndMember.member;
        }
        showResultClockInOutPopup();
    });
    }

Если это не помогло, дайте мне знать. Спасибо.

...