二つの文字列を比べるプログラム

作成したプログラムが以下です。

int main(void)
{
int i = 1;
char s1[100];
char s2[100];
printf("文字列１を入力してください > ");
gets_s(s1);
printf("文字列２を入力してください > ");
gets_s(s2);
・・・・・・・・・・・・・・・・・・・・・・・・・・・・・・・・・・・
for (i = 0;s1[i] != '\0' || s2[i] != '\0';i++)
{
if (s1[i] != s2[i])
{
printf("違う。");
return 0;
}
}
if (s1[i] == s2[i])
{
printf("同じ。");
return 0;
}
}
出力結果が　
abc　abc　同じ　
abc dfr 違う
となるプログラムなのでですが、
「同じ間繰り返しており，if文で違うことを調べる意味がなく、どちらかが'\0'なら同じとする」と指摘されました。どのようにすればコメント通りのプログラムになるのでしょうか？
お力をお借りしたいです。よろしくお願いいたします。｛・・・・より上を変更するのはなしでお願いします。｝

kazuma-s

2020/12/07 16:40

gets_s(s1); がエラーにならないのが不思議でしたが、ソースファイルの拡張子を .c ではなく、.cpp にして C++ としてコンパイルしていませんか？質問のタグが C なのに。

kennzi_665

2020/12/08 05:16

タグが間違っています。申し訳ありません。

行動規範の内容に同意します

回答2件

ベストアンサー

質問のプログラムに対して、「同じ間繰り返しており，if文で違うことを調べる
意味がなく、どちらかが'\0'なら同じとする」という指摘は間違っています。

質問のプログラムは、2つの文字列のどちらかが終端でない間繰り返しており、
if文で違うことを調べることには意味があります。
ただ、2つの文字列がどちらも終端なら繰り返さないので、
ループを抜けた後に、if文で同じことを調べるのは無意味です。
次のように書くべきです。

C
1#include <stdio.h>
2
3int main(void)
4{
5	int i = 1;
6	char s1[100];
7	char s2[100];
8	printf("文字列１を入力してください > ");
9	gets_s(s1);
10	printf("文字列２を入力してください > ");
11	gets_s(s2);
12	for (i = 0; s1[i] != '\0' || s2[i] != '\0'; i++) {  // どちらも終端なら繰り返さない
13		if (s1[i] != s2[i]) {
14			printf("違う。");
15			return 0;
16		}
17	}
18	printf("同じ。");
19	return 0;
20}

「同じ間繰り返し、どちらかが'\0'なら同じとする」コードは次のようになります。

C
1#include <stdio.h>
2
3int main(void)
4{
5	int i = 1;
6	char s1[100];
7	char s2[100];
8	printf("文字列１を入力してください > ");
9	gets_s(s1);
10	printf("文字列２を入力してください > ");
11	gets_s(s2);
12	for (i = 0; s1[i] == s2[i]; i++) {  // 同じ間繰り返す
13		if (s1[i] == '\0') {   // 同じなんだから s2[i] も '\0'
14			printf("同じ。");
15			return 0;
16		}
17	}
18	printf("違う。");
19	return 0;
20}