无法将文件中的条目正确添加到指针的二维数组中

时间:2014-09-30 01:02:49

标签: c arrays

我在向C中的2d指针数组添加项目时遇到问题。我的代码如下:

#include <stdio.h>
#include <stdlib.h>
#include <malloc/malloc.h>
#include "pg3_methods.h"

void load_input(char path[])
{
char ** input = (char**)malloc(sizeof(char));

// Variable to hold current word
char * currentWord = malloc(sizeof(char));

// Set file equal to the path
FILE * file;
file = fopen(path, "r");
int i = 0;

// Checks if file is null
if (file == NULL)
{
    fprintf(stderr, "File %s does not exist.", path);
}

// If not null, cycles through file and adds entries
while (!feof(file))
{
    fscanf(file, "%s", currentWord);
    input = (char**)malloc(sizeof(char)*1);
    input[i] = currentWord;
    printf("input[%i] = %s\n", i, currentWord);
    i++;
}
for (int i = 0; i < sizeof(input); i++)
{
    printf("%s\n", input[i]);
}
}

输出是这样的:

input[0] = The
input[1] = small
input[2] = fox
input[3] = jumped
input[4] = over
input[5] = the
input[6] = red
input[7] = fence
input[8] = bad
input[9] = The
input[10] = small
input[11] = fox
input[12] = jumped
input[13] = over
input[14] = the
input[15] = red
input[16] = fence
input[17] = bad
input[18] = The
input[19] = small
input[20] = fox
input[21] = jumped
input[22] = over
input[23] = the
input[24] = red
input[25] = fence
input[26] = bad

(null)
(null)
bad
(null)
(null)
bad
(null)
(null)

正如您所看到的,它在while循环中工作,但是一旦它退出循环,它就会搞砸了。有关如何使这项工作的任何建议?任何帮助将不胜感激! (注意:输出的第一部分中的单词来自单独的文本文件。)

1 个答案:

答案 0 :(得分:0)

char ** input = (char**)malloc(sizeof(char));

这为单个字符分配了足够的内存,这可能不是您想要的。相反,你应该做类似的事情:

char ** input = (char**)malloc(sizeof(char*) * WORD_COUNT);

将分配几个指向char的指针。您现在可以将其视为char*

的数组
char * currentWord = malloc(sizeof(char));

同样,您只分配一个字符。您可能需要分配更多内存。

现在让我们看一下while循环的主体:

fscanf(file, "%s", currentWord);
input = (char**)malloc(sizeof(char)*1);
input[i] = currentWord;

这里有几个问题。首先,你不断地在内存中写下前一个单词。第二,为input分配一个新的内存块,忘记分配的旧内存块(这是内存泄漏)。最后,input数组中的每个指针(假设您修复了分配数组而不是单个字符的东西)都指向内存中的相同字符串。

要解决这些问题,您应该遵循以下伪代码:

  1. input分配足够的内存,以存储指向您将读入的每个单词的指针。
  2. 在循环中:
    1. 为一个单词分配内存
    2. 将单词读入已分配的内存
    3. 指定input中的一个指针指向刚刚读入的
  3. 如果您在将此流程转换为C代码时遇到困难,请通过新问题告诉我们。