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

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

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

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

wants to merge 3 commits into from

Commits on Mar 23, 2023

  1. row 파일 생성

    @Hyuja
    Hyuja committed Mar 23, 2023
    Configuration menu
    Copy the full SHA
    22a9c93 View commit details
    Browse the repository at this point in the history

Commits on Mar 24, 2023

  1. .

    @Hyuja
    Hyuja committed Mar 24, 2023
    Configuration menu
    Copy the full SHA
    3c93895 View commit details
    Browse the repository at this point in the history

Commits on Mar 31, 2023

  1. refactor : numOfPeople와 ladderHeight를 의미하는 변수를 final로 변경하여 불변성을 보장 

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

refactor : drawLine 오타 수정
    @Hyuja
    Hyuja committed Mar 31, 2023
    Configuration menu
    Copy the full SHA
    8ed74ea View commit details
    Browse the repository at this point in the history