Floats using floating point representation to allow you to store decimal values. Integers are designed to store whole numbers - which is what you're doing and should be using.

float x = 34.2f;
int y = 34;

For your program, you'd be reading the input from the standard cin and then using a switch statement to match to an enum int value. This removes all the if-else-if mess and improves readability and understanding of your code.

Also, from actually looking at your code I can tell you it's invalid.

Thanks for the help, I'll have to have another look at it.

This whole programming thing is just a big headache.

Here's your program, fixed.. without the joke bit since you need to do system specific sleep calls. As for the enum, you can define HAPPY = 1 and leave the rest blank and the compiler will infer incrementation. I didn't 'use' namespace standard because that's what scrubs do. I added in the standard input algorithm expected from beginner CLI programmers (and basically anyone doing CLI - removes repetition).

ObscureCoder wrote Here's your program, fixed.. without the joke bit since you need to do system specific sleep calls. As for the enum, you can define HAPPY = 1 and leave the rest blank and the compiler will infer incrementation. I didn't 'use' namespace standard because that's what scrubs do. I added in the standard input algorithm expected from beginner CLI programmers (and basically anyone doing CLI - removes repetition).


#include <iostream>

enum Choices {
    HAPPY = 1, PUNISH = 2, TEXTBOOK = 3, MAX_CHOICES
};

int main()
{
    int choice;
    std::cout << "Enter a number..." << std::endl;
    std::cin >> choice;


    while(choice == 0 || choice >= (MAX_CHOICES))
    {
        std::cerr << "Error, invalid choice. Try again." << std::endl;
        std::cin >> choice;
    }

    switch (choice) {
    case HAPPY:
        /*
         * You're gonna need to do whatever joke you're doing here
         * Note: you'll have to use the WinAPI Sleep() function to delay your program
         * I would've added it in but I use Linux and don't wanna add in usleep() or #ifdefs to confuse you.
         */
        break;
    case PUNISH:

        break;
    case TEXTBOOK:

        break;
    default:
        std::cerr << "Unexpected error occured." << std::endl;
        break;
    }

    // Optional: waits for keypress to finish execution.
    std::cin.ignore().get();

    return 0;
}



To be completely honest. The code you provided looks completely foreign to me, I've never done some of those things.

I will try to execute it tomorrow, at school.

If you'd like, you could PM me your email (Has to be Gmail, because I would share them via Google Docs) and I could share you some of my past projects, that worked. They're screen shots in the actual program, and not just typed code on a forum. It might give you an idea of what I'm used to doing.

I really do appreciate all the help though.