BFS 당근마켓 승원이 문제 질문이 있습니다!
안녕하세요! 큰돌님. 문제를 풀다가 자꾸 visited 배열에 오버플로우가 일어나는 것 같아서 예시 코드와 비교해 봤더니, 큰돌님은 max_n = 104로 배열의 크기를 최대로 정의 해 놓으셨더라구요.
그래서 큰돌님 처럼 다음과 같이 배열의 크기를 최대로 해놓고 하니, 문제가 해결되었습니다.
const int max_n = 104;저는 입력받는 코드를 따로 만들지 않아서, 크기를 예측할 수 있는 코드여서 5X5 로 정의를 해놓았는데, 왜 오버플로우가 일어나는 지 궁금합니다!
제가 작성했던 코드 첨부하겠습니다.
// Online C++ compiler to run C++ program online
#include <bits/stdc++.h>
using namespace std;
const int max_n = 104;
int dy[4] = {-1, 0, 1, 0};
int dx[4] = {0, 1, 0, -1};
int main() {
int N = 5;
int M = 5;
int x,y;
int visited[N][M] = {0,}; //방문 노드
//시작 지점
int sx = 0, sy = 0;
//끝 지점
int ex = 0, ey = 4;
//MAP
int map[N][M] =
{
{1,0,1,0,1},
{1,1,1,0,1},
{0,0,1,1,1},
{0,0,1,1,1},
{0,0,1,1,1}
};
queue<pair<int, int>> q;
//깊이 탐색을 위한 큐
visited[sy][sx] = 1; // start 위치 방문 처리
q.push({sy,sx});
while(q.size())
{
tie(y,x) = q.front(); q.pop();
for(int i = 0; i < 4 ; i++)
{
int ny = y + dy[i];
int nx = x + dx[i];
if(ny < 0 || ny >= N || nx < 0 || nx >= M || map[ny][nx] == 0) continue;
if(visited[ny][nx]) continue;
visited[ny][nx] = visited[y][x] + 1;
q.push({ny, nx});
}
}
printf("%d\n", visited[ey][ex]);
// 최단거리 디버깅
for(int i = 0; i < N; i++){
for(int j = 0; j < M; j++){
cout << visited[i][j] << ' ';
}
cout << '\n';
}
return 0;
}
답변 1
1
안녕하세요 kamin님 ㅎㅎ
저는 입력받는 코드를 따로 만들지 않아서, 크기를 예측할 수 있는 코드여서 5X5 로 정의를 해놓았는데, 왜 오버플로우가 일어나는 지 궁금합니다!
>> 오버플로우가 아니라 이 코드 자체가 에러가 떠서 그렇습니다.
혹시 이 코드 컴파일러로부터 에러가 뜨지 않았나요?

이 코드는
int main() {
int N = 5;
int M = 5;
int x,y;
int visited[N][M] = {0,}; //방문 노드다음 부분 때문에 에러가 뜨는 코드입니다.
C++에서는 단순한 array를 기반으로 N을 정의해서 동적할당을 하려고 한다면 다른 방법을 사용해야 합니다.
대표적으로 malloc 을 사용합니다.
하지만 그럴 바에는 vector를 기반으로 resize()로 동적할당하시는 것을 추천드립니다.
그래서 저 코드는...
// Online C++ compiler to run C++ program online
#include <bits/stdc++.h>
using namespace std;
const int max_n = 104;
int dy[4] = {-1, 0, 1, 0};
int dx[4] = {0, 1, 0, -1};
int main() {
int N = 5, M = 5;
int x,y;
int visited[5][5] = {0,}; //방문 노드
//시작 지점
int sx = 0, sy = 0;
//끝 지점
int ex = 0, ey = 4;
//MAP
int map[5][5] =
{
{1,0,1,0,1},
{1,1,1,0,1},
{0,0,1,1,1},
{0,0,1,1,1},
{0,0,1,1,1}
};
queue<pair<int, int>> q;
//깊이 탐색을 위한 큐
visited[sy][sx] = 1; // start 위치 방문 처리
q.push({sy,sx});
while(q.size())
{
tie(y,x) = q.front(); q.pop();
for(int i = 0; i < 4 ; i++)
{
int ny = y + dy[i];
int nx = x + dx[i];
if(ny < 0 || ny >= N || nx < 0 || nx >= M || map[ny][nx] == 0) continue;
if(visited[ny][nx]) continue;
visited[ny][nx] = visited[y][x] + 1;
q.push({ny, nx});
}
}
printf("%d\n", visited[ey][ex]);
// 최단거리 디버깅
for(int i = 0; i < N; i++){
for(int j = 0; j < M; j++){
cout << visited[i][j] << ' ';
}
cout << '\n';
}
return 0;
}
이렇게 수정하시는게 좋습니다. (올바르게 나옵니다.)

그리고 kamin님 코드리뷰를 좀 드리면요.
int main() {
int N = 5, M = 5; 지역변수보다는 전역변수를 쓰는게 좋습니다.
해당 부분은 교안내의 다음 부분 참고해주세요.
지역변수 보다는 전역변수를, 변수명을 간결하게.
int visited[5][5] = {0,}; //방문 노드0으로 초기화하는 것은 교안내의 다음부분을 참고해주세요.
쓰지 말아야 할 초기화 방법 {0, }
다른 부분들은 잘 짜셨습니다. ㅎㅎ
또 질문 있으시면 언제든지 질문 부탁드립니다.
좋은 수강평과 별점 5점은 제게 큰 힘이 됩니다. :)
감사합니다.
강사 큰돌 올림.
6-H 체점 관련 질문
0
7
0
채점서버 연결 관련 질문입니다
0
31
1
삼성 s직군
0
31
0
삼성 코테 없어짐
0
81
1
코딩살구클럽 가입부탁드립니다
0
40
2
코딩살구클럽 가입 요청 확인부탁드립니다
0
35
2
5-S 테스트 케이스 질문
0
37
2
코살 문제풀이 환경
0
54
2
2 - T 오큰수 문제가 있는 것 같습니다.
0
46
1
추천 추가문제들
0
44
2
프로그래머스 코테 환경 관련해서 질문드립니다.
0
50
2
해당 문제에 대한 채점이 코딩살구클럽에서 올바르게 처리되지 않습니다.
0
44
2
균형 이진 트리 설명 시 높이 숫자
0
33
2
4-H 질문드립니다.
0
37
2
1-K 질문드립니다.
0
43
2
대기업 인적성 시험 질문
0
43
2
4-C 질문드립니다
0
43
2
[수학숙제 / BOJ 2870] 채점 서버 오작동
0
40
1
코테 준비 질문
0
58
1
살구클럽가입 요청드려요
0
42
2
1-I 문제 질문
0
46
2
코딩살구클럽 가입
0
61
2
AI 코딩 도구 사용 시 학습 방법 조언
0
56
2
코딩살구클럽 오류
0
67
2





