3단계 - 사다리(게임 실행) #2421
3단계 - 사다리(게임 실행) #2421
Conversation
| import java.util.concurrent.ThreadLocalRandom; | ||
|
|
||
| public class LineGenerator { | ||
| public static List<Boolean> generateLine(int countOfPerson) { |
There was a problem hiding this comment.
클래스, 메서드 이름의 의도가 그렇듯이 List 보다는 Line 객체를 생성해 반환하는 것은 어떨까?
| } | ||
|
|
||
| for(int i = 0; i < values.size() -1; i++) { | ||
| if(isConsecutiveLine(values.get(i), values.get(i + 1))) { |
There was a problem hiding this comment.
Boolean 값 두 개를 가지는 새로운 객체를 도출해 보는 것은 어떨까?
예를 들어
Point {
Boolean current;
Boolean previous;
}
위와 같이 현재 위치의 boolean 값과 이전 위치의 boolean 값을 가지는 더 작은 단위의 객체를 도출해 보면 어떨까?
더 작은 객체의 도출했을 때 현재와 비교해 어떤 장/단점이 있을까?
| } | ||
| public int countOfPeople() { | ||
| return countOfPeople; | ||
| } |
There was a problem hiding this comment.
Ladder가 사다리 타기를 위한 모든 상태 값을 가진다.
Ladder에 play()와 같은 메시지를 보내 사다리 타기를 실행하는 역할을 부여하면 어떨까?
| return false; | ||
| } | ||
|
|
||
| public int move(int index) { |
There was a problem hiding this comment.
index 값이 원시값이다보니 IndexOutOfBoundsException에 대한 체크를 계속해야 한다.
index 원시값을 포장해 보는 것은 어떨까?
| return false; | ||
| } | ||
|
|
||
| public int move(int index) { |
There was a problem hiding this comment.
LEFT, RIGHT, PASS와 같은 상태 값을 enum으로 구현할 경우 이 로직 구현이 어떻께 달라질까?
얻을 수 있는 이점이 있을까?
| assertValidLine(line); | ||
| }); | ||
| } | ||
| static Stream<List<Boolean>> validLines() { |
There was a problem hiding this comment.
ParameterizedTest 기반으로 테스트하는 것은 알지만 이렇게 많은 경우의 수를 테스트할 필요가 있을까?
좀 더 적은 경우의 수로 모두 커버할 수 있지 않을까?
|
|
||
| @Test | ||
| public void 사다리_왼쪽으로_이동불가() { | ||
| Line line = new Line(List.of(false, false)); |
There was a problem hiding this comment.
| Line line = new Line(List.of(false, false)); | |
| Line line = new Line(false, false); |
Line 객체에 가변 인자를 받은 생성자를 추가해 위와 같이 구현하도록 지원하면 어떨까?
리뷰 중점 구분
그외의 조언도 부탁드립니다.
감사합니다.
기능 요구사항
사다리 실행 결과를 출력해야 한다.
개인별 이름을 입력하면 개인별 결과를 출력하고, "all"을 입력하면 전체 참여자의 실행 결과를 출력한다.
사다리 게임에 참여하는 사람에 이름을 최대5글자까지 부여할 수 있다. 사다리를 출력할 때 사람 이름도 같이 출력한다.
사람 이름은 쉼표(,)를 기준으로 구분한다.
사람 이름을 5자 기준으로 출력하기 때문에 사다리 폭도 넓어져야 한다.
사다리 타기가 정상적으로 동작하려면 라인이 겹치지 않도록 해야 한다.
|-----|-----| 모양과 같이 가로 라인이 겹치는 경우 어느 방향으로 이동할지 결정할 수 없다.
프로그래밍 요구사항