-
Notifications
You must be signed in to change notification settings - Fork 0
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
base: main
Are you sure you want to change the base?
The head ref may contain hidden characters: "\uCD5C\uB2E8\uACBD\uB85C"
[최단경로] 11월 1일 #11
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
p3. 3085 코드 리뷰 완료
안녕하세요 영은님~
문제 푸시느라 정말 고생 많으셨어요!!👍👍
코드 중복과 관련해서 약간의 코멘트를 달았습니다. 솔루션 링크를 참고해주세요😉
궁금한 점 있으시면 리뷰어를 호출해주세요~
수고하셨습니다!!
//행의 최대값 | ||
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); | ||
} |
There was a problem hiding this comment.
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
[추가제출 확인 완료] |
There was a problem hiding this 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 범위 참고 |
There was a problem hiding this comment.
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; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
const int 로 선언해주신거 좋습니다!!👍👍
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으로 가는 경우 |
There was a problem hiding this comment.
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번 진행하지 않아도 풀이할 수 있답니다!
샘플코드와 함께 참고해주시면 좋을 것 같아요😊
내용 & 질문
<기존 제출>
<추가 제출>