Skip to content
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

merge :: (#10) MainView-Publishing #11

Merged
merged 36 commits into from
Oct 15, 2024
Merged

Conversation

uson1004
Copy link
Member

@uson1004 uson1004 commented Oct 13, 2024

homeView 제작, 캘린더 기능, 운동 리스트(recyclerView를 곁들인), view 선택 상태처리(enterSkills) 등을 구현하였습니다.
image

@uson1004 uson1004 added the enhancement New feature or request label Oct 13, 2024
@uson1004 uson1004 requested a review from Tmdhoon2 October 13, 2024 09:22
@uson1004 uson1004 self-assigned this Oct 13, 2024
@uson1004 uson1004 linked an issue Oct 13, 2024 that may be closed by this pull request
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

HomeViewModel 생성자에 아무 값도 없어서 당장은 필요없을 것 같은데 확장성 고려해서 만들어두신 건가요??

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

HomeViewModel에는 LiveData가 있는 것으로 알고있는데 생성자에 값이 없다는 말씀이 무엇을 의미하시는 지 이해를 못하였습니다ㅠ

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

스크린샷 2024-10-15 오전 8 47 28

HomeViewModel에는 생성자에 값이 없어요.
코틀린에서 생성자는 클래스 이름 옆에 열린 ()를 의미합니다.
생성자에 값이 없다는건 객체 생성 시 전달하는 값이 없다는 의미입니다.
근데 뷰모델 팩토리는 보통 뷰모델 생성자에 값을 넣어주기 위해 사용합니다.
그래서 말씀드린겁니다!

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

아하 네 맞습니다 🚀

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

이건 어떤 역할을 하는 클래스인가요??

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

하위 레벨의 폴더가 파일 없이 존재하게 되면 상위 폴더와 겹쳐져서 임의의 파일을 생성해두었습니다. 다른 파일로 대체할 예정입니다!

app/src/main/res/drawable/button_selected.xml Outdated Show resolved Hide resolved
app/src/main/res/drawable/calender_text_color.xml Outdated Show resolved Hide resolved
app/src/main/res/layout/activity_email_validation.xml Outdated Show resolved Hide resolved
@uson1004 uson1004 requested a review from Tmdhoon2 October 15, 2024 08:30
@uson1004 uson1004 merged commit e84c046 into develop Oct 15, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

MainView Publishing
2 participants