在 C++ 中计算两个不同的平均值

Question

我的任务是计算保龄球平均值。我有五名球员，每个球员三场比赛。我目前正在运行两个循环，一个用于播放器，另一个用于游戏编号。我需要显示每个循环结束时球员的平均水平，以及该循环结束时球队的平均水平。

我修复了我的代码，并用下面的新代码替换了我的旧代码。在我在这里查看每个人的 cmets 等之前，我一直在玩它，到那时我已经解决了。

但是谢谢大家！

#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

您的循环不会检查最后一个游戏号码或玩家号码。

system("pause") 只是因为打开控制台不好吗？您可以通过使用std::cin.get() 或getchar() 之类的东西来避免使用system("pause")。

您还使playerScore 成为一个指针，并在它之前没有* 使用它，因此您实际上是在尝试获取它所指向的地址（在这种情况下什么都没有——它甚至没有被分配)。

Answer 2

您将输入存储在未知位置。我很惊讶你还没有遇到段错误。

double* playerScore; 不一定要声明一个数组，它是一个“指向双精度的指针”。您可以使用它在堆上创建一个数组 (playerScore = new double[SOME_SIZE];)。

直到你真正告诉指针指向哪里使用它就像使用任何其他未初始化的变量一样，没有告诉它实际包含什么。不同之处在于不是将存储在那里的字节解释为 int、double 等，而是将其解释为内存地址，并且您尝试写入内存中的该位置。

既然你知道你需要存储多少个值，我就声明一个静态数组double playerScore[SOME_SIZE]

Answer 3

所以这里有几个问题：

• 您永远不会为您的阵列分配任何空间。你的playerScore 需要一个new 在某个地方。
• cin >> playerScore[currentGame] 只会写入数组索引 0、1 和 2。这个逻辑需要以某种方式结合 currentPlayer 和 currentGame。
• 与playerAverage += playerScore[currentGame];相同
• 在完成 playerScore 数组后，您需要使用 new 分配的空间 delete[]。

Answer 4

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 5

您声明了double* playerScore，但我看不到您在哪里分配存储空间。也许您正在覆盖某些内容。