-
Notifications
You must be signed in to change notification settings - Fork 0
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
[feat] [#58] 관심 부스 로컬 데이터베이스 환경 구축 및 CRUD 연동 #68
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
MapScreen 과 BoothLocationScreen 내에 SystemUiController 관련 설정 코드 제거
사용하지 않는 라이브러리 의존성 제거
androidx-room 을 사용
code style check
wjdtkdgns777
approved these changes
Apr 13, 2024
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
수고 많으셨습니다!! 저도 해당부분 공부해보고 있겠습니다
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
P.S
원래는 pull to refresh 를 통해 변경 내역을 반영하는 것이 기획이었으나, 데이터가 로컬 데이터이기도 하구, 메뉴 화면과의 변경 내역 반영의 통일성을 맞추기 위해 pull to refresh 가 아닌, 변경된 내역을 즉시 반영하는 방식으로 구현하였습니다. (석준님과 얘기된 내용)
Room 내에 데이터를 flow 스트림을 통해 뷰모델이 구독을 하는 방식을 사용하여, 변경이 발생한 경우, 이를 변화를 즉각 반영하는 식으로 구현하였습니다. 단, 바로 아이템을 삭제할 경우 북마크 아이콘 버튼의 변화를 확인할 수 없으므로, 북마크 버튼을 먼저 해제하고(아이콘 변경), 0.5초 후에 해당 아이템이 삭제되는 식으로 구현하였습니다. 0.5초가 너무 짧다고 생각된다면 더 늘려도 좋을 것 같습니다.
아이템이 삭제되고 추가되는 애니메이션 같은 경우에는 현재는 아이템을 하나밖에 추가하고, 삭제할수없어서 제대로 확인이 불가능한데, 이후 관심 부스를 복수로 추가할 수 있을때, 다시 확인해서 어색한 부분은 해결하면 될 것 같습니다.
module이 추가되고, 구글 권장 아키텍처를 채택하면서 각 모듈간(data, database, domain, network)의 model 을 다루는 것이 복잡해졌는데, nia 의 경우 순수한 model을 domain 내에 entity 가 아닌 model 이라는 새로운 모듈 내에 정의하여 관리하던데, 해당 내용에 대해선 아직 학습이 부족하여, 좀 더 알아보겠습니다. 클린 아키텍처를 주로 사용하다가, 구글 권장 아키텍처를 사용하려니 다소 차이점이 존재하는거 같네요