Thread: code is doing the opposite of what i tell it to do..

ok well here is my problem. my code is doing the exact opposite i want it to do. it is saying when a person enters a 1 or anything else other than a 0 it will show up as a the person rolled but i would like it to show up when a person enters a number or letter thats NOT 1 it will show up as

Code:

//adds the iostream library to the program
#include <iostream>
#include <cstdlib>
#include <stdio.h>
#include <stdlib.h>
#include <time.h>
#define BOOL_H
#undef true
#undef false 
//informs the compiler you are using the standard library set
using namespace std;

const int LOW = 1;
const int HIGH = 6;
int firstDie;
int secondDie;
int Choice;
int yes;
int no;

/*
Variables to hold random values
for the first and the second die on
each roll.
*/


int main ()    
{
   
   time_t seconds;
   time(&seconds);
   srand((unsigned int) seconds);
   for ( ;; )
   {
      cout << "Round 1 \n";
      cout << "Would you like to roll this round? [1 for yes, 0 for no]:\n\n";
   if ( cin >> yes || no == 0 )
   {}
   else
      {firstDie = rand() % (HIGH - LOW + 1) + LOW;
(secondDie = rand() % (HIGH - LOW) + 1) + LOW;
cout << "you rolled a "<< firstDie << " and a " << secondDie << endl; break;}    
   }
       system("pause\n\n\n");
   return 0;
}

cin << choice;

cin>>choice;
???

As I mentioned in your other thread, a common way to handle bad input is this:

Code:

while (!(cin >> yes) || (yes != 0 && yes != 1))
{
  // clear any fail state from the user entering characters instead of a number
  cin.clear();

  // remove all bad characters from the input stream to start fresh
  cin.ignore(numeric_limits<streamsize>::max(), '\n');

  // Prompt again here...

}

// Use yes as you would normally.

Note that you have to #include <limits> to use numeric_limits. You can also just use a large number there to make sure all bad characters are ignored.

That code will keep prompting until 0 or 1 is input. Then you can handle 0 or 1 as you wish.

Code:

Daved

i can use this one because this one doesn't actually do what i need it to do. although it has the clear values i need it to repeat the message over and i can't find a code like that yet.

Originally Posted by SlyMaelstrom

Code:

if ( cin >> yes || no == 0 )

What do you think this does? Do you tink it takes your input stream and puts it either into yes or no and then checks to see if it's zero? What it says is, wait for input and put it into yes, return true, or if no equals 0, return true. Since it's gonna wait on input and you're gonna have to input eventually, this will always return true and therefor you'll never get the else case to happen.

I think what you want is:

Code:

cin >> choice;
if (choice == 0) {}
else if (choice == 1) {}

Also don't make calls to the system.

also your code doesn't help either because when i use it makes me do the samething i had before which would make it only work with if a person enters 0 then it will repeat the same statement over... i need it to repeat the statement that it

Code:

 
//adds the iostream library to the program
#include <iostream>
#include <cstdlib>
#include <stdio.h>
#include <stdlib.h>
#include <time.h>
#define BOOL_H
#include <limits>
#undef true
#undef false 
//informs the compiler you are using the standard library set
using namespace std;

const int LOW = 1;
const int HIGH = 6;
int firstDie;
int secondDie;
int Choice;
int yes;
int no;

/*
Variables to hold random values
for the first and the second die on
each roll.
*/


int main ()    
{
   
   time_t seconds;
   time(&seconds);
   srand((unsigned int) seconds);
   for ( ;; )
   {
      cout << "Round 1 \n";
      cout << "Would you like to roll this round? [1 for yes, 0 for no]:\n\n";
cin >> Choice;
if (Choice == 0) {}
else{      {firstDie = rand() % (HIGH - LOW + 1) + LOW;
(secondDie = rand() % (HIGH - LOW) + 1) + LOW;
cout << "you rolled a "<< firstDie << " and a " << secondDie << endl; break;}  }
  
   }
       system("pause\n\n\n");
   return 0;
}

i kind of had that part figured out i just needed some help to make it like this.

example : http://img400.imageshack.us/my.php?image=pig16ft.png

>> i can use this one because this one doesn't actually do what i need it to do. although it has the clear values i need it to repeat the message over and i can't find a code like that yet.

Sure you can use that code. It wasn't meant as a finished solution, it was meant to be understood and added to your code. If you noticed, there is a comment in the code that says prompt again here. That means you add your message there. If you do that, it will of course repeat the message.

The while loop runs while the input is bad. The input is bad if cin >> yes returns false, or if it succeeds but yes does not equal 1 and yes does not equal 0. Either way, if the input is bad, then you clear the stream of any errors and ignore any bad or leftover input. Then you prompt again. That is exactly what my code does. You can add that inside your own for loop and it will do what you want.

ok but how do you make the code when a person enters 0 or anything other than 1 repeat?

Code:

"Would you like to roll this round? [1 for yes, 0 for no]:\n\n";

Originally Posted by nvoigt

My code does repeat for anything but zero. Read or run it ( better yet, both ).

yawn read my question atthe top of your post then look at your answer....

Code:

#include <iostream>
#include <cstdlib>
#include <ctime>

//informs the compiler you are using the standard library set
using namespace std;

const int LOW = 1;
const int HIGH = 6;

int main ()    
{
    int firstDie    = 0;
    int secondDie    = 0;
    int choice        = 0;
    bool running    = true;
    int  round        = 1;
   
    srand ( time(NULL) );

    while( running )
    {
        cout << "Round " << round << endl;
        cout << "Would you like to roll this round? [1 for yes, 0 for no]:" << endl;
        
        cin >> choice;

        if( choice) // any non-zero value will return true
        {
            firstDie = rand() % (HIGH - LOW + 1) + LOW;
            secondDie = rand() % (HIGH - LOW + 1) + LOW;

            cout << "you rolled a "<< firstDie << " and a " << secondDie << endl;
        }
        else /* choice == 0 */
        {
            running = false;
        }

        round++;
      }

    system("pause");
   
    return 0;
}