[Step2] 🚀 2단계 - 사다리(생성) by SimJung · Pull Request #1776 · next-step/java-ladder

SimJung · 2023-05-17T13:11:03Z

안녕하세요.
2단계 - 사다리(생성) 미션 진행하여 PR 드립니다.
생각보다 난이도가 있어 시간이 좀 걸렸습니다.

기능 요구사항

사다리 게임에 참여하는 사람에 이름을 최대5글자까지 부여할 수 있다. 사다리를 출력할 때 사람 이름도 같이 출력한다.
사람 이름은 쉼표(,)를 기준으로 구분한다.
사람 이름을 5자 기준으로 출력하기 때문에 사다리 폭도 넓어져야 한다.
사다리 타기가 정상적으로 동작하려면 라인이 겹치지 않도록 해야 한다.
|-----|-----| 모양과 같이 가로 라인이 겹치는 경우 어느 방향으로 이동할지 결정할 수 없다.

PS.
PR 올리고나니 이전 스텝 커밋까지 섞여있는걸 확인했습니다..
squash&merge 이후 로컬에서 pull받고 진행했는데 헷갈리네요 ㅜㅜ

… simjung

- 사람 이름은 쉼표를 기준으로 구분한다.

- VerticalLine 및 컬렉션 클래스 추가 - HorizontalLine 및 컬렉션 클래스 추가

- view 패키지 추가

shared-moon

정님 안녕하세요!
2단계 미션 잘 진행 해 주셨네요 :)
코멘트를 좀 남겨 두었는데, 다음 미션 전에 살펴보면 좋을 것 같아요!
확인 후 리뷰 재요청 부탁드릴게요!

shared-moon · 2023-05-18T14:44:53Z

src/main/java/ladder/Main.java

