最小値と最大値を返す関数

Question

double型の配列とint型のflgを渡してflgが0の時は配列の要素の最小値を、flgが0以外の時は配列の要素の最大値を返す関数minMaxを作成したのですがうまく動作しません。
どこが悪いのでしょうか？教えてください。

コード
#include <stdio.h>

#define N 5

double minMax(double x[],int flg){
    int i=0;
    double min=x[N],max;
    if(flg){
       for(i=0;i<N;i++){
          if(x[i]<min){
            min=x[i];
          }
       i++;
       }
     return min; 
    }
    else{
        for(i=0;i<N;i++){
          if(x[i]>max){
            max=x[i];
          }
       i++;
       }
     return max; 
    }

}


int main(){
    int flg;
    double a[N]={2.7,0.4,6.2,1.5,8.4};
    
    printf("最小値=%f",minMax(0,flg));
    printf("最大値=%f",minMax(1,flg));

}

Comments

[tiitoi]

質問欄は何回でも編集できるので、質問に関係するコードなどは質問欄に記載してください

[kaina]

コードはマークダウン形式に修正して下さい。
flgに値が設定されていません。
minMaxの引数を見直して下さい。

[Mithra]

どう「うまく動作しません」なのでしょうか？
そもそもコンパイルが通らないソースに見えますけど。

Answer 1

最大値最小値を求める関数は元々あったかと思いますので、目的はC言語の勉強と推察します。

あまりにも色々なところに余りにも色々な問題が起きているので一度しっかり勉強し直して作り直した方が良さそうです。
0. 引数に渡すべき物が間違っている(関数を勉強する)

1. i++が多過ぎ(for文を勉強する)
2. 変数が初期化されていない箇所がいくつかある(変数を勉強する)

配列の長さが違っていても動くように作れたら、なお関数の本質に近づけるかと思います。

Answer 2

ざっと見た印象で回答します

1. minMaxの引数がおかしい

　×minMax(0,flg) / minMax(1,flg)
〇minMax(a,0) / minMax(a,1)

1. minMax内のmin, maxの初期化が正しくない

　minは範囲外アクセスしています
maxは初期化されていません、自動的に0になるとは思いますが対象がdoubleなので最小値はマイナスもありうるのでは？
doubleの最大値、最小値(あるいは配列の要素として入力される最大値最小値)で初期化するなどしたほうがよいでしょう

1. for文自身と中でiをインクリメントしている

for(i=0;i<N;i++){　←ココ
if(x[i]<min){
min=x[i];
}
i++;　　　　　←ココ
}
このため1ループで2つずつiが増えて全要素が見れていません

こんなところでしょうか

Comments

[XYZNo.00]

ありがとうございます！

Answer 3

double min=x[N],

double a[N]はa[0]～a[N-1]までしかありませんので、x[N]は範囲外です。範囲外のメモリにアクセスすると、何が起きるかわかりません。

Answer 4

コード
#include <stdio.h>

#define N 5

double minMax(double x[],int flg){
    int i=0;
    double min=x[N],max;
    if(flg){
       for(i=0;i<N;i++){
          if(x[i]<min){
            min=x[i];
          }
       i++;
       }
     return min; 
    }
    else{
        for(i=0;i<N;i++){
          if(x[i]>max){
            max=x[i];
          }
       i++;
       }
     return max; 
    }

}


int main(){
    int flg;
    double a[N]={2.7,0.4,6.2,1.5,8.4};
    
    printf("最小値=%f",minMax(0,flg));
    printf("最大値=%f",minMax(1,flg));

}