c言語 for(;;)の無限ループを終わらせたいです

Question

1. 番号リスト### 無限ループを終わらせたいです

2つの整数 n, k を入力する (ただし, 0<k<n, k≤5, n≤10 とする).
n 個の整数 a0, a1, …, an-1 を入力する.
以下を繰り返す.
k 個の整数 b0, b1, …, bk-1 を入力する.
ただし, 途中で EOF が入力されれば, k 個未満の整数しか入力されない.
入力された数字が k 個未満であれば, プログラムを終了する.
そうでなければ,
b0=ai, b1=ai+1, b2=ai+2, … bk-1=ai+k-1
を満たす i をすべて (スペース区切りで) 出力し, その後に改行を出力する.

という課題を行っているのですが、配列bに要素をファイルから入力するときに
無限ループしてしまい、うまくいきません。

発生している問題・エラーメッセージ

以下の入力ファイル(match-in.txt)を入力すると、最終行の3 5 2 が永遠に実行され続けます
10 3
3 5 2 4 2 4 2 3 5 2
5 2 4
2 4 2
5 2 0
3 5 2

該当のソースコード

#include <stdio.h>

#define size 512
int compare(int, int[], int[], int, int);



int main(void)
{
    int i, n, k;
	int a[size], b[size];

	fprintf(stderr, " 0<k<n, k≤5, n≤10 となる2つの整数 n, k を入力してください");
	scanf("%d%d", &n, &k);
	
	fprintf(stderr, "%d個の整数を入力:", n);
	for(i=0; i<n;i++)
	{
		scanf("%d", &a[i]);//文字列を受け取る場合&は不要
	}
	i=0;
	for(;;)
	{

		for(i=0;i<k;i++)
		{
			fprintf(stderr, "%d個の整数を入力:", k);
			if(scanf("%d", &b[i]) == EOF ){break;}//ここのbreakが動いていないと考えられます

		}
		
		

		for(i=0;i<n-k+1;i++)
		{
			if(compare(i, a, b, n, k) == 1)
			{
				printf("%d", i);
			}
		}
		i=0;
		printf("\n");
	}
	return 0;
}

int compare(int i, int a[], int b[], int alen, int blen)
{
	int k;
	k = 0;
	
	while( a[i] == b[k] )		
	{
			
		if(k == blen-1)
		{
			return 1;
		}
		k++;
		i++;
	}

	return 0;
}

備考

プログラムはテキストファイルに書いてcygwinでコンパイルしています

gcc -Wall match.c
したのち
./a < match-in.txt
でファイルを入力しています

Comments

[itagagaki]

内側のforループは i<k の条件を満たさなくなるかまたは break文によって終了するけど、その後においても、その外側にあるforループには終了する条件もbreak文も無いので当然無限にループしますよね。

[iisan]

返信ありがとうございます。breakの意味を正しく理解していませんでした。

Answer 1

提示されているコードでは，外側の（for(;;)の）ループを抜けた後に特段やるべきことがない（return 0;しかない）ので，

if(scanf("%d", &b[i]) == EOF ){break;}//ここのbreakが動いていないと考えられます

の break; を return 0; としてしまっても良いですよね．

…そんな感じ(?)で，
多重ループの内側にある（：最も外側ではない）ループ内に多重ループ自体の終了判定があるような場合，
「多重ループ自体を関数に突っ込んでreturnで関数から抜ける」という形にすると楽な場合もあります．
（大抵，そのような処理は，適切な名称が付与できる作業なんじゃないかと思うし）

void SomeWork( /*引数群*/ )
{
  //多重ループ
  for( /*略*/ )
  {
    for( /*略*/ )
    {
      if( XXX )return;  //関数から抜ける
    }
  }
}

Answer 2

for(i=0;i<k;i++)

    {
        fprintf(stderr, "%d個の整数を入力:", k);
        if(scanf("%d", &b[i]) == EOF ){break;}//ここのbreakが動いていないと考えられます
    }

このbreakは直前のforから抜けるだけ、ですよ

Comments

[iisan]

返信ありがとうございます。breakの意味を正しく理解していませんでした。breakの書く行を変え、うまく動きました。