c - 使用免费时程序 "sometimes"崩溃

Question

我的程序有时会崩溃，有时当我对数组中的元素调用 free() 时不会。数组中的元素是一个结构。我将展示一些代码:

//This first part might be a bit messy, and hard to understand but it is working
int main(int argc, char *argv[]){
      komplexNumber komplexNumbers[antal-1]; 
      int i;

      float temp;
      int realCounter=0;
      int imaginarCounter=0;
      int numberOfElements=6 // this variable is set by the user using scanf, and can be any number>=2. Each komplexNumber consists of two numbers (i.e 9 -3j is a complex number)

       for(i=0;i<numberOfElements*2;i++){ // 
           printf("Enter number %i\n", i+1);
           scanf("%f", &temp); 
           if(i%2==0){  //a complex number consts of two parts. first entered number is first part of number1, second is second part of number1, third is first part of nr 2 etc
               komplexNumbers[realCounter].real=temp;
               realCounter++;
            }
            else{
              komplexNumbers[imaginarCounter].imaginar=temp;
              imaginarCounter++;
              }
//The struct
typedef struct komplexNumber{ 
  float real;
  float imaginar;
} komplexNumber;




//The method that mallocs memory for each element:
void calculation(float a1, float a2, float b1, float b2, komplexNumber komplexNumbers[]){ 

    float temp1 = (a1*a2)-(b1*b2);     
    float temp2 = (a1*b2)+(a2*b1);

    komplexNumber *k;
    k=(komplexNumber*)malloc(sizeof(komplexNumber));              
    k->real=temp1;
    k->imaginar=temp2;
    komplexNumbers[0]=*k;
}

//The loop, in which im calling free each iteration:

int counter=1;
for(i=0;i<(numberOfIterations-1);i++){                   
    a1=komplexNumbers[0].real;
    b1=komplexNumbers[0].imaginar;
    a2=komplexNumbers[counter].real;
    b2=komplexNumbers[counter].imaginar;
    calculation(a1, a2, b1, b2, komplexNumbers);         
    counter++;

    free(komplexNumbers[counter]);
}

这个程序有时会崩溃，有时不会。我还没有太明白它为什么会这样的模式，但它是导致崩溃的 free() 函数(因为当我删除 free 并使用相同的值运行程序时，它不会' t崩溃)。我还没有看到导致崩溃的模式。它可以处理负数

注意:每个结构元素称为 complexNumber，数组称为 complexNumbers(带有 s:))

Answer 1

至少有几个问题:

• 您总是分配 komplexNumbers[0] 并释放 komplexNumbers[count]

• komplexNumbers[0] = *k 可能意味着 komplexNumbers 是一个结构数组，而不是一个指针数组 - 您正在分配一个结构，而不是指针

编辑

根据最近的代码，向您展示要做什么比解释您做错了什么更容易。正如所怀疑的那样，komplexNumbers 是一个结构数组。在您的计算函数中，您不需要所有的 malloc 东西(因此您不需要 free 位)。改为这样做:

komplexNumbers->real = temp1;
komplexNumbers->imaginar = temp2;