c - 将名称从文件复制到 char * 数组的函数

Question

作为程序的一部分，我需要将单个名称从文件复制到定义为 char *Names[NumOfNames] 的数组中，其中 NumOfNames 是一个整数，它包含文件中名称的总数。我是数组指针的新手，这似乎是我的麻烦所在。

文件按以下格式写入:

JohnFrankJamesPeter

(即每个名字都以大写字母开头，名字之间没有空格)

这是我到目前为止未完成的功能:

void LoadNamesIntoArray()
{
    char ch;
    int NumOfNames = 0;

    FILE *fpn = fopen(NamesFilePath, "r+");

    if (fpn == NULL)
    {
        printf("Cannot open %s for reading. \n", NamesFilePath);
        printf("Set up Names file at %s and restart. \n", NamesFilePath);
        perror("Error opening Names file");
    }

    do{
        ch = fgetc(fpn);
        if(isupper(ch)){
            NumOfNames++;
        }
    }while(ch != EOF);

    char *Names[NumOfNames];
    ...
    ...
    ...
}

我尝试了几种方法将每个名称复制到数组的每个元素中，使用 fgets 函数和 islower() 函数来识别何时移动到下一个Names 数组的元素。

我希望数组是这样的

printf("%s", Names[0])

将打印“John”等。这可能吗？对于我哪里出错的任何帮助或帮助，我们将不胜感激。

编辑

我现在正试图将每个名称复制到一个临时数组中，然后复制到 char *Names[NumOfNames} 的每个元素中。但是，为了测试文件是否被正确复制到临时数组中，我尝试将其打印出来，但打印不正确。这是代码块:

do{
    ch = fgetc(fpn);
    TempName[i] = ch;
    i++;
}while(ch != EOF);

for(i = 0; i<15; i++){
    printf("%c", TempName[i]);
}

我知道这会打印出前 15 个字符而不是单个名称，但我的问题是它打印的是所有奇怪的符号而不是实际的字母。

Answer 1

好的第一步，找到名字的数量。

向其添加最大长度的确定。

size_t MaxLength = 0;
size_t CurentLength = 0;
while ((ch = fgetc(fpn)) != EOF) {
  if(isupper(ch)){
    CurrentLength = 0; 
    NumOfNames++;
  }
  CurentLength++;
  if (CurrentLength > MaxLength) {
    MaxLength = CurrentLength; 
  }
} 

倒回文件，分配缓冲区。

rewind(fpn);
char *Buffer = malloc(MaxLength + 1);
// +1 here to deal with files that do not begin with  A-Z
char **Names = malloc((NumOfNames + 1) * sizeof *Names);

然后再次读取名称并使用strdup() 为每个名称分配空间。 strdup()不是标准 C，但在 POSIX 中是标准的，因此通常可用 - 见下文。

size_t i = 0;
size_t name_index = 0;
for(;;)  {
  ch = fgetc(fpn);
  if (ch == EOF || isupper(ch)) {
    buffer[i] = '\0';
    if (i > 0) Names[name_index++] = strdup(buffer);
    if (ch == EOF) break;
    i = 0;
  }
  buffer[i++] = ch;
}

---

[编辑]

示例 strdup() 实现。

char *strdup(const char *str) {
  size_t len = strlen(str) + 1;
  char *copy = malloc(len);
  if (copy) {
    memcpy(copy, str, len);
  }
  return copy;
}