주석 달아 둔 부분을 바꾸려고 하는데 Collector.of()로 변경해보라는 리뷰 조언을 받았는데 진짜 어렵네요.
이것저것 보면서 생각 중인데 그냥 isRowContinuousTrue() 이 부분과 drawLine() 부분을
Predicate 하나 구현해서 집어넣는 식으로 accumulator() 구현하는게 편할려나요?
약간 이런 코드 응용해보려고 생각중입니다.
11개의 댓글
무분별한 사용은 차단될 수 있습니다.
주석 달아 둔 부분을 바꾸려고 하는데 Collector.of()로 변경해보라는 리뷰 조언을 받았는데 진짜 어렵네요.
이것저것 보면서 생각 중인데 그냥 isRowContinuousTrue() 이 부분과 drawLine() 부분을
Predicate 하나 구현해서 집어넣는 식으로 accumulator() 구현하는게 편할려나요?
약간 이런 코드 응용해보려고 생각중입니다.
Portrait
stream은 순차적으로 돈다고 생각하고 짜면안되는데 그렇게생각하고 짠거같다
그리고 변수명이 뭘 뜻하는지 모르겠오
그리고 돌아가는거 맞아? iob 익셉션 날거같은데...
TQQQ
아 그렇군요 병렬처리에도 사용되니까 배제를 해야되나보네요 변수명은 도메인이 이해안되시니까 그럴 수 있다고 생각합니다.
그냥 사용자대로 입력을 받으면 사다리 게임을 위한 사다리를 출력하는 건데 List points는 네이밍컨벤션에 있던 부분이고 나머지는 제가 네이밍하긴했는데 네이밍 혹시 고쳐야할 부분이 있을까요? 이게 좀 더 낫다라던가
아마 시급해보이는게 isRowContinousTrue일 거같은데 함수는 사다리 게임을 할때 사다리가 연속적으로 가로선이 그어지면 안되잖아요? 그걸 구별하기 위한 함수 네이밍으로 적었는데 별로일려나요?
Portrait
is와 True 가 같이 있는게 좀 이상하고..
사다리관련된 용어가 하나도 없어서 연속적인 선을 왜체크하는지 몰랏던거같아여
지금 코드를 이해해보려고 노력중.. 저두 고수아니라서..
좀 이상하긴한데
mapToObj 와 foreach가 섞어서 돌아간다고 가정하자 (아니면 iob날거같음)
그랬을때 isRowContinousTrue 는 결국 idx가 even이냐 아니냐로 참거짓이 갈리는거같은데 너무어렵게짠거아닐까
TQQQ
오오 조언감사합니다
스비니
strategy가 관여하니 even이냐 아니나는 아닐 것 같아요.
Portrait
내가 쓴 내용에서
isRowContinousTrue 가 even 이냐 아니냐이고
drawLine 에 있는 strategy 는 상관없엉
스비니
짝수냐 아니냐 말씀하시는 거 맞는 거죠? 제가 다른 용어인데 착각하나 싶어서 물어봐요..
drawLine이 해당 index를 true로 만들어놔야만, 다음 index에서 RowContinuous를 확인할 때 true가 반환되는 거 아닌가요?
Portrait
아 다시보니 너말이맞는거가타
Portrait
그리구 그냥 어레이리스트는 스레드세이프하지않을거라서 다른 컬렉션을 써야할거같고..
Portrait
그리고 계속 보다보니 느낀건데
사수가 말한거는 custom collector 만들라는게 아니고
미리 points 선언하는 대신 forEach 를 없애고 collect 써서 한번에 만들라는거 아닐까
그럼 동시성 이슈도 없어지니까
굳이 collector.of 를 써서 만들 필요가 있나 싶은데
혜안을 얻게 되면 나두 공유해줘
TQQQ
아 리뷰진행했는데 일단 가독성이 우선시되는 토이프로젝트라 동시성 관련하여 코멘트를 남겨주시긴했네요.
Portrait님께서 말씀하신거처럼 동시성을 해결하기 위해서는 따로 작업이 들어가야할거같습니다.