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

2주차 미션 / 서버 2조 - 전희지 #29

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

Conversation

Hyuja
Copy link

@Hyuja Hyuja commented Mar 31, 2023

No description provided.

refactor : ladder 배열에서 가로 인덱스와 세로 인덱스를 바꿔서 가로 줄과 세로 줄의 정보를 명확하게 구분
refactor : drawLine 메소드에서는 랜덤한 값을 생성하는 부분을 moveRightOrLeft메소드로 추출
refactor : run 메소드에서 numOfPeople - 1번 drawLne 메소드를 호출하는 부분은 바깥쪽 루프에서 수행되므로 중복 호출을 개선

refactor : drawLine 오타 수정
refactor : ladder 배열에서 가로 인덱스와 세로 인덱스를 바꿔서 가로 줄과 세로 줄의 정보를 명확하게 구분
refactor : drawLine 메소드에서는 랜덤한 값을 생성하는 부분을 moveRightOrLeft메소드로 추출
refactor : run 메소드에서 numOfPeople - 1번 drawLne 메소드를 호출하는 부분은 바깥쪽 루프에서 수행되므로 중복 호출을 개선

refactor : drawLine 오타 수정
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.

1 participant