C++、最小値を見つける、関数エラー：ループは最大で1回実行されます（ループの増分は実行されません）

Question

5倍を読み取る関数を作成しようとしています（main関数）、最小値を返します。

私は配列の作成とこれらの5つの番号の割り当てを検討していました。その後、配列をループし、その配列の最初の項目と新しい値を比較します。

以下のエラーが表示されます。ループは最大で1回実行されます（ループ増分は実行されません）。

以下のコードを確認してください。それが間違っていたアドバイスは？

double findLowest(double a, double b, double c, double e, double f) 
{ 
    // creating an array numberRange[] to read the 5 input double values 
    double numberRange[] = {a,b,c,e,f}; 

    // creating a variable minimum and assigning it the value of the 
    // first item in the numberRange[] array 
    double minimum = numberRange[0];  

    // looping through the numberRange array 
    for(int counter = 1; counter < sizeof(numberRange) ; counter++) 
     { 
      // checking if the numberRange value at the counter is less than 
      // the minimum value. If true, the minimum value is replaced with 
      // a new value. 
      if (numberRange[counter] < minimum) 
       { 
        minimum = numberRange[counter]; 
       } 

      return minimum; 
     } 

} 

Answer 1

あなたはreturn minimumループの内側を持っているので、その警告の理由があります。したがって、ループの最初の反復中にすぐに関数を終了します。その文は、の後にになるはずです。

また、counterのテストは正しくありません。 sizeof(numberRange)は、配列内のバイト数です。しかし、各数字は1バイト以上です。 doubleが8バイトの場合、sizeof(numberRange)は5 * 8 = 40となり、配列の終わりを過ぎて読み取ろうとします。配列の要素数を取得するには、あなたはそれが

counter < sizeof(numberRange)/sizeof(*numberRange) 

Answer 2

はsizeof（numberRange）= 40

をご希望する必要がありますので、各要素の大きさによって配列のサイズを分割する必要があり40回ループする？各ダブルは、あなたのメモリ内で8バイトを取る。 sizeof（）は配列の長さではなくスペースを教えてくれます...