[英]Cannot free nodes in my hash table when there are collisions (using separate chaining) in C
好的,所以我有一个用C语言编写的哈希表。我正在使用单独的链接(链接列表)来解决冲突。 我注意到,如果没有冲突并且每个项目都哈希到其自己的索引中,则可以释放整个表。 但是,如果发生冲突,并且索引中有多个值,那么它只能释放第一个值,而不能释放该索引中的其余值。 当试图释放该索引处的其他程序时,该程序崩溃。 我尝试对其进行调试,但我意识到那些其他值已设置为NULL,我不确定为什么,因为当我将它们插入表中时,我正在使用malloc。 我知道我想念什么。 如果有人可以提供帮助,那将是非常棒的事情,因为我已经尝试解决了几个小时的问题:/
这是代码:
int symTabSearch(struct hashTable * h, char * label);
int insertToSymTab(struct hashTable * h, char * label, int locctr);
struct listNode
{
char * label;
int address;
struct listNode * next;
};
struct hashTableNode
{
int blockCount; //number of elements in a block
struct listNode * firstNode;
};
struct hashTable
{
int tableSize;
int count; //number of elements in the table
struct hashTableNode * table;
};
struct hashTable * createHashTable(int size)
{
struct hashTable * ht;
ht = (struct hashTable*)malloc(sizeof(struct hashTable));
if (!ht)
return NULL;
ht->tableSize = size;
ht->count = 0;
ht->table = (struct hashTableNode *)malloc(sizeof(struct hashTableNode) * ht->tableSize);
if (!ht->table)
{
printf("Memory error\n");
return NULL;
}
int i;
for (i = 0; i < ht->tableSize; i++)
{
ht->table[i].blockCount = 0;
ht->table[i].firstNode = NULL;
}
return ht;
}
/*hash function: adds up the ascii values of each
character, multiplies by a prime number (37) and mods the sum wih the table size*/
int hash(char * label, int tableSize)
{
int hashVal = 0;
size_t i;
for (i = 0; i < strlen(label); i++)
hashVal = 37 * hashVal + label[i];
hashVal %= tableSize;
if (hashVal < 0)
hashVal += tableSize;
return hashVal;
}
int symTabSearch(struct hashTable * h, char * label)
{
struct listNode * temp;
temp = h->table[hash(label, h->tableSize)].firstNode; //temp points to the first listNode in table[hashedIndex]
while (temp)
{
if (strcmp(temp->label, label) == 0)
return 1; //found
temp = temp->next; //go to next link
}
return 0; //not found
}
int insertToSymTab(struct hashTable * h, char * label, int locctr)
{
int index;
struct listNode * currentNode, *newNode;
index = hash(label, h->tableSize);
currentNode = h->table[index].firstNode;
newNode = (struct listNode *)malloc(sizeof(struct listNode));
newNode->label = (char *)malloc(sizeof(char) * 7); //allocates 7 chars to store label up to 6 chars long (0-5), last one is for the '\0'
if (!newNode) //if new node is null
{
printf("Error creating new node\n");
return 0;
}
strcpy(newNode->label, label);
newNode->address = locctr;
if (h->table[index].firstNode == NULL) //if first node at table index is empty
{
h->table[index].firstNode = newNode;
h->table[index].firstNode->next = NULL;
}
else
{ //firstNode was not empty, so chain newNode to the next empty node
while (currentNode != NULL) //go to next available node
currentNode = currentNode->next;
currentNode = newNode;
currentNode->next = NULL;
}
h->table[index].blockCount++;
h->count++;
return 1;
}
void freeHashTable(struct hashTable * h) //might not free memory properly, might crash too, test later
{
int i, j;
struct listNode * current, *temp;
char * tempStr;
if (!h) //make sure table even has memory to be freed
return;
for (i = 0; i < h->tableSize; i++)
{
current = h->table[i].firstNode;
for (j = 0; j < h->table[i].blockCount; j++)
{
temp = current;
tempStr = current->label;
current = current->next;
free(temp);
free(tempStr);
temp = NULL;
tempStr = NULL;
}
}
free(h->table);
h->table = NULL;
free(h);
h = NULL;
}
当您尝试将节点追加到列表时,问题出在insertToSymTab
函数中。
这里的问题是这个循环:
while (currentNode != NULL) //go to next available node
currentNode = currentNode->next;
该循环完成后,您已通过列表的末尾,并且currentNode
值为NULL
。 更改该指针不会将新节点附加到列表的末尾。
相反,您需要将循环更改为例如
while (currentNode->next != NULL)
currentNode = currentNode->next;
然后,当循环结束时, currentNode
将成为列表中当前的最后一个节点,并通过更改currentNode->next
追加新节点:
currentNode->next = newNode;
不要忘记将newNode->next
设置为NULL
。
您的错误是在insertToSymTab中:
while (currentNode != NULL) //go to next available node
currentNode = currentNode->next;
currentNode = newNode;
currentNode->next = NULL;
您将currentNode设置为currentNode-> next(复制指针值),然后将其设置为newNode。 但是currentNode没有链接到先前的currentNode-> next,它将只是一个NULL指针,然后将其分配给newNode。
您必须为列表的最后一个节点设置currentNode-> Next = newNode,或者使用struct listnode **指针来实现类似于我在这里尝试的功能。
编辑:约阿希姆提供了更快的答案
声明:本站的技术帖子网页,遵循CC BY-SA 4.0协议,如果您需要转载,请注明本站网址或者原文地址。任何问题请咨询:yoyou2525@163.com.