전남대 Android_장수민_5주차 과제(Step2) #92
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
omjoonkim
approved these changes
Jul 28, 2024
| private val preferenceManager: PreferenceManager | ||
| ) : ViewModel() { | ||
|
|
||
| private val _placeName = MutableLiveData<String>("") |
There was a problem hiding this comment.
뷰를 표현하기 위한 데이터 클래스를 정의하여 사용하면 뷰모델의 코드가 좀 더 간결해질 수 있습니다. :)
| } | ||
| } | ||
|
|
||
| fun getLastCategory(input: String): String { |
There was a problem hiding this comment.
nit; 보통 뷰모델의 퍼블릭 함수는 리턴값이 없는 호출하는 형태의 함수만 허용합니다.
| name="viewModel" | ||
| type="campus.tech.kakao.map.SearchViewModel" /> | ||
| <variable | ||
| name="activity" |
There was a problem hiding this comment.
activity는 사용되는 곳이 없어보이는데 인자로 받은 이유가 있으신가요?
There was a problem hiding this comment.
다른 코드들을 참고하면서 작성하여 따라만들게 된 것 같습니다!!! 사용하지 않는 코드 삭제하겠습니다!
|
5주차 과제 진행하시느라 고생하셨습니다. 1단계 PR의 코멘트처럼 Room적용 부분만 좀 더 추가 부탁드릴게요~!
코드 잘 작성해주신 것으로 보여집니다. xml에서 모든 로직을 수행하는 것이 아닌, BindingAdpater를 사용하거나 혹은 앞서 언급한 ViewState등의 데이터 클래스의 함수를 정의하여 xml에서는 단순히 호출만 하는 형태로 사용하면 xml의 코드를 간결하게 만들 수 있습니다. |
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.
5주차 과제
1단계
기능 사항
2단계
step1 pr을 올리고 바로 step2를 작업했습니다!!
step1 PR
코루틴은 이전에 변경했어서 따로 변경하지 않았습니다!!
Room을 적용하려고 하는데, 기존 SharedPreference에 저장하던 데이터를 Room으로 변경하면 되는지 궁금합니다!!
💀어려웠던 부분
데이터바인딩을 올바르게 적용했는지 궁금합니다. 🥲
MVVM에 어긋난 부분이 있는지 궁금합니다.
더 개선할 부분이 있다면 알려주시면 감사드리겠습니다!!!
📷앱 사진(저번 과제와 비교했을 때 UI가 달라진 부분은 없습니다!)
온다방 까눌레 추천합니다 👍
항상 코드리뷰 해주셔서 감사합니다!!