帮忙写getstring函数

Question

我在编写 getstring 函数时遇到了一些麻烦，这就是我到目前为止所遇到的。

问候， V

const char* getstring()
{


    char *buffer;
    int i = 255;

    buffer = (char *)malloc(i*sizeof(char));

    *buffer = getchar();
    while ( *buffer != '\n' )
    {
        buffer++;
        *buffer = getchar();
    }
    *buffer = '\0';

    const char* _temp = buffer;
    return _temp;
}


int main()
{
    char* temp = getstring();

    for ( ;temp++ ; *temp != '\0')
    {
        printf("%c", *temp);
    }

    return 0;
}

Answer 1

当 _temp 指向字符串的终止 '\0' 时，您将 _temp 设置为 buffer。

将行：移动

const char* _temp = buffer;

到紧接行：之后，

buffer = (char *)malloc(i*sizeof(char));

以便 _temp 指向缓冲区的开头。

您还有其他一些问题：

1. 不要使用名称_temp - 带有前导下划线的名称被保留；

2. 您需要测试您向缓冲区写入的字节数不超过 i 个字节；

3. 您应该测试 malloc() 返回 NULL;

4. 您需要测试 getchar() 返回 EOF。这意味着您需要将 getchar() 结果存储在 int 类型的变量中，然后再将其分配给 *buffer；

5. 正如 Michael Mrozek 在评论中指出的那样，for 循环中的表达式是错误的。

...并且从风格上来说，sizeof(char) 始终为 1，因此无需乘以它；并且在 C 中强制转换 malloc() 的结果是不必要的，并且被认为是不可取的（与 C++ 不同，C++ 是必需的）。

Answer 2

为什么不直接使用

char buffer[255];
scanf("%254s", &buffer);

或

char* buffer = readline("GO GO GO:");

Answer 3

const char* _temp = buffer;

将上述语句移至 malloc 调用之后

重要：
在 main() 使用后释放分配给 buffer 的内存。

free(temp);

Answer 4

您需要在调用 malloc() 之后立即跟踪分配的指针 - 由 malloc() 返回的值，以便您可以将其传递回调用者。您还应该检查 EOF 以及换行符 - 这需要一个 int （而不是 char）来保存 getchar() 中的值。至少！