这个数组的循环有效，但我仍然得到一个编译器错误？调试错误运行时检查失败 #2 - S

Question

大家好，我刚开始使用数组。

请有人对这个话题有所了解。这是我创造的，我认为它是有道理的，但我一定错过了一些东西。 我收到调试错误运行时检查失败 #2 - S

错误类型如下：Debug Error Run-Time Check Failure #2 - S

#include<iostream>
#include<iomanip>
#include <climits>


//Prototypes:
int lowestAmount(int[]);
int highestAmount(int[]);



using namespace std;

int main() {

    const int AMOUNT = 9;

    int values[AMOUNT];
    int lowest, highest;

    cout << "Please Insert 10 Numbers of your Choice: ";
    cin >> values[0] >> values[1] >> values[2] >> values[3] >>
        values[4] >> values[5] >> values[6] >> values[7] >>
        values[8] >> values[9];
    cout << endl;
    lowest = lowestAmount(values);
    highest = highestAmount(values);

    cout << "/tThe Lowest out of all of them is: " << lowest <<
        endl;
    cout << "/tThe Highest out of all of them is: " << highest << endl ;


    return 0;
}



int lowestAmount(int val[]) {

    int lowest = INT_MAX;
    int count = 10;
    for (int i = 0; i < count; i++) {
        if (val[i] < lowest)
            lowest = val[i];
    }

    return lowest;
}

int highestAmount(int val[]) {
    int highest = INT_MIN;
    int count = 10;
    for (int i = 0; i < count; i++) {
        if (val[i] > highest)
            highest = val[i];
    }
    return highest;
}

代码有点工作，但不是一直正确，我不知道我做错了什么？我虽然我的逻辑是错误的？？？有人能解释一下吗？？？

Answer 1

数组索引从零开始，这意味着最后一个索引是元素的数量减一。您希望用户输入 10 个数字，并且您确实读取了 vales[0] 到 values[9]，但您的数组没有空间。

const int AMOUNT = 9;

int values[AMOUNT];

这意味着你声明了一个足够大的数组来容纳 9 个元素，而不是 10 个。AMOUNT 需要为 10 个。

C++ 不会自动进行边界检查。这意味着如果你在数组之外写，你会得到不可预知的行为，你甚至不能依赖程序总是崩溃。

在你的更高/更低函数中还有一个错误，即使它现在仍然忽略最后一个元素。将数组固定为 10 个元素后，您需要循环直到 i < 10 之后。

同样，在未来，您将通过发布具体什么不起作用来获得更好的帮助，这意味着您应该显示编译器/调试器错误，而不仅仅是您的代码。