작성
·
285
0
http://boj.kr/13cd857dcaae4b858119dc6057a36655
DFS를 이용해서 풀었습니다. 전체적인 로직은 큰돌님이 보여주신 코드랑 다를 바 없는 것 같은데 cnt (큰돌님 코드에서는 ret) 값이 고정 1로 나옵니다.
아무리 봐도 어디가 잘못된건지 모르겠습니다..
답변 1
0
안녕하세요 01님ㅎㅎ
if(ny < 0 || ny >= n || nx < 0 || nx >= 0) continue;
if(vis[y][x] || !a[y][x]) continue;
이부분이요. vis[ny][nx].. 이런식으로 체크해야 되지 않을까요?
감사합니다.
음.. 틀린부분을 또 하나 발견햇는데요.
이거 a[i][j].. 식으로 넘겨야 하는게 아닐까요?
for(int i = 0; i < n; i++)
for(int j = 0; j < m; j++)
if(a[y][x] && !vis[y][x]){
이렇게 고쳐보시구.
또 안될경우
void dfs(int y, int x)
{
vis[y][x] = 1;
이부분에다가 출력 디버깅을 넣어서 즉, 다음과 같이 만들어서 디버깅해보시구요.
void dfs(int y, int x)
{
vis[y][x] = 1;
cout << y << " : " << x << '\n';
그래도 잘 안될 경우 다시 코드 공유부탁드립니다.
다시 봐드릴게요.
감사합니다.
아 링크를 안남겼네요..
http://boj.kr/e8bba7b903d743b69a174e8126f0d9f5
출력초과 문제는 제가 실수로 디버깅 코드까지 넣어버려서 났던 것 같습니다. 삭제했지만 그럼에도 여전히 틀렸다고 나오네요..
주석 달았습니다. 주석 참고해주세요.
#include <bits/stdc++.h>
using namespace std;
int t, n, m, k, cnt, y, x;
int a[51][51], vis[51][51];
int dy[] = {-1, 0, 1, 0};
int dx[] = {0, 1, 0, -1};
void dfs(int y, int x)
{
vis[y][x] = 1;
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 >= 0) continue;
// 이렇게 고치셔야 합니다.
if(ny < 0 || ny >= n || nx < 0 || nx >= m) continue;
//if(vis[ny][nx] || !a[y][x]) continue;
// 이렇게 고치셔야 합니다.
if(vis[ny][nx] || !a[ny][nx]) continue;
dfs(ny, nx);
}
}
int main()
{
ios_base::sync_with_stdio(false);
cin.tie(NULL); cout.tie(NULL);
cin >> t;
while(t--)
{
cnt = 0;
fill(&a[0][0], &a[0][0] + 51 * 51, 0);
fill(&vis[0][0], &vis[0][0] + 51 * 51, 0);
cin >> m >> n >> k;
for(int i = 0; i < k; i++)
{
cin >> x >> y;
a[y][x] = 1;
}
for(int i = 0; i < n; i++)
for(int j = 0; j < m; j++)
if(a[i][j] && !vis[i][j]){
dfs(i, j);
cnt++;
}
cout << cnt << "\n";
}
}
감사합니다.
맞았습니다! 라고 뜨네요. 감사합니다.
그런데 주석 달아주신 내용 중에 dfs 함수 내부에서 a[ny][nx] 가 아니라 a[y][x]인게 이해가 가지 않습니다. 선생님 모범답안 코드에도 and, or만 다를뿐 a[ny][nx]를 사용하셨고 visited 배열은 ny와 nx를 검토하는데 a배열은 왜 y, x를 평가해야하는지 잘 이해가 가지 않습니다..
dfs 함수 내부에서 a[ny][nx] 가 아니라 a[y][x]인게 이해가 가지 않습니다.
>> ny, nx 로 하고 있습니다.
//if(vis[ny][nx] || !a[y][x]) continue;
// 이렇게 고치셔야 합니다.
if(vis[ny][nx] || !a[ny][nx]) continue;
또한, 한글로 표현해볼까요?
y, x : 지금 이 시점에서 방문처리를 한다. 라는 의미입니다.
다음 코드 : 내가 이시점에서 다음 방문지점으로 가기 전! 해당 부분의 방문처리를 확인한다는 의미입니다.
if(vis[ny][nx] || !a[ny][nx]) continue;
감사합니다.
아 그렇네요.. 감사합니다ㅠ
그런데 그렇게 바꿨는데도 틀렸다고.. 뜨네요ㅠ