动态增加字符数组的维度

Question

我在 C 语言中有这个函数，我在我的项目中使用它来读取输入中的字符：

char* readStringInp()
{
    char* string=NULL,c=NULL;
    int i=0;

    string=(char *)malloc(sizeof(char)*LGTH_STRING_INPUT);

    for(i=0;i<LGTH_STRING_INPUT && ( (c=getchar()) != '\n' && c != EOF );i++)
        string[i]=c;

    string[i]='\0';

    return string;
}

LGTH_STRING_INPUT 是一个数字常量，表示可以读取的字符串的最大长度。 读取没有固定长度的字符串的最佳方法是什么？

Answer 1

这会为 20 个字符分配内存。当使用了 18 个字符时，再分配 20 个字符由 realloc 分配。随着字符串的增长，将分配 20 个字符的额外增量。如果 realloc 失败，则返回当前字符串。当找到换行符或 EOF 时，while 循环终止。
分配的内存应该由调用函数释放。
EDIT 添加了 main() 以显示分配的内存的释放

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

char* readStringInp();

int main()
{
    char *s=readStringInp();
    if ( s != NULL)
    {
        printf ( "%s\n", s);
        free ( s);
    }
    return 0;
}

char* readStringInp()
{
    char* string=NULL, *temp = NULL;
    int c=0, available=20, used=0, increment=20;

    string=malloc(available);
    if ( string == NULL)
    {
        printf ( "malloc failed\n");
        return NULL;
    }

    while( (c=getchar()) != '\n' && c != EOF )
    {
        if ( used == available - 2)
        {
             available += increment;
             temp = realloc ( string, available);
             if ( temp == NULL)
             {
                 printf ( "realloc failed\n");
                 return string;
             }
             string = temp;
        }
        string[used]=c;
        used++;
        string[used]='\0';
    }
    return string;
}

Answer 2

读取没有固定长度的字符串的最佳方法是什么？

正如您在代码中已经完成的那样，使用malloc()动态分配内存并继续读取字符直到达到标记条件并继续增加使用realloc()在同一指针上分配的内存，除非您有空间容纳所有字符.

Don't cast malloc and family

Answer 3

calloc()（您已经在使用）和 realloc() 的组合将是用于动态调整缓冲区大小以适应不同长度的用户输入。
但是，现有存在几个问题 在继续使用该组合之前应解决的示例代码：请参阅下面的建议和 cmets：

正如所写，您现有的代码会泄漏内存，因为您从不调用 free()。 此外，您将在string 中获得缓冲区溢出。 （见下文解释）

这里有一些更正可以让您的示例代码正常工作：（参见 cmets）

#define LGTH_STRING_INPUT 20

//have written function to accept argument, easier to free memory
char* readStringInp(char *string){

    //char* string=NULL,c=NULL;//done in main
    int c; //getchar() requires int to detect EOF (-1)
    int i=0;

    //string=malloc(LGTH_STRING_INPUT);  //do not cast return value of calloc

    //   max length allow should be LGTH_STRING_INPUT - 1 
    //   to allow room for NULL after loop 
    for(i=0;i<LGTH_STRING_INPUT-1 && ( (c=getchar()) != '\n' && c != EOF );i++)
    {
        string[i]=c;
    }

    string[i]='\0';

    return string;
}

int main(void)
{
    char *buf = {0};
    char bufIn[LGTH_STRING_INPUT];

    //allocated and free memory in calling function:
    buf = malloc(LGTH_STRING_INPUT);
    if(buf)//check return of malloc
    {
        sprintf(bufIn, "%s", readStringInp(buf)); //use return value in buffer
        free (buf);//free memory that has been created
    }
    return 0;
}