마이페이지 #74
마이페이지 #74
Conversation
| val dogs: List<Dog> = emptyList(), | ||
| ) | ||
|
|
||
| data class Dog( |
There was a problem hiding this comment.
ui model인가요?
패키지 분리하거나 나중에 수정할 코드라면 TODO로 명시해도 좋습니다 !!
| fetchDummy() | ||
| } | ||
|
|
||
| private fun fetchDummy() { |
There was a problem hiding this comment.
fetchDummy에 실패할 경우 or 리스트가 없는 경우에 대한 케이스도 구상되어 있을까요?
There was a problem hiding this comment.
에러 처리에 대한 이야기를 더 자세히 나눠야할 것 같아 구현하지 않았습니다!!
| @@ -31,3 +31,27 @@ fun ImageView.bindProfile1000(bitmap: Bitmap?) { | |||
| .transform(CenterCrop(), RoundedCorners(1000)) | |||
| .into(this) | |||
| } | |||
|
|
|||
| @BindingAdapter("glideProfile1000") | |||
| <shape xmlns:android="http://schemas.android.com/apk/res/android" | ||
| android:shape="rectangle"> | ||
|
|
||
| <!-- <gradient--> |
There was a problem hiding this comment.
헉 지워야할 것 같습니다! 7e4b064
| data class MyPageUiState( | ||
| val nickname: String = "", | ||
| val email: String = "", | ||
| val profilePath: String? = null, | ||
| val dogs: List<Dog> = emptyList(), | ||
| ) |
There was a problem hiding this comment.
nickname, email에 default-value 가 있는 다른 이유가 있나요?
There was a problem hiding this comment.
닉네임과 이메일에 null값이 들어와도 화면에는 보이도록 하기 위해서 위와 같이 구현했는데, 이건 다같이 의논해봐도 좋을 것 같아요!!
| internal object DogItemDiffCallback : DiffUtil.ItemCallback<Dog>() { | ||
| override fun areItemsTheSame( | ||
| oldItem: Dog, |
There was a problem hiding this comment.
혹시 internal companion object 대신 internal object를 쓰는 이유가 있을까요..? 다들 companion object를 많이 쓰던데 object로 두신 이유가 궁금해요!
There was a problem hiding this comment.
멀티모듈이 아닌 시점에서는 수정해야할 것 같습니다!!
이슈
개발 사항