1

我有一个函数可以遍历我的字符串数组，以找出该字符串在数组中出现的次数。如果找到，该字符串将被设置为NULL，并且计数器会跟踪找到该字符串的次数。然后我在循环中调用另一个函数来为我的频率数组分配内存，以便我可以存储count. 它似乎工作正常，但是当我在 main 中创建任何其他变量时，我的程序崩溃了。这是我的两个功能：

int search(char **table, int **frequency, int wordSize)
{
//  Local Declaration
int i, j, k;
int count = 1;
int strCount = 0;
char target[25];

// Statement
for(i = 0, k = 0; i < wordSize; i++)
{
    if(table[i] != NULL)
    {
        strcpy(target, table[i]);
        for(j = i + 1; j < wordSize; j++)
        {
            if(table[j] != NULL &&
               strcmp(target, table[j]) == 0 &&
               target != table[i])
            {
                count++;
                free(table[j]);
                table[j] = NULL;
            }
        }
    strCount += makeFreq(frequency, k, count);
    k++;
    }
    count = 1;
}

return strCount;
}// search


int makeFreq(int **frequency, int k, int count)
{
//  Local Declaration
int strCount = 0;

//  Statement
frequency[k]=(int*)malloc(sizeof(int));
frequency[k][0] = count;
strCount += 1;

return strCount;
}// makeFreq

有人可以向我解释为什么我的程序崩溃了吗？

在这里，我为我的表分配了 1000 个指针。

char** getPoint(void)
{
//  Local Declaration
char **table;

//  Statement
table = (char**)calloc(MAX_SIZE + 1, sizeof(char));
if(table == NULL)
{
    MEM_ERROR, exit(100);
}

return table;
}// getPoint

比我读到的，我为文件中的字符串分配内存并将其存储到字符串数组中。

int scanFile(char **table, FILE *fpFile)
{
//  Local Declaration
int count = 0;
char temp[500];
char **ptr = table;

//  Statement

//  scan file, allocate, and copy string to array.
while(fscanf(fpFile, "%s", temp) != EOF)
{
    *(ptr + count) =(char*)calloc(strlen(temp)+1, sizeof(char));
    strcpy(*(ptr + count), temp);
    count++;
}

return count;
}// scanFile

这是我为频率数组分配指针数组的方式。

void aloFreqAry(int **frequency, int wordSize)
{
//  Local Declaration

//  Statement
frequency =(int**)calloc(wordSize + 1, sizeof(int));
if(frequency == NULL)
{
    MEM_ERROR, exit(103);
}

return;
}// aloFreqAry
4

2 回答 2

3

除了分配中的大小问题（应该sizeof(char*)在分配中table，sizeof(int*)在分配中frequency），

void aloFreqAry(int **frequency, int wordSize)
{
//  Local Declaration

//  Statement
frequency =(int**)calloc(wordSize + 1, sizeof(int));
if(frequency == NULL)
{
    MEM_ERROR, exit(103);
}

return;
}// aloFreqAry

不frequency为调用者分配任何东西。它只是为该指针的本地副本分配内存，并在函数返回时丢失该句柄。

函数应该返回一个，而不是int**作为参数，

frequency = calloc(wordSize + 1, sizeof(int*)); // size of a _pointer_ to int
if(frequency == NULL)
{
    MEM_ERROR, exit(103);
}

return frequency;

您在调用者中分配的。

于 2013-02-27T22:13:14.200 回答
1

这个语句看起来很可疑（你说“我在这里为我的表分配了 1000 个指针”）：

table = (char**)calloc(MAX_SIZE + 1, sizeof(char));

这看起来不像是指针分配，而是 char 缓冲区的分配。

也许你的意思是：

table = (char**)calloc(MAX_SIZE + 1, sizeof(char*));
于 2013-02-27T22:06:40.897 回答