C++で2つの異なる平均を計算する

Question

私の割り当てはボウリング平均を計算することです。私は5人の選手と各選手に3人の選手がいます。私は現在、2つのループを実行しています.1つはプレイヤー用、もう1つはゲーム番号用です。私は、各ループの終わりにプレーヤーの平均を表示する必要があり、チームはそのループの最後に平均します。

私のコードを修正し、私の古いコードを以下の新しいコードに置き換えました。私はここですべての人のコメントなどを見るためにここをチェックする前にそれを試していましたが、それまでに解決しました。

皆さんありがとうございます！

#include <iostream> 

using namespace std; 

int main() 
{ 
//DECLARATIONS 
const int PLAYER_NUMBER = 5; //There are five players total 
const int GAME_NUMBER = 3; //There are three games total 
const int MIN = 0; //Min number 
const int MAX = 300; //Max number 
double* playerScore; //The players' score of current game 
double playerAverage = 0; //The current players' average 
double teamAverage = 0; //The teams' average 

//INPUT 

for (int currentPlayer = 0; currentPlayer < PLAYER_NUMBER; currentPlayer++) 
{//Set the current player number 

    for (int currentGame = 0; currentGame < GAME_NUMBER; currentGame++) 
    {//Set the current game number 
      //Get scores 

      cout << "For Player " << (currentPlayer + 1) << ", enter score for game " << (currentGame + 1) << ": "; 
      cin >> playerScore[currentGame]; 


      if(playerScore[currentGame] < MIN || playerScore[currentGame] > MAX) 
      {//Check range 
        cout << "The score must be between 0 and 300!\n"; 
        currentGame--; //If there is an error, subtract the game number by one 
      }//End If statement 

      playerAverage += playerScore[currentGame]; 

      if(currentGame == 2) 
      {//Current player average 
       cout << endl << "The average for player " << (currentPlayer + 1) << " is: " << (playerAverage/3) << endl << endl; 
       teamAverage += playerAverage; 
       playerAverage = 0; 
      }//End If statement 

    }//End game for-statement 

}//End player for-statement 

    cout << endl << "The average for the team is: " << (teamAverage/15) << endl << endl; 

//ENDING  
system("Pause"); 
return 0;  
}//Close main 

しかし、まだそこに誰のために、単に端末が開いたままにし、「SYS（」PAUSE「）を使用する必要がない持っている方法はあり;」？私はそれを使用して本当に嫌いです。

Answer 1

double* playerScoreと宣言していますが、ストレージの割り当て先はわかりません。おそらくあなたは何かを上書きしているでしょう。

Answer 2

int main() 
{ 
/* ... */ 
double* playerScore; //The players' score of current game 

for (int currentPlayer = 0; currentPlayer < PLAYER_NUMBER; currentPlayer++) { 
    for (int currentGame = 0; currentGame < GAME_NUMBER; currentGame++) { 
      cout << "For Player " << (currentPlayer + 1) << ", enter score for game " << (currentGame + 1) << ": "; 
      cin >> playerScore[currentGame]; 

playerScore[currentGame]に書き込むと、割り当てられていないメモリに書き込んでいます。あなたが何を書いているのか分かりませんが、書くのはあなたのものではありません。

playerScoreのメモリを割り当てる必要があります。メモリを割り当てる最適な方法を決定する必要がありますが、次のようなものがあります。

double playerScore[PLAYER_NUMBER]; 

は良い出発点になるかもしれません。

ちなみに、これはおそらくあなたのコンパイラが警告するものです。 （-Wall -Wextraはgccへの私のお気に入りのフラグです - あなたのコンパイラは別のものが必要かもしれませんが）これについて警告することができます。 をコンパイラの警告ごとに修正する必要はありませんが、無視するだけではありません。何千年ものプログラミング経験が現代のコンパイラで掘り下げられています。

Answer 3

あなたのループは、最後のゲーム番号またはプレーヤー番号を確認しません。

system("pause")コンソールを開いているだけで悪くないのですか？ std::cin.get()またはgetchar()のようなものを使用して、system("pause")の使用を避けることができます。

あなたが実際にそれが（この場合は何もない - それはさえ割り当てられませんでした）を指しているものは何でものアドレスを取得しようとしているので、また、それ以前playerScoreポインタと*せずにそれを使用して作られました。

Answer 4

だからここにいくつかの問題があります。

• あなたはアレイの任意の領域を割り当てることはありませんが。 playerScoreにはnewが必要です。
• cin >> playerScore[currentGame]は、配列の添え字0,1,2を書き込みます。このロジックは、currentPlayerとcurrentGameを何とか組み合わせる必要があります。あなたがdelete[]に、あなたのplayerScoreアレイに終わった後、あなたはnewに割り当てるスペースが必要になりますplayerAverage += playerScore[currentGame];
• と同じ
• 。

Answer 5

入力を不明な場所に保存しています。私はあなたがまだセグメンテーションに遭遇していないのに驚いています。

double* playerScore;は、必ずしも配列を宣言しているわけではありません。「ダブルへのポインタ」です。これを使用してヒープ上に配列を作成することができます（playerScore = new double[SOME_SIZE];）。

ポインタを使用してポインタを使用する場所に、他の初期化されていない変数を使用するように指示するまで、実際に何が含まれているかはわかりません。違いは、そこに格納されているバイトをint、doubleなどと解釈するのではなく、メモリアドレスとして解釈され、メモリ内のその場所に書き込もうとします。

あなたはCINが正常に実行された場合、私はちょうど静的配列double playerScore[SOME_SIZE]