🐛 dialog 두 번 띄우면 앱이 종료되는 오류 해결

Author: peter-j0y

presentation/src/main/java/com/whyranoid/presentation/myrun/MyRunFragment.kt

@@ -1,33 +1,24 @@
 package com.whyranoid.presentation.myrun
 
 import android.os.Bundle
-import android.view.LayoutInflater
 import android.view.View
-import android.view.ViewGroup
+import android.widget.EditText
 import androidx.appcompat.app.AlertDialog
-import androidx.databinding.DataBindingUtil
 import androidx.fragment.app.viewModels
 import com.whyranoid.presentation.R
 import com.whyranoid.presentation.base.BaseFragment
-import com.whyranoid.presentation.databinding.DialogEditNickNameBinding
 import com.whyranoid.presentation.databinding.FragmentMyRunBinding
 import com.whyranoid.presentation.util.loadImage
+import com.whyranoid.presentation.util.repeatWhenUiStarted
 import dagger.hilt.android.AndroidEntryPoint
 
 @AndroidEntryPoint
 internal class MyRunFragment : BaseFragment<FragmentMyRunBinding>(R.layout.fragment_my_run) {
 
     private val viewModel by viewModels<MyRunViewModel>()
-    private lateinit var dialogViewBinding: DialogEditNickNameBinding
 
-    override fun onCreateView(
-        inflater: LayoutInflater,
-        container: ViewGroup?,
-        savedInstanceState: Bundle?
-    ): View? {
-        dialogViewBinding =
-            DataBindingUtil.inflate(inflater, R.layout.dialog_edit_nick_name, container, false)
-        return super.onCreateView(inflater, container, savedInstanceState)
+    private val builder: AlertDialog.Builder by lazy {
+        AlertDialog.Builder(requireContext())
     }
 
     override fun onViewCreated(view: View, savedInstanceState: Bundle?) {
@@ -46,31 +37,37 @@ internal class MyRunFragment : BaseFragment<FragmentMyRunBinding>(R.layout.fragm
     }
 
     private fun observeInfo() {
-        viewModel.nickName.observe(viewLifecycleOwner) {
-            binding.tvNickName.text = it
+        repeatWhenUiStarted {
+            viewModel.nickName.collect { nickName ->
+                binding.tvNickName.text = nickName
+            }
         }
 
-        viewModel.profileImgUri.observe(viewLifecycleOwner) {
-            binding.ivProfileImage.loadImage(it)
+        repeatWhenUiStarted {
+            viewModel.profileImgUri.collect { profileImgUri ->
+                binding.ivProfileImage.loadImage(profileImgUri)
+            }
         }
     }
 
     private fun popUpEditNickNameDialog() {
-        val newNickName = dialogViewBinding.etChangeNickName
-
-        val builder = AlertDialog.Builder(requireContext())
-            .setView(dialogViewBinding.root)
-            .setPositiveButton(
-                getString(R.string.my_run_edit_nick_name_dialog_positive)
-            ) { _, _ ->
-                viewModel.updateNickName(newNickName.text.toString())
-            }
-            .setNegativeButton(
-                getString(R.string.my_run_edit_nick_name_dialog_negative)
-            ) { dialog, _ ->
-                dialog.dismiss()
-            }
+        val dialogView = layoutInflater.inflate(R.layout.dialog_edit_nick_name, null)
 
-        builder.show()
+        if (dialogView.parent != null) {
+            builder.show()
+        } else {
+            builder.setView(dialogView)
+                .setPositiveButton(
+                    getString(R.string.my_run_edit_nick_name_dialog_positive)
+                ) { dialog, _ ->
+                    viewModel.updateNickName(dialogView.findViewById<EditText>(R.id.et_change_nick_name).text.toString())
+                    dialog.dismiss()
+                }
+                .setNegativeButton(
+                    getString(R.string.my_run_edit_nick_name_dialog_negative)
+                ) { dialog, _ ->
+                    dialog.dismiss()
+                }.show()
+        }
     }
 }

presentation/src/main/java/com/whyranoid/presentation/myrun/MyRunViewModel.kt

@@ -1,13 +1,14 @@
 package com.whyranoid.presentation.myrun
 
-import androidx.lifecycle.LiveData
-import androidx.lifecycle.MutableLiveData
 import androidx.lifecycle.ViewModel
 import androidx.lifecycle.viewModelScope
 import com.whyranoid.domain.usecase.GetNicknameUseCase
 import com.whyranoid.domain.usecase.GetProfileUriUseCase
 import com.whyranoid.domain.usecase.UpdateNicknameUseCase
 import dagger.hilt.android.lifecycle.HiltViewModel
+import kotlinx.coroutines.flow.MutableStateFlow
+import kotlinx.coroutines.flow.StateFlow
+import kotlinx.coroutines.flow.asStateFlow
 import kotlinx.coroutines.launch
 import javax.inject.Inject
 
@@ -18,13 +19,15 @@ class MyRunViewModel @Inject constructor(
     private val updateNickNameUseCase: UpdateNicknameUseCase
 ) : ViewModel() {
 
-    private val _nickName = MutableLiveData<String>()
-    val nickName: LiveData<String>
-        get() = _nickName
+    private val EMPTY_STRING = ""
 
-    private val _profileImgUri = MutableLiveData<String>()
-    val profileImgUri: LiveData<String>
-        get() = _profileImgUri
+    private val _nickName = MutableStateFlow(EMPTY_STRING)
+    val nickName: StateFlow<String>
+        get() = _nickName.asStateFlow()
+
+    private val _profileImgUri = MutableStateFlow(EMPTY_STRING)
+    val profileImgUri: StateFlow<String>
+        get() = _profileImgUri.asStateFlow()
 
     fun getNickName() {
         viewModelScope.launch {