Skip to content

[백트래킹] 5월 1일#9

Open
song122333 wants to merge 2 commits intomainfrom
09-assignment
Open

[백트래킹] 5월 1일#9
song122333 wants to merge 2 commits intomainfrom
09-assignment

Conversation

@song122333
Copy link
Collaborator

###인적사항
학번: 2391035
이름: 한송희

###과제제출
기존 제출: 14888,10971
추가 제출: 20055

Copy link

@sforseohn sforseohn left a comment

Choose a reason for hiding this comment

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

[백트래킹 구현 문제 추가 제출 확인]

송희님 안녕하세요! 구현 문제 추가 제출해주신 것 확인했습니다. 주석 보니 잘 이해하신 것 같아요~! 나중에 비슷한 유형의 문제도 풀어보시는 거 추천 드려요 👍
https://www.acmicpc.net/workbook/view/1152

Copy link

@ZERO-black ZERO-black left a comment

Choose a reason for hiding this comment

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

알고리즘 문제 코드리뷰가 완료되었습니다.
리뷰에 대한 질문이 있으시면 언제든지 슬랙 DM으로 찾아와주세요~!

  1. 지금 기본 배열을 사용해주셨는데, 기본적으로 벡터를 사용하시는 습관을 들이시는 것을 추천드립니다. 벡터가 훨씬 유연한 자료구조이기 때문에, 활용성이 높아요!
  2. 알튜비튜의 코딩 컨벤션을 따라주시면 좋겠습니다. 변수 이름을 규칙에 맞추는 연습도 중요해요!

Comment on lines +34 to +39

Choose a reason for hiding this comment

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

p2. 문제의 조건은 한바퀴 도는 것이었어요!
만약 cycle이 존재한다면, 어느 도시에서 출발하든 한 바퀴 돌아 다시 돌아올 수 있어요.
다시 말해, 출발을 어디서 하는지는 중요하지 않답니다!

Choose a reason for hiding this comment

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

P2. 지금 목표가 최솟값을 찾는 것이니, 그보다 큰 값은 끝까지 탐색하지 않아도 괜찮아요!
이 조건을 넣어주면 탐색의 효율이 더 높아질 수 있습니다!

Comment on lines +18 to +24

Choose a reason for hiding this comment

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

메인 로직 아주 깔끔합니다!

Comment on lines +20 to +34

Choose a reason for hiding this comment

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

p3. 이 부분을 함수로 빼주면 조금 더 깔끔한 코드가 될 것 같아요!

Comment on lines +30 to +32

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.

3 participants