Segmentation fault: 11というエラーが出ます・・・

Question

以下のプログラムを作成したのですが、実行するとき、一応求めていた物を得ることはできるのですが、実行結果の最後に、一定数の文字数を超えるとSegmentation fault: 11と表示されてしまいます・・・
結果は一応正しいものが得れるのですが、これが表示されるのはあまり良くない気がします・・・
どなたかご教授いただけると嬉しいです。
＃include <stdio.h>
＃include <stdlib.h>

int main(void){
char ip1;
char ip2;
char IP1;
char IP2;
ip2 = &ip1;
IP2 = &IP1;
int i = 0;
int I = 0;
int counter = 0;
printf("英単語を入力>>");
scanf("%s",ip2);
printf("入力された英単語は");
while((ip2 + i) != '\0'){
printf("%c",(ip2 + i));
i++;
counter++;
}
printf("です。\n");
printf("逆方向に英単語を出力>>");
for(i = counter - 1;i >= 0;i--){
*(IP2 + I) = (ip2 + i);
printf("%c",(IP2 + I));
}
printf("\n");

return 0;

}

Answer 1

上限をなくすためにはどのようにしたら良いのでしょうか。

#include <stdio.h>  // getchar, putchar, printf

struct S { int c; struct S *p; } t;

void g(struct S *p)
{
	struct S s = { getchar() };
	p->p = &s;
	if (s.c == EOF || s.c == '\n') {
		printf("入力された英単語は: ");
		for (p = t.p; p != &s; p = p->p) putchar(p->c);
		printf("\n逆方向に単語を出力: ");
	}
	else g(&s), putchar(s.c);
}

int main(void)
{
	printf("英単語を入力: "), g(&t), putchar('\n');
}

Answer 2

動的メモリ確保についてあまり詳しくはありませんが、
while文の中で、動的確保するのは難しいのでしょうか。

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

int main(void){
    char* t;
    int n = 1;
    t = (char*)malloc(n* sizeof(char));
    
    
    char c;
    int idx = 0;
    
    do {
        scanf("%c", &c);
        if (idx+1 == n) {
            n*=2;
            char* s = (char*)malloc(n* sizeof(char));
            for (int i = 0;i < idx;++i) s[i] = t[i];
            free(t);
            t = s;
        }
        t[idx] = c;
        idx++;
    } while (c != '\n');
    printf("%s\n", t);
    free(t);
    
    return 0;
}

Answer 3

こんにちは。
突っ込みどころ満載のコードですね。

• 先ずコードはマークダウンで記載しましょう。
• C言語はcase-sensitiveなのでip1とIP1は別物ですが、紛らわしいので目視ではっきり区別できる変数名にしましょう。
• ポインタ変数ip2とIP2に割り当てられているメモリ実体はchar変数ip1,IP1で共に１バイトの領域しかありません。

短い文字列入力で動いたのはたまたま（運が良かったから）です。
scanfはnull終端するので、実際には１バイト文字入力した時点でバッファオーバーフローが発生します。

これが表示されるのはあまり良くない気がします・・・

あまり良くないではなく絶対ダメなコードです。
配列やポインタをもっと学習することを強くお勧めします。

### ■追記
余裕のあるバッファを確保して、かつscanfの結果を保護するのであれば、
上記なら８０バイトまでの文字列はNULL終端されますし、
入力が80バイトを超えた場合、80バイトで打ち切ってnull終端した文字列を返却してくれます。
これならバッファーオーバーフローを心配しなくて済みます。
※scanfのフォーマット指定子で文字列%sに小数を指定すると小数の値が最大文字列となります。
### ■追記２
文字列を入力させる前に、何文字入力するか問い合わせてその文字数＋１の領域を動的に確保する。
という方法を思いつきましたが、もうすこしマシな方法がありました。
Cのコンパイル環境がないのでエラーが出たらごめんなさい。

■追記３

