-
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
refactor: 피드 조회 뷰 api 수정 #213
Conversation
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.
lgtm
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.
산군이 돌아와따
@SerialName("title") | ||
val title: String?, |
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.
a: 타이틀은 왜 널러블하게 두신건가요?!
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.
서버가 널러블하게 내려주니까요 ..?
@@ -16,6 +16,7 @@ data class FeedModel( | |||
val categories: String = relevantCategories.joinToString(prefix = "", postfix = ""), | |||
) { | |||
val formattedCreatedDate: String = " · $createdDate" | |||
val isEmptyOfRelevantCategories: Boolean = relevantCategories.isEmpty() |
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.
c: 그냥 isEmptyRelevantCategories
는 어떠신지요? 그쪽이 더 읽기 매끄러운 느낌?!
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.
음... 그대로 쓰는게
val ratingCount: Int?, | ||
) { | ||
|
||
val isNothing: Boolean = id == null |
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.
a: isNothing
은 어떤 역할을 하나요? 변수명으로 봐서는 아무것도 아니다! 라는 느낌인데 🤔
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.
소설 모델 자체가 없다라는 뜻으로 만들어두었습니다.
변수 이름만 보기보단 객체 내부의 '객체 상태'값이니 객체의 이름과도 같이 유추해보면 좋을 것 같습니다.
사용부분에선 FeedModel.novel.isNothing으로 쓸 수 있긴한데 더욱 좋은 이름 있으면 추천좀용
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.
존재하지 않는다라는 느낌으로 가면 어떨까요
- 카테고리 변경할 때 피드 호출하는 함수 아예 분리 - 모든 피드 조회전에 로딩 로직 추가
📌𝘐𝘴𝘴𝘶𝘦𝘴
📎𝘞𝘰𝘳𝘬 𝘋𝘦𝘴𝘤𝘳𝘪𝘱𝘵𝘪𝘰𝘯
📷𝘚𝘤𝘳𝘦𝘦𝘯𝘴𝘩𝘰𝘵
123123.webm
💬𝘛𝘰 𝘙𝘦𝘷𝘪𝘦𝘸𝘦𝘳𝘴