loop the menu automatically
the pattern for what you ask looks like this:
do
{
get user input
perform action(s) on user input if not the quit action (just do nothing for that)
//the above is often a switch statement, but it does not have to be.
}while(input is not quit option)
do
{
get user input
perform action(s) on user input if not the quit action (just do nothing for that)
//the above is often a switch statement, but it does not have to be.
}while(input is not quit option)
it goes around the menu options, which I will be honest, I don't know what the menu is in your code. Its just a wall of print statements to me.
I think, again roughly, you want:
do
cin choice
switch on choice
end of switch //does anything else go in here besides the switch?
while(choice != whatever to exit)
What that is doing is looping once no matter what, get the input, process it (again, ignore if if the choice is to quit) and then see if it needs to quit (while check). If its not the quit, it loops back and does it again...
I think, again roughly, you want:
do
cin choice
switch on choice
end of switch //does anything else go in here besides the switch?
while(choice != whatever to exit)
What that is doing is looping once no matter what, get the input, process it (again, ignore if if the choice is to quit) and then see if it needs to quit (while check). If its not the quit, it loops back and does it again...
You need to add the whole loop that jonnin mentioned.
To prompt until the user gives a valid value, I prefer to use the pattern:
It's hard to get used to this pattern because most people don't like breaking out of the middle of a loop, but to me, this expresses the actual logic very clearly.
Once you have the input you need a switch statement:
Case 3 just sets a variable to indicate that you're done with the loop.
Have you learned about arrays? What about functions? Either one of these would help shrink your code for case 1 dramatically.
To prompt until the user gives a valid value, I prefer to use the pattern:
|
|
It's hard to get used to this pattern because most people don't like breaking out of the middle of a loop, but to me, this expresses the actual logic very clearly.
Once you have the input you need a switch statement:
|
|
Case 3 just sets a variable to indicate that you're done with the loop.
Have you learned about arrays? What about functions? Either one of these would help shrink your code for case 1 dramatically.
|
|
just a function would do wonders.
at the top of the code before main:
string domino(int checkdigit)
{
string result;
if (checkdigit == 0)
result= "||:::";
else if (checkdigit == 1)
result= ":::||";
else if (checkdigit == 2)
result= "::|:|";
else if (checkdigit == 3)
result= "::||:";
else if (checkdigit == 4)
result= ":|::|";
else if (checkdigit == 5)
result= ":|:|:";
else if (checkdigit == 6)
result= ":||::";
else if (checkdigit == 7)
result= "|:::|";
else if (checkdigit == 8)
result= "|::|:";
else if (checkdigit == 9)
result= "|:|::";
}
…
down in the code
fifth1 = domino(fifth);
checkdigit1 = domino(checkdigit);
etc..
at the top of the code before main:
string domino(int checkdigit)
{
string result;
if (checkdigit == 0)
result= "||:::";
else if (checkdigit == 1)
result= ":::||";
else if (checkdigit == 2)
result= "::|:|";
else if (checkdigit == 3)
result= "::||:";
else if (checkdigit == 4)
result= ":|::|";
else if (checkdigit == 5)
result= ":|:|:";
else if (checkdigit == 6)
result= ":||::";
else if (checkdigit == 7)
result= "|:::|";
else if (checkdigit == 8)
result= "|::|:";
else if (checkdigit == 9)
result= "|:|::";
}
…
down in the code
fifth1 = domino(fifth);
checkdigit1 = domino(checkdigit);
etc..
Are you sure you haven't learned about functions and arrays? This is so much easier with them. Using Jonnin's names:
If you really can't do it this way then here's a method without arrays:
|
|
If you really can't do it this way then here's a method without arrays:
|
|
Topic archived. No new replies allowed.