散列函数 C

Question

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

我想对我的单词进行哈希处理，使得 A = 1、B = 2、C = 3 等等。单词中字母的位置无关紧要，因为我们将考虑单词的排列。此外，字母的大小写在这个问题中也无关紧要，所以 a 的值 = A 的值 = 1。

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

对于 ab = 3 和 aaa = 3 的情况，我已经有了处理这种情况的方法。现在我只想获取哈希值。

我现在遇到的问题是 aaa 给我 1，而 ab 给我 2。

下面是我的代码：

int hash(char *word)
{
   int h = 1;
   int i, j;

   char *A;
   char *a;
   // an array of 26 slots for 26 uppercase letters in the alphabet
   A = (char *)malloc(26 * sizeof(char));
   // an array of 26 slots for 26 lowercase letters in the alphabet
   a = (char *)malloc(26 * sizeof(char));

   for (i = 0; i < 26; i++) {
      A[i] = (char)(i + 65); // fill the array from A to Z
      a[i] = (char)(i + 97); // fill the array from a to z
   }

   for (i = 0; i < strlen(word); i++) {
      //printf("HIT\n");
      for (j = 0; j < 26; j++) {
         // upper and lower case have the same hash value
         if (word[i] == A[j] || word[i] == a[j]) {
            h = h + j; // get the hash value of the word
            //printf("HIT 2\n");
            break;
         }
      }
   }

   printf("H: %d\n", h);

   return h;
}

Answer 1

我认为改变

int h = 1;

到

int h = 0;

和

h = h + j;

到

h = h + j + 1;

将解决问题。

---

另一个问题是您忘记释放malloced 内存。另外，C 中有no need to cast the result of malloc（和家人）。

这个

for (i = 0; i < strlen(word); i++) {

将在循环的每次迭代中调用strlen。这会降低程序的性能。使用

int len = strlen(word);
for (i = 0; i < len; i++) {

相反，这要快得多，因为 strlen 在每次迭代中都不会被调用。最后，sizeof(char) 是 1。所以你可以省略它。

Answer 2

将h=h+j 更改为 h=h+j+1 和h=1 到h=0。

您还应该释放分配的内存，因此在返回之前包含这些行：

free(A);
free(a);

但是我不明白为什么要为如此简单的任务编写如此复杂的代码。

可以编写更简单的代码：

int hash(char *word)
{
   int sum=0;
   while(*word != '\0')
   {

       if(*word >='A' && *word < 'A'+26)
            sum=sum+(*word -'A' + 1);
       else if(*word >='a' && *word < 'a'+26)
            sum=sum+(*word -'a' + 1);
       else
            return -1;

       word++;
   }

   return sum;
}

Answer 3

多个问题：

1. 你仍然没有释放你分配的数组
2. h 的初始值为 1 没有意义
3. 您将索引添加到哈希中。 'A' 和 'a' 在索引 0 处，因此在这种情况下您要添加 0（所以无论您给代码提供多少 'a' 都将返回 1）

4. 为什么是动态数组？你知道大小，它不会改变。你可以使用

   char A[26];
   char a[26]; // you can also add initialisation, e.g. = {'a', 'b', ...
   

5. 为什么是数组？

所以，here 是快速修复，与您的代码保持密切联系。

考虑到以上所有因素，您可以简化为：

int hash(char const * string) {
  int h = 0;
  for (; *string; ++string) {
    int index = tolower(*string) - 'a' + 1;
    if ((index > 0) && (index < 27)) {
      h += index;
    }
  }
  return h;
}

Live

---

当只对带有非特殊字符的单词进行哈希处理时，您需要以某种方式处理调用者中被忽略的单词。

char hash(char const * string, int * h) {
  *h = 0;
  for (; *string; ++string) {
    int index = tolower(*string) - 'a' + 1;
    if ((index > 0) && (index < 27)) {
      *h += index;
    } else {
      return 0;
    }
  }
  return 1;
}

这样你就可以使用返回值来测试这个词是否应该被忽略。