いくつかの行を印刷した後にCプログラムがクラッシュする

Question

このコードはCで書かれていますが、実行すると数行印刷した後にプログラムがクラッシュします。問題を解決してください。

コード：

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


void main() { 

    char *str[] = {"Dog", "Cat", "Hen", "Goat"}; 
    char *sentence; 
    int i; 
    srand(time(NULL)); 

    for(i=0; i<20; i++) { 
     sentence = strdup(""); 
     strcat(sentence, str[rand()%4]); 
     strcat(sentence, str[rand()%4]); 
     strcat(sentence, str[rand()%4]); 
     printf("%s\n", sentence); 
    } 

} 

Answer 1

sentenceに文字列を格納するのに十分なメモリが割り当てられていなかったため、プログラムがクラッシュしました。

void main() { 

    char *str[] = {"Dog", "Cat", "Hen", "Goat"}; 
    char *sentence = NULL; //initialize the string 
    int i; 
    srand(time(NULL)); 

    for(i=0; i<20; i++) { 
     sentence=malloc(13); // longest string would be GoatGoatGoat + terminating null 
     sentence[0]='\0'; 
     strcat(sentence, str[rand()%4]); 
     strcat(sentence, str[rand()%4]); 
     strcat(sentence, str[rand()%4]); 
     printf("%s\n", sentence); 
     free(sentence); //always free the allocated memory 
    } 


} 

Answer 2

sentence = strdup(""); 

あなただけsentenceのための1 charを割り当てています。 3つすべての動物の名前（13 =最長の動物の名前の3倍、ヤギ+ 1のヌル文字）を格納するのに十分なメモリを割り当てる必要があります。また、文字列をゼロにするにはcallocを使用します。作業が完了したときにも、あなたのメモリを解放していない

sentence = calloc(13, sizeof(char)); /* CORRECT */ 

：

free(sentence); 

をところでそれが標準規格に準拠していないとして、あなたはvoid main()を使用しないでください。代わりにint main()を使用してください。

Answer 3

この場合、ループ内にメモリを割り当てて解放するのは悪い考えです。無害ですが、この場合は割り当てられたメモリを再利用する方がはるかに優れています。

また、mallocとfreeを使う必要はなく、代わりに固定長のchar配列を宣言することもできます。

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

int main() 
{ 
    char *str[] = {"Dog", "Cat", "Hen", "Goat"}; 
    char sentence[64]; 
    int i; 

    srand(time(NULL)); 

    for(i = 0; i < 20; i++) 
    { 
     strcpy(sentence, str[rand() % 4]); 
     strcat(sentence, str[rand() % 4]); 
     strcat(sentence, str[rand() % 4]); 
     printf("%s\n", sentence); 
    } 
    return 0; 
}