Question

您好我正在创建一个程序，该程序使用哈希来存储文本文件中的单词及其出现次数。这按预期工作。我遇到的问题来自于释放分配的内存。

这是我的哈希

#include<stdio.h>
#include<stdlib.h>
#include<string.h>
#include<ctype.h>
#include"hash.h"

/*
struct listnode{
    char * word;
    int count;
};
*/

void hashCreate(struct listnode * hashTable[], int size){
    int i;
    for(i=0;i<size;i++){
        hashTable[i]=(struct listnode *)malloc(sizeof(struct listnode));
        hashTable[i]->count=0;
    }
}

int hash(char * data, int size) {
  unsigned long hash = 5381;
  char * p;
  for (p = data; *p != '\0'; ++p) {
    hash = (hash * 33) + *p;
  }
  return (int)(hash % size);
}

void hashAdd(char * data, struct listnode * hashTable[], int size){
    int key=hash(data, size);
    hashTable[key]->word=strdup(data);
    hashTable[key]->count+=1;
}

void hashPrint(struct listnode * hashTable[], int size){
    int i;
    for(i=0;i<size;i++){
        if(hashTable[i]->count!=0)
        printf("%s: %d \n",hashTable[i]->word,hashTable[i]->count);
    }
}

void hashDelete(struct listnode * hashTable[],int size){
    int i;
    for(i=0;i<size;i++){
        free(hashTable[i]->word);
        free(hashTable[i]);
    }
}

这是使用它的原因

#include<stdio.h>
#include<stdlib.h>
#include<string.h>
#include<ctype.h>
#include"hash.h"

/*
int hash(char * data, int size) {
  unsigned long hash = 5381;
  char * p;
  for (p = data; *p != '\0'; ++p) {
    hash = (hash * 33) + *p;
  }
  return (int)(hash % size);
}
*/

#define SIZE 1500


void removePunct(char * str);
void fileRead(char * filename);


struct listnode * hashTable[1500];

int main(int argc, char ** argv){
    int i;
    if(argc<2)
        fprintf(stderr,"Enter filename \n");

    hashCreate(hashTable, SIZE);

    for(i=1; i<argc; i++){
        fileRead(argv[i]);
    }

    hashPrint(hashTable,SIZE);
    hashDelete(hashTable, SIZE);
    return 0;
}

void fileRead(char * filename){
    FILE * file = fopen(filename,"r");
    char word[80];
    if(!file){
        fprintf(stderr,"Error opening file \n");
        return;
        }
    while(fscanf(file, "%s", word)==1){
        removePunct(word);
        hashAdd(word,hashTable,SIZE);
    }
    fclose(file);
}

void removePunct(char * str){
    int i,p=0;
    for(i=0; i<strlen(str);i++){
        if(isalpha(str[i]) || str[i]==' '){
            str[p]=tolower(str[i]);
            p++;
        }
    }
    str[p]='\0';
}

在我的hashDelete函数中，没有释放字符串，导致内存泄漏。我通过释放hashAdd函数中的字符串来测试它，并且没有内存泄漏，但也没有打印字符串。我无法找到不让我释放所有记忆的问题。任何帮助将不胜感激。

Answer 1

在此代码中

void hashAdd(char * data, struct listnode * hashTable[], int size){
    int key=hash(data, size);
    hashTable[key]->word=strdup(data);
    hashTable[key]->count+=1;
}

您使用strdup获取新字符串（malloc＆＃39; ed by strdup）。如果您已经为给定的key做了一次，那么您将泄漏内存。

所以你需要检查：

if (hashTable[key]->word == NULL) hashTable[key]->word=strdup(data);

但是，这需要在创建表时将word初始化为NULL。

主题：通常，您需要使用一些额外的代码处理相同的key值。导致data的{{1}}值可能与已存储的key相同或不同。你要检查的东西。如果它们相同，您可以增加word。如果它们不同，您必须有一种方法来存储两个具有相同count值的不同单词。

它可能看起来像：

key

free（）不从结构数组中释放字符串

1 个答案: