Расчет двух разных средних значений в C++

Question

Мое задание - рассчитать средние боулинг. У меня пять игроков и три игры для каждого игрока. В настоящее время у меня две петли, одна для игрока, а другая для номера игры. Мне нужно показать средние игроки в конце каждого из этих циклов, а команды - в конце этого цикла.

Я исправил свой код и заменил старый код на новый код ниже. Я играл с ним, прежде чем я проверил здесь, чтобы увидеть все комментарии и т. Д., И я решил это к тому времени.

Но спасибо всем!

#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") Плохо только для того, чтобы держать консоль открытой? Вы можете избежать использования system("pause"), используя что-то вроде std::cin.get() или getchar().

Вы также указали playerScore указатель и используете его без * перед тем, как вы на самом деле пытаетесь получить адрес того, на что он указывает (в данном случае ничего - он даже не получил выделение).

Answer 4

Итак, есть несколько проблем здесь:

• Вы никогда не распределяющие любое пространство для вашего массива. Ваш playerScore нуждается в new.
• cin >> playerScore[currentGame] будет записывать только массивные цифры 0, 1 и 2. Эта логика должна как-то совместить currentPlayer и currentGame.
• То же самое с playerAverage += playerScore[currentGame];
• Вам нужно delete[] пространство вы выделить с new после того как вы сделали с вашей playerScore массива.

Answer 5

Вы храните вход в неизвестном месте. Я удивлен, что вы еще не встретили segfault.

double* playerScore; не обязательно объявляет массив, это «указатель на двойной». Вы можете использовать его для создания массива в куче (playerScore = new double[SOME_SIZE];).

Пока вы на самом деле не указали указатель, куда следует указывать, используя любую другую неинициализированную переменную, нет никакой информации о том, что она на самом деле содержит. Разница заключается не в интерпретации байтов, хранящихся там как int, double и т. Д., Это интерпретируется как адрес памяти, и вы пытаетесь записать в это место в памяти.

Поскольку вы знаете, сколько значений вам нужно хранить, я бы просто объявить статический массив double playerScore[SOME_SIZE]