[xcloud]

Hi! I'm having trouble w/ writing this guessing game program I am writing on c++, these the instructions:

# Let diff = the absolute value of (guess - theNumber). If diff is 0, then guess is correct and the program outputs a message indicating that the user guessed the correct number (and this part is unchanged from the original code). But if diff is not 0, then the program outputs the message as follows.

1. If diff is greater than or equal to 50, the program outputs the message indicating that the guess is either "very high" or "very low" depending on if the guess is higher/lower than theNumber.

2. If diff is greater than or equal to 30 (and less than 50), the program outputs the message indicating that the guess is either "high" or "low" depending on if the guess is higher/lower than theNumber.

3. If diff is greater than or equal to 15 (and less than 30), the program outputs the message indicating that the guess is either "moderately high" or "moderately low" depending on if the guess is higher/lower than theNumber.

4. If diff is greater than 0 (and less than 15), the program outputs the message indicating that the guess is either "somewhat high" or "somewhat low" depending on if the guess is higher/lower than theNumber.

# Give the user at most five tries to guess the number.

This is the neutral/ functioning code:


----------------------------------------------------

#include <iostream>
#include <cstdlib>  // to use a random number generator
#include <ctime>    // to get the current time
using namespace std;

int main()
{
  srand(time(0)); // seed randm number generator

  int theNumber = rand() % 100 + 1; // random number between 1 and 100
  int tries = 0; // variable for the number of tries
  int guess;     // variable for the user's guess
  bool done = false; // a bool variable which indicates the game is/isn't done

  // Welcome message first.
  cout << "***********  Welcome to Guess My Number **********\n\n";

  while (done == false)
  {
    cout << "Enter a guess (between 1 and 100): ";
    cin >> guess;

    if (guess == theNumber)
    {
      cout << "  Congratulations!! You guessed the correct number!\n";
      done = true;
    }
    else if (guess < theNumber)
    {
      cout << "==> Too low.  Guess again.\n";
    }
    else
    {
      cout << "==> Too high.  Guess again.\n";
    }
  }

  // Closing message
  cout << endl 
       << "****** THANK YOU. COME BACK AGAIN SOON.******\n";

  system("pause");
  return 0;
}

-----------------------------------------------------------------------
This is where I am stuck, if anyone can point me in the right directions, it would be greatly appreciated!

#include <iostream>
#include <cstdlib>  // to use a random number generator
#include <ctime>    // to get the current time
using namespace std;

int main()
{
  srand(time(0)); // seed randm number generator

  int theNumber = rand() % 100 + 1; // random number between 1 and 100
  int tries = 0; // variable for the number of tries
  int guess;     // variable for the user's guess
  bool done = false; // a bool variable which indicates the game is/isn't done

  // Welcome message first.
  cout << "***********  Welcome to Guess My Number **********\n\n";

  while (done == false)
  {
    cout << "Enter a guess (between 1 and 100): ";
    cin >> guess;

    if (guess == theNumber)
    {
      cout << "  Congratulations!! You guessed the correct number!\n";
      done = true;
    }
    else if (guess <= 50)
    {
      cout << "==> Very Low.  Guess again.\n";
    }
    else
    {
      cout << "==> Very High.  Guess again.\n";
    }
    else if (guess <= 30 > 50)
    {
      cout << "==> Low.  Guess again.\n";
    }
    else
    {
      cout << "==> High.  Guess again.\n";
    }
    else if (guess <= 15 > 30)
    {
      cout << "==> Moderately Low.  Guess again.\n";
    }
    else
    {
      cout << "==> Moderately High.  Guess again.\n";
    }
    else if (guess <= 0 > 15)
    {
      cout << "==> Somewhat Low.  Guess again.\n";
    }
    else
    {
      cout << "==> Somewhat High.  Guess again.\n";
    }
  }

  // Closing message
  cout << endl 
       << "****** THANK YOU. COME BACK AGAIN SOON.******\n";

  system("pause");
  return 0;
}

MOD EDIT: When posting code...USE CODE TAGS!!!

This post has been edited by JackOfAllTrades: 30 September 2010 - 03:54 AM

[janotte]

Welcome to DIC!

Please edit your posting (see the "EDIT" button in lower right).
( a ) Delete all your code.
( b ) Get a fresh copy of the code with formatting in place from your editor / IDE.
( c ) Paste the formatted code between code tags like this
( d ) Use the "Preview Post" button to check it's all good.
( e ) Use the "Submit Modified Post" button to finish the editing.



Please give us some more details of your problem.
( a ) Does your code compile?
( b ) Any errors or warnings? If there are then share them with us.
( c ) Is the program producing any output?
( d ) How is the actual output different to what you want / expect? Give details and, ideally, examples.
( e ) What have you already tried to fix it?

This post has been edited by janotte: 29 September 2010 - 06:57 PM

[Alex6788]

Like janotte said you didn't say what was wrong.

Please post in code tags

and a better alternative to system("pause"); is cin.get(); and that sometimes doesn't work if there already input in the buffer than use

cin.ignore();
cin.get();

Read this it explains why system("pause"); is bad http://www.gidnetwork.com/b-61.html
I've made a really nice guess a number game and could help you with this if you tell us what's wrong.

Hope this was helpful :-)

This post has been edited by Alex6788: 29 September 2010 - 07:42 PM

[JackOfAllTrades]

Your logic in general is flawed because you're testing against the guess, rather than the difference, but you definitely can't do this kind of thing:

else if (guess <= 30 > 50)

you have to use multiple boolean statements, like this:

else if (guess <= 30 || guess > 50)

[aaa111]

Look at your else if chain:

 if (guess == theNumber)
    {
      cout << "  Congratulations!! You guessed the correct number!\n";
      done = true;
    }
    else if (guess <= 50)
    {
      cout << "==> Very Low.  Guess again.\n";
    }
    else
    {
      cout << "==> Very High.  Guess again.\n";
    }
--->else if (guess <= 30 ||guess> 50)
    {
      cout << "==> Low.  Guess again.\n";
    }

Here you are not following the else if chain properly.After an else you can't use else if,an else if followed by if statement.
An else if chain start it with a if statement and end with else statement with else if chain inbetween them,like this:

if(exp)
 statement
else if(exp)
 statement
else if(exp)
 statement
.
.
else
 statement

This post has been edited by aaa111: 30 September 2010 - 04:28 AM

[Salem_c]

Consider some examples:
theNumber is 50
guess is 30, print low
guess is 10, print very low
guess is 50, print correct

Now,
theNumber is 90
guess is 50, print very low

You need to measure the distance between theNumber and guess.

[Larsonator]

would a switch statement not be better here?

[janotte]

Larsonator, on 30 September 2010 - 10:39 PM, said:

would a switch statement not be better here?

No. Not really.