C言語での計算の結果が大きく出る

Question

最小値、最大値、合計値を求める問題です。
c言語はあえて使っています。
どこが間違っているか教えてください...

 int main(){
     int n,i;
     int num,MAX,MIN,SUM;
    scanf("%d",&n);
    for(i=1;i<n;i++){
        scanf("%d",&num);
        if(MAX<num){
            MAX=num;
            SUM=SUM+num;
        }else if(MIN>num){
            MIN=num;
            SUM=SUM+num;
        }else{
            SUM=num+SUM;
        }
    }
    printf("%d %d %d %d\n",MIN,MAX,SUM,i);
 }

Answer 1

初期化が必要ですが
なにを初期値とするか　を考える必要がありますね

int main(){
    int n, int i = 0;
    int MIN, MAX, SUM = 0, num;
    scanf("%d", &n);
    for(i=1;i<n;i++){
        scanf("%d", &num);
        if(i == 1) // 初期化
            MIN = MAX = num;
        if(MIN > num) MIN = num;
        else if(MAX < num) MAX = num;
        SUM += num;
    }
}

Comments

[re_refuse]

ありがとうございます！

[re_refuse]

if(i==1)の部分は
MINを初期値0としたときに、MINよりnumが小さくなることを防ぐということですか？

[re_refuse]

numよりMINが小さくなるでした...

[asm]

最初の一回の値を暫定最大最小に置いています

[re_refuse]

ありがとうございます！

Answer 2

初期値はこんな感じで与えます。
他のおかしい点も直しました。

#include <stdio.h>
#include <limits.h>

int main () {
  int n, i;
  int num, MAX = INT_MIN, MIN = INT_MAX, SUM = 0;
  scanf("%d", &n);
  for (i = 0; i < n; i++) {
    scanf("%d", &num);
    if (num > MAX) {
      MAX = num;
    }
    if(num < MIN) {
      MIN = num;
    }
    SUM += num;
  }
  printf("%d %d %d %d\n", MIN, MAX, SUM, i);
}

Comments

[re_refuse]

ありがとうございます！

[re_refuse]

if文はelseifは使わないほうがよかったですか？

[otn]

1件目は、最小値かつ最大値なので、両方実行が必要です。
asmさんの回答のように1件目だけ特別扱いするなら、else if でいいです。

[re_refuse]

勉強になりました！ありがとうございます！。

Answer 3

こうですね。C言語は初期化が命なので...

int main(){
     int n = 0,i = 0;
     int num = 0,MAX = 0,MIN = 0,SUM = 0;
    scanf("%d",&n);
    for(i=1;i<n;i++){
        scanf("%d",&num);
        if(MAX<num){
            MAX=num;
            SUM+=num;
        }else if(MIN>num){
            MIN=num;
            SUM+=num;
        }else{
            SUM+=SUM;
        }
    }
    printf("%d %d %d %d\n",MIN,MAX,SUM,i);
 }

Comments

[otn]

初期化というとゼロを入れると思っていませんか？

[re_refuse]

ありがとうございます！

[stdio]

otnさん、初心者相手に仕組んだ罠に玄人が突っ込むべきではないですよ。それぐらい考えたさせる方が身につくと思ったのでね...
実際にやるならint型の最大値(limits.hの中に存在するINT_MAX)でも入れておく方が無難でしょうね。

Answer 4

MAX、MIN、SUMが初期化無しで使われてます
ローカル変数は初期化しないとでたらめな値が入ってるのでこれではダメですね

Comments

[re_refuse]

ありがとうございます！

Answer 5

scanf()を取って初期化した元のソース

#include <stdio.h>

static int number[] = { 0, 1, 2, 3, 4, 5, 6, 7, 8, 9 };

int main()
{
    int num, MAX = 0, MIN = 10, SUM = 0;
    //    scanf("%d", &n);
    int n = 10;

    for (int i = 1; i < n; i++) {
        //        scanf("%d", &num);
        num = number[i];
        if (MAX < num) {
            MAX = num;
            SUM = SUM + num;
        } else if (MIN > num) {
            MIN = num;
            SUM = SUM + num;
        } else {
            SUM = num + SUM;
        }
    }
    printf("%d %d %d \n", MIN, MAX, SUM);

}
結果
usr ~/Project/test/teratail % ./a.out
10 9 45

修正版

#include <stdio.h>

static int number[] = { 0, 1, 2, 3, 4, 5, 6, 7, 8, 9 };

int main()
{
    int num, MAX = 0, MIN = 10, SUM = 0;
    //    scanf("%d", &n);
    int n = 10;

    for (int i = 1; i < n; i++) {
        //        scanf("%d", &num);
        num = number[i];
        if (MAX < num) {
            MAX = num;
        }
		if (MIN > num) {
            MIN = num;
        }
        SUM = SUM + num;
    }
    printf("%d %d %d \n", MIN, MAX, SUM);
    //
    return 0;
}

結果
usr ~/Project/test/teratail % ./a.out
1 9 45 

理由はわかりますか？

Comments

[cateye]

かぶったｗ