我的问题是我的整数数组的值在传递给函数 calculate 时会发生变化。索引0和2-> 5的值是正确的。
由于某种原因,索引1和6+不是正确的值。
以下是我的代码。
#include <stdio.h>
#include <time.h>
#include <stdlib.h>
int* generate_rand (int length, int MAX_ARRAY);
void calculate (int *array_ptr, int length, int *mean, int *sd);
main() {
srand(time(NULL));
int a;
printf("\nArray length?: ");
scanf("%d", &a);
int* array_ptr2;
array_ptr2 = generate_rand(a, 100);
//int mean, sd;
int* *mean;
int* *sd;
int i = 0;
for (i = 0; i < 10; i++) {
printf("Array2: %d\n", *(array_ptr2 + i));
}
calculate(array_ptr2, a, *mean, *sd);
//printf("Mean: %d\n", (int)*mean);
}
int* generate_rand (int length, int MAX_ARRAY) {
int arr[length];
int i;
for (i = 0; i < 10; i++) {
int r = rand()%MAX_ARRAY;
arr[i] = r;
printf("Rand: %d\n", arr[i]);
}
int *arrPtr;
arrPtr = &arr[0];
return arrPtr;
}
void calculate (int *array_ptr, int length, int *mean, int *sd) {
int sum;
int i;
for (i = 0; i < length; i++) {
printf("Array: %d\n", *(array_ptr + i));
sum += *(array_ptr + i);
//array_ptr++;
printf("Sum: %d, i:%d\n", sum, i);
}
//*mean = sum / length;
}
你知道我做错了吗?
答案 0 :(得分:3)
generate_rand(
)返回一个指向arr[0]
的指针,其中arr
是一个局部变量。一旦函数generate_rand()
返回,局部变量就会超出范围。一旦变量超出范围,就无法保证其值,事实上,访问超出范围的变量是未定义的。
一种可能的解决方案:generate_rand()
可以在堆上分配数组arr
,即使用malloc()
分配内存。不用说,一旦不再需要,记忆必须free()
。
答案 1 :(得分:3)
问题是您要返回函数的局部变量的地址。函数的局部变量在函数退出后不再存在,因此返回的指针突然变为悬空(如,不指向任何有用的东西)。
您需要使用malloc
为数组分配内存,因为从malloc
返回的内存将一直存在,直到它被free
释放(之后可以随时调用)。< / p>
int* generate_rand (int length, int MAX_ARRAY) {
int *arr = malloc(sizeof(int) * length);
int i;
for (i = 0; i < 10; i++) {
int r = rand()%MAX_ARRAY;
arr[i] = r;
printf("Rand: %d\n", arr[i]);
}
return arr;
}
当您不再需要从此函数返回的数组时,将其传递给free
以释放内存,例如:
int *arr = generate_rand(10, 100);
// do something with arr here...
free(arr);
答案 2 :(得分:1)
正如您在注释中所述,generate_rand()
需要返回一个指向充满随机值的数组的指针。此时,您将返回指向本地数组的第一个元素的指针。正如其他人所说,这个数组的内存只能保证在generate_rand()
函数内的程序可用。要创建可在程序中的任何位置使用的数组,需要使用malloc()
分配内存。我建议你进一步研究内存管理。您需要了解malloc()
和free()
函数。