+
+public class Main {
+    public static void main(String[] args) {
+        Players players = PlayersGenerator.create(InputView.getPlayerNames());


InputView.getPlayerNames() 는 변수로 한번 빼는 건 어떨까요 ?

shared-moon · 2023-05-18T14:45:57Z

src/main/java/ladder/Main.java

+    public static void main(String[] args) {
+        Players players = PlayersGenerator.create(InputView.getPlayerNames());
+        int width = players.getPlayerNumber();
+        System.out.println();


줄바꿈은 꼭 여기서 해 줘야 할까요 ?

shared-moon · 2023-05-18T14:46:19Z

src/main/java/ladder/Main.java

+        System.out.println();
+
+        Ladder ladder = new LadderGenerator().generate(height, width);
+        ResultView.showResultMessage();


이 세 개를 묶는 함수를 따로 제공해주는 건 어떨까요 ?

shared-moon · 2023-05-18T14:48:17Z

src/main/java/ladder/exception/ContinuousHorizontalLineException.java

@@ -0,0 +1,9 @@
+package ladder.exception;
+
+public class ContinuousHorizontalLineException extends RuntimeException {


상세한 예외 선언 좋네요 👍

shared-moon · 2023-05-18T14:55:22Z

src/main/java/ladder/domain/LadderGenerator.java

+import java.util.HashSet;
+import java.util.Set;
+
+public class LadderGenerator {


이 클래스는 유틸 클래스로 선언 해 보는 건 어떨까요 ? Strategy를 generate에서 받아도 될 것 같아서요!

shared-moon · 2023-05-18T15:16:59Z

src/main/java/ladder/domain/HorizontalLines.java

+import java.util.stream.Collectors;
+
+public class HorizontalLines {
+    private final Set<HorizontalLine> horizontalLineSet;


혹시 자료구조를 Set으로 정하신 이유가 있을까요 ?

shared-moon · 2023-05-18T15:20:25Z

src/main/java/ladder/domain/LadderGenerator.java

+            VerticalLine nowVerticalLine = verticalLines.getVerticalLineByIndex(i);
+            VerticalLine nextVerticalLine = verticalLines.getVerticalLineByIndex(i + 1);
+
+            boolean notExistPreviousHorizontalLine = sameHeightHorizontalLineHashSet.stream()


이렇게 다 비교해야 할까요 ? 어차피 순차처리를 하는거라면 이전 칸에서 생성을 했는지만 확인하면 될 것 같아서요!

shared-moon · 2023-05-18T15:25:08Z

src/main/java/ladder/domain/LadderGenerateStrategy.java

@@ -0,0 +1,5 @@
+package ladder.domain;
+
+public interface LadderGenerateStrategy {


인터페이스 활용 좋네요! 혹시 domain 패키지 내부에서도 관계있는 클래스끼리 따로 패키징 할 수는 없을까요 ?

shared-moon · 2023-05-18T15:30:28Z

src/main/java/ladder/domain/Ladder.java

+        }
+    }
+
+    private void checkValidLines(VerticalLines verticalLines, HorizontalLines horizontalLines) {


요 유효성검사 혹시 꼭 필요 할까요 ? 앞에 VerticalLines와 HorizontalLines를 만들 때 이미 검증 되고 있는 것 같아서요!

shared-moon · 2023-05-18T15:37:08Z

src/main/java/ladder/domain/Ladder.java

+
+import java.util.Set;
+
+public class Ladder {


사다리를 가로선 / 세로선의 개념으로 접근하면서 로직이 많이 복잡해 진 것 같아요!
사다리 한칸 한칸을 점으로 바라보고, 각 점에서 사다리의 유무를 판단하는 식으로 개념을 조금 바꾸면 쉽게 접근 할 수 있지 않을까요 ?
4명의 플레이어, 높이 4의 사다리라면
[x][o][x]
[o][x][o]
[o][x][o]
[x][x][o]
이런식으로요!

SimJung added 16 commits May 13, 2023 02:56

람다 실습 1 - 익명 클래스를 람다로 전환

4919b56

람다 실습 2 - 람다를 활용해 중복 제거

7060fe1

map, reduce, filter 실습 1 - sumOverThreeAndDouble 테스트 pass

c7a9148

map, reduce, filter 실습 2 - printLongestWordTop100() 메서드 구현

c9f1605

요구사항 1 - Optional을 활용해 조건에 따른 반환

51532c0

요구사항 2 - Optional에서 값을 반환

79be2ed

요구사항 3 - Optional에서 exception 처리

67146b7

Merge branch 'simjung' into step1

9e2df9e

Merge branch 'simjung' of https://github.com/SimJung/java-ladder into…

793a092

… simjung

feat : Conditional을 Predicate로 변경

ef3f71c

refactor : matchExpression static 제거

d1728d4

fix : age의 null 체크 추가

5170347

docs : README.md 추가

226645d

feat : 사다리 게임에 참여하는 사람에 이름을 최대 5글자 까지 부여할 수 있다.

20e0660

- 사람 이름은 쉼표를 기준으로 구분한다.

feat : Ladder 관련 로직 구현

3bd6f44

- VerticalLine 및 컬렉션 클래스 추가 - HorizontalLine 및 컬렉션 클래스 추가

feat : LadderGenerator 구현 및 main 구성

59d1c9b

- view 패키지 추가

SimJung changed the title ~~Step2~~ [Step2] 🚀 2단계 - 사다리(생성) May 17, 2023

shared-moon requested changes May 18, 2023

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[Step2] 🚀 2단계 - 사다리(생성)#1776

[Step2] 🚀 2단계 - 사다리(생성)#1776
SimJung wants to merge 16 commits intonext-step:simjungfrom
SimJung:step2

SimJung commented May 17, 2023 •

edited

Loading

Uh oh!

shared-moon left a comment

Uh oh!

shared-moon May 18, 2023

Uh oh!

shared-moon May 18, 2023

Uh oh!

shared-moon May 18, 2023

Uh oh!

shared-moon May 18, 2023

Uh oh!

shared-moon May 18, 2023

Uh oh!

shared-moon May 18, 2023

Uh oh!

shared-moon May 18, 2023

Uh oh!

shared-moon May 18, 2023

Uh oh!

shared-moon May 18, 2023

Uh oh!

shared-moon May 18, 2023

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

		@@ -0,0 +1,9 @@
		package ladder.exception;

		public class ContinuousHorizontalLineException extends RuntimeException {

		@@ -0,0 +1,5 @@
		package ladder.domain;

		public interface LadderGenerateStrategy {

Conversation

SimJung commented May 17, 2023 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

shared-moon left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

SimJung commented May 17, 2023 •

edited

Loading