C：將字符串讀入動態數組

Question

嗨即時嘗試讀取「無限」長度的用戶輸入到char數組。它適用於較短的字符串，但超過30個字符左右的程序崩潰。爲什麼會發生這種情況，我該如何解決這個問題？

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

char* read_string_from_terminal()//reads a string of variable length and returns a pointer to it 
{ 
    int length = 0; //counts number of characters 
    char c; //holds last read character 
    char *input; 

    input = (char *) malloc(sizeof(char)); //Allocate initial memory 

    if(input == NULL) //Fail if allocating of memory not possible 
    { 
     printf("Could not allocate memory!"); 
     exit(EXIT_FAILURE); 
    } 

    while((c = getchar()) != '\n') //until end of line 
    { 
     realloc(input, (sizeof(char))); //allocate more memory 
     input[length++] = c; //save entered character 
    } 

    input[length] = '\0'; //add terminator 
    return input; 

} 

int main() 
{ 
    printf("Hello world!\n"); 
    char* input; 
    printf("Input string, finish with Enter\n"); 
    input = read_string_from_terminal(); 
    printf("Output \n %s", input); 
    return EXIT_SUCCESS; 
} 

Answer 1

realloc(input, (sizeof(char))); //allocate more memory只分配1 char。不是1 更多char。 @MikeCAT

(sizeof(char)*length+1)在語義上是錯誤的。應該是(sizeof(char)*(length+1))，但由於sizeof (char) == 1，它沒有功能上的差異。

空字符需要空格。 @MikeCAT

應該測試重新分配失敗。

char c不足以區分來自getchar()的所有257個不同響應。使用int。 getchar()可能會返回EOF。 @Andrew Henle

輕微：對於數組索引，最好使用size_t而不是int。 int也許太窄了。

在端代碼需要做類似：

size_t length = 0; 
char *input = malloc(1); 
assert(input); 
int c; 
... 
while((c = getchar()) != '\n' && c != EOF) { 
    char *t = realloc(input, length + 1); 
    assert(t); 
    input = t; 
    input[length++] = c; 
} 
... 
return input; 

int main(void) { 
    ... 
    input = read_string_from_terminal(); 
    printf("Output \n %s", input); 
    free(input); 
    return EXIT_SUCCESS; 
}