從陣列中刪除最低的輸入？

Question

我必須找到給定的最低輸入，然後平均減去最低分。我遇到了一些麻煩，我的averageScore函數發現陣列中得分最低。我得到非常奇怪的數字作爲我的輸出。任何建議如何調整這將不勝感激。提前致謝。

#include <iostream> 
#include <cstdlib> 
using namespace std; 

//function prototypes 
double* allocate(int&); 
double averageScore(int&); 

int main() 
{ 
    double* testArray; 
     int numOfScores; 
    double average; 

    testArray = allocate(numOfScores); 
    average = averageScore(numOfScores); 

    //delete memory created 
    delete[] testArray; 

    return 0; 
} 

//function to collect user info, dynamically allocate 
double* allocate(int &numOfScores) 
{ 
    double* testArray; 

    //prompt user for scores 
    cout << "How many test scores would\n"; 
    cout << "you like to process: "; 

    //user input validation 
    if(!(cin >> numOfScores)) 
    { 
     cout << "Invalid input!\n"; 
     cout << "Program termination, please\n"; 
     cout << "restart the program." << endl; 
     exit(0); 
    } 
    else if(numOfScores < 0) 
    { 
     cout << "Invalid input!\n"; 
     cout << "Program termination, please\n"; 
     cout << "restart the program." << endl; 
     exit(0); 
    } 

    //dynammically allocate an arrray to hold the scores 
    testArray = new double[numOfScores]; 

    //get the scores from user 
    for (int count = 0; count < numOfScores; count++) 
    { 
     cout << "Enter Score: "; 

     //user input validation 
     if(!(cin >> testArray[count])) 
     { 
      cout << "Invalid input!\n"; 
      cout << "Program termination, please\n"; 
      cout << "restart the program." << endl; 
      exit(0); 
     } 
     else if(testArray[count] < 0.0) 
     { 
      cout << "Invalid input!\n"; 
      cout << "Program termination, please\n"; 
      cout << "restart the program." << endl; 
      exit(0); 
     } 


    } 

    return testArray; 
} 

//function to calculate the average score 
double averageScore(int &numOfScores) 
{ 
    double* testArray; 

    double total, 
      average, 
      scores[0], 
      lowest; 

    lowest = scores[0]; 

    //calculate total scores entered 
    for(int count = 0; count < numOfScores; count++) 
    { 
     total += testArray[count]; 

     //find lowest score entered 
     for(int count = 1; count < numOfScores; count++) 
     { 
      if (testArray[numOfScores] < lowest) 
       lowest = scores[numOfScores]; 
     } 
    } 

    //average the total amount of scores drop the lowest 
    average = (total - lowest)/numOfScores; 

    cout << "The average test score is: " << average << endl; 
    cout << "Lowest is: " << lowest << endl; 

    return average; 
} 

Answer 1

有很多概率的與您的averageScore函數一起使用，但現在我將介紹最基本的函數。

首先，你應該傳遞一些數據。現在你正在使用testArray我什至不知道它在哪裏分配。我很驚訝你在運行時沒有出現分段錯誤。

但它也沒有初始化。在C++中，當你聲明一個指針時，它指向的變量就有一個值。它有一個垃圾值，如果你用這個垃圾值執行算術運算，那麼你的輸出也是垃圾。

你必須讓你的averageScore函數可用的分數列表，最好是將它們作爲參數傳入。

的平均函數的開頭如下所示：

double averageScore(int &numOfScores) 
{ 
    double* testArray; 
    ... 

相反，它應該是這樣的

double averageScore(double*testArray, int numOfScores) 
{ 
    ... 

當您使用&numOfScores而不是numOfScores，這意味着，如果你改變numOfScores在你的averageScore函數中，它的功能也會改變，你不應該那樣做。

現在，在double* testArray;行中，您聲明瞭一個全新的指針，名爲「testArray」，並且其中沒有任何有意義的數據，儘管它可能充滿垃圾。可能還有其他雙指針變量，在您的代碼中名爲「testArray」，但它們都不在您的averageScore函數的範圍內。如果您在方法調用中通過testArray，則可以使用它。例如：double someNumber = testArray[i]。

請記住，您的數組也被引用傳遞。如果你寧願按值傳遞它，你可以嘗試

`double averageScore(double testArray[], int numOfScores)` 

但不要引用我一個

一旦你這樣做，你的代碼將仍然有一些問題，但輸出應該足夠有意義，以至於你希望能夠自己解決這些問題。

Answer 2

我想你想改變這一行：

if(testArray[numOfScores] < lowest) 

這樣：

if(testArray[count] < lowest) 

此外，作爲@jzworkman指出的，平均的分母應該是（numScores - 1）因爲你正在消除分子的最低分數。 （如果適用，您可能希望測試只有一個分數的邊緣情況，一旦您消除最低分數，則不會得到平均值。）

Answer 3

幾個問題。你不應該嵌套這兩個for循環（而只是使用if語句檢查值是否低於最低值）。

由於這是家庭作業，我會給你的步驟，然後你可以通過修復你的代碼

1. 環路和計算總，同時
2. 找到最低分數計算平均爲（總最低）/（numScores -1）
3. 返回平均

Answer 4

std::vector<double> scores = {1.2,6.5,3.0,8.3,4.8,6,7.7}; 

// drop lowest score 
scores.erase(min_element(begin(scores),end(scores))); 

double average = accumulate(begin(scores),end(scores),0.0)/scores.size();