配列から最低の入力を削除しますか？

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

PROBがたくさんありますあなたのaverageScore機能では、私は今のところ最も基本的なものをカバーします。

まず、何らかのデータを渡す必要があります。今すぐ使用していますtestArray私はどこに割り当てられているかわかりません。私はあなたがこれを実行する際にセグメンテーションフォールトを取得していないことに驚いています。

ただし、初期化されていません。 C++では、ポインタを宣言すると、ポインタが指す変数に値が設定されます。それはガベージ値を持ち、そのガベージ値を使って算術演算を実行すると、出力もガベージになります。

averageScoreの機能で利用可能なスコアのリストを、好ましくはパラメーターとして渡すことによって、スコアリストを作成する必要があります。

あなたの平均化関数の先頭には、以下のようになります。つまり、あなたが代わりにnumOfScoresの&numOfScoresを使用するときに代わりに、それはこの

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

のようになります

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

つまり、あなたはnumOfScoresを変更した場合averageScore関数では、main関数でも変更されるよりも、そうするべきではありません。

今、double* testArray;の行では、 "testArray"という名前の新しいポインタが宣言されています。意味のあるデータはありませんが、ガベージがいっぱいです。あなたのコードに "testArray"という名前の他のダブルポインタ変数があるかもしれませんが、それらのどれもaverageScore関数のスコープにはありません。 testArrayを渡すと、メソッド呼び出しでそれを使用できるようになります。たとえば、double someNumber = testArray[i]です。

あなたの配列も参照渡しされていることに注意してください。あなたではなく値によって、それを通過する場合は、

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

を試すことができますが、あなたはそれをやった後は、あなたのコードは、まだいくつかの問題を持っていますが、出力されます、1つの

で私を引用しないでくださいあなた自身の上でそれらをうまくやっていくことができるほど意味があるはずです。

Answer 2

：これまで

if(testArray[numOfScores] < lowest) 

を：

if(testArray[count] < lowest) 

@jzworkmanが指摘するようにまた、平均化のための分母は（numScoresする必要があります - 1）分子から最低スコアを取り除くからです。 （該当する場合は、スコアが1つしかないエッジケースをテストしたい場合は、スコアを最低にすると平均化されません）

Answer 3

カップルの問題。ネストされたループのためにこれらの2つを持つべきではありません（代わりに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();