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주차 미션 / 서버 3조 장익환 #12

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

Conversation

JangIkhwan
Copy link

@JangIkhwan JangIkhwan commented Mar 22, 2024

미션 요구사항2는 시간이 부족해서 시도해보는데 그쳤지만, 미션 요구사항1인 리팩토링까지는 겨우 했습니다. 테스트 코드를 수정해서 기능이 동작하는지도 확인해보았습니다. 이전에는 자바 코드를 작성할 때 고민이 부족했던 것 같습니다. 하지만 이번 과제를 통해서 원시값과 유효성 검증을 하나의 클래스로 묶는 습관을 들여야 겠다고 생각했습니다. 그리고 한 클래스에 책임이 과중되지 않도록 책임을 여러 클래스에 분산시켜야 한다는 점을 배울 수 있었습니다. 미션 결과를 확인해주셔서 감사합니다.

Copy link

@kmw10693 kmw10693 left a comment

Choose a reason for hiding this comment

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

1주차 코드 작성하느라 정말 수고하셨습니다!
객체지향은 정답이 없는 것 같습니다! 1주차 코드와 설명을 정리해 보시면서 객체지향에 대해 생각 해보시는 시간을 가져서 다행입니다!
정말 수고하셨습니다! :)

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