辞書に載っている最大長の英単語が４５文字（pneumonoulttamicroscopicsilicovolcanoconiosis）なので、46バイト以上の配列を確保すれば、一応命題はクリアできると思います。

Comments

[TSUNA-omarilove]

ご回答ありがとうございます。
先の回答者様にも同様に返信させていただいたのですが、私も初めは解答者様のおっしゃるように1バイトではなくip[256]のようにしてプログラムを作成していたのですが、問に英単語の文字数はプログラム上で上限がないように作成しろ、とありました。おそらく動的メモリの確保を用いて解答していくのだろうなとは見当がついたのですがそこから全くわからなくなってしまい、このようなめちゃくちゃなコードになってしまいました。
どのようにすれば題意に沿ってプログラムが作成できるのか、お忙しいとは思いますがお時間ある時にでもご回答いただけると嬉しいです。

[DreamTheater]

成程、追記が参考になるかと思いましたが、そもそも命題に無理がありますね。
『英単語の文字数はプログラム上で上限がないように作成しろ』
malloc関数で動的に巨大なメモリを確保すること自体は可能ですが、そもそも入力が終わらなければ何文字（何バイト）入力されたかわからない訳で、問題作成者の意図を測りかねます。

まあ命題が「英単語」ということなので実存する英単語の最大長＋１の配列を定義すれば要件を満足していると主張できますねWw

他には無理やりな方法ですが、回答に「追記２」として掲載します。

[kazuma-s]

> ※scanfのフォーマット指定子で文字列%sに小数を指定すると小数の値が最大文字列となります。

これは間違いです。scanf の書式に "." はありません。"%80s" です。
コンパイル環境を用意して確かめてみてください。

[kazuma-s]

> (void)scanf("%c", wordBuff + pos);
> if (*(wordBuff + pos) = '\0') break;

scanf でキーボードから '\0' を読み込むことは通常ありません。

[DreamTheater]

kazuma-sさんご指摘ありがとうございます。
回答を修正します。

Answer 4

まず、

scanf("%s",ip2);

文字列を入力した場合の格納先が、１文字分しか存在してません。これではダメです
C言語での文字列というのはどういうものなのかを確認しよう

Comments

[TSUNA-omarilove]

無知で申し訳ありません。
ip2 = &ip1;
を入力すれば格納先が存在することにはなりませんか...?

[y_waiwai]

ip1というのは、char変数です。
そのアドレスを入れたところで、１文字分の格納先しか存在しません。

[y_waiwai]

どーにかするなら、
char ip1[100];
ぐらいのサイズの配列にしましょう

[tatsu99]

>ip2 = &ip1;
>を入力すれば格納先が存在することにはなりませんか...?
1バイトの格納先になりますね。
格納先はたっぷりとっておいた方が安全です。
char ip1[256];
ip2 = ip1;

[TSUNA-omarilove]

ご丁寧に解説して下さり、ありがとうございます。
私も初めそのようにサイズを指定して作成していたのですが、扱える英単語の文字数にプログラム上の上限があってはいけない(メモリや環境による上限はok)プログラムを作成しろ、という問題をといておりましてそこで完全にわからなくなってしまいこのようなプログラムにしてしまいました。
上限をなくすためにはどのようにしたら良いのでしょうか。
お時間がある時にご回答いただけると嬉しいです。

[y_waiwai]

上限があってはいけないというテーマであるなら、scanfで入力を行うというのはほとんど不可能です
まあ、初心者のうちは、あまりこういう事を考えないで、取りあえず動くコードを組んでいけばいいかとはおもいますが。。

こういう制限のないコードを組もうとするとかなりレベルが高くなります
やり口としては、一定サイズのバッファを用意して、そのサイズ分の文字列を順番に読み出していくというコードになりますね

[Daregada]

提示されたプログラムを見る限り、質問者のレベルは「扱える英単語の文字数にプログラム上の上限があってはいけない」プログラムを作成できるレベルに達していないので、まずはサイズに上限のあるプログラムを問題なく作成することを目指してください。