Skip to content

Conversation

Inryu
Copy link
Member

@Inryu Inryu commented Nov 7, 2021

필수 #15685, 선택 #1504 추가제출하겠습니다!

Copy link

@bsa0322 bsa0322 left a comment

Choose a reason for hiding this comment

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

p2. 코드가 깔끔하네요!! 추가제출 해주신다 한 부분도, 혹시 힌트 필요하시면 언제든 말씀해주세요!
1063은 틀렸습니다가 나오는데 확인 부탁드려요! 관련 코멘트 남겼습니다. 나머지 부분은 모두 제출 확인했어요! p2가 붙은 부분은 여유 되신다면 수정해보셔도 좋을 것 같아요 😊

수정 후, 저 다시 리뷰어로 호출해주세요~!! 수고하셨습니다

Comment on lines +13 to +31
Copy link

Choose a reason for hiding this comment

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

p2. 함수 사용도 좋지만, "LB"에 접근하기까지 8번 연산을 해야 하니 검색에 O(logN)이 걸리는 map을 사용해봐도 좋을 것 같아요!

Comment on lines +39 to +40
Copy link

Choose a reason for hiding this comment

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

king_r과 rock_r은 1~8사이의 수 일테니 절댓값 함수는 없어도 괜찮겠네요!

Comment on lines +66 to +75
Copy link

Choose a reason for hiding this comment

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

킹은 범위 안이고, 킹이 움직인 자리에 돌이 있어서 이동했을 때 만약 돌이 범위를 벗어났다면 어떻게 해야 할까요? 킹이 움직이는 게 맞을까요?

이 경우만 처리해주시면 주석 다신 부분 해결될 것 같아요!

Comment on lines +10 to +11
Copy link

Choose a reason for hiding this comment

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

visited는 bfs 함수에서만 사용하니 지역변수로 선언해도 괜찮겠네요~!!!

Comment on lines +14 to +15
Copy link

Choose a reason for hiding this comment

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

p2. deque로 정점만 관리하는 방법이 있어요! visited 함수를 활용해서 방문처리와 함께 시간을 관리해보면 어떨까요?

Comment on lines +5 to +6
Copy link

Choose a reason for hiding this comment

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

넵! 그래도 힌트 하나만 드릴께요...!

주어진 정점 2개를 시작 또는 도착 지점으로 생각하고, 여러 경로를 이어서 생각해보면 돼요!

Comment on lines +10 to +11
Copy link

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