a few questions

i am creating a game of Rock, paper, Scissors,

Code:

---

#include <iostream>
#include <string>

using namespace std;

string a;
int b;
int main()
{
    cout << "Welcome to Rock, Paper, Scissors\n";
    cout << "To play, Simply type R,P or S\n";
    cout << "\n";
    system("PAUSE");
    system("CLS");
 //Game Begins
 
  cout << "Your turn, R,P or S?\n";
  cin >> a;
 
  if (a == "R")
      {cout << "You chose Rock..\n";}
  else if (a == "P")
      {cout << "You chose Paper..\n";}
  else if (a == "S")
      {cout << "You chose Scissors\n";}
  else
      {cout << "Please enter R,P or C only\n";}
  system("PAUSE");
 
     
   
   
   
}

---

First off, How can i get the code to Start again after "Please enter R,P or C only" so it dosent have to restart?,


And, How can i make a random number generator, that randomly picks one number, from 3, so 1,2,or 3,?

For the program to repeat itself, you should put the whole thing inside of a while loop which runs until a variable (I prefer using the word "end" as the variable name) changes to 1, like so:

Code:

---

int main ()
{
    int end;
    int everything_else;
    while ( end == 0 )
    {
          blah blah blah
          if ( a == "R" || a == "P" || a == "S" )
              end = 1;
    }
    return 0;
}

---

This will have the program continually loop until the input is either R, P, or S. Oh, and its better to have the input looked for be lower case, just because nobody's going to want to hold shift and most will try the program without putting it in caps.

For the random number, your going to need something like srand() (http://www.cplusplus.com/reference/c...cstdlib/srand/), though thats not completely random (they refer to it as pseudo-random the whole time), and actually tends towards the lower part of the numbers your trying to get as output and has predictable output.

Hope this helped! :P

Quote:

---

Originally Posted by momrocker

For the program to repeat itself, you should put the whole thing inside of a while loop which runs until a variable (I prefer using the word "end" as the variable name) changes to 1, like so:

Code:

---

int main ()
{
    int end;
    int everything_else;
    while ( end == 0 )
    {
          blah blah blah
          if ( a == "R" || a == "P" || a == "S" )
              end = 1;
    }
    return 0;
}

---

This will have the program continually loop until the input is either R, P, or S. Oh, and its better to have the input looked for be lower case, just because nobody's going to want to hold shift and most will try the program without putting it in caps.

---

You're right, but stylistically, I'd use a bool. Don't forget to initialize it, and watch the and/or logic there.

Code:

---

int main ()
{
    bool end = false;
    int everything_else;
    while (!end )
    {
          blah blah blah
          if ( a != "R" && a != "P" && a != "S" )
              end = true;
    }
    return 0;
}

---

Quote:

---

Originally Posted by GCDEF

You're right, but stylistically, I'd use a bool. Don't forget to initialize it, and watch the and/or logic there. ...

---

I nomally don't use a variable in the while condition but use an infinite loop and break the loop when the error condition occurs:

Code:

---

int main()
{

    while (true) 
    {
        ...
        if (input == 'q')  // break the loop when the user types q for quit.
              break;
    }
    return 0;
}

---

Though the above seems to be more dangerous it actually is less error-prone cause the break would work immediately while the check on 'end' variable works at begin of next loop cycle.

It's undisputable that rand() isn't well-suited for cryptographic applications, but this didn't let me sleep well:

Quote:

---

Originally Posted by momrocker

[...] and actually tends towards the lower part of the numbers your trying to get as output [...].

---

I don't think that it actually tends anywhere, as illustrated by the attatched image, sampled from the VC++ 2010 implementation of rand().