char数组赋值[关闭]

Question

我正在尝试将文本文件中的行读入 char 数组，但出现了问题。请查看代码，让我知道我做错了什么。 谢谢。

#include <stdio.h>
#include <stdlib.h>
int main(void) {
    int i=0,j;
    char* string[100];
    char line[100];
    FILE *file; 
    file = fopen("patt", "r"); 
    while(fgets(line, sizeof line, file)!=NULL) {
        printf("%d %s",i, line);
        string[i]=line;
        i++;
    }
    for (j=0 ; j<i ; j++) {
        printf("string[%d] %s",j, string[j]);
    }
    fclose(file);
    return 0;
}

输入文件patt有以下内容。

rec
cent
ece
ce
recent
nt

在执行上面的代码时，我得到了这个

0 rec
1 cent
2 ece
3 ce
4 recent
5 nt
string[0] nt
string[1] nt
string[2] nt
string[3] nt
string[4] nt
string[5] nt

我期待的是这个

0 rec
1 cent
2 ece
3 ce
4 recent
5 nt
string[0] rec
string[1] cent
string[2] ece
string[3] ce
string[4] recent
string[5] nt

Answer 1

为了帮助您理解您的代码：

• 您每次都在向string[i] 写入line 的地址，而数组line 的内容在整个循环中不断变化。 这导致所有string[i] 都包含相同变量的地址，即line。 在循环结束时，从文件中读取的最后一行存储在line 数组中。现在，当您打印 string[i] 时，您正在打印变量 line 中存在的数据。由于 string[i] 的所有实例都包含 line 的相同地址，它们都在打印相同的值，即nt 在您的输出中。

int main(void) {
    int i=0,j;
    char* string[100];
    char line[100];
    FILE *file; 
    file = fopen("patt", "r"); 
    while(fgets(line, sizeof(line), file)!=NULL) {//You missed the ()'s in the sizeof
        printf("%d %s",i, line);
        string[i]=line;
        i++;
    }

*你应该做的是：

int main(void) {
int i=0,j;
char* string[100];
char line[100];
FILE *file; 
file = fopen("patt", "r"); 
while(fgets(line, sizeof(line), file)!=NULL) {//You missed the ()'s in the sizeof
    printf("%d %s",i, line);
    string[i]=malloc(strlen(line)+1); ///You need to allocate some memory to char* string[i] here
    if(string[i] == NULL)//Do a NULL check if malloc failed
      //Handle NULL here
    strncpy(string[i],line,strlen(line)+1);//Use strncpy to copy line into the malloced memory in string[i]
    i++;
}

现在，由于您已分配内存来存储数据，因此您也需要释放内存。 所以，而不是在你的代码中：

    for (j=0 ; j<i ; j++) {
        printf("string[%d] %s",j, string[j]);
    }
    fclose(file);
    return 0;
}

这样做：

    for (j=0 ; j<i ; j++) {
    printf("string[%d] %s",j, string[j]);
    free(string[j]);
    }
   fclose(file);
   return 0;
}

现在，这应该会给你你想要的。

Answer 2

您的string[i]=line; 表示您将指向同一缓冲区 (line) 的指针重复存储到 string 中的每个连续项目中。像这样的：

为了使事情顺利进行，您需要为每个指向的缓冲区分配一个新缓冲区，例如：string[i] = dupe_string(line);，其中dupe_string 将类似于：

char *dupe_string(char const *in) { 
    char *dupe;
    if (NULL != (dupe = malloc(strlen(in) + 1)))
        strcpy(dupe, in);
    return dupe;
}

请注意，由于这使用malloc 为每个字符串分配空间，您最终需要为每个字符串调用free 以避免内存泄漏。

Answer 3

您重复写入同一个数组。

while(fgets(line, sizeof line, file)!=NULL) {

在这一行中，您反复写入数组line。

---

我认为您误解了 char* string[100]; 的作用。您将它用作数组数组，而实际上它是一个包含 100 个指针的数组。

要纠正这个问题，你需要在每次迭代中首先分配一个新的内存块：

 string[i] = malloc(strlen(line)+1);

那你需要把line的内容复制到string[i]：

strcpy(string[i], line); 

另外，在程序结束时，您需要使用free 来释放内存。