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

Assignment of Day05 Uploaded By JongYoonBae #28

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

Conversation

jy-b
Copy link

@jy-b jy-b commented Apr 17, 2023

User class 의 registerTime의 타입이 따로 명시 되어 있지 않아서
System.currentTimeMillis()로 넣어주었는데 상관 없을까요?
설계 사양에 따라 달라질것(?) 같아서 임의로 지정하였습니다

입력받는 부분이 없어서
User와 Electronic의 필드들에 대한 validation 검사는 구현하지 않았으나 여유되면 추가하겠습니다

jy-b added 3 commits April 17, 2023 21:49
JongYoonBae
by JongYoonBae

add missing method
by JongYoonBae

add Electronic[] electronicArray, User[] userArray
add method for arrays
add findByUserId test
@jy-b
Copy link
Author

jy-b commented Apr 17, 2023

Electronics와 Users의 필드를
배열로 구현하라고 되어있어 배열로 구현한 코드를 추가하였습니다

[Modify toString() of Electronics List]
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