C - ダイナミックメモリ割り当て

Question

#include<stdio.h> 
#include<malloc.h> 

int main() 
{ 
    char *string1; 
    int length; 

    scanf("%d", &length); 

    string1 = (char *)malloc(sizeof(char) * length); 


    printf("\n Enter the First String : "); 
    fgets(string1, length, stdin); 

    printf("\n The First String : %s ",string1); 

    free(string1);  

    return 0; 
} 

誰かが上記のコードを助けてくれますか？私は文字列と文字列の長さを入力として取得しようとしています。しかし、私は弦の長さだけを入力することができます。その後、文字列入力部分をスキップします。

これは私が取得しています出力されます：

sh-4.3$ main 
10 

Enter the First String : 
The First String : 
sh-4.3$ 

Answer 1

• あなたはそれを削除scanf に後getchar()を使用する必要がありますので、 標準入力バッファに残ります"10<enter>"<enter>または"\n"を入力した後。

• さらに、malloc.hの代わりに#include <stdlib.h>を指定する必要があります。

• You malloc 0文字ではありません.0ターミネーターが原因です。 string1 = malloc(length + 1);が仕事をする、キャストが必要とsizeof(char) ではありません改行など でSTDIN /キーボードからの読み込み最も単純な答えは単に「\ nを追加することで、常に1

Answer 2

あなたは、文字列入力用のstdinを使用する必要がある場合は、fgetlnを使用することができます。私はあなたの例を編集し、今ではそのようになっています

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

int main() 
{ 
    char *string1; 
    size_t length; 

    //scanf("%d", &length); 

    //string1 = (char *)malloc(sizeof(char) * length); 


    printf("\n Enter the First String : "); 
    //fgets(string1, length, stdin); 
    string1 = fgetln(stdin, &length); 
    string1[length] = '\0'; 

    printf("\n The First String : %s ",string1); 

    free(string1); 

    return 0; 
} 

注：fgetlnはC文字列、あなたが最後にNULL文字を追加する必要がありません返します。

Answer 3

です「つまり、scanf関数へ：

scanf("%d\n", &length); 

は、問題を解決します;）

PS：scanfの

の他のすべてのセキュリティ/バッファオーバーフローの問題に注意してください