文字が打てなくなる原因がわからない。

Question

###実現したいこと
2つの文字列を入力して比較する。
###問題点
sは順調に入力できるのだがtをいれようと思ったら、入力できなくなる。
その原因を教えてください。

文字列を入力してください
s: AbcDe
t:　　　　　　　　　<-ここで文字が打てなくなる

###コード

//文字列の比較1

#include<stdio.h>
#include<ctype.h>

int main(void)
{
    char s[5],t[5];
    printf("文字列を入力してください\n");
    printf("s: ");
    fgets(s,5,stdin);
    printf("t: ");
    fgets(t,5,stdin);

    int cnt,cnsup,cnlow,cdiff=0;
    int i;
    while(1){
        if(s[i]==t[i]){ //小文字大文字もすべて一致
            cnt++;
        }
        if(isupper(s[i])){ //s[i]が大文字でt[i]が小文字の時
            if(islower(t[i])){
                cnsup++;
            }
        }
        if(islower(s[i])){ //s[i]が小文字でt[i]が大文字の時
            if(isupper(t[i])){
                cnlow++;
            }
        }
        if(s[i]!=t[i]){ //もはや文字が違う時
            cdiff++;
        }
    }
    if(cnt==5){
        printf("same\n");
    }else if(cnsup>0||cnlow>0){
        printf("case-insensitive\n");
    }else if(cdiff>0){
        printf("differnt\n");
    }

    return 0;
}

Answer 1

取得する行の最大文字数 (fgets の第二引数）が小さすぎて次の fgets が実行されちゃって次の変数の方に入っちゃってます。文字列の長さ + 改行コード + \0 で 7（以上）にしたらとりあえず動くと思いますが、

fgets(s, 7, stdin);

バッファも含めて余裕を持った数にした方がいいと思います。

Comments

[langhtorn]

長さ5の文字が入力したかったので7がちょうどよかったです。ありがとうございます。

Answer 2

sは順調に入力できるのだがtをいれようと思ったら、入力できなくなる。

fgets(s,5,stdin);でsには"AbcD"が格納され(第2引数に指定するサイズは'\0'を格納するための領域も含む)、
fgets(t,5,stdin);でtには残りの"e\n"が格納されるためです。

---

cnt,cnsup,cnlow,iが不定値のまま使用されています。
大文字と小文字であれば別の文字でも、cnsupやcnlowを加算しています。

Comments

[langhtorn]

理由がよくわかりました。ありがとうございます。

Answer 3

配列sは大きさ5の配列ですね。
fgets実行時、配列sに入る文字列の長さは、4文字です。終端のヌル文字も含めて5文字分になるので。
提示されている例だと「AbcD」までがsに入れられて、fgetsが終了します。

では、残りの「e」と、入力を終えるための改行はどこに入るのかというと、次のfgets実行時に、配列tに入ります。提示されている例だと「e」と改行と終端のヌル文字がtに入っています。

ということで、最初に入力した文字列が配列sとtに分割して格納されるので、そのまま2回目のfgetsも終了してしまいます。対策としては、配列の大きさを十分に大きくしてください。

そこから後のコードも突っ込みたいのですが、質問にないので省略いたします。

Comments

[langhtorn]

そうですね。見えてない部分があるのですね。ありがとうございます。