Loops & Switch Statements

[kwozz48]

Hey everyone,

I'm working out some of the practice problems that are on this website, and I just had a question about this one.


"Write a program that ccontinues to asks the user to enter any number other than 5 until the user enters the number 5.
Then tell the user "Hey! you weren't supposed to enter 5!" and exit the program.

★ Modify the program so that after 10 iterations if the user still hasn't entered 5 will tell the user "Wow, you're more patient then I am, you win." and exit.

★★ Modify the program so that it asks the user to enter any number other than the number equal to the number of times they've been asked to enter a number. (i.e on the first iteration "Please enter any number other than 0" and on the second iteration "Please enter any number other than 1"m etc. etc. The program must behave accordingly exiting when the user enters the number they were asked not to.)"


Now I've almost figured out everything (though it may not be the best way to do everything) except for 2 star portion of the question. I used a switch statement to determine if the input is valid, and I'm not sure how to make a 'case' that allows me to refer to whatever number 'i' is. Obviously 'i' changes depending on how many times the loop has been run through, so it the case number has to change every time. Is there a way to do this?

And also, even though the problem didn't ask for it, I wanted to make it where if an invalid character (such as a letter, or '@') was entered, the program would end. What's the best way to do this?

I know this is pretty basic stuff, but I've been teaching myself and between school and a job, it's tough to learn as quickly as I'd like.

Thank you for any feed back!

  // Picking Numbers

#include <iostream>

using std::cout;
using std::cin;

int main()
{
    int number = 0;                 // Integer entered by user
    int i = 0;                      // Loop counter
    
    
    
    for (i = 0; i < 10; ++i)        // Loop that stops program when a number has succesfully been entered 10 times
    {
        cout << "Please enter any number other than a " << i << " or a 5.\n\n";
        
        cin >> number;              // User input
        
        switch (number)             // Switch statement to determine if what the user entered is valid
            {
                case 5: cout << "You weren't supposed to enter a 5.";
                    return 0;       // Ends program
                
                case i: cout << "You weren't supposed to enter a " << i << " !";
                    return 0;       // Ends program
            
                default: break;     // Any number other than a 5 or 'i' entered causes the program to loop back until 10 valid numbers have been entered
            }
    }
    
    cout << "Good job you entered a valid number 10 times in a row!";
    
}

[Yay295]

Just use an if statement instead of that switch.

[xenovia12]

@Yay295 what kind of answer is that?

[xenovia12]

i see the problem now
change line 26 into this case !'i' : cout << "You weren't supposed to enter a " <<number << " !";

[TwilightSpectre]

@justinelandichoruiz
What? That doesn't even make much sense... what @Yay295 was suggesting is correct. So, rather than having the switch, you would do something like this:

// ...

for (i = 0; i < 10; ++i) {
    cout << "Please enter any number other than a " << i << " or a 5.";
    cin >> number; 

    // the || means 'or', in case you didn't know
    if (number == 5 || number == i) {
        cout << "You weren't supposed to enter a " << number << ".\n";
        return 0;
    }
}

// ... 

[xenovia12]

and also this is my fix code

#include <iostream>
using namespace std;
int main()
{
    int i=0;
    int number;
    for(i=0;i<=10;i++)
  {

        cout << "Please enter any number other than a " << i << " or a 5.\n\n";
        cin>>number;
        switch(number)
        {
            case 5:
            cout<<"Hey! you weren't supposed to enter 5!";
           return 0;


        }
if(number==i)
{
    cout<<"Hey you were not supposed to enter "<<i ;
    return 0;
}
}    }

[xenovia12]

@NT3 see my new code above this comment
its not what you think. i said that because maybe the OP wants to do in his way

[kwozz48]

Thanks for the answers guys! I know a switch statement isn't really necessary, but the problem said that I should use one and that was the only way I think I could. Thanks for the help though!

[xenovia12]

Why dont you mark this thread as solved