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주차 미션 / 서버 2조 박성현 #17

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

sh1220
Copy link

@sh1220 sh1220 commented Mar 22, 2024

2-2는 다하지 못해서 아쉽네요..

다른분들 코드보니, LadderPostion으로 하나 만들면 좀 더 가독성 좋아보일것 같네요..
만약 추후에 2-2도 다하게 된다면 올려보겠습니다!

리팩토링 한것)

  • 미션 힌트에 있는것들
  • NaturalNumber를 Row와 Person에 따라 처리하고 싶어서, 따로 팩토리 메서드 만들기

@sh1220 sh1220 changed the title Until misson 2-1 1주차 미션 / 서버 2조 박성현 Mar 22, 2024
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