Функция, которая возвращает более одной вещи

Question

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

struct num { 
    int min; 
    int max; 
}; 

struct num Some(int Array[]) { 
    int i; 
    struct num result; 
    result.max = Array[0]; 
    result.min = Array[0]; 
    for(i = 0; i<=3; i++) { 
     if(result.min > Array[i]) { 
      result.min = Array[i]; 
     } else if(result.max < Array[i]) { 
      result.max = Array[i]; 
     } 
    } 
    return result; 
} 

int main() 
{ 
    int Array[] = {7,8,3}; 
    struct num something; 
    something = Some(Array); 
    printf("Total is %d and max is %d",something.min,something.max); 
} 

Программа печатает правильное минимальное значение массива, но максимальное значение дает 4200784.

Answer 1

for(i = 0;i<=3;i++){ 
//  ^^^^ 

Вы цикл в четыре раза.

Над массивом, который имеет три элемента.

Answer 2

sizeof 

Для надлежащего разрешения вы должны использовать выше, чтобы получить точный размер массива использовать вместо I < = 3.

Answer 3

Вы используете неправильный диапазон индексов. Поскольку исходный массив имеет только 3 элемента, то допустимый диапазон индексов равен [0, 2]. Таким образом, в цикле внутри функции

for(i = 0; i<=3; i++) { 

вы пытаетесь получить доступ к памяти за пределами массива, когда я равно 3.

Но в любом случае дизайн функция не является хорошим. Он использует магическое число 3. Поэтому его можно применять только к массивам из трех элементов. Также вы не можете определить, какой элемент является минимальным, а какой - максимальным.

Лучше подход заключается в следующем

typedef struct minmax 
{ 
    size_t min; 
    size_t max; 
} minmax_t; 

minmax_t minmax_element(const int a[], size_t n) 
{ 
    minmax_t result = { 0, 0 }; 

    for (size_t i = 0; i < n; i++) 
    { 
     if (a[i] < a[result.min]) 
     { 
      result.min = i; 
     } 
     else if (a[result.max] < a[i]) 
     { 
      result.max = i; 
     } 
    } 

    return result; 
} 

И главная функция может быть вызвана следующим образом

int main() 
{ 
    int a[] = { 7, 8, 3 }; 
    minmax_t something; 

    something = minmax_element(a, sizeof(a)/sizeof(*a)); 

    printf("Minimum is %d and max is %d\n", a[something.min], a[something.max]); 

    int b[] = { 7, 8, 3, 0, 1, 5, 9 }; 

    something = minmax_element(b, sizeof(b)/sizeof(*b)); 

    printf("Minimum is %d and max is %d\n", b[something.min], b[something.max]); 
} 

Примите во внимание, что в C++ существует соответствующий алгоритм std::minmax_element, объявленная в заголовке <algorithm> Он возвращает пару указателей (итераторов) первого минимального элемента и последнего максимального элемента. Вы можете изменить функцию, которую я показал таким образом, чтобы она также возвращала индекс последнего максимального элемента в массиве вместо первого максимального элемента.