Thread: Slot Machine program c++ bug

Well, i started my C++ book again after a year to learn more about the hardware of a computer, binary and others so i would understand more about my code and so i understand what i am doing.

So here is my code and i will explain the bug at the bottom.

Code:

#include <iostream>
#include <cstdlib>
#include <time.h>

using namespace std;

typedef unsigned short int USHORT;

const USHORT tokensThree = 3;
const USHORT tokensTwo = 2;
USHORT tokens = 12;
const char playY = 'y';
char inputYes;
const USHORT highRandom = 6;
USHORT randomNumber;
USHORT randomNumberTwo;
USHORT randomNumberThree;

int winThreeTokens(int firstNumber, int secondNumber, int thirdNumber)
{
    if(firstNumber==secondNumber==thirdNumber)
    {
        cout << "Congratulations! You WON 3 Tokens!" << endl;
        tokens = tokens + tokensThree;
        cout << "You know have " << tokens << "tokens.";
    }
    return(firstNumber, secondNumber, thirdNumber);
}

int winTwoTokens(int firstNumber, int secondNumber, int thirdNumber)
{
    if(firstNumber==secondNumber || firstNumber==thirdNumber || secondNumber==thirdNumber)
    {
        cout << endl << "Congratulations! You WON 2 Tokens!" << endl;
        tokens = tokens + tokensTwo;
        cout << "You know have " << tokens << "tokens.";
    }
    return(firstNumber, secondNumber, thirdNumber);
}

int Lose(int firstNumber, int secondNumber, int thirdNumber)
{
    if(firstNumber!=secondNumber!=thirdNumber)
    {
        cout  << endl << "HAHAHA Sorry, You lost THREE Tokens!";
        tokens = tokens - 3;
        if(tokens==0)
        {
            cout << endl << "You have " << tokens << "tokens.";
            cout << endl << "Game Over!";
            cout << endl << "Now Exiting...";
            exit(0);
        }
        cout << endl << "You know have " << tokens << "tokens.";
    }
    return(firstNumber, secondNumber, thirdNumber);
}

void lever()
{
    time_t clockTimeSeconds;
    time(&clockTimeSeconds);
    srand((unsigned int) clockTimeSeconds);
    randomNumber = rand() % highRandom + 1;
    randomNumberTwo = rand() % highRandom + 1;
    randomNumberThree = rand() % highRandom + 1;
    cout << endl << "Do you want to pull the BIG Lever right beside you?" << endl;
    cout << "If so, Please type 'y' and press Return." << endl;
    cin >> inputYes;
    if(inputYes==playY)
    {
        cout << "Your numbers are:\t" << randomNumber << "\t" << randomNumberTwo << "\t" << randomNumberThree;
        winThreeTokens(randomNumber, randomNumberTwo, randomNumberThree);
        winTwoTokens(randomNumber, randomNumberTwo, randomNumberThree);
        Lose(randomNumber, randomNumberTwo, randomNumberThree);
    }
}

void welcome()
{
    cout << "Welcome to the most amazing text-based Slot Machine!" << endl;
    cout << "Our fine Slot Machines are rigged so that you almost never win and so that we make BILLIONS of DOLLARS!" << endl;
    cout << "Anyways, You have " << tokens << "tokens" << endl;
    cout << "Soooo, have fun with these few tokens as you will have none left very, very, very SOON!" << endl;
}

int main()
{
    welcome();
    for(int playTime = 0; playTime<30001; playTime++)
    {
        lever();
    }
    return 0;
}

the bug is that when you execute the application and you type "y" for your first set of random numbers and you get like "6 6 5" for example it will grant you your 2 tokens but then think that you lost and take away 3. After that happens once and get another set of 3 random numbers like "3 3 4", you get your 2 tokens and it doesn't take away the 3 like ti should work.

I just finished lesson 3 where it went over brief explanation of a function, how to call one, what it is to get you familiar with it but not to explain the whole 9 yards, which is in another lesson.

This: if(firstNumber==secondNumber==thirdNumber) doesn't do what you think; whichever of the == gets executed first (I think it's the second one, but I'd have to look it up) will return either 0 or 1, and then the first number is compared to that result (not the values of the other variables). Ditto in your lose routine.

I believe your problem might be in this condition:

Code:

firstNumber!=secondNumber!=thirdNumber

I would rewrite it to use the && operator.

Another problem is that you always call the functions for winning two tokens, for winning three tokens, and for losing tokens. So, if you managed to get the numbers 5 5 5, you would get five tokens. I don't know if this is by design or not (I'd have to assume not).

As a further exercise, I would modify your program so that you call a function based upon how many numbers match, not call each function and have each function determine what we can do.

Here is my new, with 3 functions, clean, using a while loop instead of for which i think was the problem.

Code:

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

using namespace std;

typedef unsigned short int USHORT;

const USHORT tokensThree = 3;
const USHORT tokensTwo = 2;
USHORT tokens = 24;
const char playYes = 'y';
char inputYes;
const USHORT highRandom = 6;
const char playNo = 'n';
USHORT randomNumber;
USHORT randomNumberTwo;
USHORT randomNumberThree;

void welcome()
{
    cout << "Welcome to the most amazing text-based Slot Machine!" << endl;
    cout << "Our fine Slot Machines may or maynot grant you wishes!" << endl;
    cout << "Anyways, You have " << tokens << "tokens." << endl;
    cout << "Soooo, have fun with these few tokens as you will have none left very, very, very SOON!" << endl;
}

int outcome(int firstNumber, int secondNumber, int thirdNumber)
{
    if(firstNumber == secondNumber == thirdNumber)
    {
        tokens += tokensThree;
        cout << "Congrats! You won 3 Tokens!" << endl;
        cout << "You have " << tokens << "tokens." << endl;
    }
    if(firstNumber == secondNumber || firstNumber == thirdNumber || secondNumber == thirdNumber)
    {
        tokens += tokensTwo;
        cout << endl << endl << "Congrats! You won 2 Tokens!" << endl;
        cout << "You have " << tokens << "tokens." << endl;
    }
    else
    {
        tokens -= tokensThree;
        cout << endl << endl << "Sorry! You lost 3 Tokens." << endl;
        cout << "You have " << tokens << "tokens." << endl;
    }
    return (firstNumber, secondNumber, thirdNumber);
}

void lever()
{
    time_t clockTime;
    time(&clockTime);
    srand((unsigned int) clockTime);
    randomNumber = rand() % highRandom + 1;
    randomNumberTwo = rand() % highRandom + 1;
    randomNumberThree = rand() % highRandom + 1;
    cout << endl << "Do you want to pull the Lever?" << endl;
    cout << "Press 'y' and Press Return to play or Press 'n' and Return to Quit." << endl;
    cin >> inputYes;
    if (inputYes == playYes)
    {
        cout << endl << endl << "Your random numbers are:\t" << randomNumber << "\t" << randomNumberTwo << "\t" << randomNumberThree;
        outcome(randomNumber, randomNumberTwo, randomNumberThree);
    }
    else
    {
        exit(0);
    }
}

int main()
{
    welcome();
    while(tokens != 0)
    {
        lever();
    }
}

You've made some little improvements in the new version of your program, but some bugs still remain that have been previously pointed out.

Check this line:

Code:

firstNumber == secondNumber == thirdNumber

I believe that if you have the numbers X X 1, this statement will be true. Thankfully you also don't generate 0 as a number, because that would cause an even more interesting case: X Y 0 results in true.

After you've fixed that line, you still have an issue with your functions. If you have 3 of the same number, you will be rewarded with 3 tokens. Your program then points out that you have 2 matching numbers and gives you a further 2 tokens. Check your if statements. As far as each individual statement goes, they are correct (save for the above).

I am not trying to be rude here but if you tell em that there is still 2 bugs in my program, can you at least give me some sample program or a suggestion on how to fix it. Thanks for the tips but it didn't learn anything but getting knwoledge of 2 bugs in my program and i am here not knowing how to fix it or any suggestions to resolve those bugs.

thanks for your, it made me relax, focus on the code and to be all jumpy. Try to run the code through your head. I am still stumbled and i tried many ways to resolve it, i will come back to it and go on with the lessons in the book.

Well, now your program should work more or less as expected, according to my quick scan. That is, except for the overflow error that you are aware of. With a little bit of thought, you'll be able to solve that issue, too. Wait: after a reread of your program, I just noticed something: a mistake that I already pointed out above.

You saw how incorrect usage of a condition can end up in totally incorrect statements being regarded as true or false, causing all sorts of problems. Well, you did it again. Let's have a look.

Code:

tokens != 0 != tokens > 65000

After examining some test values that statement might actually work, but it probably doesn't work how you think it does. It is executed like this:

(tokens != 0) != (tokens > 65,000), tokens > 65,000 is executed first, then tokens != 0, and then the comparison of these two answers.



Now that we've got your program working, we can focus on all of the bad practices you engage in. Yay us!

It is generally bad practice to declare global variables, depending upon the circumstances. You always want to strive for your variables to be in the smallest scope possible. For example, you wouldn't do this:

Code:

int x = 0; //Only used in someFunction
double y = 0.0; //Also only used in someFunction

void someFunction()
{
//Code here
}

A better version of this would be like this:

Code:

void someFunction()
{
int x = 0; //only used in someFunction, and can only  be used in someFunction
double y = 0.0 //only available in this function

}

That's one issue you have. I would also declare your outcome function as void, since you apparently don't actually need it to return anything. It can still accept the values you pass it, and you can still use return, but you just can't return values. You may have noticed compiler warnings about outcome's value not actually being used.

Perhaps someone else will help you on whatever else this program needs.