2단계 - 사다리 (생성) #2319
Open
2단계 - 사다리 (생성) #2319
+541
−1
Conversation
This file contains 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
ksy90101
reviewed
Apr 6, 2025
ksy90101
requested changes
Apr 6, 2025
| import java.util.stream.Collectors; | ||
| import java.util.stream.IntStream; | ||
|
|
||
| public class LadderLineGenerator { |
There was a problem hiding this comment.
랜덤이다보니 테스트 및 제어하기가 어려울것으로 생각됩니다.
따라서 인터페이스를 통해 제어해보시면 어떨까요?
| public class Ladder { | ||
| private final List<LadderLine> ladderLines; | ||
|
|
||
| public Ladder(int height, int width) { |
There was a problem hiding this comment.
지금 여기서 잘 만들어주셨는데,
사다리를 고민해보면
사다리 갯수가 있을꺼고. (이게 높이의 갯수로 보입니다.)
그리고 width인 넓이들이 존쟇라거 같아요.
이걸 한번 나타내봐도 좋을거 같습니다.
실제 사다리 하나가 어떠한 결과를 반환해야 하지 않을까 생각이 듭니다.
ksy90101
reviewed
Apr 6, 2025
ksy90101
reviewed
Apr 6, 2025
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.
이번 단계에서는 기존에 적용하던 원칙을 지키는 것과 더불어 Stream 적용에 집중했습니다.
List 를 그대로 반환하는 대신, 리스트의 크기를 반환하는 메소드와 리스트의 인덱스에 해당하는 원소에 접근하는 메소드를 구현하여 IntStream 을 적용했는데요.
리스트를 그대로 반환하는 것에 비해 객체 내부의 필드에 직접 접근하는 것을 막을 수 있었지만 코드의 양 자체는 늘었고 이것이 정말 효율적인지는 좀 더 생각해볼 여지가 있을 거 같습니다.
감사합니다.