哈希表 - 链表 - 分段错误

时间:2012-04-11 15:55:25

标签: c function segmentation-fault arguments hashtable

我正在尝试使用链表链接实现哈希表。以下代码有效 -

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

#define TABSIZ 200

struct record {
    struct record *next;
    char name[BUFSIZ];
    int data;
};

static struct record *htable[TABSIZ];

unsigned hash(char *s)
{
    unsigned h;

    for (h = 0; *s; s++)
        h = *s;
//printf("%d", h%TABSIZ);
//I know its not a good hash function but i wanted to check chaining
    return h % TABSIZ;
}

struct record *find(char *name)
{
    struct record *item;

    for (item = htable[hash(name)]; item; item = item->next)
    {
        if (strcmp(name, item->name) == 0)
            return item;
    }

    return NULL;
}

struct record *insert(char *name,int value)
{
    struct record *item;
    unsigned h;

    if ((item = find(name)) == NULL)
    {
        if ((item = malloc(sizeof (*item))) == NULL)
            return NULL;

        strcpy(item->name, name);
        item->data=value;
        h = hash(name);
        item->next = htable[h];
        htable[h] = item;
    }

    return item;
}
void printTable()
{
    int i=0;
    struct record *temp;
    for(i=0;i<=TABSIZ;i++)
    {
        temp=htable[i];
        while(temp!=NULL)
        {
            printf("\n%d - %s - %d\n", i,temp->name, temp->data);
            temp=temp->next;
            }
    }
}
int main(void)
{
    char buf[BUFSIZ];int value;
    struct record *item;
    do{
    printf("Enter the name of the student:\n");
    scanf("%s", buf);
    if(strcmp(buf,"stop")==0) break;
    printf("Enter the marks of the student:\n");
    scanf("%d", &value);
    if(insert(buf, value)==NULL)
    {
        break;
    }
}while((strcmp(buf,"stop"))!=0);

    printf("Enter a name to find: ");
    scanf("%s", buf);
    if((item=find(buf))!=NULL)
        printf("The marks of the student is %d\n", item->data);
    else printf("\n Not Found\n");
    printTable();
    return 0;
}

现在我试图删除全局变量并使用局部变量作为结构数组。我删除了htable的全局声明,并在main中声明为

struct record *htable[TABSIZ];

并将功能更改为

struct record *find(struct record *htable, char *name);
struct record *insert(struct record *htable, char *name,int value);

我将这些函数称为

find(htable, name);
insert(htable,name,value);

但现在我的程序是segfaulting。我是否正确地传递了一系列结构?并且我已正确宣布。任何帮助将不胜感激。

2 个答案:

答案 0 :(得分:2)

我在早些时候的回答中走错了路。

当它是全局时,它会自动初始化为0。

当它在main堆栈上声明时,它没有被初始化。

memset( htable, 0, sizeof(htable))中添加main(),这应该会将其恢复为之前的行为。

答案 1 :(得分:0)

在printable()中:

for(i=0;i<=TABSIZ;i++)
看起来很可疑。你可能想要:

void printTable()
{
    unsigned int i;
    struct record *temp;

    for(i=0; i < TABSIZ;i++)
    {
        for (temp=htable[i]; temp!=NULL; temp=temp->next )
        {
            printf("\n%d - %s - %d\n", i,temp->name, temp->data);

        }
    }
}