为什么我总是得到我的字符串数组的最后一个元素？

Question

我正在尝试从文本文件中读取所有字符串并将它们保存在字符串数组中。 但是，当我尝试打印我的字符串数组的内容时，只打印最后一部分。 为什么我的代码只复制最后一个字符串？

我的代码

# include <stdio.h>
# define BUFFERSIZE 100

int main(int argc, char *argv[]){
   char buffer[BUFFERSIZE];

   int i = 0;
   char *text[BUFFERSIZE];

   while(fgets(buffer, BUFFERSIZE, stdin) != NULL){
          text[i] = buffer;
          i++;
   }

   int j = 0;
   for (j=0; j<sizeof(text)/sizeof(char); i++){
          printf("%s\n", text[j]);
   }

   return 0;
}

我的文本文件

ESD can create spectacular electric sparks (thunder and lightning is a large-scale ESD       event), but also less dramatic forms which may be neither seen nor heard, yet still be large enough to cause damage to sensitive electronic devices.
Electric sparks require a field strength above about 4 kV/cm in air, as notably occurs in lightning strikes. Other forms of ESD include corona discharge from sharp electrodes and brush discharge from blunt electrodes.

输出

>>> make 1_17; ./1_17 < example.txt
m blunt electrodes.

m blunt electrodes.

m blunt electrodes.

m blunt electrodes.

...

Answer 1

有两个问题。首先，对于所有i个，text[i]包含您多次使用的相同缓冲区。第二，在您的打印代码中，您只打印text[0]。

使用相同的缓冲区

只声明了一个缓冲区，

char buffer[BUFFERSIZE];

虽然你在循环中多次修改它的内容，但它总是相同的缓冲区（即内存中的存储区域相同），所以

text[i] = buffer;

使text的每个元素都包含相同buffer的（地址）。您需要将<{1}}的内容复制到新字符串中，然后将其存储在buffer中。如果可以使用POSIX函数，则可以使用例如strdup复制字符串，如

text[i]

text[i] = strdup(buffer); 使用strdup为字符串分配空间，因此如果您在较大的应用程序中使用此空间，请务必稍后malloc这些字符串。但是，在您的简单应用程序中，当应用程序退出时它们将被释放，因此您不会遇到太多麻烦。

如果你只能使用标准的C函数，你可能需要strcpy，这会让你做更多的工作。 （您需要分配一个足够大的字符串来保存free的当前内容，然后将buffer的内容复制到其中。之后您仍然需要buffer他们

仅打印free

但是，您的打印代码也存在问题。您使用text[0]将text编入索引，但从不修改j（您使用j递增i），因此您始终打印相同的字符串（实际上是数组中的 first ，而不是最后一个，但其内容与您从文件中读取的lsat字符串相同）：

i++

在第一个循环之后， int j = 0; for (j=0; j<sizeof(text)/sizeof(char); i++){ printf("%s\n", text[j]); } 是你得到的字符串数，所以你可能只想要：

i

Answer 2

此外，除了已发布的答案之外，您还在for循环中递增i，而不是j。

Answer 3

看看这个循环：

while(fgets(buffer, BUFFERSIZE, stdin) != NULL){
    text[i] = buffer;
    i++;
}

您将数组的每个元素设置为完全相同的值：buffer的值，它是指向一块内存的指针。在每次迭代中，您都会在上次读取数据的顶部读取数据。

要解决此问题，您需要在每次迭代时分配 new 缓冲区，并将数组值设置为指向该新缓冲区的指针。这样，数组的每个元素都将指向不同的内存部分。