Функции массива не работают должным образом (C++)

Question

Я не могу для жизни меня выяснить, что не так, но следующие функции findLargest и findSmallest (а также функции findAcer, я считаю) работают не так, как должны и вернуть неверные значения. Что не так?

#include <iostream> 
#include <cstdlib> 
#include <ctime> 

using namespace std; 

int findLargest(int array[], int arraySize){ 
int largest = array[0]; //I set the largest to the first member of the array initially 
for (int i = 0; i < arraySize; ++i){ 
    if (array[i] > largest){ 
     largest = i; 
    } 
} 
return largest; 
} 

int findSmallest(int array[], int arraySize){ 
int smallest = array[0]; //I set the smallest to the first member of the array initially 
for (int i = 0; i < arraySize; ++ i){ 
    if (array[i] < smallest){ 
     smallest = i; 
    } 
} 
return smallest; 
} 

int findAverage(int array[], int arraySize){ 
int total = 0; 
for (int i = 0; i < arraySize; ++i){ 
    total += array[i]; 
} 
int average = total/arraySize; 

return average; 
} 

void display(int array[], int arraySize){ 

cout << "\nThe values for the array are: \n\n"; 

for (int i = 0; i < arraySize; ++i){ 
    cout << array[i] << endl; 
    } 
} 



int main(){ 

const int size = 50; 
int taker[size]; 
srand(time(NULL)); 

for (int i = 0; i < size; ++i){ 
    taker[i] = rand() % 100; //generate 50 random numbers for the array taker 
} 

int largest = findLargest(taker, size); 
int smallest = findSmallest(taker, size); 
int average = findAverage(taker, size); 

cout << "The largest entry was " << largest << endl; 
cout << "The smallest entry was " << smallest << endl; 
cout << "The average for all the entries is " << average << endl; 

display(taker, size); 

} 

Answer 1

если вы хотите вернуть указатель.

int findLargest(int array[], int arraySize){ 
int largest = array[0]; //I set the largest to the first member of the array initially 
int largestindex=0; 
for (int i = 0; i < arraySize; ++i){ 
    if (array[i] > largest){ 
     largestindex=i; 
     largest = array[i]; 
    } 
} 
return largestindex; 
} 

если вы хотите вернуть значение.

int findLargest(int array[], int arraySize){ 
int largest = array[0]; //I set the largest to the first member of the array initially 
for (int i = 0; i < arraySize; ++i){ 
    if (array[i] > largest){ 
     largest = array[i]; 
    } 
} 
return largest; 
} 

Answer 2

В findLargest вы устанавливаете largest к i не array[i]

Answer 3

Ваша функция для среднего выглядит хорошо, однако, самый большой и самый маленький вам нужно использовать значение по индексу, а не только индекс.

Самый большой:

int findLargest(int array[], int arraySize){ 
    int largest = array[0]; //I set the largest to the first member of the array initially 
    for (int i = 0; i < arraySize; ++i){ 
     if (array[i] > largest){ 
      largest = array[i]; 
     } 
    } 
    return largest; 
} 

Наименьший:

int findSmallest(int array[], int arraySize){ 
    int smallest = array[0]; //I set the smallest to the first member of the array initially 
    for (int i = 0; i < arraySize; ++ i){ 
     if (array[i] < smallest){ 
      smallest = array[i]; 
     } 
    } 
    return smallest; 
} 

Ваша функция для среднего выглядит хорошо.

Answer 4

вы должны установить smallest = array[i];largest=array[i]; в их соответствующих функциях.