Just one question now!

[katielynnsdad]

Can I turn all of this into one function?

#include <iostream>
#include <vector>
#include <algorithm>
#include <ctime>
#include <windows.h>
#include <cstdlib>

using namespace std;

      vector<int> roll;
      int myRoll[5];
      const int NUMDICE = 5;
      const int NUMSIDES = 6;
 
int getScore(int score)
{    
         sort(roll.begin(), roll.end());
         for(int i =0; i < 5; i++)
         {
                 cout << roll[i];
                 score = score + roll[i];                                 
         }
         
         return score;
}     
int displayRoll()
{
    int myNewRoll[5];
    for(int i =0; i < 5; i++)
    {
            myNewRoll[i] = roll[i];
            cout << myNewRoll[i];
    }
    return myNewRoll[5];
}

int main()
{
        
    
    char keepIn;
    char keep;
    
    srand((unsigned)time(NULL));
    for(int i = 0; i < NUMDICE; i++)
         {
                 myRoll[i] = (rand() %NUMSIDES) +1;
                 cout << myRoll[i];                                  
         }
         cout << endl << endl;
         cout << "keep dice 1 ?" << endl;
         cin >> keepIn;
         keep = toupper(keepIn);
                  
         if(keep =='Y')
         {                 
                 roll.push_back(myRoll[0]);                
         }
         else if(keep == 'N')
         {
              myRoll[0] = 0;                            
         }
         cout << endl << endl;
         cout << "keep dice 2 ?" << endl;
         cin >> keepIn;
         keep = toupper(keepIn);
                  
         if(keep =='Y')
         {
              roll.push_back(myRoll[1]);                             
         }
         else if(keep == 'N')
         {
              myRoll[1] = 0;
         }
         cout << endl << endl;         
         cout << "keep dice 3 ?" << endl;
         cin >> keepIn;
         keep = toupper(keepIn);
                  
         if(keep =='Y')
         {                 
              roll.push_back(myRoll[2]);                                 
                            
         }
         else if(keep == 'N')
         {
              myRoll[2] = 0;              
         }
         cout << endl << endl;
         cout << "keep dice 4 ?" << endl;
         cin >> keepIn;
         keep = toupper(keepIn);
                  
         if(keep == 'Y')
         {
                 roll.push_back(myRoll[3]);                                    
                                                  
         }
         else if(keep == 'N')
         {
              myRoll[3] = 0;              
         }
         cout << endl << endl;       
         cout << "keep dice 5 ?" << endl;
         cin >> keepIn;
         keep = toupper(keepIn);
                          
         if(keep == 'Y')
         {         
                roll.push_back(myRoll[4]);                                   
                
         }
         else if(keep == 'N')
         {
              myRoll[4] = 0;              
         }
         cout << endl << endl;
         
         
         
         sort(roll.begin(), roll.end());
                                         
         cout << endl << endl;
         
         for(int i = 0; i < 5; i++)
         {
                 cout << myRoll[i];
         }
         
         cout << endl << endl;
         
         int myRoll2[5];
         int newDice;
         newDice = 5 - roll.size();
         for(int i = 0; i < newDice; i++)
         {
                myRoll2[i] = (rand() %NUMSIDES) +1;
                cout << myRoll2[i];
                roll.push_back(myRoll2[i]);                                               
         }
         
         
         cout << endl << endl;
         
         int score;
         cout << endl << endl;      
         cout << endl << getScore(score) - newDice << endl;
         
         cout << endl << endl;
                                         
                             
         
         system("pause");
         return 0;
}

[Faldrax]

Hi, looking back at your previous posts it look like you are trying to get a quick prog to roll 5 dice, then have 3 rounds of accept or re-roll each die to try an maximise the score.

I am relativly new to C++, but have been programming (mainly Delphi) for many years now, and one 'rule' I try and stick to is to keep things as simple as possible - it seems to work for C++ too!
(Another 'rule' is that rules are guidelines not absolutes:-)

I would make a few mods to the code you have in order to put it into a function (or, to be more exact, a bunch of functions).

Where you have the same (or very similar) code repeated it is often the sign that it would be a good chunk to split out as a function - in you code, for example, for the 5 dice you have virtualy the same code to keep or discard the roll - if you pull this out the code become much shorter, and as a result easier to follow.

I would also stick to just the single int array - there is no need for the vector or second array, so why add complexity?
You can also use the NUMDICE constant for declaring the array, loop boundaries, etc, which both helps show what is intended and make changing the code easier.

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

const int NUMDICE = 5;
const int NUMSIDES = 6;
int myRoll[NUMDICE];
 
int getScore(void)
{    
    int score=0;
    for(int i =0; i < NUMDICE; i++)
    {
          score = score + myRoll[i];                                 
    }
    return score;
} 

void displayRoll(void)
{
    for(int i =0; i < NUMDICE; i++)
    {
          cout << myRoll[i];
    }
    cout << endl;
    return;
}

bool keep(int i)
{
    char keepin;
    cout << "\nKeep dice " << i+1 << " ?" << endl;
    cin >> keepin;
    return (toupper(keepin)=='Y');
}

void rollTheDice()
{
    srand((unsigned)time(NULL));
    for(int i = 0; i < NUMDICE; i++)
    {
        myRoll[i] = (rand() %NUMSIDES) +1;
    }
    for (int j=0; j<3; j++)
    {
        displayRoll();
        cout << "Score: " << getScore() << endl;
        for(int i = 0; i < NUMDICE; i++)
        {
            if (!keep(i)) 
            {
                myRoll[i] = (rand() %NUMSIDES) +1;
            }
        }
    }
    displayRoll();
    cout << "Score: " << getScore() << endl;
}

int main()
{
    rollTheDice();
    system("pause");
    return 0;
}

Hope that helps -

[katielynnsdad]

Thank you very much! You helped tremendously. I can see where the fuctions make it alot easier coding.
Thanks Again!