使用 scanf 分配指针的 char 数组

Question

我正在尝试使用 scanf 填充 char 指针数组以将输入存储为字符串。变量 T 用于动态构建大小为 T 的数组。然后输入并显示 T 数量的字符串，但是当我填写数组时，例如如果 T = 2 第一行可能是狗，第二行是猫，它会打印出“cdog”和“cat”。所以第一个字符串的第一个字母然后是第二个字符串的全部。我不确定我在使用 char* 时的错误在哪里。任何帮助，将不胜感激。

#include <stdio.h>
#include <string.h>
#include <math.h>
#include <stdlib.h>

int main()
{
  int T;
  int i;
  scanf("%d",&T);


  char *new_array = (char*)malloc(T * sizeof(char));

  for (i = 0; i < T; ++i)
  {
    scanf("%s", new_array+i);

  }

  for (i = 0; i < T; i++)
  {
    printf("%s\n", new_array+i);
  }
}

Answer 1

1. 始终检查scanf() 的返回值。

2. 您不是为指针分配空间，而是为字节分配空间，这是主要问题，您需要

   char **new_array = malloc(T * sizeof(char *));
   /*    ^                                   ^             */
   /* allocate pointer to poitners         sizeof(pointer) */  
   if (new_array == NULL)
       handleThisErrorAndDoNotTryToWriteTo_new_array();
   

   你还需要为每个字符串留出空间

   new_array[i] = malloc(1 + lengthOfTheString);
   if (new_array[i] == NULL)
       handleThisErrorAndDoNotTryToWriteTo_new_array_i();
   

   在scanf() 之前，而不是scanf("%s", new_array + i) 这样做

   scanf("%s", new_array[i]);
   

如果您启用编译器警告，编译器应该警告您将不兼容的类型传递给printf()。

最好使用scanf() 的长度修饰符来防止缓冲区溢出，并且在不再需要指针时不要忘记调用free()。

Answer 2

在您的代码中，new_array 的类型为 char *，这不是您想要的。您必须将定义更改为

char *new_array[T] = malloc(T * sizeof(char*));

然后，您可以按照之前的代码使用scanf()。

Answer 3

改为与身体的其他部分一起这样做：

int string_size;
//this begins just after reading T
scanf("%d", &string_size);
char **new_arrays = malloc(T * sizeof(char*));
for(i = 0; i < T; i++)
{
    new_arrays[i] = malloc(string_size * sizeof(char));
}

第一个malloc是指定你想要多少个字符串，第二个malloc是指定一个字符串可以有多大。

更多提示：

1. 当您使用 C 语言编写时，不要强制转换由 malloc 和 realloc 生成的 void*。

2. 您应该以相反的方式取消分配使用的内存：

   for (i = 0; i < T; ++i)
   {
       free(new_array[i]);
   }
   free(new_array);
   

3. 始终检查内存分配过程是否（不）成功：

   char **new_arrays = malloc(T * sizeof(char*));
   if(new_arrays == NULL)
      exit(0) //e.g.
   for(i = 0; i < T; i++)
   {
       new_arrays[i] = malloc(string_size * sizeof(char));
       if(new_arrays[i] == NULL)
           exit(0) //e.g.
   }
   

4. 检查用户是否通过scanf提供了有效值。

Answer 4

谢谢大家。 char* 数组中字符串的长度不能大于 1000 个字符，因此是硬编码的。这是最终的工作代码...

#include <stdio.h>
#include <string.h>
#include <math.h>
#include <stdlib.h>

int main()
{
  int T;
  int i;
  scanf("%d",&T);


  char **new_array = malloc(T * sizeof(char*));
  for (i = 0; i < T ; i++)
  {
    new_array[i] = malloc(1000 * sizeof(char));
    scanf("%s", new_array[i]);
  }

  for (i = 0; i < T; i++)
    printf("%s\n", new_array[i]);
}