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.

Sign up for GitHub

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

Jump to bottom

2주차 미션 / 서버 2조 이준영 #45

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

2주차 미션 / 서버 2조 이준영 #45

wants to merge 17 commits into from

Conversation

lee-june-young
Copy link

  • 2주차 실습 영상 기반으로 리팩토링 진행
  • 요구사항 2 구현:
  • 사다리 출력 기능 추가
  • 사다리 출력을 내려갔을 때 print하게 짜서 왼쪽 오른쪽으로 이동한거 보여주는 부분 리팩토링 필요
  • 요구사항 3 구현:
  • 사다리 랜덤 생성 기능 추가
  • 인터페이스 활용한 의존성 주입 미완성
  • 정적 팩토리 메서드 패턴 활용 미완성

@lee-june-young
Ladder init
d0d3fb5
@lee-june-young
객체지향 생활체조 원칙적용
c1fabc3 
- 3번: 모든 원시 값과 문자열을 포장한다. (rows)
- 5번: 일급 컬렉션을 사용한다. (row)
- 1번: 한 메서드에 오직 한단계의 들여쓰기만 한다.(범위 체크를 메서드로
  빼주었다.)
@lee-june-young
LadderTest 추가 및 중복 제거
2817939
@lee-june-young
row수, person수 범위 체크(음수 에러 처리)
d0d8ee9
@lee-june-young
사다리 그릴 때 중복발생이나 이어서 그려질 경우 예외처리 & Test
e56cae1
@lee-june-young
Docs: added README.md for To Do List
a7a3fdd
@lee-june-young
Refactor: use Direction enum for ladder line direction handling
21414b2
@lee-june-young
Refactor: ExceptionMessage enum for centralized error messages
c2a7c9f
@lee-june-young
Refactor: wrap int with Position wrapper in Row & Ladder
9081967
@lee-june-young
Refactor: replace int array with Node array
c1a03fe
@lee-june-young
Test: add test cases for Node class
b1abb9d
@lee-june-young
Refactor: Delegate Ladder responsibilities
b302b38
@lee-june-young
Feat: printLadder 함수 구현
c1889f6
@lee-june-young
Feat: 사다리 자동 생성
7ecb299
@lee-june-young
Refactor: 사다리 이동 과정까지 출력
435e49b
@lee-june-young
LadderGame에 인터페이스를 활용한 의존성 주입을 통해 LadderCreator를 변경할 수 있게함
94508cf
@lee-june-young
정적 팩토리 메서드 패턴 활용 LadderGame 생성..?
4bd515e
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@lee-june-young