用scanf分配指针的char数组

Question

我正在尝试使用scanf填充char指针数组，以将输入存储为字符串。 变量T用于动态构建大小为T的数组。 然后输入并显示T数量的字符串，但是当我填写数组时，例如，如果T = 2，则第一行可以跟踪，第二行显示cat，它会打印出“ 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编写时，请勿realloc 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) //eg for(i = 0; i < T; i++) { new_arrays[i] = malloc(string_size * sizeof(char)); if(new_arrays[i] == NULL) exit(0) //eg } 

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]);
}