Question

我在为哈希表实现插入函数时遇到了麻烦。

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

我在下面发布一些输出。表看起来很奇怪的原因是因为我的哈希函数。它散列的词是A = 1，B = 2，C = 3等等。字母在单词中的位置是无关紧要的，因为我会考虑单词的排列。此外，字母的情况也与此问题无关，因此a的值= A = 1的值。

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

总的来说，哈希函数很好。问题是插入功能。

我的本地词典的前4个单词是A，A＆A，A＆C，AA＆＃39; 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;
}

Answer 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++;
}

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

注意：未经过测试

Answer 2

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

插入函数哈希表C.

2 个答案: