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

배정환 과제1 제출 #5

Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open

배정환 과제1 제출 #5

wants to merge 5 commits into from

Conversation

hwan04
Copy link

@hwan04 hwan04 commented Oct 6, 2024

No description provided.

@SeonghwiShin SeonghwiShin self-requested a review October 7, 2024 05:06
@SeonghwiShin
Copy link
Contributor

이번에 mvvm을 처음 해봤을텐데 어려운건 없으셨나요 아무 설명이없네 ㅜㅜ

viewmodel에서는 원칙적으로 context를 사용하지 않는게 맞습니다. 하지만 repository 패턴을 사용하지 않으면 어쩔수 없죠.
mvvm에서 저장하는 로직은 viewmodel은 비즈니스 로직이 들어가야해서 sharedpreference로직이 들어가야할 것 같은데, context가 필요하면 view로 가야하나? 고민이 많이 되는게 맞습니다.
그래도 viewmodel에 넘겨주는 applicationContext를 이용해서 넘겨주신건 잘했습니다. (이부분은 톡방에서 설명드리겠습니다)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants