[Team04] Stitch&스타크 - 네트워크 통신 및 드래그 앤 드롭, 스와이프 구현 - 최종 PR #210

jminie-o8o · 2022-04-15T06:38:10Z

구현 상황

팝업 메뉴를 리스트 아이템에 연결
- 메뉴는 3가지로 구성 (완료한 일로 이동, 수정하기, 삭제하기)
서버와 네트워크 통신 구현
- Retrofit2, Coroutine, Gson 사용
- 응답 객체 관리를 위한 Result.Success, Result.Error 구현
- 모든 카드 조회
- 모든 기록(History) 조회
- 카드 추가
- 카드 수정
- 카드 삭제
  - 스와이프를 이용한 삭제, 팝업 메뉴를 이용한 삭제
드래그 앤 드롭, 스와이프 구현
- ItemTouchHelper 로 RecyclerView 내에 드래그 앤 드롭, 스와이프 구현
- 리사이클러뷰들 간의 드래그 앤 드롭을 위해 ItemTouchHelper 의 onMove 메서드를 구현을 제거하고 DragListener 로 구현

질문

 private val _todoTask = MutableLiveData<MutableList<TaskDetailResponse>>()
    val todoTask: LiveData<MutableList<TaskDetailResponse>>
        get() = _todoTask
...
...

val tasks = taskRemoteRepository.loadTask()
_todoTask.value.add = tasks.data.todo

위와 같이 직접 LiveData에 add를 시키는 경우 LiveData에 데이터가 추가되지 않고

 private var todoItem: MutableList<TaskDetailResponse> = mutableListOf()
 private val _todoTask = MutableLiveData<MutableList<TaskDetailResponse>>()
    val todoTask: LiveData<MutableList<TaskDetailResponse>>
        get() = _todoTask
...
...

val tasks = taskRemoteRepository.loadTask()
todoItem = tasks.data.todo
_todoTask.value = todoItem

이런식으로 새로운 MutableList를 만든 후 MutableLiveData에 동기화 하는 방법으로 구현했을 때만 정상적으로 LiveData에 데이터가 추가되는 경험을 했습니다.
해당 부분의 이유가 궁금합니다.

Screenshot

2022-04-15.3.33.59.mov

[Android] Stitch - 리사이클러 뷰 목록 팝업메뉴 연결, 메뉴 별 기능 구현

…e-#39 [Android] 스타크 - 레트로핏 카드 저장, 조희, 수정 기능 구현

[Android] stitch - drag&drop&swipe 기능 구현

[Android] Stitch&스타크 - 2주차 수요일 피드백

[Android] Stitch - 카드 모든 조회 네트워크 통신

[Android] 스타크 - 추가 수정 네트워크 통신 구현

[Android] Stitch - 모든 활동 로그기록(history) 네트워크 통신

[Android] Stitch - task 삭제 네트워크 통신

[Android] Stitch - Task 이동하기 네트워크 통신 기능 구현

…into feature/recyclerview-drag-drop-#42

[Android] Stitch - 드래그 앤 드롭

taewooyo · 2022-04-15T08:58:25Z

@renovatio0424 스타크와 스티치와 같이 진행했습니다. drag&drop&swipe, 네트워크 책임과 분리에 대해 따끔한 채찍 부탁드립니다.

renovatio0424 · 2022-04-17T13:24:13Z

질문

이런식으로 새로운 MutableList를 만든 후 MutableLiveData에 동기화 하는 방법으로 구현했을 때만 정상적으로 LiveData에 데이터가 추가되는 경험을 했습니다.
해당 부분의 이유가 궁금합니다.

답변

좋은 질문이네요 ㅎㅎ
이런 경우에는 구현체를 확인해보시는게 좋아요
MutableLiveData 내부에 어떻게 구현이 되어있는지 보면 알 수 있을 것 같네요 ㅎㅎ
혹시 내부 구현 코드를 보고도 이해가 안된다면 DM 주세요~

renovatio0424 · 2022-04-17T13:25:58Z

android/app/src/main/java/com/example/todolist/common/extensions.kt

-fun String.htmlToString() : String {
-    return if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.N) {
-        Html.fromHtml(this, Html.FROM_HTML_MODE_LEGACY).toString()
-    } else {
-        Html.fromHtml(this).toString()
-    }
+fun String.htmlToSpanned() : Spanned {
+    return HtmlCompat.fromHtml(this, FROM_HTML_MODE_LEGACY)
 }


