글 읽기 - 너무 복잡하게 해서 런타임에러가 뜨는걸까요ㅠㅠ

ljs921026 6년 전 0

출력은 맞게 나오는데 런타임에러가 계속 나오네요ㅠㅠ

#include<stdio.h>
#include<stdlib.h>
int main(void)
{
	int Testcase;
	scanf("%d", &Testcase);

	int** Score = (int**)malloc(sizeof(int*)*Testcase);
	double* Average = (double*)calloc(Testcase, sizeof(double));
	double* Ratio = (double*)calloc(Testcase, sizeof(double));

	for (int i = 0; i < Testcase; i++)
	{
		int Num;
		scanf("%d", &Num);
		Score[i] = (int*)malloc(sizeof(int)*(Num + 1));
		Score[i][0] = Num;
		for (int j = 1; j <= Num; j++)
		{
			int NumScore;
			scanf("%d", &NumScore);
			Score[i][j] = NumScore;
			Average[i] += Score[i][j];
			if (j == Num)
				Score[i][++j] = '\0';
		}
		Average[i] /= Num;
	}

	for (int i = 0; i < Testcase; i++)
	{
		double sum = 0;
		double num = 0;
		for (int j = 1; Score[i][j] != '\0'; j++)
		{
			sum++;
			if (Score[i][j]>Average[i])
				num++;
		}
		Ratio[i] = (num / sum)*100;
	}

	for (int i = 0; i < Testcase; i++)
		printf("%.3lf\n", Ratio[i]);

	free(Score);
	free(Average);
	free(Ratio);

	return 0;
}

bupjae 6년 전 0

25번째 줄에서 할당받은 메모리를 넘어서는 곳을 사용하고 있습니다.

배열의 끝을 0으로 표기하기 위해 사용한 것으로 보이는데, 이 문제의 경우 0점 또한 입력 데이터로 들어올 수 있기 때문에 이런 방식을 사용하면 안 됩니다.

jh05013 6년 전 1

저는 저걸 어떻게 돌려야 되는지도 모르겠습니다.

저렇게 복잡하게 malloc과 calloc을 할 필요가 없습니다. 그냥 충분히 큰 크기의 전역변수 배열을 만들면 됩니다.

jh05013 6년 전 0

(추가) 실무에서는 전역변수 배열을 잡는 것이 바람직하지 못할 지도 모르나 (이쪽은 잘 모릅니다), 적어도 알고리즘 문제에서는 그렇습니다.

댓글을 작성하려면 로그인해야 합니다.

문제

문제

출처

ICPC

너무 복잡하게 해서 런타임에러가 뜨는걸까요ㅠㅠ

ljs921026 6년 전 0

bupjae 6년 전 0

jh05013 6년 전 1

jh05013 6년 전 0

Baekjoon Online Judge

채점 현황

문제

유저 대회

고등학교 대회

출처

대학교 대회

도움말