我正在尝试使用链表链接实现哈希表。以下代码有效 -
#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。我是否正确地传递了一系列结构?并且我已正确宣布。任何帮助将不胜感激。
答案 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);
}
}
}