为什么 Dev C++在运行整个代码来查找 10 个元素之间的最大和最小数字之前停止工作吗？

Question

这是我编写的用于查找 10 个元素之间的最大和最小数字的代码：

#include <iostream>
using namespace std;

#define SIZE 10
void acceptNumbers (int Numbers[SIZE]);
int High(int Numbers[SIZE]);
int Low(int Numbers[SIZE]);

void acceptNumbers(int Numbers[SIZE])
{int i, *max, *min;
    for (i=0; i<SIZE; i++)
    cin>>Numbers[i];

    max = Numbers;
    min = Numbers;
}

int High(int Numbers[SIZE])
{int i, *max;
    for (i = 0; i < SIZE; i++)
    {
        if (*(Numbers+i) > *max)
            max = (Numbers+i);
    }
    cout<<*max;
}

int Low(int Numbers[SIZE])
{int i, *min;
    for (i = 0; i < SIZE; i++)
    {
        if (*(Numbers+i) < *min)
        min = (Numbers+i);
    }
    cout<<*min;
}

int main()
{
    int arr[SIZE];

    cout<<"Enter Elements: \n";
    acceptNumbers(arr);

    cout<<"Highest Number: ";
    High(arr);
    cout<<endl;

    cout<<"Lowest Number: ";
    Low(arr);
    cout<<endl;
    }

在我的 PC 上，它不会运行整个程序。当它到达查找最大和最小数字的代码时它就停止了

这是我正在做的作业，我不能使用任何快捷方式和诸如此类的东西，因为给出了需要使用的代码行。因此，我尝试在另一台电脑上运行此代码，效果很好：

我也尝试在在线 c++ 编译器上运行它，但它仍然不起作用。我在网上找不到解决方案，所以这是我最后的手段，请帮助我

Answer 1

您的代码表现出未定义的行为。

在每个函数中，您将 min 和 max 声明为局部变量。因此，当您在 acceptNumbers() 中初始化 min/max 时，它们仅在该函数中初始化。您根本没有初始化 High()/Low() 中的 min/max 变量。当这些函数尝试取消引用那些未初始化的变量时，您的代码就会崩溃。

此外，您的 High()/Low() 函数具有非 void 返回类型，但实际上并不是 return'ing任何值。

试试这个：

#include <iostream>
using namespace std;

#define SIZE 10
void acceptNumbers (int Numbers[SIZE]);
int High(int Numbers[SIZE]);
int Low(int Numbers[SIZE]);

void acceptNumbers(int Numbers[SIZE])
{
    for (int i=0; i<SIZE; i++)
        cin>>Numbers[i];
}

int High(int Numbers[SIZE])
{
    int *max = Numbers;
    for (int i = 1; i < SIZE; i++)
    {
        if (*(Numbers+i) > *max)
            max = (Numbers+i);
    }
    return *max;
}

int Low(int Numbers[SIZE])
{
    int *min = Numbers;
    for (int i = 1; i < SIZE; i++)
    {
        if (*(Numbers+i) < *min)
            min = (Numbers+i);
    }
    return *min;
}

int main()
{
    int arr[SIZE];

    cout<<"Enter Elements: \n";
    acceptNumbers(arr);

    cout<<"Highest Number: ";
    cout<<High(arr);
    cout<<endl;

    cout<<"Lowest Number: ";
    cout<<Low(arr);
    cout<<endl;
}

Answer 2

函数 Low() 和 High() 仅声明 int* max、min 但从未初始化。尽管他们持有无效地址。

if (*(Numbers + i) > *max)

此函数会导致分段错误，因为您的程序尝试取消引用无效的内存地址。

如果您想存储int *min, *max，请将它们设置为全局或在 main() 中声明它们。