配列の中で最大の数を見つけると、最大のint値が返されます。

Question

関数を使って配列内の最大値と最小値を探したいと思います。ここに私のコードだ：

#include <stdio.h> 

void findLowHigh(int* numbers, int size, int* min, int* max) { 
    for(int i = 0; i < size; i++) { 
     if (*min > numbers[i]) *min = numbers[i]; 
     else if (*max < numbers[i]) *max = numbers[i]; 
    } 
} 

int main() { 
    printf("##### Find lowest and highest number in collection #####"); 
    printf("\n Array checked: [ 1, 2, 3, 4, 5, 6, 7, 8, 9, 10 ]"); 
    int min, max; 
    int numbers1[10] = { 1, 2, 3, 4, 5, 6, 7, 8, 9, 10 }; 
    findLowHigh(numbers1, 10, &min, &max); 
    printf("\nMin: %d\nMax: %d", min, max); 

    printf("\n Array checked: [ 10, 9, 8, 7, 6, 5, 4, 3, 2, 1 ]"); 
    int numbers2[10] = { 10, 9, 8, 7, 6, 5, 4, 3, 2, 1 }; 
    findLowHigh(numbers2, 10, &min, &max); 
    printf("\nMin: %d\nMax: %d", min, max); 

    return 0; 
} 

と出力があります：

##### Find lowest and highest number in collection ##### 
Array checked: [ 1, 2, 3, 4, 5, 6, 7, 8, 9, 10 ] 
Min: 1 
Max: 32767 
Array checked: [ 10, 9, 8, 7, 6, 5, 4, 3, 2, 1 ] 
Min: 1 
Max: 32767 
Process finished with exit code 0 

私はポインタに何か問題があると推測しているが、それは何ですか？

Answer 1

if (*min > numbers[i])*minには既に有効な値が含まれているものとします。

ただし、アドレスが渡された整数は初期化されません。 （limits.h（またはclimitsを含めた後、あなた本当には、C++）をコンパイルしている場合）が可能修正：

int min=INT_MAX; 
int max=INT_MIN; 

---

ネイサンオリバーはコメントで示唆したようにします。より頑強な解決策は、配列の最初の要素にminとmaxを設定することです。

if (min && max && numbers && size > 0) { 
    *min = numbers[0]; 
    *max = numbers[0]; 
} 

Answer 2

あなたの最小/最大を初期化する必要があります。

int min, max; 

のように：Cで

int min=INT_MAX, max=INT_MIN; 

もっと++あなたが#include <limits>および使用するスタイル：

int min = std::numeric_limits<int>::max(); 
int max = std::numeric_limits<int>::min(); 

[編集]

を

また、（それがコメントにあったように）から、あなたのロジックでelseを削除する必要があります。

if (*min > numbers[i]) *min = numbers[i]; 
    else if (*max < numbers[i]) *max = numbers[i]; 

そうでない場合、あなたはあなたの結果で最大値を見つけないかもしれません。

Answer 3

値を関数に渡す前に、min = INT_MAXおよびmax = INT_MINの値を初期化する必要があります。

Answer 4

int min, max;のような変数を初期化するときは、数値ではなく空白のままにします。これは、minとmaxが自動的に未定義数として設定されることを意味します。

Answer 5

int min, max;は初期化されていませんが、任意の値を含めることができます。 findLowHighの冒頭で初期化する必要があります。 findLowHighを呼び出す前にリセットすることが重要です。過去の結果が将来の結果に影響を与えます。

両方の要素を最初の要素numbersに設定し、ループ内のその要素をスキップできます。最初にsizeがゼロでないことを確認してください。

void findLowHigh(int* numbers, int size, int* min, int* max) 
{ 
    if(size > 0) 
    { 
     *min = numbers[0]; 
     *max = numbers[0]; 

     for(int i = 1; i < size; i++) { 
      if (*min > numbers[i]) *min = numbers[i]; 
      if (*max < numbers[i]) *max = numbers[i]; // Also, remove this else 
     } 
    } 
} 

編集：sizeがゼロの場合に何が起こるかを決める必要があります。空リストの最小値と最大値は何ですか？