Skip to content

Conversation

@pci2676
Copy link
Member

@pci2676 pci2676 commented May 1, 2019

No description provided.

pci2676 added 15 commits April 13, 2019 18:45
각 기둥간 동기화를 통해 사다리를 완성시켜야함
todo
사다리 출력을 위해 if문이 떡칠이 되어있다.
이 부분에 대한 수정이 필요하다.
2개만 그릴때 무한루프발견
불필요한 객체 전달 제거
--> factory자체를 넘겼는데 그냥 List<pillar>를 넘김
Point를 찾을때 x로만 찾는데 y까지 넘기는 경우를 삭제했음
1. 위치에 맞게 메소드를 다른 클래스로 이동
2. 멤버변수를 추가해서 get을 많이 호출하지 않게함
pci2676 added 2 commits May 5, 2019 14:31
추가요구사항 작성

LadderResult로 결과 저장 후 출력

추가 요구사항 작성 완료
jyami-kim added a commit that referenced this pull request May 16, 2019
import java.util.stream.Collectors;
import java.util.stream.IntStream;

public class FixedLadderFactory implements LadderFactory {

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

이쪽 로직이 너무 몰려있고 복잡해서 의도 파악하기가 힘들어 한번설명들어야 알수 있을거같아

}

private List<Point> createTopFixedPoints(List<Location> locations, Direction direction) {
return IntStream.range(0, height / 3)

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

이건 왜하는거야?? 의도를모르겠어

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants