인프런 커뮤니티 질문&답변

안녕하세요. 30:40 강의 구간에서 발생한, 메서드 추출 이슈에 관하여, 원인이 무엇인지 궁금해서 질문 드립니다.

발생한 문제는

• 지뢰를 밟았을 때, 모든 지뢰 구간이 노출 된다거나,

• 셀을 열었을 때 엉뚱한 곳도 같이 열리는 현상이 나타납니다.

문제 발생 지점은 updateCellAt 메서드 사용 구간 입니다.

  private void initializeEmptyCells(CellPositions cellPositions) {
    List<CellPosition> positionList = cellPositions.getPositionList();
    updateCellsAt(positionList, new EmptyCell());
  }

  private void initializeLandMineCells(List<CellPosition> landMinePositionList) {
    updateCellsAt(landMinePositionList, new LandMineCell());
  }

  ...

  private void updateCellsAt(List<CellPosition> positionList, Cell cell) {
    for (CellPosition position : positionList) {
      updateCellAt(position, cell);
    }
  }  

상위 호출 부분으로 넘어가서,

제 생각에 initializeEmptyCells 와 initializeLandMineCells 메서드 호출로 넘기는 인자가, 공유될 대상이었나 라고 생각해볼 수 있었습니다.

cellPositions 나 landMinePositionList 모두 새로운 컬렉션으로 리턴되어서, 공유되지 않다고 생각되었습니다.

  public void initializeGame() {
    CellPositions cellPositions = CellPositions.from(board);
    initializeEmptyCells(cellPositions);

    List<CellPosition> landMinePositionList = cellPositions.extractRandomPositions(landMineCount);
    initializeLandMineCells(landMinePositionList);

    ...
  }

그러면 어느 부분이 메서드 추출로 인해, 공유될 대상이 발생한 원인이었나가 궁금합니다.