Использование наблюдателя внутри диалогового фрагмента вызывает программу cra sh после повторного открытия диалогового окна.

Question

Я недавно задал этот вопрос, но он немного запутался, поэтому я подумал, что я немного лучше объединил всю информацию и попробую снова.

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

Вот основной фрагмент, содержащий одну кнопку, при нажатии на которую открывается фрагмент диалога.

public class MainFragment extends Fragment {

    private MainViewModel mViewModel;
    private View fragmentView;
    private SharedViewModel sharedViewModel;


    public static MainFragment newInstance() {
        return new MainFragment();
    }

    @Override
    public View onCreateView(@NonNull LayoutInflater inflater, @Nullable ViewGroup container,
                             @Nullable Bundle savedInstanceState) {
        fragmentView = inflater.inflate(R.layout.main_fragment, container, false);
        return fragmentView;
    }

    @Override
    public void onActivityCreated(@Nullable Bundle savedInstanceState) {
        super.onActivityCreated(savedInstanceState);
        mViewModel = ViewModelProviders.of(this).get(MainViewModel.class);

        sharedViewModel = new ViewModelProvider(requireActivity()).get(SharedViewModel.class);

        Button aButton = fragmentView.findViewById(R.id.button);
        aButton.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View v) {
                sharedViewModel.setColor("Green");
                NavDirections action = MainFragmentDirections.actionMainFragmentToBlankFragment();
                Navigation.findNavController(getView()).navigate(action);

            }
        });

    }

}

Вот код фрагмента диалога, который появляется, когда я нажимаю кнопку.

public class BlankFragment extends DialogFragment {
    private View dialogView;
    private SharedViewModel sharedViewModel;

    @Override
    public Dialog onCreateDialog(Bundle savedInstanceState) {
        LayoutInflater layoutInflater = requireActivity().getLayoutInflater();
        dialogView = layoutInflater.inflate(R.layout.blank_fragment, null);
        AlertDialog.Builder builder = new AlertDialog.Builder(getActivity());
        builder.setView(dialogView);

        return builder.create();
    }

    @Override
    public void onActivityCreated(@Nullable Bundle savedInstanceState) {
        super.onActivityCreated(savedInstanceState);
        sharedViewModel = new ViewModelProvider(requireActivity()).
                get(SharedViewModel.class); //gets the shared view model from the associsated fragment.
        MutableLiveData<String> tableColor = sharedViewModel.getColor();
        tableColor.observe(getParentFragment().getViewLifecycleOwner(), new Observer<String>() {
            @Override
            public void onChanged(String s) {
                Toast.makeText(getActivity(), "IF I GET HERE TWICE I DIE", Toast.LENGTH_SHORT).show();
            }

        });
    }
}

Как вы, наверное, видели, я использовал навигационный график для навигации между ними.

Модель общего представления выглядит следующим образом

public class SharedViewModel extends ViewModel {
    private MutableLiveData<String> color = new MutableLiveData<>();


    public void setColor(String color){
        this.color.setValue(color);
    }
    public MutableLiveData<String> getColor(){
        return color;
    }

}

Сообщение об ошибке гласит:

E / AndroidRuntime: FATAL EXCEPTION: main Процесс: com.example.dialogfragmentcrashing, PID: 32620 java .lang.NullPointerException: попытка вызвать виртуальный метод 'java .lang.String android .content.Context.getPackageName ()' для пустой ссылки на объект в android .widget.Toast. (Toast . java: 121) в android .widget.Toast.makeText (Toast. java: 286) в android .widget.Toast.makeText (Toast. java: 276) в com.example. dialogfragmentcrashing.BlankFragment $ 1.onChanged (BlankFragment. java: 47) в com.example.di alogfragmentcrashing.BlankFragment $ 1.onChanged (BlankFragment. java: 44) в androidx.lifecycle.LiveData.considerNotify (LiveData. java: 131) в androidx.lifecycle.LiveData.dispatchingValue (LiveData. java 14): 14 androidx.lifecycle.LiveData.setValue (LiveData. java: 307) в androidx.lifecycle.MutableLiveData.setValue (MutableLiveData. java: 50) в com.example.dialogfragmentcrashing.SharedViewModel *setMode: 10: Shared (Shared). 11) в com.example.dialogfragmentcrashing.MainFragment $ 1.onClick (MainFragment. java: 49) в android .view.View.performClick (View. java: 7155) в android .view.View. executeClickInternal (просмотр. java: 7124) при android .view.View.access $ 3500 (просмотр. java: 808) при android .view.View $ PerformClick.run (просмотр. java: 27370 ) в android .os.Handler.handleCallback (Обработчик. java: 883) в android .os.Handler.dispatchMessage (Обработчик. java: 100) в android .os.Looper.l oop (Looper. java: 359) в android .app.ActivityThread.main (ActivityThread. java: 7418) в java .lang.reflect.Method.invoke (Native M и т. д.) в com. android .internal.os.RuntimeInit $ MethodAndArgsCaller.run (RuntimeInit. java: 492) в com. android .internal.os.ZygoteInit.main (ZygoteInit. java: 935)

Answer 1

getParentFragment().getViewLifecycleOwner() всегда неправильно использовать LifecycleOwner, поскольку он не уничтожается при уничтожении DialogFragment. Переместите весь этот код в onCreate() и используйте this:

@Override
public void onCreate(@Nullable Bundle savedInstanceState) {
    super.onCreate(savedInstanceState);
    sharedViewModel = new ViewModelProvider(requireActivity()).
            get(SharedViewModel.class); //gets the shared view model from the associsated fragment.
    MutableLiveData<String> tableColor = sharedViewModel.getColor();
    tableColor.observe(this, new Observer<String>() {
        @Override
        public void onChanged(String s) {
            Toast.makeText(getActivity(), "IF I GET HERE TWICE I DIE", Toast.LENGTH_SHORT).show();
        }

    });
}