for文を使用し、文字列を比べるプログラム

Question

二つの文字列を比べ、同じもしくは違うと表示するプログラム作成しいたのですが、（例ABC,
ABC 同じ、ABC,DEF 違う）「同じ間繰り返すようにし．どちらかが'\0'なら同じとする」「FOR文が間違っている」と言われたのですがですが、どのように直せばよいでしょうか？お力を貸して頂きたいです。
以下コードです。
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;
	}
	return 0;	

}
よろしくお願いいたします。

Answer 1

for文は間違っていません。

get_s関数は Linux にはないので、Windows でコンパイラが Visual C++ だと
すると、get_s の引数の個数が異なることが問題です。

get_s(s1); を gets_s(s1, 100); に、
get_s(s2); を gets_s(s2, 100); に変更すると、正しく動きます。

if (s1[i] == s2[i]) や printf("同じ。"); のあとの return 0; は不要ですが、
あっても正しく動きます。

#include <stdio.h>

int main(void)
{
	int i = 1;  // 1 に初期化する必要はありませんが
	char s1[100];
	char s2[100];
	printf("文字列１を入力 > ");
	gets_s(s1, 100);
	printf("文字列２を入力 > ");
	gets_s(s2, 100);

	for (i = 0; s1[i] != '\0' || s2[i] != '\0'; i++)
		if (s1[i] != s2[i]) {
			printf("違う。");
			return 0;
		}
	printf("同じ。");  // ここでは s1[i] も s2[i] も '\0'
	return 0;
}

Answer 2

まず繰り返したいfor文に括弧がついていないです。
繰り返したい処理は｛｝をつけないといけません。
↓これを

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;
    }

こうしてください。

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;
    }
}

次に今のままだと文字列を比較しているというより、各文字列の一文字目だけ見てfor文から抜けてしまっています。
例) s1 = ABC ,s2 = ADE　でも"同じ。"と出る。

// 1回目のループ (i = 0)
if (s1[0] != s2[0]) // s1の1文字目とs2の1文字目が違う時
{
    printf("違う。"); // "違う。"と出力して
    return 0; // for文から抜け出す
}
if (s1[0] == s2[0]) // s1の1文字目とs2の1文字目が等しい時
{
    printf("同じ。"); // "同じ"と出力して..?
    return 0; // for文から抜け出す..?
}

"同じ"と出力するのは良いとして次のreturn 0;でfor文から抜けてしまっているので
合っていようと間違っていようと今のままだと2文字目の判定をしてくれません。
文字が同じだった時にはreturn 0;は不要です。

Comments

[kennzi_665]

大変詳しくありがとうございます！二人ともベストアンサーにしたいのですがそうも行かないのでより詳しく説明してくださったMUSHIPEＮさんをベストアンサーにさせていただきます。

[kennzi_665]

ベストアンサーついた後で申し訳ないのですが、「同じ間繰り返しているので，if文で違うことを調べる意味がなく、どちらかが'\0'なら同じとする」と言われたのですがどのようにすればいいか分かりますか？もし、ここで返すのがよくないのなら新しい質問で投稿します。

[dodox86]

＞@kennzi_665さん

横からですみませんが、本回答はkennzi_665さんの元のコードに対する間違いと修正案の指摘であり、それに対応して修正したはずであるkennzi_665さんの最終的なコードはどこにも提示されていません。

> 「同じ間繰り返しているので，if文で違うことを調べる意味がなく、どちらかが'\0'なら同じとする」と言われた

とありますが、その文面だけ読んでも状況が分からないですね。コードが適切に修正されていないように思え、どうすべきかは誰にも分からないように思います。いったんはBAを付け、納得して質問を閉じたはずなのですから、修正したコードと共に新たに質問を立てるべきと考えます。

[kennzi_665]

そうですよね。焦ってしまいましたアドバイスありがとうございます。

Answer 3

forの後に中括弧がないので、forでループする対象は1文のみとなります。

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;
}

これは意図したものでしょうか？