Question

#include <stdio.h>
#define SIZE 10
void function(int array[], int size, int compare, int* min, int* max);

int main() {
    int max1, min1, n, m, array1[SIZE];
    printf("Please enter an array size: ");
    scanf("%d", &n);
    printf("Enter numbers for array:");
    for (int i = 0; i < n; i++) {
        printf("enter number %d", i + 1);
        scanf("%d", &array1[i]);
    }
    printf("Enter a number to be compared:");
    scanf("%d", &m);
    function(array1, n, m, &min1, &max1);
    printf("There are %d numbers less than %d and there are %d numbers greater than %d",min1, m, max1, m);
}

void function(int array[], int size, int compare, int *min, int *max) {
    for (int i = 0; i < size; i++) {
        if (array[i] < compare)* min++;
        if (array[i] > compare)* max++;
    }
}

需要帮助，为什么它只返回最小值和最大值的随机数。通过引用传递可能是搞砸了，但是我不知道该如何解决。

Answer 1

您的代码具有未定义的行为。

由于operator precedence（++的优先级高于取消引用运算符*）

* min++;

被翻译为

*(min++);

您需要的是

(*min)++;

更好的是，更改功能以接受引用类型，使您的生活更轻松。

void function(int array[], int size, int compare, int& min, int& max) {
    for (int i = 0; i < size; i++) {
        if (array[i] < compare) min++;
        if (array[i] > compare) max++;
    }
}

此外，请确保初始化max1和min1。否则，您的代码将使用未初始化变量的值，这将导致未定义行为。

int max1 = 0;
int min1 = 0;
int n, m, array1[SIZE];

Answer 2

min1和max1从未初始化，包含垃圾值。 function然后将其递增，得出的垃圾值略有不同-但仍然是垃圾。

您的程序通过访问未初始化的对象而表现出未定义的行为。

查找大于和小于数组中的数字

2 个答案: