最小値、最大値、合計値を求めるプログラム

Question

前提・実現したいこと

n 個の整数 を入力し、それらの最小値、最大値、合計値を求めるプログラムを作成する。

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

最大値、または最小値どちらかの値が入力したものと違う

該当のソースコード

C++
ソースコード
（略）
int main()
{
int n,a,i,x;

cin >> n;
int number[n];
for (i = 1; i <= n; i++)
{
cin >> number[i];
}

int max,min,all = 0;
max = min = number[0];
//最小値
for(x = 1; x <= n; x++)
{
if(number[x] > max)
max = number[x];
if(number[x] < min)
min = number[x];

all = all+number[x];

}

cout << "max:" << max << endl;
cout << "min:" << min << endl;
cout << "all:" << all << endl;

（略）

試したこと

for文の中でこまめに出力させたりして経過を確認してみた。

補足情報（FW/ツールのバージョンなど）

editor : atom
compiler : MinGW

Answer 1

ちょと見ですが・・・
int number[n];
で、nが１０の時、number[9];までしか有効でない。その後の
for文のfor (i = 1; i <= n; i++)がおかしい。cやc++の場合、配列のインデックスは０から始まる。

Answer 2

int number[n];
for (i = 1; i <= n; i++)
...
for(x = 1; x <= n; x++)

配列の添え字は 0 から始まるのだが...

Answer 3

for(x = 1; x <= n; x++)

配列の要素番号は1ではなく0から始まります。

max = min = number[0];

初期値をnumber[0]にするのは変ですね。
maxにはintの最小値、minにはintの最大値が初期値として保持されるべきです。

後者については問題なしです。

Comments

[BluOxy]

for (i = 1; i <= n; i++)
ここも0から始まります。

nが4のとき、numberで指定できる要素の番号は0,1,2,3です。

[LouiS0616]

回答後半についてはnumber[0]の存在が保証される限り全く問題無いように思います。

[episteme]

> 初期値をnumber[0]にするのは変ですね。

別におかしくはないような。
最小値/最大値を「暫定的に」number[0]ってことにしておいて、
それより小さい/大きい値を見つけたら更新するんだから。

[BluOxy]

理解しました。LouiS0616さん、epistemeさん、ありがとうございます。

[yshk0402]

皆さんありがとうございました！！
期待通りの結果を実行できました。
ベストアンサーは一番早く回答してくださった、BluOxyさんにさせていただきました。