配列の最大値を求める関数を使用する

Question

誰かが間違っていることを説明できますか？配列を使ってパーセンテージ配列の最大値を探し、その量と年を対応する年配列に表示する必要があります。

#include <stdio.h> 
#include <stdlib.h> 
#include <ctype.h> 

int findHighNumber (double percentages[], int elements); 

int main (void) 
{ 

    int index; 
    int elements; 


    int years[] = {2000, 2002, 2004, 2006, 2008, 2010, 2012}; 
    double percentages[] = {6.7, 6.6, 8, 9, 11.3, 14.7, 14.6}; 
    int sampleSizes[] = {187761, 444050, 172335, 308038, 337093, 1000, 346978}; 


    int high = findHighNumber(percentages, elements); 
    printf ("%i had the highest percentage with autism at %.2lf%%.\n", years[high], percentages[high]); 


    return 0; 
} 

int findHighNumber (double percentages[], int elements) 
{ 
    int index, high; 
    high = percentages[0]; 

    for (index = 1; index < elements; index++) 
     if (high < percentages[index]) 
     { 
     high = percentages[index]; 
    } 
    return high; 
} 

Answer 1

まず、あなたはmain()でelements変数を宣言する必要があります。

ループ内の変数elementsを変更しないでください。 これを行うと、現在の最大値が変更されるたびに、次の要素の後でループが停止するようにプログラムされます。実際の最大値が不足している可能性があります。

次に、Bobさんが言ったように、最大​​ののインデックスを返す必要があります。この方法で、対応するsampleSize、またはyearを取得できます。
years[high]を実行すると、アレイインデックスとしてのパーセンテージが使用されています。これは{0,1,2,3,4,5にとどまるインデックスではなく[0,100]で実際の値をとることができます、6}。

さらにhighをの変数に格納すると、値が切り捨てられます。 doubleという変数に格納することをお勧めします。

のでfindHighNumber()はなることができますいくつかの小さなアドバイス、そして、

int main (void) 
    { 
      int years[] = {2000, 2002, 2004, 2006, 2008, 2010, 2012}; 
      double percentages[] = {6.7, 6.6, 8, 9, 11.3, 14.7, 14.6}; 
      int sampleSizes[] = {187761, 444050, 172335, 308038, 337093, 1000, 
        46978}; 
      int elements = sizeof(percentages)/sizeof(*percentages); 
      int high_index = findHighNumber(percentages, elements); 

      printf ("%i had the highest percentage with autism at %.2lf%% (sample" 
        "size was %d).\n", years[high_index], percentages[high_index], 
        sampleSizes[high_index]); 

      return 0; 
    } 

：

int findHighNumber(double percentages[], int elements) 
    { 
      int index; 
      double high; 
      int high_index; 

      high_index = 0; 
      high = percentages[high_index]; 

      for (index = 1; index < elements; index++) 
        if (high < percentages[index]) { 
          high = percentages[index]; 
          high_index = index; 
        } 

      return high_index; 
    } 

main()機能は、今のようなものになることができます

1. がfindHighNumber()のプロトタイプを入れています必要ではない、ちょうどその場所の上に関数を定義するeded。このように、の場合、実際には関数のプロトタイプを置く必要があります。これは、（例えば、相互に再帰的な関数のように）強制されたことを示唆し、コードを短縮します。

2. は、あなたがそのように、例えば、struct autism_sampleを定義する必要があります。

   struct autism_sample autism_samples[] = { 
       { 
        .year = 2000, 
        .sample_size = 187761, 
        .percentage = 6.7, 
       }, 
       { 
        .year = 2002, 
        .sample_size = 444050, 
        .percentage = 6.6, 
       }, 
       ... 
   }; 
   

   この方法では、あなたのデータがで構成されています

   struct autism_sample { 
        int year; 
        int sample_size; 
        double percentage; 
   }; 
   

   この方法は、あなただけの1つの配列を定義する必要がありますより論理的な方法、エラーが発生しにくい、findHighNumber（）の実装では、最大または直接のインデックスを返すために、最大値を保持する要素へのポインタを返すようにすると、インデックスは役に立たなくなります。
   しかも、それがシリアライズ（UN）に簡単です...

Answer 2

これはここにタイプミスである場合、私はわからないんだけど、

int high = findHighNumber(percentages, elements); 

を呼び出している間、それはあなたが変数elementsが定義されていない表示されます。それはfindHighNumber()内、elementsは、関数にローカルなので、elements = index+1;としてのみ使い方は無用である、あなたは

最後に

if (high < percentages[index]) 
     { 
     high = percentages[index]; ///after this. 

そば（終値}が不足して）別の構文エラーを持っている、と述べました。

Answer 3

データを格納して使用する方法を考えれば、上位の要素がある場合はその値ではなく、値のインデックスを返すべきです。 また、初期化されていない値を渡している間は、配列の適切なサイズを渡す必要があります（関数内で変更しないでください）。

Answer 4

値の代わりにインデックスを送信する必要があります。自動的に配列のサイズを計算した

int elements = sizeof(percentages)/sizeof(*percentages); 

：

int elements = 7; 

またはより良い：

int findIndexOfMax (double percentages[], int elements) { 

    int index = 0; 
    int highIndex = 0; 
    double high = 0.0; 
    high = percentages[highIndex]; 

    for (index = 1; index < elements; index++) 
    { 
     if (high < percentages[index]) 
     { 
      high = percentages[index]; 
      highIndex = index; 
     }  
    } 

    return highIndex; 
} 

Answer 5

あなたは最高値に何を返すべきではありませんが、あなたはそれのインデックスを返す必要があります。..間違った値を返すされています。

あなたが呼んでいる機能にあなたが望む結果を単に印刷することができます。

また、可変要素を初期化しているとは限りません。