0

我用指针和函数制作了这个程序，它应该是一个链表。我不断收到“访问冲突读取位置 0xcdcdcded”。在下面的最后一部分。我认为可能是我下一步没有初始化，但我是指针新手，不知道该怎么做。任何帮助是极大的赞赏。

typedef struct temp 
{
    char name[20];
    char telephone[10];
    temp *next;
} node;


node* creation1 ()
{    
    node *NEW = NULL;
    NEW = (node*)malloc(sizeof(node)); 
    return NEW;
}

node* creation2 ()
{   
    node *start= NULL;
    node *NEW = creation1();
    start= NEW;
    return start;
}

node* creation3 ()
{    
    node *NEW = creation1();
    node *current = NULL;
    current=NEW;
    return current;
} 

void consult ()
{   
    node *NEW= creation1();
    node *start= creation2();
    node *current = creation3();
    int exit;
    printf("How many contacts do you wish to add? ");
    scanf("%i",&exit);

    for(int i=1; i<=exit; i++)
    {
        NEW = (node*)malloc(sizeof(node));
        current->next=NEW;                 
        current = NEW; 
        fflush(stdin);
        puts("NAME: ");
        gets(NEW->name); 
        puts("TELEPHONE: ");
        gets(NEW->telephone);
        NEW->next=NULL;
    } 

    current=start->next;

    int i = 0;
    do 
    {
        i++;
        current = current->next; //this is where it stops and gives me the access reading violation
    }while (current != NULL);
}

int main(int argc, char** argv)
{  
    consult();
}
4

1 回答 1

0

由于这似乎是家庭作业，我不想透露太多，但您的基本问题是您首先使用node *start= creation2();. 在执行的这一点上， 的值start->next是垃圾，可以是任何东西。

然后，在你的 for 循环中，start节点根本没有被触及，这意味着它start->next仍然可以是任何东西。

接下来，在 行current=start->next;中，您将设置current为start->next的垃圾值。

然后最后一行current = current->next;你取消引用垃圾值并跳转到内存中的一个随机位置。

通常，如果您有一个指针值（例如start->next），并且在创建它时没有将指针设置为的好值，则应将其设置为NULL. 然后，在取消引用值之前（使用->运算符），您应该检查左侧的变量->是否等于NULL，如果是，则不执行->操作。我真的很难给出任何更具体的建议，因为您的代码中没有注释来解释应该发生的事情。

于 2012-10-21T08:04:16.007 回答