文字列が一致している判定がわからない。

Question

###実現したいこと
入力した文字列において一致する文字の数を求める。
###問題点
1つめの実行結果と3つめの実行結果が間違っている。
s[7]は'\0'だと思います。それが判定に入らないようにif(s[i]!='\0')にしましたができませんでした。
実行結果

s : Abcdefg
t : tochori
s[2]  t[2]
s[7]  t[7]
Answer:  2
---------------
s : ABCDE
t : aABCDE
Answer:  0
---------------
s : Ab43
t : cd13
s[3]  t[3]
s[4]  t[4]
Answer:  2

###コード

//文字列の比較2

#include<stdio.h>

int main(void)
{
    char s[21],t[21];
    int i=0;
    int cnt=0;
    printf("s : ");
    fgets(s,21,stdin);
    printf("t : ");
    fgets(t,21,stdin);
    while(t[i]!='\0'){
        if(s[i]==t[i]){
            if(s[i]!='\0'){
            printf("s[%d]  t[%d]\n",i,i);
            cnt++;
            }
        }
        i++;
    }
    printf("Answer:  %d\n",cnt);

    return 0;
}

Answer 1

他に回答がありますが、入力した文字列の最後に改行コードがあるため余計にカウントされていますね。
対策としては２つ。

入力文字列から改行文字を削除（NULL文字に変換）してから比較する。
または比較するときに改行コードを除外する。

で、手っ取り早いのは後者で、if文を次のようにすればよいかと。

if(s[i]!='\0'){
　↓
if(s[i]!='\0' && s[i]!='\n'){

Comments

[langhtorn]

見えない改行コードがあるのですね。よくわかりました。ありがとうございます。

Answer 2

なぜにすべてをmainに詰め込もうとするんだろう。
やりたいことを関数で表現し、mainはそれ呼ぶだけにすればいいのに。

#define _CRT_SECURE_NO_WARNINGS // disable warning (only for Visual C++)
#include <stdio.h>
#include <string.h>

// 一致する文字の数を返す
int count_same(const char* s, const char* t) {
  int count = 0;
  // 両者共に終端に達していない間くりかえす
  // (どっちかが終端ならloopを抜ける)
  while ( *s != '\0' && *t != '\0' ) {
    // 一致したら count +1
    if ( *s == *t ) {
      ++count;
    }
    ++s; // 次の文字
    ++t; // 次の文字
  }
  return count;
}

// 末尾の改行文字を、あれば(あるけど)取り除く 
void trim_lf(char* s) {
  char* p = strchr(s, '\n');
  if ( p != NULL ) *p = '\0';
}

int main(void) {
    char s[21], t[21];
    int cnt = 0;

    printf("s : ");
    fgets(s, 21, stdin);
    trim_lf(s);

    printf("t : ");
    fgets(t, 21, stdin);
    trim_lf(t);

    cnt = count_same(s, t);
    printf("Answer:  %d\n", cnt);

    return 0;
}

Comments

[langhtorn]

関数バージョンも作ってみます。ありがとうございます。

[episteme]

いやホント疑問なんだけど、なぜにすべてをmainに詰め込もうとするんです?
ひとつのことを確実に行う部品を作り、それらを組み上げたほうがずっと楽じゃないんですか?

Answer 3

if(s[i]==t[i]){

        if(s[i]!='\0'){
        printf("s[%d]  t[%d]\n",i,i);
        cnt++;
        }

文字を比較して同一ならprintfを実行してますが、それは意図した動作なんでしょうか。

---

s : Ab43

t : cd13
s[3] t[3]
s[4] t[4]
Answer: 2

これ、双方の文字列の後尾にスペースが入ってるのでは。

Comments

[episteme]

fgetsで読んでるから、ケツに改行があるせいですきっと。

[langhtorn]

見えない部分にこれから気を付けます。ありがとうございます。

Answer 4

//文字列の比較2

#include<stdio.h>

int main(void)
{
    char s[21],t[21];
    int i=0;
    int cnt=0;
    printf("s : ");
    fgets(s,21,stdin);
    printf("t : ");
    fgets(t,21,stdin);
    while(t[i]!='\0'){
        if(s[i]==t[i]){
            if(s[i]!='\0'&&s[i]!='\n'){
            //printf("s[%d]  t[%d]\n",i,i);
            cnt++;
            }
        }
        i++;
    }
    printf("Answer:  %d\n",cnt);

    return 0;
}

/*実行結果
s : Imahori
t : teacher
Answer:  1
------------
s : ChuoUniv2020
t : student123456
Answer:  3
------------------
s : ABCDE
t : aABCDE
Answer:  0
*/