Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

경북대 Android_유지예_5주차과제_Step2 #67

Open
wants to merge 23 commits into
base: yjy1220
Choose a base branch
from

Conversation

YJY1220
Copy link

@YJY1220 YJY1220 commented Jul 26, 2024

어려웠던 점 & 느낀 점

  • 개인적으로 coroutine을 미리 적용시켜뒀었는데 그때 당시에는 이게 왜 필요한지, 왜 이렇게 해야 돌아가는지 잘몰랐습니다. 목요일 클론코딩 시간에 멘토님으로부터 배우고 나서 보니 제대로 왜 돌아가는지 이해할 수 있어서 새로운 느낌이었습니다.
  • 코드를 보면서 굉장히 더럽다는 생각을 많이 했는데 이번 리팩토링 과정에서 효율적이고 깔끔한 코드를 어떻게 짜는지, 어떠한 방식으로 돌아가는지 잘 알 수 있었던 거 같습니다.

[MVVM 패턴 적용 - LiveData, Databinding]

  • SearchActivity.kt에서 ActivitySearchBinding을 사용하여 DataBinding을 구현하였습니다.
  • MapViewModel.kt에서 MutableLiveData와 LiveData를 사용하여 구현하였습니다.

[비동기 처리 - Coroutine으로 변경]

  • MapViewModel.kt의 searchItems에서 viewModelScope.launch를 사용하여 비동기처리를 완료하였습니다.
  • MapRepository.kt에서 searchItems에 suspend함수로 비동기 작업을 처리하였습니다.

멘토님께서 중점적으로 봐주셨으면 좋겠는 점

  • coroutine이 현업에서도 중점적으로 사용되는지 궁금합니다.
  • android studio를 5주째 써보았는데 갈수록 노트북이 느려지는 느낌입니다. 원래 이런건지 궁금합니다.
  • 항상 감사드립니다!

@acious
Copy link

acious commented Jul 28, 2024

데이터바인딩은 적용되지 않은것으로 보이네요!

@acious
Copy link

acious commented Jul 28, 2024

추가로 Step 1을 선 리뷰하고 머지했는데, 컨플릭이 나는데요. 이거 수정해주시면 머지 가능합니다!

@YJY1220
Copy link
Author

YJY1220 commented Jul 29, 2024

databinding 적용과 step1 수정사항을 반영하여 새로운 pr을 올렸습니다! 다만, 충돌이 너무 많이 나서 웹에서 수정이 안된다고 하여 방법을 찾아보는 중입니다...!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants