C++：尝试打印指针值时程序崩溃

Question

我正在编写一个程序，它应该允许用户输入数组的大小，输入数组每个索引的值，并使用指针打印出数组的最小值和最大值。程序成功确定数组的最大值和最小值，并使用指针打印出最大值，但在对最小值指针执行完全相同的操作时会崩溃。

代码如下：

int main()
{
    //Variable Declaration
    int arsize  = 0;
    int i       = 0;
    int range   = 0;
    int armin, armax;
    int *ptrmin, *ptrmax;

    //User Prompt for array size, saved to variable arsize
    printf("How many elements should be stored in the array?: ");
    scanf("%d", &arsize);
    fflush(stdin);

    //The array ar is declared according to the user input
    int ar[arsize];

    for (i = 0; i < arsize; i++)
    {
        printf("Enter value for element at index %d:\t", i);
        scanf("%d", &ar[i]);
        fflush(stdin);
    }

    //For loop with if statement to determine biggest value in array 'ar'
    armax = ar[0];
    for (i = 0; i < arsize; i++)
    {
        if (armax < ar[i])
        {
            armax = ar[i];
            ptrmax = &ar[i];
        }
    }

    //For loop with if statement to determine the smallest value in array 'ar'
    armin = ar[0];
    for (i = 0; i < arsize; i++)
    {
        if (armin > ar[i])
        {
            armin = ar[i];
            ptrmin = &ar[i];
        }
    }


    //The Min and Max is printed using pointers, Range is printed regularly
    printf("\nMax:\t%d\n", *ptrmax);
    printf("Min:\t%d\n", *ptrmin);

输出如下：

How many elements should be stored in the array?: 2
Enter value for element at index 0:     50
Enter value for element at index 1:     100

Max:    100

Process returned -1073741819 (0xC0000005)   execution time : 4.438 s

程序成功打印最大值，但不是最小值？

Answer 1

对于像这样的初学者可变长度数组

int ar[arsize];

不是标准的 C++ 功能。而是使用标准容器std::vector<int>。

这个电话

fflush(stdin);

具有未定义的行为。删除它。

在这两个 for 循环中

armax = ar[0];
for (i = 0; i < arsize; i++)
{
    if (armax < ar[i])
    {
        armax = ar[i];
        ptrmax = &ar[i];
    }
}

指针ptrmax 和ptrmin 未初始化。因此，取消引用指针通常会导致未定义的行为。

这个输入就是这种情况

Enter value for element at index 0:     50
Enter value for element at index 1:     100

因为最小元素是未设置指针的数组的初始元素。

您可以通过以下方式重写循环

ptrmax = ar;
for (i = 1; i < arsize; i++)
{
    if ( *ptrmax < ar[i])
    {
        ptrmax = ar + i;
    }
}

变量armax 和armin 是多余的。

还请记住，可以使用标准算法 std::min_element、std::max_element 和 std::minmax_element 来代替循环。