Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[최단경로] 2071031 유서현 #346

Open
wants to merge 3 commits into
base: 2071031-유서현2
Choose a base branch
from

Conversation

ruruisryu
Copy link
Member

@ruruisryu ruruisryu commented May 27, 2023

인적사항

학번: 2071031
이름: 유서현


과제 제출

기존 제출 : 1238, 2458
추가 제출 : 15685

기존에 제출했던 두문제가 11주차 파일에 들어가 있었더라구요! 그부분도 수정했습니다.


@Dong-droid
Copy link
Contributor

[코드 리뷰 완료]
수고하셨습니다! 😸

Comment on lines 44 to 47
for(int i=1; i<=n; i++){
// tmp는 시작점에서 파티장소까지 갈 떄의 거리와 파티장소에서 시작점으로 돌아올 떄의 거리 합
tmp = dijkstra(nodes, i, dest, n) + dijkstra(nodes, dest, i, n);
// result와 tmp중 값이 더 큰 것으로 result를 갱신
Copy link
Contributor

@Dong-droid Dong-droid May 30, 2023

Choose a reason for hiding this comment

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

P3. 파티장소는 dest로 고정이기 때문에, 파티 장소에서 다른 정점들가지 걸리는 시간은 1번만 구해줘도 됩니다. 그래서, dijkstra(nodes,dest,i,n)은 한 번만 실행하는 게 더 효율적일 것 입니다!

Comment on lines 24 to 30
// 조건을 만족한다면 i와 j 사이의 대소 관계는 서로 알 수 있다는 뜻
if(graph[i][j] != INF || graph[j][i] != INF){
count++;
}
}
// 서로의 키 대소관계를 아는 사람의 수가 n-1 + 자기 자신은 항상 알수 있으므로 1
// 즉 count가 n이라면 자신의 키 순위를 알 수 있을 것이다.
Copy link
Contributor

@Dong-droid Dong-droid May 30, 2023

Choose a reason for hiding this comment

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

P3. 설명좋습니다 👍 근데 이부분은 floydwarshall과 크게 상관없기 때문에 다른 함수로 만들어주면 더 좋아요. 예를 들면, countStudent() 어떤가요?? 😄

Copy link
Contributor

@jk0527 jk0527 left a comment

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