🚀 4단계 - 사다리(리팩터링)

[mingulee-devel]

안녕하세요! 정말 오랜만에 PR 드립니다...
지금도 리뷰를 진행하시는지 모르겠지만 한번 올려봅니다!

구현하면서 궁금했던 부분에 대해 질문 남겨봅니다!

1. ResultView에서
   getAllResult에 사람 수만큼 반복이 필요한데, crossSize 혹은 countOfPerson을 넘겨주는 것은 불필요하다고 생각해서
   아래와 같이 lines.get(0).getCrossSize()로 작성하였습니다.
   이러한 방식으로 구현을 해도 되는지, 더 좋은 방법이 있는지 궁금합니다!

  public Map<Integer, Integer> getAllResult() {
      return IntStream.range(0, lines.get(0).getCrossSize())
              .boxed()
              .collect(Collectors.toMap(i -> i, i -> move(i)));
  }

2. 리팩토링을 진행할 때 들었던 강의에서 Line 클래스에 아래와 같이 move가 구현되어있어서

    public int move(int position) {
        return this.crosses.get(position).move();
    }

Usernames에서 username을 가져오기 위해서 아래와 같이 인덱스를 넘겨받아 내부 username을 반환하도록 메서드를 추가 구현하였는데 이렇게 해도 괜찮은지 궁금합니다.

public String getUsername(int index) {
        return usernames.get(index).getUsername();
    }

3. 로또까지 진행하면서 일급 컬렉션을 분리할 때 항상 도메인명s를 붙였는데,
   구현된 내용에서 Line에는 List만 존재하고, Ladder에는 List만 존재한다면
   Line이 Cross의 일급 컬렉션, Ladder가 Line의 일급 컬렉션이라고 보면 되는 걸까요?

강의를 들으면서 내부 객체를 꺼내서 사용하는 것은 최대한 지양해야하고 List가 존재할 때 일급 컬렉션으로 분리해야된다는 내용을 배웠는데, 제가 여기에만 너무 신경을 쓰는 건지 위에 질문 드린 get(~)이나 List만 보면 마음이 불편합니다 ㅠㅠ
아직 판단 기준이 명확하지 않은 것 같습니다...

바쁘신 와중에 질문이 많아 송구스러운 마음입니다
수강신청 미션도 최대한 빠르게 진행해보도록 하겠습니다...
오늘도 감사합니다. 😄