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

[최단경로] 11월 1일 #11

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

[최단경로] 11월 1일 #11

wants to merge 2 commits into from

Conversation

yekim8
Copy link
Collaborator

@yekim8 yekim8 commented Nov 6, 2022

내용 & 질문

제출합니다!

<기존 제출>

1504, 1719, 3085

<추가 제출>

16235

Copy link
Member

@dooli1971039 dooli1971039 left a comment

Choose a reason for hiding this comment

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

p3. 3085 코드 리뷰 완료
안녕하세요 영은님~
문제 푸시느라 정말 고생 많으셨어요!!👍👍
코드 중복과 관련해서 약간의 코멘트를 달았습니다. 솔루션 링크를 참고해주세요😉
궁금한 점 있으시면 리뷰어를 호출해주세요~
수고하셨습니다!!

Comment on lines +13 to +41
//행의 최대값
int r_cnt = 0;
for (int i = 0; i < n; i++) {
int cnt = 1;
for (int j = 0; j < n - 1; j++) {
if (candy[i][j] != candy[i][j + 1]) {
r_cnt = max(r_cnt, cnt);
cnt = 1;
continue;
}
cnt++;
}
r_cnt = max(r_cnt, cnt);
}

//열의 최대값
int c_cnt = 0;
for (int j = 0; j < n; j++) {
int cnt = 1;
for (int i = 0; i < n - 1; i++) {
if (candy[i][j] != candy[i + 1][j]) {
c_cnt = max(c_cnt, cnt);
cnt = 1;
continue;
}
cnt++;
}
c_cnt = max(c_cnt, cnt);
}
Copy link
Member

Choose a reason for hiding this comment

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

p3. 코드 중복을 조금 줄일 수 있을 것 같습니다! 이 부분은 솔루션 링크를 참고해주세요! (지금도 너무 좋습니다!)
https://github.com/Altu-Bitu-3/Notice/blob/main/11%EC%9B%94%2001%EC%9D%BC%20-%20%EC%B5%9C%EB%8B%A8%20%EA%B2%BD%EB%A1%9C/%ED%95%84%EC%88%98/3085.cpp

@dooli1971039
Copy link
Member

[추가제출 확인 완료]
안녕하세요 영은님~ 16235번 추가제출 확인 완료되셨습니다🥰
이번 과제도 수고 많으셨습니다:)

Copy link

@0321minji 0321minji left a comment

Choose a reason for hiding this comment

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

1504 코드리뷰 완료
안녕하세요 영은님!
제출해주신 코드 잘봤습니다😊😊

INF 값 선언과 다익스트라 실행에 대해 간단한 코멘트 남겨드렸으니 샘플코드와 함께 참고 부탁드려요!
혹시 코드 수정하신다면 리뷰어로 호출해주세요~!
수고하셨씁니다🙌🙌

#include <vector>
#include <queue>

//https://steady-coding.tistory.com/82 92%에서 실패가 떠 INF 범위 참고
Copy link

@0321minji 0321minji Nov 9, 2022

Choose a reason for hiding this comment

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

p3. 참고하신 부분 주석 달아주신거 너무 좋아요! 저도 들어가서 읽어보았는데, 해당 간선의 최대개수 * 가중치를 INF 값으로 선언하여도 좋지만,
해당 문제가 최단 경로를 구하는 문제라는 것을 고려하면, 지나가는 간선은 최대 (N-1)개이며(N개의 정점) 중복 순회가 가능하므로 (3) 을 곱하여 1e5 * 8 * 3 을 INF 값으로 설정할 수 있어보이네요!


using namespace std;
typedef pair<int, int> ci;
const int INF = 200000000;

Choose a reason for hiding this comment

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

const int 로 선언해주신거 좋습니다!!👍👍

Comment on lines +57 to +61
int st_to_v1 = dijkstra(1, n, v1, graph); //1에서 v1으로 가는 경우
int st_to_v2 = dijkstra(1, n, v2, graph); //1에서 v2로 가는 경우
int v1_to_v2 = dijkstra(v1, n, v2, graph); //v1에서 v2 혹은 v2에서 v1으로 가는 경우
int v1_to_n = dijkstra(v1, n, n, graph); //v1에서 n으로 가는 경우
int v2_to_n = dijkstra(v2, n, n, graph); //v2에서 n으로 가는 경우

Choose a reason for hiding this comment

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

p2. 해당 풀이로 다익스트라를 각각 구하셔도 괜찮지만 1, v1, v2에 대해서 3번 다익스트라를 진행한 후 결과를 배열로 받아오면 5번 진행하지 않아도 풀이할 수 있답니다!
샘플코드와 함께 참고해주시면 좋을 것 같아요😊

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