配列の要素から誤った最大値を取得してしまう。

Question

前提・実現したいこと

配列の要素から最小値、最大値を取得したいです。
最小値も最大値も同様の方法で取得しているにも関わらず、最小値は意図したものが、
最大値のみ誤ったものが出力されます。

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

最小値は16
最大値は42

該当のソースコード

#include <stdio.h>
int main(void){
    int array[10]={50,38,21,54,83,23,94,67,16,42};
    int min = array[0];//仮の最小値
    int max = array[0];//仮の最大値
    
    for(int i=0;i<=9;i++){//最小値を求める
        if(array[i]>array[i+1]){
            min = array[i+1];
        }
    }
    
    for(int j=0;j<=9;j++){//最大値を求める
        if(array[j]<array[j+1]){
            max = array[j+1];
        }
    }
    
    printf("最小値は%d\n",min);
    printf("最大値は%d",max);
}

試したこと

サンプルの値を下記のようの代入した場合は最小値、最大値ともに正しい実行結果が得られました。
int array[10]={80,70,31,24,88,90,4,63,116,12};

最小値は12
最大値は116

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

本来、サンプルの値に関わらず最小値と最大値を求められることを想定しています。
実行環境はpaiza.ioです。

Answer 1

ええと、現在のソースで最大値と最小値が正しく求められているのは偶然そうなっているだけです。
(たとえば、「試したこと」のサンプルの最後の2つの値が全体の最大値と最小値になっている)

最小値で説明すると、minに「それまでの最小値」を入れるためには、if文は「minと配列の要素の値」を比較し、minよりも小さければ、minにその配列の要素の値を代入する、ように書かなければいけません。

ついでに言うと、元のソースは array[10] (配列の範囲外の値)を使っています。

Comments

[ceg34ry]

回答ありがとうございます。偶然もっともらしい値が得られてしまうことも勉強になりました。

Answer 2

最小値、最大値の検索内で、配列内の値の大小を比較していますが、比較対象は、仮の最小値、最大値である、min、maxです。

#include <stdio.h>
int main(void){
    int array[10]={50,38,21,54,83,23,94,67,16,42};
    int min = array[0];//仮の最小値
    int max = array[0];//仮の最大値

    for(int i=1;i<=9;i++){//最小値を求める
        if(array[i]<min){
            min = array[i];
        }
    }

    for(int j=1;j<=9;j++){//最大値を求める
        if(array[j]>max){
            max = array[j];
        }
    }

    printf("最小値は%d\n",min);
    printf("最大値は%d",max);
}

なお、元のコードは、配列外へのアクセスがある危険なコードです。
また、試したことの場合の最小値は４が正解なので、誤った結果です。

Comments

[ceg34ry]

回答ありがとうございます。わかった事は以下のとおりです。
・比較対象は配列同士ではなく仮の最小値（または仮の最大値）であること
・配列外アクセスの危険なコードであること
・「試したこと」も正常に動作していなかった事

Answer 3

プログラムが間違っていますので、正しく求まったり、求まらなかったりするのは、データによってたまたまです。

隣と比べて大きいか小さいかを見ても無意味です。
今までの最小値や最大値と比べましょう。

Comments

[ceg34ry]

回答ありがとうございます。

Answer 4

    for(int i=0;i<=9;i++){//最小値を求める
        if(array[i] < min) { // minより小さいのが見つかったらminを更新 
            min = array[i];
        }
    }

じゃなくて?
※ maxも同様

Comments

[ceg34ry]

回答ありがとうございます。

Answer 5

forループの中のifで配列要素同士ではなくminやmax変数と比較する必要があります。
最小値の方もたまたま正常に動作してるように見えるだけです。

Comments

[ceg34ry]

回答ありがとうございます。