我正在制作这个程序,其中我的main函数调用一个函数,该函数在计算后返回一个数组.我已经检查过计算是否在本地函数内部.但是当我将该数组返回到'main'函数时,我只能打印一次正确的值,并且在其他时间打印错误的值.
#include <math.h>
#include <stdio.h>
#include <string.h>
#include <stdlib.h>
#include <assert.h>
#include <limits.h>
#include <stdbool.h>
int* getJoinedPipes(int input1_size, int* input1,int* output_size){
int i,j,temp;
int op1[input1_size-1];
*output_size = input1_size - 1;
for(i=0; i<input1_size; i++){
for(j=i+1; j<input1_size; j++){
if(input1[i] > input1[j]){
temp = input1[i];
input1[i] = input1[j];
input1[j] = temp;
}
}
}
op1[0]=input1[0] + input1[1];
for(i=1;i<input1_size-1;i++){
op1[i] = op1[i-1]+input1[i+1];
}
//printf("%d\n",op1[2]);
return op1;
}
int main() {
int output_size;
int* output;
int ip1_size = 0;
int ip1_i;
scanf("%d\n", &ip1_size);
int ip1[ip1_size];
for(ip1_i = 0; ip1_i < ip1_size; ip1_i++) {
int ip1_item;
scanf("%d", &ip1_item);
ip1[ip1_i] = ip1_item;
}
output = getJoinedPipes(ip1_size,ip1,&output_size);
printf("a==%d\n",output[0]);
printf("a==%d\n",output[0]);
printf("a==%d\n",output[0]);
int output_i;
for(output_i=0; output_i < output_size; output_i++) {
printf("%d\n", output[output_i]);
}
return 0;
}
Run Code Online (Sandbox Code Playgroud)
输出应该是
5
9
15
Run Code Online (Sandbox Code Playgroud)
a==5
a==1943372821
a==1943372821
1943372821
17
6356632
Run Code Online (Sandbox Code Playgroud)
您可以看到它第一次给出正确的值(5),然后在同一打印时看到它给出的垃圾值.
op1是一个自动数组.您不能将其退回并在其范围之外使用它.
op1仅存在于其中getJoinedPipes,如果返回,则结果为Undefined Behavior.
要修复它,您可以:
op1作为参数传递给getJoinedPiped op1在动态上分配在堆上.你做到了,你可以放心地返回,op1但是free当你不需要它时你必须记住它.