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

Question

我的程序有时会崩溃，有时在im对数组中的元素调用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 %in", 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并用相同的值运行程序时，它不会崩溃）。我还没能太清楚造成这次事故的原因。它可以处理负数

注意：每个结构元素都被称为complexNumber，数组被称为complexNumbers（带s：）

Answer 1

至少有几个问题：

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

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

编辑

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

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

Answer 2

因为在计算中，您总是设置第0个项的值（通过复制k指向的值），所以您希望释放未分配的数组项。

Answer 3

我发现这部分代码有点混乱：-komplexNumbers[0]=*k;内部计算。因为它将一次又一次地将指针设置为仅在索引0中的复杂类型。当您尝试释放此While itearting in a for loop using counter时，如果counter的值>0，它将失败。我想是的，但会更好如果你用一种很好的方式框出问题并发布，这样我们就可以调查它`

Answer 4

komplexNumbers（数组）是如何分配的？您不能对malloc未返回的任何内容调用free。并且您正在泄漏内存，k的内存在从calculation返回时丢失。