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

Question

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

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'なら同じとする」と指摘されました。どのようにすればコメント通りのプログラムになるのでしょうか？
お力をお借りしたいです。よろしくお願いいたします。｛・・・・より上を変更するのはなしでお願いします。｝

Comments

[kazuma-s]

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

[kennzi_665]

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

Answer 1

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

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

#include <stdio.h>

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;
		}
	}
	printf("同じ。");
	return 0;
}

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

#include <stdio.h>

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] == s2[i]; i++) {  // 同じ間繰り返す
		if (s1[i] == '\0') {   // 同じなんだから s2[i] も '\0'
			printf("同じ。");
			return 0;
		}
	}
	printf("違う。");
	return 0;
}

これはこれで正しいコードです。

Comments

[kennzi_665]

詳しく説明してくれたかずまさんをベストアンサーにしました。二人ともご回答ありがとうございました。

Answer 2

ループが終わった時点のif文は必ず成立するので無駄です

if (s1[i] == s2[i])

{
printf("同じ。");
return 0;
}
}

は、

printf("同じ。");
return 0;
}

でよろしいかと