最大输出不正确

Question

试图计算数组中随机数的最大值和最小值，我不确定我在这里做错了什么。我一直在试图弄清楚，但我束手无策。也许你们中的一个可以帮忙？

#include <iostream>
#include <stdlib.h>   
#include <ctime>

using namespace std;

int main() {
    int *sora;
    int num;
    int min;
    int max;

    cout<< "Enter the size :  " ;
    cin >> num;
    cout<< endl;
    sora = new int [num];   

    max= sora[0];
    min= sora[0];

    srand((unsigned)time(0));

    for(int x=0;x<num; x++) {
        sora[x]=rand()%100;

        if(min > sora[x]) {
            min=sora[x];        
        }
        else if (max < sora[x]) {
            max=sora[x];
        }

        cout<< sora[x] << endl;     
    }

    cout<< "Maximum value is: " << max << endl << "Minimum value is: " << min;
}

Answer 1

if(min > sora[x]) {
    min=sora[x];
}
else if (max < sora[x])
{
    max=sora[x];
}

有问题。仅当满足第一个 if 时，才输入第二个 else if 语句。但是第二个if 必须始终执行。请把上面的块改成如下

if(min > sora[x]) {
    min = sora[x];
}
if (max < sora[x]) {
    max = sora[x];
}

而且您还使用未初始化的区域进行比较。如果您只使用正数

max = 0;
min = UINT_MAX;

如果你也使用负数，那么

max = INT_MIN;
min = INT_MAX;

您必须为这些宏添加<limits.h>。

Answer 2

当您执行max= sora[0]; 和min= sora[0]; 时，您将最小值和最大值初始化为一个未定义的值

另外，你有一个else，你不一定想要一个。第一个值将是最小值 和 最大值。 您可以从设置max=0 和min=99 开始，这应该可以解决该特定问题。

Answer 3

sora = new int [num];   

max= sora[0];
min= sora[0];

这一行用垃圾初始化一个 int 数组，并将 max 和 min 设置为垃圾值。

你有两个选择：

1. 在开始时将 max 初始化为可能的最小值 (INT_MIN)，并将 min 初始化为可能的最大值 (INT_MAX)。

2. 首先创建一个随机值并将其分配到 sora[0] 并将 max 和 min 初始化为该值。之后初始化 [1..num]

Answer 4

您正在读取一个未初始化的内存位置：

sora = new int [num];   

max= sora[0];
min= sora[0];

使用new 创建数组不会初始化其内容。

这会导致所谓的未定义行为，这意味着几乎任何事情都可能发生。在这种情况下，它可能很普通：一些任意值（很可能在 rand()%100 的 0-99 范围之外）存储在 min 和 max 中，经常导致不正确的结果。

相反，做

// must #include <limits>

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

甚至只是

int max = 0;
int min = 99;

因为您的范围是预定义的。