[ 사다리 - 함수형 프로그래밍 ] 박채연 미션 제출합니다. #57
Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
RIANAEH
suggested changes
May 30, 2025
Comment on lines
+7
to
+8
| public class Ladder { | ||
| private List<Floor> floors; |
There was a problem hiding this comment.
사다리는 N개의 층을 가질 수 있고, 층은 N개의 연결여부를 가지고 있게 구성해주셨군요! 좋습니다😎
Comment on lines
+10
to
+20
| public Ladder(int height, int width, Random random) { | ||
| this.floors = generateFloors(height, width, random); | ||
| } | ||
|
|
||
| private List<Floor> generateFloors(int height, int width, Random random) { | ||
| List<Floor> floors = new ArrayList<>(); | ||
| for (int i = 0; i < height; i++) { | ||
| floors.add(new Floor(width, random)); | ||
| } | ||
| return floors; | ||
| } |
There was a problem hiding this comment.
List floors를 넣어주는 생성자를 만들어주고, 높이와 넓이 connection 생성전략을 넣어주는 정적 팩토리메서드를 만들어주면 어떨까요?
Comment on lines
+10
to
+18
| public Floor(int width,Random random) { | ||
| connections = new ArrayList<>(); | ||
| boolean previousConnection = false; | ||
| for (int i = 0; i < width-1; i++) { | ||
| Connection connection =Connection.of(previousConnection,random); | ||
| connections.add(connection); | ||
| previousConnection = connection.isConnected(); | ||
| } | ||
| } |
There was a problem hiding this comment.
List를 넣어주는 생성자를 만들어주고, 넓이와 connection 생성전략을 넣어주는 정적 팩토리메서드를 만들어주면 어떨까요?
생성자에서는 어떤것을 검증해볼 수 있을까요? (Floor가 어떤 규칙을 가져야하는지 생각해보면?)
Comment on lines
+5
to
+15
| public class Connection { | ||
| private final boolean connected; | ||
|
|
||
| private Connection(boolean connected) { | ||
| this.connected = connected; | ||
| } | ||
|
|
||
| public static Connection of(boolean previousConnected, Random random) { | ||
| boolean connect = !previousConnected && random.nextBoolean(); | ||
| return new Connection(connect); | ||
| } |
There was a problem hiding this comment.
이건 선택의 영역이지만 Floor 내의 커낵션을 결정하는 로직은 Floor가 가지고 있어도 무방하다고 생각하긴 합니다!
그래도 값들을 최대한 값객체로 만들어보려는 시도 좋습니다👍
Comment on lines
+30
to
+35
| public void printResult(LadderResult result, int width) { | ||
| for (int i = 0; i < width; i++) { | ||
| int end = result.move(i); | ||
| System.out.println(i + " -> " + end); | ||
| } | ||
| } |
Comment on lines
+3
to
+4
| public class LadderResult { | ||
| private final Ladder ladder; |
There was a problem hiding this comment.
LadderResult라는 이름을 가지고 있지만, Ladder 내에서의 움직임 정책을 담당하고 있는 것 같아요.
움직임을 수행시키고 결과를 만들어 반환하는 객체로 바꿔보면 좋을 것 같고, LadderResult는 정말 결과만 들고 있으면 좋을 것 같네요!
| } | ||
|
|
||
| private int movePosition(Floor floor, int index) { | ||
| if (floor.getConnections().get(index).isConnected()) { |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
[ enum ] - 고정된 상태에 따른 행동/속성 정의하고 싶을 때 사용
[ 일급 컬렉션 ] - 컬렉션이 도메인 의미 갖고 있을 때 사용
Enum 과 일급 컬렉션은 둘 다 정보 은닉을 목적으로 한다.
[ 원시값 포장 ] - 원시값이 도메인 의미 갖고 있을 때 사용
사다리 구성요소가 무엇인지 생각하며 어떤 클래스들이 필요할지 고민했습니다.
Ladder -> Floor -> Connection
Ladder : List < Floor > - 일급 컬렉션
Floor : List < Connection > - 일급 컬렉션
Connection : boolean isConnected, public static Connection.of(boolean) - 정적 팩토리 메서드 (생성 제약 있음)
원시값 포장이 필요한 부분은 없는 것 같다.
3단계 게임 결과 출력하기
정리