0 
#include<stdio.h>
#include<stdlib.h>
#include<malloc.h>

#include <math.h>
typedef struct label{

int id;
double p,*t,q,c;

int V[45];
struct label *next;
struct label *prev;
struct path *tail;
struct path *head;

  }label;

 typedef struct path{
int i;

struct path *Pperv;
struct path *Pnext;
}path;




void main (){

int i,j,k;
struct label *Current,*FCurrent,*Current2,*Head,*Tail,*FHead,*FTail;
struct path *cur,*test3,*test2,*test1,*path_head,*path_tail;

Head=(struct label*)malloc(1*sizeof(struct label));
Tail=(struct label*)malloc(1*sizeof(struct label));



Head->next=Tail;
Tail->prev=Head;

FHead=(struct label*)malloc(1*sizeof(struct label));
FTail=(struct label*)malloc(1*sizeof(struct label));

FHead->next=FTail;
FTail->prev=FHead;



for (i=0;i<250000;i++)
{
    //printf("%d",i);
    Current=(struct label*)malloc(1*sizeof(struct label));
    Current->t=(double*)malloc(15*sizeof(double));

    Current->head=(struct path*)malloc(1*sizeof(struct path));
    Current->tail=(struct path*)malloc(1*sizeof(struct path));
    Current->head->Pnext=Current->tail;
    Current->tail->Pperv=Current->head;

    for (j=0;j<15;j++)
    {
        test1=(struct path*)malloc(1*sizeof(struct path));

        test1->Pperv=Current->head;
        test1->Pnext=Current->head->Pnext;

        Current->head->Pnext->Pperv=test1;
        Current->head->Pnext=test1;


        test1->i=1;
        Current->t[j]=23123.43;

    }
    if (i % 4!=0)
    {
    Current->next=Tail;
    Current->prev=Tail->prev;
    Tail->prev->next=Current;
    Tail->prev=Current;
    Current->p=54545.323241321;
    }
    else 
    {   


    Current->next=FTail;
    Current->prev=FTail->prev;
    FTail->prev->next=Current;
    FTail->prev=Current;


    }
}



Current=Head->next;
while(Current->next!=Tail)
{   

    Head->next->next->prev=Head;
    Head->next=Head->next->next;

    test1=Current->head->Pnext;
    while(test1!=Current->tail)
        {
            test2=test1;
            test1=test1->Pnext;
            free(test2);


        }

    free(Current->t);
    free(Current->head);
    free(Current->tail);
    free(Current);
    Current=Head->next;
}


Current=FHead->next;
while(Current->next!=FTail)
{   

    FHead->next->next->prev=FHead;
    FHead->next=FHead->next->next;
    k=0;
    test1=Current->head->Pnext;
    while(test1!=Current->tail)
        {

            test2=test1;
            test1=test1->Pnext;
            free(test2);
            k++;


        }

    free(Current->t);
    free(Current->head);
    free(Current->tail);
    free(Current);
    Current=FHead->next;
}







}

我遇到以下问题。这不是我的实际问题，它只是我创建的一个示例，以便更容易查看问题。正如你在这个例子中看到的那样，我有两个结构，一个在另一个里面，这个例子的作用是创建标签类型的新结构，并用指针 FHead 和 FTail 将它们放在列表中 4 次中的 3 次和 1 次列表中的 4 次指针是 Head 和 Tail。问题是，当我尝试释放结构时它没有发生，语法是 100% 正确的，因为当我将所有结构保存在两个列表之一中时，费用函数工作得很好。这导致我在使用指针时出现了问题，不幸的是我是 C 的新手，所以我不太清楚整个指针是如何进行的。因此，如果有人能解释使用指针出了什么问题，以及我应该如何创建这些列表以使 frre 函数正常工作，我将非常感激。先感谢您...

4

1 回答 1

0

您的代码很难阅读，因为您只有一个很长的函数，并且其中发生了很多事情。即使是有经验的 C 也很难阅读，所以对于 C 初学者来说一定很难。如果将此函数拆分为更小的、非常专业的函数，则可能更容易理解和发现错误。

附带说明（即使错误可能不是来自这里，不这样做也是一种非常糟糕的做法）：每次调用 malloc 时，都必须检查它是否失败。Malloc 可能无法给您一些内存，在这种情况下它返回 NULL。例如 ：

test1=(struct path*)malloc(1*sizeof(struct path));

可能变成

test1 = (struct path*)malloc(1*sizeof(struct path));
if (test1 == NULL) {
    abort(); /* Makes the program stop abruptly */
}

如果您不检查并得到 NULL 结果，您的程序将出现不稳定的行为；这些错误很难找到。

编辑：正如评论所建议的，更好的风格是

test1 = malloc(sizeof(path));
if (test1 == NULL) {
    abort();   /* Or something else that makes sense */
}
于 2012-06-20T17:21:52.827 回答