오 버전 분기 코드를 개선했군요!
좋습니다~
xxxCompat 클래스들이 버전 분기 처리를 해주느니 기억해두면 좋겠네요~

renovatio0424 · 2022-04-17T13:26:15Z

android/app/src/main/java/com/example/todolist/model/History.kt

-    val createData: String,
+    val createDateTime: String,


네이밍 개선 좋습니다~

renovatio0424 · 2022-04-17T13:27:09Z

android/app/src/main/java/com/example/todolist/network/Result.kt

+sealed class Result<out T : Any> {
+    data class Success<out T : Any>(val data: T) : Result<T>()
+    data class Error(val error: String) : Result<Nothing>()
+}


오 sealed class 를 사용했네요 ??
sealed class 를 사용했던 이유가 무엇인가요?

renovatio0424 · 2022-04-17T13:27:37Z

android/app/src/main/java/com/example/todolist/network/RetrofitAPI.kt

+import retrofit2.converter.gson.GsonConverterFactory
+
+object RetrofitAPI {
+    private const val BASE_URL = "http://www.louie-03.com/"


https 를 사용하지 않고 http 를 사용했던 이유가 무엇인가요?

renovatio0424 · 2022-04-17T13:33:43Z

android/app/src/main/java/com/example/todolist/ui/DragListener.kt

+import androidx.recyclerview.widget.RecyclerView
+import com.example.todolist.R
+
+class DragListener : View.OnDragListener {


DragListener 이름만 보았을땐 모든 DragEvent 를 핸들링해줄 수 있는 Listener 같네요
이렇게 네이밍과 실제 구현체의 동작이 다르면 혼동을 일으킬 수 있어요 ㅎㅎ
이럴땐 같이 작업하는 팀원들과 어떻게 하면 혼동이 되지 않을지 이야기해보면 좋겠네요 ㅎㅎ

renovatio0424 · 2022-04-17T13:34:05Z

android/app/src/main/java/com/example/todolist/ui/ItemTouchCallback.kt

+import kotlin.math.max
+import kotlin.math.min
+
+class ItemTouchCallback(private var clamp: Float) : ItemTouchHelper.Callback() {


DragListener와 마찬가지네요 ㅎㅎ

renovatio0424 · 2022-04-17T13:35:31Z

android/app/src/main/java/com/example/todolist/ui/ItemTouchHelperListener.kt

+interface ItemTouchHelperListener {
+    fun onItemMove(fromPosition: Int, toPosition: Int): Boolean
+
+    fun onItemSwipe(position: Int)
+
+    fun add(type: Int, index: Int, task: TaskDetailResponse)
+
+    fun remove(index: Int, task: TaskDetailResponse)
+}


인터페이스 구현으로 미리 api 구조를 설계하는 게 좋네요 ㅎㅎ
미리 설계하고 만드는 것도 좋지만 만들어진 클래스들을의 공통 API 를 뽑아 인터페이스를 만들어보는것도 연습해보시면 도움이 많이 되실거에요 ㅎㅎ

renovatio0424

리뷰 확인해주세요~

taewooyo and others added 30 commits April 11, 2022 13:23

feat: 팝업메뉴 리스트 아이템에 연결, task 삭제, 이동 구현-#35

5e669dd

feat: Task 수정 구현-#35

25ea26d

Merge pull request #43 from bn-tw2020/feature/popup_menu-#35

5d44ced

[Android] Stitch - 리사이클러 뷰 목록 팝업메뉴 연결, 메뉴 별 기능 구현

feat: 카드 저장 조회 기능 API 구현

e8af100

feat: 카드 수정 기능 API 구현

f1ed922

fix: 실행오류제거-#39

0c6e127

Merge pull request #56 from bn-tw2020/feature/retrofit_network_packag…

161060a

…e-#39 [Android] 스타크 - 레트로핏 카드 저장, 조희, 수정 기능 구현

feat: drag-drop-swipe-#40

50a3d75

Merge pull request #57 from bn-tw2020/feature/drag-drop-#40

fd30c11

[Android] stitch - drag&drop&swipe 기능 구현

feat: ItemTouchCallback 책임 분리-#59

4d297df

fix: setOnTouchListener 경고문고해결-#59

09cf433

chore: LoadTaskResponse.kt 불필요한 파일제거-#59

2b86c38

Merge pull request #60 from bn-tw2020/feature/feedback-week2-1-#59

aae4853

[Android] Stitch&스타크 - 2주차 수요일 피드백

feat: todo, inProgress, done 활동 조회-#62

32fc36a

feat: todo, inProgress, done 활동 조회-#62

573f0e8

feat: 히스토리 네트워크 통신-#65

edf6f70

chore: 불필요한 로그삭제-#65

a8d32cb

feat: delete-api-#66

0b0e3af

feat: move-api-#67

2635df0

feat: 리사이클러뷰간 데이터 이동(메모리 이동)-#42

0ad17e2

Merge pull request #72 from bn-tw2020/feature/활동_조회-#62

e3a2a90

[Android] Stitch - 카드 모든 조회 네트워크 통신

feat: 추가 수정 네트워크 통신 구현-#63

e9f5045

Merge pull request #74 from jminie-o8o/feature/add-delete-api-#63

01998b7

[Android] 스타크 - 추가 수정 네트워크 통신 구현

feat: history 기능 + 추가,수정 동기화-#65

bffef65

Merge pull request #76 from bn-tw2020/feature/history-api-#65

1690d23

[Android] Stitch - 모든 활동 로그기록(history) 네트워크 통신

chore: 동기화-#66

9369a9d

Merge pull request #77 from bn-tw2020/feature/delete-api-#66

fe8f161

[Android] Stitch - task 삭제 네트워크 통신

feat: 이동하기 네트워크 통신 요청 객체 수정 및 동기화-#67

8ea5fd3

Merge pull request #79 from bn-tw2020/feature/task-move-api-#67

78e4762

[Android] Stitch - Task 이동하기 네트워크 통신 기능 구현

Merge branch 'test_android' of https://github.com/bn-tw2020/todo-list …

20bb8ca

…into feature/recyclerview-drag-drop-#42

taewooyo and others added 2 commits April 15, 2022 14:50

feat: 드래그 앤 드롭 에러 수정(메모리 관리)-#42

ac8a775

Merge pull request #80 from bn-tw2020/feature/recyclerview-drag-drop-#42

70169cf

[Android] Stitch - 드래그 앤 드롭

jminie-o8o changed the title ~~Test android~~ [Team04] Stitch&스타크 - 네트워크 통신 및 드래그 앤 드롭, 스와이프 구현 - 최종 PR Apr 15, 2022

hwicode requested a review from renovatio0424 April 15, 2022 06:44

hwicode added the review-android Further information is requested label Apr 15, 2022

hwicode assigned jminie-o8o Apr 15, 2022

renovatio0424 reviewed Apr 17, 2022

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[Team04] Stitch&스타크 - 네트워크 통신 및 드래그 앤 드롭, 스와이프 구현 - 최종 PR #210

[Team04] Stitch&스타크 - 네트워크 통신 및 드래그 앤 드롭, 스와이프 구현 - 최종 PR #210

jminie-o8o commented Apr 15, 2022

taewooyo commented Apr 15, 2022

renovatio0424 commented Apr 17, 2022

renovatio0424 Apr 17, 2022

renovatio0424 Apr 17, 2022

renovatio0424 Apr 17, 2022

renovatio0424 Apr 17, 2022

renovatio0424 Apr 17, 2022

renovatio0424 Apr 17, 2022

renovatio0424 Apr 17, 2022

renovatio0424 left a comment

[Team04] Stitch&스타크 - 네트워크 통신 및 드래그 앤 드롭, 스와이프 구현 - 최종 PR #210

Are you sure you want to change the base?

[Team04] Stitch&스타크 - 네트워크 통신 및 드래그 앤 드롭, 스와이프 구현 - 최종 PR #210

Conversation

jminie-o8o commented Apr 15, 2022

구현 상황

질문

Screenshot

taewooyo commented Apr 15, 2022

renovatio0424 commented Apr 17, 2022

질문

답변

renovatio0424 Apr 17, 2022

Choose a reason for hiding this comment

renovatio0424 Apr 17, 2022

Choose a reason for hiding this comment

renovatio0424 Apr 17, 2022

Choose a reason for hiding this comment

renovatio0424 Apr 17, 2022

Choose a reason for hiding this comment

renovatio0424 Apr 17, 2022

Choose a reason for hiding this comment

renovatio0424 Apr 17, 2022

Choose a reason for hiding this comment

renovatio0424 Apr 17, 2022

Choose a reason for hiding this comment

renovatio0424 left a comment

Choose a reason for hiding this comment