[테스트] 샘플 앱에서 같은 데이터를 바라보는 상황에서 Player를 remove했을 때 발생하는 ConcurrentModificationException 이슈 처리

Author: yeon-kyu

app/src/main/java/com/yeonkyu/holdableswipehandler/ui/list_adapter_recyclerview/ListAdapterRecyclerViewFragment.kt

@@ -20,7 +20,7 @@ class ListAdapterRecyclerViewFragment : Fragment() {
     private lateinit var binding : FragmentRecyclerviewBinding
     private lateinit var adapter : MyListAdapter
 
-    private lateinit var playerList : ArrayList<Player>
+    private lateinit var playerList : MutableList<Player>
 
     override fun onCreateView(
         inflater: LayoutInflater,
@@ -30,7 +30,7 @@ class ListAdapterRecyclerViewFragment : Fragment() {
         binding = DataBindingUtil.inflate(inflater,R.layout.fragment_recyclerview, container, false)
         binding.lifecycleOwner = this
 
-        playerList = DataLoader.initPlayer(requireContext())
+        playerList = DataLoader.initPlayer(requireContext()).toMutableList()
 
         setUpRecyclerView()
         setUpHoldableSwipeHandler()

app/src/main/java/com/yeonkyu/holdableswipehandler/ui/normal_recyclerview/NormalAdapter.kt

@@ -11,10 +11,10 @@ import com.yeonkyu.holdableswipehandler.databinding.ItemPlayerBinding
 
 class NormalAdapter : RecyclerView.Adapter<PlayerViewHolder>() {
 
-    private lateinit var playerList : ArrayList<Player>
+    private lateinit var playerList : MutableList<Player>
 
-    fun setPlayer(list : ArrayList<Player>) {
-        playerList = list
+    fun setPlayer(list : List<Player>) {
+        playerList = list.toMutableList()
     }
 
     fun removePlayer(position : Int) {

app/src/main/java/com/yeonkyu/holdableswipehandler/ui/normal_recyclerview/NormalRecyclerViewFragment.kt

@@ -13,13 +13,16 @@ import androidx.recyclerview.widget.DividerItemDecoration
 import androidx.recyclerview.widget.ItemTouchHelper
 import com.yeonkyu.HoldableSwipeHelper.deprecated.HoldableSwipeHelper
 import com.yeonkyu.HoldableSwipeHelper.SwipeButtonAction
+import com.yeonkyu.holdableswipehandler.data.Player
 import com.yeonkyu.holdableswipehandler.databinding.FragmentRecyclerviewBinding
 
 class NormalRecyclerViewFragment : Fragment() {
 
     private lateinit var binding : FragmentRecyclerviewBinding
     private lateinit var adapter : NormalAdapter
 
+    private lateinit var playerList : List<Player>
+
     override fun onCreateView(
         inflater: LayoutInflater,
         container: ViewGroup?,
@@ -28,6 +31,8 @@ class NormalRecyclerViewFragment : Fragment() {
         binding = DataBindingUtil.inflate(inflater,R.layout.fragment_recyclerview, container, false)
         binding.lifecycleOwner = this
 
+        playerList = DataLoader.initPlayer(requireContext()).toList()
+
         setUpRecyclerView()
         setUpHoldableSwipeHandler()
 
@@ -38,9 +43,8 @@ class NormalRecyclerViewFragment : Fragment() {
         val swipeHelper = HoldableSwipeHelper(requireContext(), object : SwipeButtonAction {
             override fun onClickFirstButton(absoluteAdapterPosition: Int) {
                 adapter.removePlayer(absoluteAdapterPosition)
-                //adapter.notifyDataSetChanged()
                 adapter.notifyItemRemoved(absoluteAdapterPosition)
-                adapter.notifyItemRangeChanged(absoluteAdapterPosition,adapter.itemCount)
+                adapter.notifyItemRangeChanged(absoluteAdapterPosition, adapter.itemCount)
             }
         })
 
@@ -56,7 +60,7 @@ class NormalRecyclerViewFragment : Fragment() {
     private fun setUpRecyclerView() {
         binding.recyclerView.layoutManager = LinearLayoutManager(context, LinearLayoutManager.VERTICAL, false)
         adapter = NormalAdapter().apply {
-            setPlayer(DataLoader.initPlayer(requireContext()))
+            setPlayer(playerList)
         }
         binding.recyclerView.adapter = adapter
     }

app/src/main/res/values-night/themes.xml

@@ -1,13 +1,13 @@
 <resources xmlns:tools="http://schemas.android.com/tools">
     <!-- Base application theme. -->
-    <style name="Theme.HoldableSwipeHandler" parent="Theme.MaterialComponents.DayNight.DarkActionBar">
+    <style name="Theme.HoldableSwipeHandler" parent="Theme.AppCompat.Light.NoActionBar">
         <!-- Primary brand color. -->
-        <item name="colorPrimary">@color/purple_200</item>
+        <item name="colorPrimary">@color/purple_500</item>
         <item name="colorPrimaryVariant">@color/purple_700</item>
-        <item name="colorOnPrimary">@color/black</item>
+        <item name="colorOnPrimary">@color/white</item>
         <!-- Secondary brand color. -->
         <item name="colorSecondary">@color/teal_200</item>
-        <item name="colorSecondaryVariant">@color/teal_200</item>
+        <item name="colorSecondaryVariant">@color/teal_700</item>
         <item name="colorOnSecondary">@color/black</item>
         <!-- Status bar color. -->
         <item name="android:statusBarColor" tools:targetApi="l">?attr/colorPrimaryVariant</item>

app/src/main/res/values/themes.xml

@@ -1,6 +1,6 @@
 <resources xmlns:tools="http://schemas.android.com/tools">
     <!-- Base application theme. -->
-    <style name="Theme.HoldableSwipeHandler" parent="Theme.MaterialComponents.DayNight.DarkActionBar">
+    <style name="Theme.HoldableSwipeHandler" parent="Theme.AppCompat.Light.NoActionBar">
         <!-- Primary brand color. -->
         <item name="colorPrimary">@color/purple_500</item>
         <item name="colorPrimaryVariant">@color/purple_700</item>