인프런 커뮤니티 질문&답변
작성자 없음
작성자 정보가 삭제된 글입니다.
3-J 주난이의 난
작성
·
50
0
안녕하세요 큰돌선생님
선생님 덕분에 점차 백준 문제들을 생각하고 풀수 있게 되고 골드 문제까지도 한번씩 맞추다 보니 행복하게 코테 준비를 하고 있습니다. 감사합니다
https://www.acmicpc.net/submit/14497/83745509
위의 링크처럼 코드를 작성하면 시간초과가 발생하지만
dfs() 함수 부분에서 if(arr[y][x] == '0') 이부분을 없애고 아래와 같이 고치면 맞다고 뜨는데 어떤 이유에서 그런걸까요?
for(int i=0; i<4; i++){
int ny = y + dy[i];
int nx = x + dx[i];
if(ny < 0 || nx < 0 || ny >= n || nx >= m) continue;
if(visited[ny][nx]) continue;
if(arr[y][x] == '1'){
v.push_back({y,x});
continue;
}
dfs(ny,nx);
}
답변 3
0
안녕하세요 미나리님 ㅎㅎ
먼저 코드리뷰를 드리면요.
if(arr[ny][nx] == '0')
dfs(ny,nx);
if(arr[y][x] == '1'){
v.push_back({y,x});
continue;
}이런 코드는 좋지 않습니다. ny, nx로 방문처리 등을 판단하는데 -> 그다음 갑자기 지금의 y, x를 기반으로 로직이 있는 것은 이상합니다. 로직자체가 꼬일 가능성이 높은 코드입니다.
예를 들어 00011 이라고 가정했을 때.
ny, nx가 0일 떄 -> dfs를 거는데 만약 왼쪽 끝 0에서 시작을 하면 ->
dfs -> dfs -> dfs 까지 왔고 그 다음 1까지 dfs가 안갈 것 같습니다. ny, nx가 1을 가리키는데 -> arr[y][x]는 0만을 가리키기 때문입니다. 이 때문에 올바른 continue가 일어나지 않을 수 있습니다.
if(visited[ny][nx]) continue;
if(arr[y][x] == '1'){
v.push_back({y,x});
continue;
}
dfs(ny,nx);
}이 코드 또한 이상하지만, 만약 지금의 y, x가 1이라면 -> 지워야 하기 때문에 일단 넘어가 -> 그게 아니라면 -> 0이기 때문에 dfs 계속 호출 -> 이런 꼴이기 때문에 continue도 잘 일어나서 맞는 것 같습니다.
또 질문 있으시면 언제든지 질문 부탁드립니다.
좋은 수강평과 별점 5점은 제게 큰 힘이 됩니다. :)
감사합니다.
강사 큰돌 올림.
0
0
