无法将节点添加到链接列表

时间:2015-05-14 07:46:38

标签: c linked-list

我做了改动但是 我不能添加超过2个节点,它将freez但如果1或2节点将运行良好是什么原因???我放弃 我无能为力 这是我的代码直到时间

#include <stdio.h>
#include <stdlib.h>
#include <ctype.h>

struct info{
    int num;
    char name[15];
    struct info *next;
};

struct info *first,*current,*new_s;
int struct_num;
void add_struct(void);

int main(){
    first=NULL;
    add_struct();
    puts("done");
    add_struct();
    puts("done");
    add_struct();
    puts("done");

    return(0);
}

// struct add function

void add_struct(void){

new_s= malloc (sizeof(struct info));
if(!new_s){
    puts("error");
    exit (1);
}
if(first==NULL){
   first = current= new_s;
   first->next = NULL;
}else{
    current=first;

    while(current->next!=NULL){
        current=current->next;
    }

    current->next=new_s;
    current=new_s;
}

struct_num++;
}

2 个答案:

答案 0 :(得分:5)

代码中的问题是

$group = array(1=>0,2=>0,3=>0,4=>0,5=>0);
$base_u_group = array('1,2,4','3','3,4','1,4');
foreach($base_u_group as $gr) {
       $split = explode(',', $gr);
       if (!empty($split)) {
          foreach($split as $val) {
            if ($val) {
              $group[$val] =  $group[$val] + 1;
            }
          }
       }
}
var_dump($group);

如果if( first==NULL){ first->next=new_s; 为NULL,则应对其进行解除反复。这在逻辑上是错误的,并调用undefined behaviour

我认为,你想要的是(伪代码)

first

那就是说,

if(first == NULL){
    first = new_s;
    first->next = NULL;

看起来也有问题。第二个声明是错误的而不是必需的,相反,你可以添加类似

的内容
    current->next=new_s;
    current=new_s;

最后,根据当前使用情况,您的 current->next = new_s; current->next->next = NULL; 变量应全球

注意:

  1. struct_num的推荐签名为main()
  2. int main(void) malloc()C家人的回复价值do not cast
  3. 在使用返回的指针之前,请务必检查malloc()成功。

答案 1 :(得分:0)

函数add_struct错误 例如,如果first等于NULL,那么您可能不会写

    first->next=new_s;

考虑到声明函数struct_num的局部变量没有任何意义,因为它在退出函数后总是被销毁,而且它甚至没有在函数中初始化。

int struct_num;

如果您需要列表中的节点数,请将其放在函数外部。

该功能本身可以采用以下方式

int struct_num;

int add_struct( void )
{
    new_s = ( struct info * )malloc( sizeof( struct info ) );

    if ( new_s == NULL ) return 0;

    // initialize data members num and name of the allocated structure
    new_s->next = NULL;

    if ( first == NULL )
    {
        first = new_s;
    }
    else
    {
        current->next = new_s ;
    }

    current = new_s;
    ++struct_num;

    return 1;
}