Question

我制作了一个程序，从文件english_dictionary.txt中读取英语中最常用的200个单词，并在foreign_dictionary.txt中将各个单词的翻译用外语翻译。

.txt文件的内容放在两个char *类型的200元素数组中。

然后程序从文件text_to_translate.txt读取，其中包含英文文本，如果找到匹配项，则会用英文替换英文字符串（单词）（这发生在compute_text内功能）。但它不起作用，我对堆内存的断言失败了，所以我想我对malloc或类似的东西有些错误，但我无法理解错误的地方。

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

void foreign_dictionary(char **foreign, FILE *fp){

    char *buffer = malloc(30 * sizeof(char));
    char *tok;
    int i;

    for (i = 0; i < 200; ++i){
        fgets(buffer, 29, fp);
        tok = strtok(buffer, "\n");
        foreign[i] = malloc(strlen(tok) + 1);
        strcpy(foreign[i], tok);
    }

    free(tok);
    free(buffer);
}

void dictionary(char **english, FILE *fp){

    int i;
    char *tok;

    char *buffer = malloc(30 * sizeof(char));

    for (i = 0; i < 200; ++i){
        fgets(buffer, 29, fp);
        tok = strtok(buffer, " \n");
        english[i] = malloc(strlen(tok) + 1);
        strcpy(english[i], tok);
    }
    free(buffer);
    free(tok);
}

void compute_text(char **text,FILE *fp){

    char *buffer;
    int i, j, flag = 0, words_number = 0, cnt_letters_word = 0;

    buffer = malloc(100 * sizeof(char));

    while (fgets(buffer, 100, fp) != NULL){
        for (i = 0; i < 100; ++i){
            if (buffer[i] == ' ' || buffer[i] == '\0'){
                text[words_number] = malloc((cnt_letters_word + 1)* sizeof(char));
                for (j = 0; j < cnt_letters_word; ++j){
                    if (isupper(buffer[flag + j]))
                        text[words_number][j] = tolower(buffer[flag + j]);
                    else
                        text[words_number][j] = buffer[flag + j];
                }
                text[words_number][cnt_letters_word] = '\0';
                flag = i + 1;
                cnt_letters_word = 0;
                ++words_number;
            }
            else if (buffer[i] == '\n' || buffer[i] == ',' || buffer[i] == '.' || buffer[i] == ';' || buffer[i] == ':')
                ;
            else
                ++cnt_letters_word;
        }
        flag = 0;
        cnt_letters_word = 0;
    }
    free(buffer);
}

int main(void){

    char *foreign[200], *english[200], *text[50];

    FILE *fp = fopen("foreign_dictionary.txt", "r");
    foreign_dictionary(foreign, fp);
    fclose(fp);
    fp = fopen("english_dictionary.txt", "r");
    dictionary(english, fp);
    fclose(fp);
    fp = fopen("text_to_translate.txt", "r");
    compute_text(text, fp);
    fclose(fp);

    return 0;
}

Answer 1

在您的代码中，tok未指向动态分配的内存。你不需要（也不能）free()它。删除

 free(tok);

来自代码。

来自man page of free()

free(ptr)函数释放了ptr指向的内存空间，该内存空间必须在之前调用malloc()，calloc()或realloc()时返回。否则，或者之前已经调用过free(ptr) undefined behaviour。发生。

也就是说，在使用返回的指针之前，总是检查fopen()和malloc()（以及可能的所有库函数）的返回值以确保成功。

基于字符串比较的Word翻译程序 - 堆内存断言失败

1 个答案: