코드 리뷰 좀 부탁드려요

Question

/*https://codeup.kr/problem.php?id=1805다음 문제에 대해 코드를 작성했습니다.입력은 예시와 동일하고, 제 코드에 대한 출력은 나오지 않았습니다.굳이 포인터, 사용자 지정 함수를 쓴 이유는 포인터와 함수의 사용에 익숙해지기 위함이었습니다.gpt4에게도 물어보고 노트로 자체 트래킹을 통한 디버깅도 했지만 문제점이 발견되지 않았습니다.무슨 문제가 있는 건지 알려주시면 감사하겠습니다.(구조체가 아니라 2차원 배열과 배열 포인터를 썼다면 더 좋았겠다는 생각은 드네요...)*/ #include typedef struct { int a; int b; int n; } air; int trychan(air* n2, int i) { air t; if (n2[i].a > n2[i + 1].a) { t = *(n2 + i); *(n2 + i) = * (n2 + i + 1); *(n2 + i + 1) = t; return 1; } return 0; } void selection(air* n2) { for (int i = 0; i < n2[0].n; i++) { for (int j = 0; j < n2[0].n - i - 1; j++) { trychan(n2, j); } } } int main() { air n2[101] = { 0 }; int i; scanf_s("%d", &n2[0].n); for (i = 0; i < n2[0].n; i++) { scanf_s("%d %d", &n2[i].a, &n2[i].b); } selection(n2); for (i = 0; i < n2[0].n; i++) { printf("%d %d ", n2[i].a, n2[i].b); } return 0; } 코드가 붙어 나온 거는 죄송합니다ㅠㅠ

비정규분포 · Accepted Answer

왜 n을 구조체에다 넣으셨나요. selection()에서 셔플링하고 나면 0으로 초기화된 엉뚱한 구조체의 n을 다시 읽게되는군요.scanf_s() 사용법도 잘못됬어요. 컴파일러 경고가 떴을건데, MSVC가 호들갑 떠는거고 솔직히 안써도 되요. 다른 os는 전혀 불평불만 안합답니다.#include #include typedef struct { int a; int b; } air; int cnt; int trychan(air* n2, int i) { air t; if (n2[i].a > n2[i + 1].a) { t = *(n2 + i); *(n2 + i) = * (n2 + i + 1); *(n2 + i + 1) = t; return 1; } return 0; } void selection(air* n2) { for (int i = 0; i < cnt; i++) { for (int j = 0; j < cnt - i - 1; j++) { trychan(n2, j); } } } int main() { // GNU C 초기화 방법 // air n2[101] = { 0, }; air n2[101]; int i; // 혹은 이렇게 초기화 memset(n2, 0, sizeof(n2)); scanf("%d", &cnt); for (i = 0; i < cnt; i++) { scanf("%d %d", &n2[i].a, &n2[i].b); } selection(n2); for (i = 0; i < cnt; i++) { printf("%d %d ", n2[i].a, n2[i].b); } return 0; }

mirheeoj · Answer

scanf_s가 빌드가 안되는 환경이 있을 수 있겠고, scanf로 대체해도 코테에는 문제 없는 상황. 그리고 selection(n2); 를 수행한 뒤에 n2[0].n이 0으로 바뀝니다. 즉 trychan내에서 잘못된 값을 덮어씌우는 게 아닐까 생각이 되네요. 디버거로 스텝실행하면서 뭐가 문제인지 확인해 보시는게 좋겠습니다.

코드 리뷰 좀 부탁드려요

2개의 답변

기술 목록