What is wrong with this code (C++)

[Blodrast]

Say the word to be guessed is named wtbg.

guessed_chars_so_far = 0;
while (guessed_chars_so_far < strlen (wtbg)) {
boolean guessed = false;
while (guessed == false) {
cout << "Guess character in position " << guessed_chars_so_far << endl;
cin >> c;
if (c == wtbg[guessed_chars_so_far]) {
cout << "Correct !" << endl;
guessed = true;
guessed_chars_so_far ++;
}
else {
cout << "Wrong guess, let's try again " << endl;
}
}
}


To answer your question, to compare two characters you don't need strcmp, you can simply compare them (with ==). If you want to compare two strings, you need to use strcmp.

edit: even though my code was indented, the board ignores that...

[The_Doctor]

Great, I will try it tomorrow.

[Sigurd]

edit: even though my code was indented, the board ignores that

guessed_chars_so_far = 0;
while (guessed_chars_so_far < strlen (wtbg))


{

boolean guessed = false;

while (guessed == false)


{

cout << "Guess character in position " << guessed_chars_so_far << endl;
cin >> c;
if (c == wtbg[guessed_chars_so_far])
{

cout << "Correct !" << endl;
guessed = true;
guessed_chars_so_far ++;

}
else


{

cout << "Wrong guess, let's try again " << endl;

}

}

}

------------------------------------------------------------------
Like this? ...

[Blodrast]

gasp!
what is the secret ?! It took me longer to indent the stuff than it took to write the damn thing...

[The_Doctor]

Update:
It works!!! (well more or less)

There are a few bugs to work out, I should be able to work these out by myself, if I can't there is always you guys.

[Blodrast]

great, good to hear :)