🔀 #79 from boostcampwm-2022/feat/community_create_running_post

인증글 작성하기 버그 수정, 운동 내역 단일 선택 기능 버그 수정, 전체적인 UI 수정

Author: peter-j0y

presentation/src/main/java/com/whyranoid/presentation/community/runningpost/CommunityRunningHistoryAdapter.kt

@@ -1,5 +1,6 @@
 package com.whyranoid.presentation.community.runningpost
 
+import android.content.res.ColorStateList
 import android.graphics.Color
 import android.view.LayoutInflater
 import android.view.View
@@ -16,6 +17,7 @@ class CommunityRunningHistoryAdapter(private val selectRunningHistoryListener: R
     ListAdapter<RunningHistoryUiModel, CommunityRunningHistoryViewHolder>(
         MyRunningHistoryDiffCallback()
     ) {
+
     override fun onCreateViewHolder(
         parent: ViewGroup,
         viewType: Int
@@ -26,7 +28,7 @@ class CommunityRunningHistoryAdapter(private val selectRunningHistoryListener: R
     }
 
     override fun onBindViewHolder(holder: CommunityRunningHistoryViewHolder, position: Int) {
-        holder.bind(getItem(position))
+        holder.bind(getItem(position), position)
     }
 }
 
@@ -36,30 +38,49 @@ class CommunityRunningHistoryViewHolder(
 ) : RecyclerView.ViewHolder(view) {
     private val binding = ItemRunningHistoryBinding.bind(view)
 
-    fun bind(runningHistory: RunningHistoryUiModel) {
+    fun bind(runningHistory: RunningHistoryUiModel, itemPosition: Int) {
         // 아이템이 선택된 개체인지 확인
 
         binding.runningHistory = runningHistory
+        val isSelected = listener.checkRunningHistoryId(itemPosition)
 
-        binding.root.setBackgroundColor(Color.TRANSPARENT)
+        if (isSelected) {
+            binding.root.setBackgroundResource(R.drawable.background_rounded)
+            binding.root.backgroundTintList = ColorStateList.valueOf(
+                ContextCompat.getColor(
+                    binding.root.context,
+                    R.color.mogakrun_secondary
+                )
+            )
+        } else {
+            // 선택되어 있지 않던 아이템이면
+            binding.root.setBackgroundColor(Color.TRANSPARENT)
+        }
 
         binding.root.setOnClickListener {
-            val isSelected = listener.checkRunningHistoryId(runningHistory)
+            val isAlreadySelected = listener.checkRunningHistoryId(itemPosition)
 
-            // 이미 선택되어 있던 아이템이면
-            if (isSelected) {
+            // 이미 선택되어 있던 아이템을 누른 경우
+            if (isAlreadySelected) {
                 listener.unSelectRunningHistory()
                 it.setBackgroundColor(Color.TRANSPARENT)
             } else {
-                // 선택되어 있지 않던 아이템이면
-                listener.selectRunningHistory(runningHistory)
-                it.setBackgroundColor(
+                // 선택되어 있지 않던 아이템을 누른 경우
+                listener.selectRunningHistory(runningHistory, itemPosition)
+                binding.root.setBackgroundResource(R.drawable.background_rounded)
+                binding.root.backgroundTintList = ColorStateList.valueOf(
                     ContextCompat.getColor(
                         binding.root.context,
-                        R.color.gray
+                        R.color.mogakrun_secondary
                     )
                 )
             }
         }
     }
 }
+
+interface RunningHistoryItemListener {
+    fun checkRunningHistoryId(itemPosition: Int): Boolean
+    fun selectRunningHistory(runningHistory: RunningHistoryUiModel, itemPosition: Int)
+    fun unSelectRunningHistory()
+}

presentation/src/main/java/com/whyranoid/presentation/community/runningpost/CreateRunningPostViewModel.kt

@@ -28,7 +28,7 @@ class CreateRunningPostViewModel @Inject constructor(
 
     val createPostButtonEnableState: StateFlow<Boolean>
         get() = runningPostContent.map { content ->
-            content != null
+            content.isNullOrBlank().not()
         }.stateIn(
             initialValue = false,
             started = SharingStarted.WhileSubscribed(5000),

presentation/src/main/java/com/whyranoid/presentation/community/runningpost/RunningHistoryItemListener.kt

@@ -96,7 +96,11 @@ internal class SelectRunningHistoryFragment :
                         true
                     }
                     R.id.warning_select_running_history_button -> {
-                        Snackbar.make(binding.root, getString(R.string.community_select_running_history_snack_bar), Snackbar.LENGTH_SHORT).show()
+                        Snackbar.make(
+                            binding.root,
+                            getString(R.string.community_select_running_history_snack_bar),
+                            Snackbar.LENGTH_SHORT
+                        ).show()
                         true
                     }
                     else -> {
@@ -107,14 +111,27 @@ internal class SelectRunningHistoryFragment :
         }
     }
 
-    override fun checkRunningHistoryId(runningHistory: RunningHistoryUiModel): Boolean =
-        viewModel.getSelectedRunningHistory()?.historyId == runningHistory.historyId
+    override fun checkRunningHistoryId(itemPosition: Int): Boolean {
+        // 아무것도 선택이 되지 않은 상황이거나 선택한 운동내역이 이미 선택되어있는 운동내역과 다를 때 false 반환 -> 일반 배경색
+        // true 반환 -> 선택된 배경색
+        return (viewModel.selectedItemPosition.value == NOTHING_SELECTED_ITEM_POSITION || viewModel.selectedItemPosition.value != itemPosition).not()
+    }
 
-    override fun selectRunningHistory(runningHistory: RunningHistoryUiModel) {
+    override fun selectRunningHistory(runningHistory: RunningHistoryUiModel, itemPosition: Int) {
+        // 현재 선택되어 있는 아이템의 position을 가져옴
+        val currentSelectedItemPosition = viewModel.selectedItemPosition.value
         viewModel.setSelectedRunningHistory(runningHistory)
+        viewModel.setSelectedItemPosition(itemPosition)
+        // 이전에 선택되어 있던 아이템에게 배경색을 다시 불러오도록 명령
+        runningHistoryAdapter.notifyItemChanged(currentSelectedItemPosition)
     }
 
     override fun unSelectRunningHistory() {
         viewModel.setSelectedRunningHistory(null)
+        viewModel.setSelectedItemPosition(NOTHING_SELECTED_ITEM_POSITION)
+    }
+
+    companion object {
+        private const val NOTHING_SELECTED_ITEM_POSITION = -1
     }
 }

presentation/src/main/java/com/whyranoid/presentation/community/runningpost/SelectRunningHistoryFragment.kt

@@ -26,11 +26,14 @@ class SelectRunningHistoryViewModel @Inject constructor(private val getRunningHi
     val runningHistoryListState: StateFlow<UiState<List<RunningHistoryUiModel>>>
         get() = _runningHistoryListState.asStateFlow()
 
-    // TODO 인증글 작성 넘어가도록 하는 더미데이터 -> 인증글 작성 로직이 완료되면 삭제하겠습니다
-    private val _selectedRunningHistory = MutableStateFlow<RunningHistoryUiModel?>(RunningHistoryUiModel("seungmin_history_id", 8995875L, 2452, 24524L, 134L, 124.0, 23.0))
+    private val _selectedRunningHistory = MutableStateFlow<RunningHistoryUiModel?>(null)
     val selectedRunningHistory: StateFlow<RunningHistoryUiModel?>
         get() = _selectedRunningHistory.asStateFlow()
 
+    private val _selectedItemPosition = MutableStateFlow(NOTHING_SELECTED_ITEM_POSITION)
+    val selectedItemPosition: StateFlow<Int>
+        get() = _selectedItemPosition.asStateFlow()
+
     private fun getRunningHistoryList() {
         viewModelScope.launch {
             _runningHistoryListState.value = UiState.Loading
@@ -53,4 +56,12 @@ class SelectRunningHistoryViewModel @Inject constructor(private val getRunningHi
     fun setSelectedRunningHistory(runningHistoryUiModel: RunningHistoryUiModel?) {
         _selectedRunningHistory.value = runningHistoryUiModel
     }
+
+    fun setSelectedItemPosition(itemPosition: Int) {
+        _selectedItemPosition.value = itemPosition
+    }
+
+    companion object {
+        private const val NOTHING_SELECTED_ITEM_POSITION = -1
+    }
 }

presentation/src/main/java/com/whyranoid/presentation/community/runningpost/SelectRunningHistoryViewModel.kt

@@ -77,7 +77,7 @@ class CalendarDayBinder(
                 container.binding.root.background =
                     ContextCompat.getDrawable(
                         calendarView.context,
-                        R.drawable.calendar_kong
+                        R.drawable.kong
                     )
             }
         }

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

@@ -0,0 +1,9 @@
+<?xml version="1.0" encoding="utf-8"?>
+<shape xmlns:android="http://schemas.android.com/apk/res/android"
+    android:padding="10dp"
+    android:shape="rectangle" >
+    <solid android:color="@color/mogakrun_primary" />
+    <corners
+        android:topLeftRadius="12dp"
+        android:topRightRadius="12dp" />
+</shape>

presentation/src/main/res/drawable/calendar_kong.png

@@ -1,4 +1,4 @@
-<vector android:height="24dp" android:tint="#ACD182"
+<vector android:height="24dp" android:tint="@color/gray"
     android:viewportHeight="24" android:viewportWidth="24"
     android:width="24dp" xmlns:android="http://schemas.android.com/apk/res/android">
     <path android:fillColor="@android:color/white" android:pathData="M19.77,5.03l1.4,1.4L8.43,19.17l-5.6,-5.6 1.4,-1.4 4.2,4.2L19.77,5.03m0,-2.83L8.43,13.54l-4.2,-4.2L0,13.57 8.43,22 24,6.43 19.77,2.2z"/>

presentation/src/main/res/drawable/community_create_group_edit_text_background.xml

@@ -1,4 +1,4 @@
-<vector android:height="24dp" android:tint="#ACD182"
+<vector android:height="24dp" android:tint="?attr/colorOnPrimary"
     android:viewportHeight="24" android:viewportWidth="24"
     android:width="24dp" xmlns:android="http://schemas.android.com/apk/res/android">
     <path android:fillColor="@android:color/white" android:pathData="M9,16.2L4.8,12l-1.4,1.4L9,19 21,7l-1.4,-1.4L9,16.2z"/>