这段代码有问题吗？我无法让它工作。请帮帮我[关闭]

Question

目的是对用户输入的分数进行排序、显示和平均。 我还没有弄清楚。 如果你已经解决了这个问题，请帮忙。 到目前为止，我已经尝试过这段代码，但它不起作用。

#include<iostream>
using namespace std;

void sortArr(int [], int);
void getAvg(const int [], int);

int main()
{
    int *scorePtr = nullptr;
    int size;

    cout << "Enter the number of test scores: ";
    cin >> size;
    while(size < 0)
    {
        cout << "Invalid entry. Enter the number of scores: ";
        cin >> size;
    }
    if(size == 0)
    {
        cout << "No scores. Terminating program...";
        return 0;
    }

    scorePtr = new int[size];

    cout << "Enter the scores earned on each test: ";
    for (int index = 0; index < size; index++)
    {
        cin >> scorePtr[index];
        while(scorePtr[index] < 0)
        {
            cout << "Entry not valid. Enter score #" << (index + 1) << ": ";
            cin >> scorePtr[index];
        }
    }

    sortArr(scorePtr, size);
    getAvg(scorePtr, size);

    return 0;
}

void sortArr(int *arr[], int SIZE)
{
    int startScan, minIndex;
    int *minElem;

    for(startScan = 0; startScan < (SIZE - 1); startScan++)
    {
        minIndex = startScan;
        minElem = arr[startScan];
        for(int index = startScan + 1; index < SIZE; index++)
        {
            if(*(arr[index]) < *minElem)
            {
                minElem = arr[index];
                minIndex = index;
            }
        }
        arr[minIndex] = arr[startScan];
        arr[startScan] = minElem;
    }
    cout << "Here are the scores you entered, sorted in ascending order: " << endl;
    for(int count = 0; count < SIZE; count++)
        cout << *(arr[count]) << " ";
    cout << endl;
}

void getAvg(const int arr[], int Size)
{
    double total = 0.0;
    double avg = 0.0;
    for (int index = 0; index < Size; index++)
    {
        total += arr[index];
    }
    avg = (total/Size);
    cout << "Your average score is: " << avg << endl;
}

我从这段代码中得到的只是编译器错误。 有没有人有什么建议？请和谢谢。

Answer 1

正如其他人所指出的，您对sortArr() 的声明与定义不符。

宣言：

void sortArr(int [], int);

定义：

void sortArr(int *arr[], int SIZE) {...}

如您所见，定义说sortArr() 接受ints 和int 数组的参数。但是，您的定义说sortArr() 采用int*s 和int 的数组的参数。编译器得出结论，它们一定是不同的函数。

当编译器尝试编译时，它与您的声明一致，但找不到定义。你可以按照其他人所做的来修复它，我也会在这里列出：

首先，将sortArr()的声明改为

void sortArr(int*, int);

int[] 可以转换为int*，所以这就是我们需要的。

接下来，转到sortArr() 的定义，并更改它。 （cmets解释原因）

void sortArr(int* arr, int SIZE) //we only need a pointer, as it is basically the same thing as an array
{
    int startScan, minIndex;
    int minElem; //if minElem was a pointer, it would get overwritten every time by arr, so use a regular int

    for (startScan = 0; startScan < (SIZE - 1); startScan++)
    {
        minIndex = startScan;
        minElem = arr[startScan];
        for (int index = startScan + 1; index < SIZE; index++)
        {
            if (arr[index]/*No need for dereferencer here, because indexing already dereferences.*/ < minElem)
            {
                minElem = arr[index];
                minIndex = index;
            }
        }
        arr[minIndex] = arr[startScan];
        arr[startScan] = minElem;
    }
    cout << "Here are the scores you entered, sorted in ascending order: " << endl;
    for (int count = 0; count < SIZE; count++)
        cout << arr[count] << " "; //Again, no dereferencing here because we are using the [] operator
    cout << endl;
}

我已经针对多个案例测试了此代码，并且它有效。希望这会有所帮助！

Answer 2

您将 sortArr 定义为将一个指向 int 的指针数组作为其参数，但您向其传递了一个实际的 int 数组。听起来后者是您真正想要的，因此您需要重写sortArr 才能在int 的数组上正常工作，首先将定义更改为

void sortArr(int arr[], int SIZE) {
...
}

没有*。这也意味着该函数中的大部分 * 取消引用将被删除。

Answer 3

要添加 Nate 的答案，请将指向数组的指针传递给 sortArr 。编译器将数组“转换”为指针。

void sortArr(int* arr, int SIZE) {
...
}

还有更精细的方法来检查输入错误。随心所欲地使用

while(!(cin >> size) || size < 0 || size == 0)
{
// Error message
cin.clear();
// A huge constant
cin.ignore(999,'\n');
}