Segmentation faultを解決したい。

Question

写真の問題を解くようなプログラムを作ったのですが、長い文字列(具体的には26文字以降)を入力するとエラーが出てしまいます。どのように解決したらよろしいでしょうか？
文字列の長さおよび配列のサイズをあらかじめ指定することは許されていません。

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

int main(void) {
int i, len;
char *str, *word, cp;

printf("Input An English word > ");
scanf("%s", word);
len = strlen(word);

str = (char *)malloc(len * sizeof(*str));
strcpy(str, word);

for(i=0; i<(len/2); i++) {
cp = str[i];
str[i] = str[len-i-1];
str[len-i-1] = cp;
}

printf("%s\n", str);
return 0;
}

---execution results ---

$ ./test
Input An English word > dfjahdfhkjhdjkhfkhakjhdkjfhas
Segmentation fault (core dumped)
$ ./test
Input An English word > adhfkjhakhdjkhfkjahskjdhjfk
Segmentation fault (core dumped)
$ ./test
Input An English word > fakjdlkfhkljahsjkdhfkhalsd
Segmentation fault (core dumped)
$ ./test
Input An English word > fajkhdkhfkjahskjdhfkjhaks
skahjkfhdjkshajkfhkdhkjaf
$ ./test
Input An English word > fhakdjhfkjhakjsdhfkjhskjh
hjkshjkfhdsjkahjkfhjdkahf
$ ./test
Input An English word > flkdhfhakjhdkjfhkajhdkjfh
hfjkdhjakhfjkdhjkahfhdklf

Answer 1

(うっわ、めちゃくちゃめんどくさいやつだこれ…初学者にやらせる内容じゃないだろこれ…)

おっと失礼、心の声が漏れてました。さて本題ですが、いくつかの手順に分けて考えましょう。

1. 可変長の長さの文字列を読み取る

ファイルからの読み込みであれば、事前に長さを調べる方法がありますが、今回はキーボードからなので無理です。環境によっては、scanf関数のオプションで領域を自動確保することもできますが、標準の機能ではないです。

→ 何回かに分けて文字列を読み込んで、それを連結させることを考える。

1. メモリ確保の方法

malloc関数で確保して、領域が足りなくなったらrealloc関数で継ぎ足しましょう。それで足りなくなったら…諦めましょう(異常終了も、合理的な選択肢です)。

1. 文字列連結

領域を継ぎ足して、継ぎ足した領域に直接読み込めば、連結のことは考えなくてよさそうです。

というわけで、ヒントをば。(答えそのものではないですよ?)

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

char* gets_a(void)
{
  size_t len = 512, read = 0;
  char* str = (char *)malloc(len * sizeof(*str));
  char* temp;
  while(str){
    str[len-2] = '\0';
    fgets(str + read, len - read, stdin);
    if(str[len-2] == '\n' ||  ​str[len-2] == '\0'){
      ​str[len-2] = '\0';
      ​return realloc(str, read + strlen(str + read) + 1));
    ​}
    ​read = len - 1;
    ​len *= 2;
    ​temp = (char *)realloc(str, len * sizeof(*str));
    if(!temp){
      free(str);
    }
    str = temp;
  ​}
  ​return NULL;
}

Answer 2

scanf("%s", word);

wordには何が入ってるのか考えてみる。
ここでメモリ破壊が起こっている。これをどーにかしよう

Answer 3

あ、あいきゃんとすぴーくいんぐりっしゅ。

可変長配列というのが狭義の int a[n]; のことだけであれば、それに相当するモノを作れば良いのではないでしょうか。

Answer 4

The pointer variable word has not been initialized.
Therefore, it is unknown where this variable points.
Nevertheless, you are using this for scanf().

---

If you can't make any assumptions about input length, you'll need to pick up dynamically the typed characters and find the word length, I think.

Following sample code is simple one.
It checks each input character one by one until it finds the end of a word.

int main(void)
{
	unsigned int WordLength = 0;

	while( 1 )
	{
		char c = fgetc( stdin );
		if( !isalpha(c) )break;

		++WordLength;

		//You can add any code here.
		//e.g. any work releated to memory allocation to remember the input characters.
	}

	printf( "WordLength = %d\n", WordLength );
	return 0;
}

Answer 5

scanf("%s", word);

scanf doesn't allocate memory automatically.