插入函数哈希表 C答案

【问题标题】：Insert function hash table C插入函数哈希表 C
【发布时间】：2023-03-04 11:41:01
【问题描述】：

我在为我的哈希表实现插入函数时遇到问题。

所以我实现了一些测试调用，我只是单独调用函数。对于实际使用，我在一个while循环中调用该函数。出于测试目的，我只运行了 4 次循环。

我在下面发布了一些输出。表格看起来很奇怪的原因是因为我的哈希函数。它对单词进行哈希处理，使得 A = 1、B = 2、C = 3 等等。单词中字母的位置无关紧要，因为我会考虑单词的排列。此外，字母的大小写在这个问题中也无关紧要，所以 a 的值 = A 的值 = 1。

对于字符串，abc = 1 + 2 + 3 = 6，bc = 2 + 3 = 5，等等。

总的来说，哈希函数很好。问题在于插入函数。

我本地词典的前 4 个单词是 A、A's、AA's、AB's。

我的预期输出应该是（我在运行测试调用时得到了相同的输出）：

0: 
1: [W: A, Len:1] 
2: 
3: 
...
18: 
19: 
20: [W: A's, Len:3] 
21: [W: AA's, Len:4] 
22: [W: AB's, Len:4]

但是当我在循环中调用函数时，列表中最后的任何内容都会覆盖其他条目。如果我运行循环 100 次，那么最后一个条目仍然会替换前面的条目（注意单词的长度是如何不变的，但只有单词被替换了）：

0: 
1: [W: AB's, L:1] 
2: 
3: 
...
18: 
19: 
20: [W: AB's, Len:3] 
21: [W: AB's, Len:4] 
22: [W: AB's, Len:4]

下面是我的代码：

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

int hash(char *word)
{
   int h = 0;

   while(*word != '\0')
   {
      if(*word >='A' && *word < 'A'+26) {
         h=h+(*word -'A' + 1);
      }
      else if(*word >='a' && *word < 'a'+26) {
         h=h+(*word -'a' + 1);
      }
      //else { // special characters
      // return -1;  
      //}

      word++;
   }

   return h;
}

typedef struct Entry {
   char *word;
   int len;
   struct Entry *next;
} Entry;

#define TABLE_SIZE 1000 // random numbers for testing

Entry *table[TABLE_SIZE] = { NULL }; // an array of elements

void init() {
   int i;

   for (i = 0; i < TABLE_SIZE; i++) {
      // initialize values
      struct Entry *en = (struct Entry *)malloc(sizeof(struct Entry));

      en->word = "";
      en->len = 0;
      en->next = table[i];
      table[i] = en;
   }
}

//Insert element
void insertElement(char *word, int len) {
   int h = hash(word);
   int i;

   // because all words are different so there is no need to check for duplicates
   struct Entry *en = (struct Entry *)malloc(sizeof(struct Entry));

   en->word = word;
   en->len = len;
   en->next = table[h];
   table[h] = en;
}

void cleanTable()
{
   struct Entry *p, *q;
   int i;

   for( i=0; i<TABLE_SIZE; ++i )
   {
      for( p=table[i]; p!=NULL; p=q )
      {
         q = p->next;
         free( p );
      }
   }  // for each entry
}

int main() {
   init(); // create hash table

   // test calls produce correct output
   //insertElement("A", (int)strlen("A"));
   //insertElement("A's", (int)strlen("A's"));
   //insertElement("AA's", (int)strlen("AA's"));
   //insertElement("AB's", (int)strlen("AB's"));

   int i;
   i = 0;

   FILE* dict = fopen("/usr/share/dict/words", "r"); //open the dictionary for read-only access 
   if(dict == NULL) {   
      return;                    
   }

   // Read each line of the file, and insert the word in hash table
   char word[128];      
   while(i < 4 && fgets(word, sizeof(word), dict) != NULL) {
      size_t len = strlen(word); 
      if (len > 0 &&  word[len - 1] == '\n') {
         word[len - 1] = '\0'; // trim the \n
      }

      insertElement(word, (int)strlen(word));

      i++;
   }

   for ( i=0; i < 50; i++)
   {
      printf("%d: ", i);

      struct Entry *enTemp = table[i];

      while (enTemp->next != NULL)
      {
         printf("[W: %s, Len:%d] ", enTemp->word, enTemp->len);
         enTemp = enTemp->next;
      }

      printf("\n");
   }

   cleanTable();

   return 0;
}

【问题讨论】：

标签： c insert hashtable

【解决方案1】：

尝试在这部分代码的每个循环中重新分配内存：

char* word = malloc(sizeof(char)*128);      
while(i < 4 && fgets(word, sizeof(word), dict) != NULL) {
    size_t len = strlen(word); 
    if (len > 0 &&  word[len - 1] == '\n') {
        word[len - 1] = '\0'; // trim the \n
    }
    insertElement(word, (int)strlen(word));
    word = malloc(sizeof(char)*128);
    i++;
}

您忘记为每个字符串重新分配内存，导致所有指针指向同一点

注意：未测试

【讨论】：

【解决方案2】：

请注意，您的 insertElement 获得了一个指向字符串的指针，并将该指针分配给当前 Entry，但它的主要功能是传递 word 指向堆栈分配字符串的参数（一个指针），并且该字符串在每次读取一个单词后都会更改。您必须使用 malloc 以便每个 word 指向自己的内存区域

【讨论】：