正常にプログラムを終了できない理由

[メル]

下記のプログラムはやりたい動作はできましたが９９９を押したときに終了してほしいのですが正常にプログラムを終了してくれません。エラーが出ているわけではないのでどこがよくないのかわかりません。どこかおかしな点はありますか？
#include <stdio.h>

int main(void)
{
//a=生徒の数、sum=合計、ave=平均
float a[20];
int sum=0, i, j=0;
for(i=1; i<=20; i++)
{
printf("順番にタイムを記録してください\n");
scanf("%f",&a);
sum=sum+a;
}
printf("２０人の平均は%5.1fです\n",(double)sum/20);

while(1){
printf("1から20の順番で出力したい記録を入力してください(999で終了)");
scanf("%d",&j);
printf("%5.1f秒です\n",a[j]);
if(j==999)
break;
}
return 0;
}

[みけCAT]

どこかおかしな点はありますか？

• for文の終了条件が間違っており、確保されていないa[20]にアクセスしてしまいます。(これは前回も指摘しました)
• while文のループの中でjの範囲をチェックしていないので、999などの0以上20未満の値以外が入力された時、確保されていない領域にアクセスしてしまいます。
• scanfの戻り値をチェックしていないため、特にwhile文の中のscanfで無効な入力を読み込もうとすると無限ループになる可能性があります。

オフトピック

同じようなコードに関する質問が、前のトピックが解決にならないうちに連続して出てくるのも少しおかしい気がします。
解決したトピックは、解決チェックをお願いします。

[メル]

[*]for文の終了条件が間違っており、確保されていないa[20]にアクセスしてしまいます。(これは前回も指摘しました)
[*]while文のループの中でjの範囲をチェックしていないので、999などの0以上20未満の値以外が入力された時、確保されていない領域にアクセスしてしまいます。
[*]scanfの戻り値をチェックしていないため、特にwhile文の中のscanfで無効な入力を読み込もうとすると無限ループになる可能性があります。[/list]

とりあえず２番？までは直しました（あってないかもしれません）。３番の戻り値がわからないです。戻り値とはreturnのことですよね？これはどうすればいいのでしょうか。戻り値について調べてもよくわかりません。

#include <stdio.h>

int main(void)
{
//a=生徒の数、sum=合計、ave=平均
float a[20], sum=0;
int i, j=0;
for(i=0; i<20; i++)
{
printf("順番にタイムを記録してください\n");
scanf("%f",&a);
sum=sum+a;
}
printf("２０人の平均は%5.1fです\n",(double)sum/20);

while(1){
if(0<=j<20)
printf("1から20の順番で出力したい記録を入力してください(999で終了)");
scanf("%f",&j);
printf("%5.1f秒です\n",a[j-1]);
if(j==999)
break;
}
return 0;
}

[メル]

だいぶ完成しました。あとは結果で９９９を打ち込んだときそのまま終了してほしいのですが０．０秒ですと表記されて終了してしまいます。助言よろしくお願いします。

#include <stdio.h>

int main(void)
{
//a=生徒の数、sum=合計、ave=平均
float a[20], sum=0;
int i, j=0;
for(i=0; i<20; i++)
{
printf("順番にタイムを記録してください\n");
scanf("%f",&a);
sum=sum+a;
}
printf("２０人の平均は%5.1fです\n",(double)sum/20);

while(1){
　　 0<=j;
j<20;
printf("1から20の順番で出力したい記録を入力してください(1000で終了)");
scanf("%d",&j);
printf("%5.1f秒です\n",a[j-1]);
if(j==999){
printf("終了します");
break;}}
return 0;
}

[みけCAT]

あとは結果で９９９を打ち込んだときそのまま終了してほしいのですが０．０秒ですと表記されて終了してしまいます。助言よろしくお願いします。

scanf("%d",&j);の後、printf("%5.1f秒です\n",a[j-1]);の前にjが999だったら終了する処理を入れてください。

また、

0<=j;
j<20;

というコードは全く意味がなく、コンパイラとオプションによっては警告が出るはずです。

３番の戻り値がわからないです。戻り値とはreturnのことですよね？これはどうすればいいのでしょうか。戻り値について調べてもよくわかりません。

戻り値が期待した値かをチェックし、そうでなければ強制終了するのが簡単でしょう。

scanf("%f",&a); → if(scanf("%f",&a)!=1)return 1;
scanf("%d",&j); → if(scanf("%d",&j)!=1)return 1;

[メル]

言われた通り999で終了の位置を前に持って行ったら完成出来ました！ありがとうございます。