C++：ソート、max_elementのような関数に建てられた任意のC++を使用せずに、不適切

Question

私は、配列内の最大値を求める関数を記述しようとしています作業マックス配列値の関数など

私はこれを開発しました以下のコードは、それが最終的には、「最大値」を決定しないが、それは常に正しいではありません。

double dispMax(int farray[], int maxValue) 
{ 

    int max = farray[0]; 

    for (int x = 0; x < maxValue; x++) //maxValue is size of farray 
    { 

     if (farray[x] > max) 
     { 

      max = farray[x]; 
      return max; 

     } 

    } 

} 

farray []はソートされていない、とのデータ型（例えば（123.25、234.3のリストだけで構成されています、345.12））。

アレイ全体で繰り返し処理しないようなことがあります。ほとんどの場合、最大値とみなされるものを見つけた場合と同じように、すぐにその値を返します。

また、最小値を見つけるために関数を調整すると、最小値のALMOSTと似た結果が得られますが、それほどではありません。

大変助かりました！

Answer 1

リターンは間違った場所

Suposeアレイはそう最大1 2 3

= 1です。

そしてそれが第2の値のため

farrayになったとき[1] = 2が1より大きくなるように戻す（？）

ループは、すべてトラバースさせ、forループ外のリターンを入れ配列

double dispMax(int farray[], int maxValue) 
{ 
    // need to be double 
    double max = farray[0]; 
    for (int x = 0; x < maxValue; x++) //maxValue is size of farray 
    { 
     if (farray[x] > max) 
     { 
      max = farray[x];  
     } 
    } 
    return max; 
}