最も高い身長を表示するプログラム

Question

5人分の身長から最も高い身長を表示するプログラムをつくったのですが、エラーがでて実行できません。改善すべきところを教えてください。お願いします。
int main(void)
{
	double a[5], max = 0;
	int i;

	for (i = 1; i <= 5; i++)
	{
		a[i] = 0;
	}
	printf("数値を5つ入力してください。
");

	for (i = 1; i <= 5; i++)
	{
		scanf("%lf", &a[i]);
	}

	for (i = 1; i <= 5; i++)
	{
		if (a[i] > max)
			max = a[i];
	}

	printf("最も高い身長は%fです
", max);

	return 0;
}

kmee · Accepted Answer

double a[5] ;
と宣言したら、安全に使えるのは a[0] 〜 a[4] です。

なので、このプログラムを安全なものにするなら
・double a[5]はそのまま、 i=0;i<5;i++ にして、 i=0〜4 の範囲で使う
・「i = 1; i <= 5; i++は変えない」 のなら
　・ i=1〜5を、 0〜4 に対応させて使う
　　　単純明快なのは、 a[i-1] とすること
　・double a[5+1] と宣言して、a[5] を安全に使えるようにする。
　　　a[0] が無駄になるけど気にしない

tatsu99 · Answer

>i = 1; i <= 5; i++は変えない方針でいきたいのですがその場合どうすればいいのでしょうか？
それでしたら
a[i]を
a[i-1]
に全て変えてください。

tatsu99 · Answer

どこが悪いのかは、他の方が既に説明してますので、修正個所だけ。
for (i = 1; i <= 5; i++)
を
for (i = 0; i < 5; i++)
に変えてください。（３か所全て変えます）

kmee · Answer

これ、C言語だと「コンパイルエラー」にならないし、実行も正常に動作する(こともある)、というものです。

○コンパイル時にメッセージが出るものには、次の2つがあります。
　・エラー(error): 致命的な間違いでコンパイルできない
　・警告(warning): 致命的ではないので、コンパイル自体はできる。
　　　最終的に問題の原因になる可能性がある
○「コンパイルエラーが出て実行できない」と「実行したらエラーになる」とは別です。

「エラーがでて実行できません。」というのが、どんな状況なのか、正確にわからないと、答える側も答えが難しくなります。
(今回くらいのものなら、なんとかなりますが)

質問するにしろ、自分でなんとかするにしろ、エラーや警告を正確に捉えることが大事です。

(1) printf,scanf が「未定義」という「警告」または「エラー」になる。
C言語だと、宣言していない関数は自動で「引数任意、戻り値int」で宣言されたものと見做し、コンパイルを進めます。
その旨を警告として表示されることがあります。
今回の場合、たまたま、scanfとprintf は「引数任意、戻り値int」(と同等)なので、そのまま正常にコンパイルできます。

C++だと、そのような「暗黙の宣言」をしないので、「未定義」のエラーとなります。

printf等の関数を使う場合は、適切なヘッダファイルを#includeしましょう。
C言語なら #include <stdio.h>
C++なら #include <cstdio>

(2) 既に指摘がありますが、 a[5] と宣言した配列で使えるのは a[0]〜a[4] の5つです。
対して、for( i=1;i<=5;i++) { 〜 a[i] 〜 } と、 a[1]〜a[5] を使っています。

C/C++ の場合、a[5] を指定すること自体は間違いではありません。
その結果がどうなるかわからない、というだけです。
たまたまa[5]に該当するのが他に影響の出ない領域で、プログラムが期待通りに動作する、ということ有り得ます。
エラーになるときも、全然関係箇所で実行時エラーになって、本当の原因がよくわからない、ということもあります。

配列を使うときは、プログラム作成者が注意して使うしか、対策はありません。

解決策は 0〜4 になるようにするのが一般的ですが、
a[5+1] にして、a[0]〜a[5] を使えるようにして、 a[0] は使わない、というやり方もあります。

don_go · Answer

エラー箇所は明白ですが...(a[5])
エラー発生時の対処するには、どんなエラーメッセージが出ているかが、重要な
ヒントになります。
質問する際にも、エラーメッセージを略さず訳さず提示する方が良いです。

acombun · Answer

どこでエラーが出るのかわからないけど、ループの部分はfor(i=0;i<5;i++)でないと配列外を見ちゃうよ。

最も高い身長を表示するプログラム

double a[5] ;

>i = 1; i <= 5; i++は変えない方針でいきたいのですがその場合どうすればいいのでしょうか？

どこが悪いのかは、他の方が既に説明してますので、修正個所だけ。

これ、C言語だと「コンパイルエラー」にならないし、実行も正常に動作する(こともある)、というものです。

エラー箇所は明白ですが...(a[5])

どこでエラーが出るのかわからないけど、ループの部分はfor(i=0;i<5;i++)でないと配列外を見ちゃうよ。

似たような質問が見つかりました

このQ&Aを見た人はこんなQ&Aも見ています

関連するカテゴリからQ&Aを探す

このQ&Aを見た人がよく見るQ&A

デイリーランキングこのカテゴリの人気デイリーQ&Aランキング

マンスリーランキングこのカテゴリの人気マンスリーQ&Aランキング