전남대 Android_최영빈_5주차 과제_step2 #81
Open
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
- 기본 코드 준비
- 구현할 기능 목록 정리
- DataSearchActivity에서 databinding 사용 - SearchViewModel가 LiveData 관리하도록 코드 수정
- 폴더명을 activiy에서 view로 변경하면서 데이터 바인딩 관련 미완성 코드가 포함됨 - 오류 방지를 위해 관련 변경사항 커밋함 (참고 커밋: 514a764) - 주의사항 : 데이터 바인딩 부분은 현재 미완성 상태로 추가 수정이 필요함
- 에러 화면에 에러 메시지가 표시되지 않는 문제 해결 참고 커밋: b9171d4 변경 사항: [HomeMapActivity] - 복잡도를 낮추기 위해서 view가 하나의 viewModel을 사용하도록 수정 [MapErrorActivity] - 데이터 바인딩 관련 코드 추가 - 기능별로 함수 분리 [MapErrorViewModel] - setErrorMsg로 함수명 변경 - 에러 메세지가 null이 아닌지 검사하는 코드 추가, 에러 메세지의 공백을 없애는 코드 추가 [activity_map_error] - 데이터 바인딩 관련 코드 추가 [MapErrorViewModelTest] - 바뀐 setErrorMsg()함수 반영
- 코드 통일성을 위해 의존성 주입 관련 코드 추가
참고 사항: - 파일의 폴더를 변경하는 과정에서 의도치 않게 파일이 복사 된 것 같음 - 관련 커밋: 3531fc6
omjoonkim
requested changes
Jul 28, 2024
| app:layout_constraintTop_toTopOf="parent" /> | ||
| <data> | ||
| <variable | ||
| name="homeMapActivity" |
There was a problem hiding this comment.
이 variable을 사용하는 곳이 있나요?
뷰를 표현하기 위한 데이터를 정의하고, xml에서 해당 데이터를 직접사용하여 뷰가 변화되도록 만들어보세요.
|
|
||
| class SearchHistoryRepository(context: Context) { | ||
|
|
||
| private val searchHistoryDao: SearchHistoryDao = |
There was a problem hiding this comment.
context를 통해 DB에 접근하는 것이 아닌, DAO를 생성자 주입으로 받도록 의존성 주입 코드를 작성해보세요.
|
1주차 과제 코멘트도 이 PR에 적용 부탁드립니다~! |
|
step1에 Hilt 적용하였습니다. 피드백을 토대로 단계별로 진행했어야 했는데 룰을 지키지 못해 정말 죄송합니다! 그리고 commit시에도 최소 단위로 커밋을 했어야 하는데, 앞으로 더 유의해서 진행하겠습니다. 정말 죄송합니다! |
…archActivity 참고 커밋 : 518a06c
…ation Information
|
피드백 반영했습니다! |
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.
코드 작성하면서 어려웠던 점
멘토님이 중점적으로 리뷰해 주셨으면 하는 부분
현재 수정해야 할 부분이 많이 남은 상황으로 보입니다. 이른 시일 내에 보충하겠습니다.
😰 죄송합니다!