Simon Says game

[MeltyGoblin]

Keineahnung that worked, thank you very much.

if you are all still interested this is my working code

// HW6 - Simon says game
// Author - Tyler Humphries
// 10/5/11

#include <iostream>
#include <time.h>
#include <stdlib.h>
#include <string>

using namespace std;
int main () {
// variable declaration, initialization
string seq = "";
string userseq = "";
char color[4];
color[0]='R';
color[1]='G';
color[2]='B';
color[3]='Y';
int round;
round=1;
srand(time(NULL));

// loop of 15 rounds
while (round<16){

// Displays and erases sequence
seq +=color[rand()%4];

cout << seq << flush;
sleep (5);
cout << "\010." << flush <<'\n';



//user input of sequence
cout << "Please enter " << round << " characters to match:";
cin >> userseq;




//if match continue with next round otherwise quit the loop
if (userseq != seq){
cout << "Sorry, you lose. The correct sequence was: " << seq;
break;
}



if (round == 15){
cout << "Congratulations! you win!";
break;
}
round++;
}


return 0;
}

[10oTURfi]

Indent isnt supposed to get "fixed", your text editor does the indent for you. Whenever text editor enters new {} body it indents it for 4x spaces, when it exits the body it removes 4x spaces of indent.
Also I am worried about that teacher who isnt caring for coding style.

Since you are using linux system("cls") wont work, and neither will that chunk of code I posted work.

PS. I was just trying to help, and you got all mad on me beacuse I didnt solve your problem in an instant. Understanding random number generation, the gotos, if statement, the fact that code should be more readable is much more usefull than making that ugly piece of code work.

Good luck with your teacher *insert alot of sarcasm here*

[keineahnung]

Hi MeltyGoblin
But didn't you want the whole sequence to be hidden after each user entry?
So the first time you've got to remember just 1 letter, second time 2, third time 3, etc. That's not happening, at least not on my system. I'm just trying to think how you'd fix it...

[MeltyGoblin]

PS. I was just trying to help, and you got all mad on me beacuse I didnt solve your problem in an instant. Understanding random number generation, the gotos, if statement, the fact that code should be more readable is much more usefull than making that ugly piece of code work.

I apologize for getting upset with you, but you also should understand my frustration, I kind of outlined that in a post towards TheMeerkat

Good luck with your teacher *insert alot of sarcasm here*

you don't know the half of it, she barely speaks English, I can't understand her heavy accent and she thinks some words mean other things, thus I make many simple coding mistakes.

Hi MeltyGoblin
But didn't you want the whole sequence to be hidden after each user entry?
So the first time you've got to remember just 1 letter, second time 2, third time 3, etc. That's not happening, at least not on my system. I'm just trying to think how you'd fix it...

The program I have is acceptable, but not ideal. My professor didn't really specifically state where to erase the whole sequence or just the most recent. Though she did specifically tell us to use flush. If there is a way I could erase the whole sequence I will do it. Otherwise I am satisfied with this program.

[MeltyGoblin]

@ keineahnung

I thought of a way to erase the entire sequence. Here is my block of code for displaying and erasing the sequence.

cout << seq << flush;
sleep (3);
cout << "\014" << flush <<'\n';

this erases the entire screen, so not only does it prevent you from using your past answers to help with future answers, it cleans up the game quite a bit. So all you see on screen is


please enter (round variable) number of characters to match: (player entry)


I think it works much better.

[keineahnung]

this erases the entire screen, so not only does it prevent you from using your past answers to help with future answers, it cleans up the game quite a bit.

Great stuff, well done ;)
And good luck with your, erm, dodgy teacher.