Merge pull request #56 from YAPP-Github/refactor/calendar

[ Fix ] : 기록화면 코드리뷰 반영

Author: LeeOhHyung

data/src/main/java/com/yapp/android2/data/repository/RecordRepositoryImpl.kt

@@ -5,7 +5,6 @@ import com.yapp.android2.domain.entity.Product
 import com.yapp.android2.domain.entity.User
 import com.yapp.android2.domain.repository.record.Item
 import com.yapp.android2.domain.repository.record.RecordRepository
-import kotlinx.coroutines.CoroutineScope
 import kotlinx.coroutines.Dispatchers
 import kotlinx.coroutines.withContext
 import javax.inject.Inject
@@ -18,11 +17,11 @@ class RecordRepositoryImpl @Inject constructor(
         val cacheDates = mutableMapOf<String, List<String>>()
 
 
-        val savingList = withContext(CoroutineScope(Dispatchers.IO).coroutineContext) {
-                recordRemoteDataSource.fetchRecords(recordMM)
-            }
+        val savingList = withContext(Dispatchers.IO) {
+            recordRemoteDataSource.fetchRecords(recordMM)
+        }
 
-        val summaryList = withContext(CoroutineScope(Dispatchers.IO).coroutineContext) {
+        val summaryList = withContext(Dispatchers.IO) {
             recordRemoteDataSource.fetchSummaryRecord(recordMM)
         }
 

presentation/record/src/main/java/com/yapp/android2/record/RecordViewModel.kt

@@ -26,16 +26,12 @@ class RecordViewModel @Inject constructor(
     private val _action = MutableSharedFlow<Action>()
     val action: SharedFlow<Action>
         get() = _action.asSharedFlow()
-
-    var recordJob: Job? = null
-
+    
     /**
      * @param date format yyyyMM
      */
     fun fetchRecords() {
-        recordJob?.cancel()
-
-        recordJob = viewModelScope.launch {
+        viewModelScope.launch {
             try {
                 val response = getRecordUseCase.execute(Unit)
 
@@ -52,12 +48,6 @@ class RecordViewModel @Inject constructor(
         }
     }
 
-
-    override fun onCleared() {
-        recordJob?.cancel()
-        super.onCleared()
-    }
-
     sealed class Action {
         object ItemClick : Action()
     }