작성한 질문수 3

0

안녕하세요 ㅎㅎ

잘 짜셨는데요. ㅎㅎ 살짝 틀리신 부분이 있습니다. 한번 다듬어봤는데요. 참고부탁드립니다.

#include <bits/stdc++.h>
using namespace std;

int day;
int n, l, r;
int a[54][54];
int visited[54][54];
int dy[] = {-1, 0, 1, 0};
int dx[] = {0, 1, 0, -1};
int tot;
int cnt;
vector <pair<int, int>> v;

void dfs(int y, int x) {
    visited[y][x] = 1;
    tot += a[y][x];
    cnt++;
    v.push_back({y, x});
    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 >= n) continue;
        if (visited[ny][nx]) continue;
        if (abs(a[ny][nx] - a[y][x]) > r || abs(a[ny][nx] - a[y][x]) < l) continue;
        dfs(ny, nx);
    }
}

int main() {
    cin >> n >> l >> r;
    for (int i = 0; i < n; i++) {
        for (int j = 0; j < n; j++) {
            cin >> a[i][j];
        }
    }

    while (true) {
        int flag = 0;
        fill(&visited[0][0], &visited[0][0] + 54*54, 0); // 초기화
        for (int i = 0; i < n; i++) {
            for (int j = 0; j < n; j++) {
                if (!visited[i][j]) {
                    tot = 0;
                    cnt = 0;
                    v.clear();
                    dfs(i, j);
                    if (cnt > 1) { // 연합이 형성된 경우
                        tot /= cnt;
                        for (auto it : v) {
                            a[it.first][it.second] = tot;
                        }
                        flag = 1;
                    }
                }
            }
        }
        if (!flag) break; // 더 이상 인구 이동이 없으면 종료
        day++;
    }

    cout << day << '\n';
    return 0;
}

그리고 선생님 풀이코드를 봤는데, 전역스코프에있는 벡터 v사용하시는데 dfs 파라미터에 넣으신 이유가 있을까요??

>> 넣어도 되고 안넣어도 됩니다. 사실 저건 불필요하게 넣은 부분입니다. 수정하도록 하겠습니다.

이 질문을 드린게 함수 파라미터에서 '&'를 이용한 참조에 의한 전달을 하는 방식은 함수 파라미터에 해당 자료구조를 명시하지 않아도 상위 스코프를 순차적으로 순회하면서 동일한 이름의 자료구조를 발견하면 알아서 참조해서 사용한다고 알고있어서 그랬습니다.

>> 전역변수로 선언할 경우 해당 변수를 참조하기 때문에 이 때는 매개변수로 하는게 불필요하며 함수전에 지역변수로 설정할 경우 참조로 넘기는게 좋습니다.

ds를 비워주는(fill, memset, clear, ...) 로직을 수행하지 않고 코드를 작성할 수 있을것 같은데, 이렇게 매번 빈 자료구조를 사용하는 로직에서는 값전달방식이 나을까요 아니면 그냥 무조건

참조 + flush가 나을까요?

>> 전역이 아니라 지역으로 선언 -> ex) vector<int> v 선언매번 -> pushback 이렇게 하는걸 비교하는것이죠? 문제지문마다 성능차이가 다를 것 같지만.. 일반화해서 한번 제가 테스팅해봤습니다.

결과는 매번 선언보다 -> clear가 더 빠릅니다.

테스팅코드

#include <bits/stdc++.h>
#include <profile.hpp>

std::vector<int> global_vec;

void test_vector_local() {
    for (int i = 0; i < 1000; i++) {
        if (i % 100 == 0) {
            PROFILE_START("local vector insert 1000");
            std::vector<int> v;
            for (int j = 0; j < 1000; j++) {
                v.push_back(j);
            }
            PROFILE_STOP("local vector insert 1000");
        }
    }
}

void test_vector_global() {
    for (int i = 0; i < 1000; i++) {
        if (i % 100 == 0) {
            PROFILE_START("global vector insert and clear 1000");
            for (int j = 0; j < 1000; j++) {
                global_vec.push_back(j);
            }
            global_vec.clear();
            PROFILE_STOP("global vector insert and clear 1000");
        }
    }
}

int main() {
    PROFILE_RUN_ALL(1000, 100,
        test_vector_local();
        test_vector_global();
    );

    return 0;
}

https://perfbench.com/

또 질문 있으시면 언제든지 질문 부탁드립니다.

좋은 수강평과 별점 5점은 제게 큰 힘이 됩니다. :)

감사합니다.

강사 큰돌 올림.

0

안녕하세요, 인프런 AI 인턴입니다.

ghtkdrla321님의 질문에 대해, 비슷한 문제를 해결하는 데 유용한 자료를 찾았습니다.

질문하신 16234 문제의 경우, 특히 dfs 함수에서 전역 변수 벡터 v 사용과 관련하여, 다음의 답변이 도움이 될 수 있습니다:

• “dfs 참조 관련 질문이 있습니다.”
  

이 글에서는 dfs 함수에 매개변수로 v를 참조로 명시하는 이유에 대해 설명하고 있습니다. 매개변수로 넘기지 않고 전역 변수로 사용하는 경우와 비교하여 설명하고 있기 때문에, ghtkdrla321님의 질문에 대한 해답을 찾는 데 도움이 될 것입니다.

다만, 정확히 어느 부분에서 문제가 발생했는지에 대해서는 더욱 구체적인 코드 검사가 필요할 수 있으며, 이는 답변에서 직접적으로 해결되지 않을 수 있습니다. 그럼에도 이 정보가 문제의 해결점을 찾는 데 있어 유용한 출발점이 되길 바랍니다.

저는 질문자님의 이해를 돕고자 지식공유자님의 기존 답변을 참고하여 유사한 답변 링크를 추천해드리고 있습니다.
현재 베타 기능으로 답변이 만족스럽지 않을 수 있는 점 양해 부탁드립니다. 🙏
추가적으로 궁금한 점이 있으시면, 이어서 질문해 주세요. 곧 지식공유자께서 답변해 주실 것입니다.