-
Notifications
You must be signed in to change notification settings - Fork 9
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
#02 인기영화 목록 보기 및 영화 상세보기 화면 제공 #33
base: 2giwon
Are you sure you want to change the base?
Conversation
#1 프로젝트 초기 구성
ToyProject/app/build.gradle
Outdated
|
||
android { | ||
compileSdkVersion 29 | ||
buildToolsVersion "29.0.3" |
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.
buildToolsVersion은 없어도 됩니다.
한번 찾아보시면 공부하는데 도움이 될꺼같습니다.
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.
gradle 3.x 이후 부터 적용하지 않아도 된다고 하는데 이게 맞나요?
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.
네 맞습니다
import androidx.recyclerview.widget.ListAdapter | ||
import androidx.recyclerview.widget.RecyclerView | ||
|
||
abstract class BaseRecyclerView { |
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.
BaseRecyclerView
class 가 필요한지가 의문입니다
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.
맞네요 습관처럼 만들다보니 인지하지 못하고 있었습니다. 수정했습니다
disposable 관리에 대해서는 좀 더 이야기가 필요 하다고 생각됩니다 |
리뷰 부탁드립니다