[다리 건너기] 김민겸 미션 제출합니다.

[Mingyum-Kim]

No description provided.

[Mingyum-Kim]

함수가 너무 길다! 어떻게 해결할 수 있을까?

[Mingyum-Kim]

Suggested change

	public void printMap(List<Move> moves) {
	List<String> up = new ArrayList<>();
	List<String> down = new ArrayList<>();
	for (Move move : moves) {
	if (move.direction().equals("U")) {
	up.add(move.success());
	down.add(NONE);
	}
	if (move.direction().equals("D")) {
	up.add(NONE);
	down.add(move.success());
	}
	}
	ConsoleWriter.printlnMessage(generateSingleMapRow(up));
	ConsoleWriter.printlnMessage(generateSingleMapRow(down));
	ConsoleWriter.println();
	}
	public void printMap(List<Move> moves) {
	List<String> up = new ArrayList<>();
	List<String> down = new ArrayList<>();
	for (Move move : moves) {
	moveUp(up, down, move);
	moveDown(up, down, move);
	}
	ConsoleWriter.printlnMessage(generateSingleMapRow(up));
	ConsoleWriter.printlnMessage(generateSingleMapRow(down));
	ConsoleWriter.println();
	}
	private void moveDown(List<String> up, List<String> down, Move move) {
	if (move.direction().equals("D")) {
	up.add(NONE);
	down.add(move.success());
	}
	}
	private void moveUp(List<String> up, List<String> down, Move move) {
	if (move.direction().equals("U")) {
	up.add(move.success());
	down.add(NONE);
	}
	}

[Mingyum-Kim]

메서드의 파라미터는 최대 3개까지만 허용한다.

[Mingyum-Kim]

이러한 경우 에러 메시지를 범위 검증에 맞는 것을 따로 만들고, 에러 메시지에 대한 파라미터를 지우는 것을 차선책으로 한다

[Mingyum-Kim]

컨트롤러에서 뷰 단의 출력까지 책임지고 있는 형태이다.
boolean 등의 변수로 넘긴 다음에 뷰에서 분기하여 O, X를 출력하는 것이 바람직하다

[Mingyum-Kim]

StringJoiner로 간결하게 작성할 수 있다.

[Mingyum-Kim]

Suggested change

	public String generateSingleMapRow(List<String> map) {
	int size = map.size();
	if (size == 1) {
	return "[ " + map.get(0) + " ]";
	}
	String head = "[ " + map.get(0);
	for (int i = 1; i < size; i++) {
	head += String.format(SINGLE_MAP_FORMAT, map.get(i));
	}
	return head + " ]";
	}
	public String generateSingleMapRow(List<String> map) {
	String result = new StringJoiner(" | ", "[", "]");
	for(String mp : map) {
	result.add(mp);
	}
	}