DaniWeb Forum Index > Software Development > C++

Correct Answers in Hangman

Please support our C++ advertiser: Intel Parallel Studio Home

Thread Solved

•

Join Date: Jul 2009

Posts: 4

Reputation: Cromarte888 is an unknown quantity at this point

Solved Threads: 0

Cromarte888 Cromarte888 is offline

Offline

Newbie Poster

Correct Answers in Hangman

Jul 19th, 2009

Alright, I'm down to the nitty gritty absolute end of a fully functional hangman game. There is just one part where I am absolutely stuck. When you get a wrong guess, the game successfully draws the gallows. I have no issues with wrong guesses. It's the right guesses that has me stumped. Here is what I have for this:

 Help with Code Tags 
 C++ Syntax (Toggle Plain Text) 
guess(); //call member function of letter chosen
ltrpos = gameWord.find(gameGuess);
cout << "You have chosen the letter " << gameGuess << '.' << endl; 
if (ltrpos < gameWord.length()) //checks letter existance in chosen random word (correct choice)
{ 
for (i = 0; i < gameWord.length(); i++)
{
if (gameGuess == gameWord[i])
{
cout << gameGuess;
}
else
cout << '_'; 
}
cout << endl;
cout << "Wow, nice guess!!! Guess another letter." << endl;
guess(); //call member function of letter chosen
ltrpos = gameWord.find(gameGuess);
cout << "You have chosen the letter " << gameGuess << '.' << endl; 
if (ltrpos < gameWord.length()) //checks letter existance in chosen random word (correct choice)
{ 
for (i = 0; i < gameWord.length(); i++)
{
if (gameGuess == gameWord[i])
{
cout << gameGuess;
}
else
cout << '_'; 
}
cout << endl;
cout << "Wow, nice guess!!! Guess another letter." << endl;

When you guess the correct letter it DOES display ALL the instances of that character in the correct places. The problem I'm having is, when it loops around to guess again, how do I keep the correct guess that was just made and add to that guess with the next correct guess while putting the NEW letters in their correct instances as well.
Example of how it is now:
Loop 1: n_n__n__
Loop 2: _i______
Loop 3: ___t____
The word here should be nintendo and it should ALSO do this:
Loop 1: n_n__n__
Loop 2: nin__n__
Loop 3: nint_n__

Suggestions?????

•

Join Date: Jun 2009

Posts: 830

Reputation: wildgoose is a name known to all

Solved Threads: 94

wildgoose

Offline

Practically a Posting Shark

Re: Correct Answers in Hangman

Jul 19th, 2009

You don't seem to be stuffing the letter! To save it!
Or some indication you have that letter within the word that was already tested!

 Help with Code Tags 
 C Syntax (Toggle Plain Text) 
if (gameGuess == gameWord[i])   // matching letter found?
{
   myWord[i] = gameGuess;  // <---- Stuff new letter
   cout << gameGuess;
}
else if (myWord[i] )
   cout << myWord[i];
else
  cout << '_'; 
}
if (gameGuess == gameWord[i])   // matching letter found?
{
   myWord[i] = gameGuess;  // <---- Stuff new letter
   cout << gameGuess;
}
else if (myWord[i] )
   cout << myWord[i];
else
  cout << '_'; 
}

Last edited by wildgoose; Jul 19th, 2009 at 5:20 am. Reason: alteration

•

Join Date: Oct 2008

Posts: 45

Reputation: zalezog is an unknown quantity at this point

Solved Threads: 11

zalezog

Offline

Light Poster

Re: Correct Answers in Hangman

Jul 19th, 2009

Another approach could be
1.Make another userGuess string.
2. First , fill all the characters in userGuess string
with underscores( _ ).
3.Make changes in the userGuess string.

 Help with Code Tags 
 C++ Syntax (Toggle Plain Text) 
guess(); //call member function of letter chosen
ltrpos = gameWord.find(gameGuess);
cout << "You have chosen the letter " << gameGuess << '.' << endl; 
if (ltrpos < gameWord.length()) //checks letter existance in chosen random word (correct choice)
{ 
for (i = 0; i < gameWord.length(); i++)
{
    if (gameGuess == gameWord[i])
    {
        //Instead  of cout << gameGuess . I hope gameGuess is a 
       // a single character.
        userGuess[i] = gameGuess;    
    }
 
} //After the for loop display the changed string altogether
cout  << userGuess; 
//This will retain changes
guess(); //call member function of letter chosen
ltrpos = gameWord.find(gameGuess);
cout << "You have chosen the letter " << gameGuess << '.' << endl; 
if (ltrpos < gameWord.length()) //checks letter existance in chosen random word (correct choice)
{ 
for (i = 0; i < gameWord.length(); i++)
{
    if (gameGuess == gameWord[i])
    {
        //Instead  of cout << gameGuess . I hope gameGuess is a 
       // a single character.
        userGuess[i] = gameGuess;    
    }

} //After the for loop display the changed string altogether
cout  << userGuess; 
//This will retain changes

4. Search for any underscores remaining in the userGuess string.
If there are , the word is stilll incomplete. Loop again.

Edit:
@wildgoose: I think gameWord is the original string.

Last edited by zalezog; Jul 19th, 2009 at 5:17 am.

•

Join Date: Jun 2009

Posts: 830

Reputation: wildgoose is a name known to all

Solved Threads: 94

wildgoose

Offline

Practically a Posting Shark

Re: Correct Answers in Hangman

Jul 19th, 2009

Yes, simultaneous post. It's late here, and I noticed that after the post and editted then saw your post!

Cromarte88, you need either to store the characters you picked correctly, or store a flag for each correctly chosen character position so that letters previously chosen will be displayed!

•

Join Date: Jul 2009

Posts: 4

Reputation: Cromarte888 is an unknown quantity at this point

Solved Threads: 0

Cromarte888 Cromarte888 is offline

Offline

Newbie Poster

Re: Correct Answers in Hangman

Jul 19th, 2009

•

Originally Posted by zalezog

Another approach could be
1.Make another userGuess string.
2. First , fill all the characters in userGuess string
with underscores( _ ).
3.Make changes in the userGuess string.
Help with Code Tags

C++ Syntax (Toggle Plain Text)
guess(); //call member function of letter chosen
ltrpos = gameWord.find(gameGuess);
cout << "You have chosen the letter " << gameGuess << '.' << endl; 
if (ltrpos < gameWord.length()) //checks letter existance in chosen random word (correct choice)
{ 
for (i = 0; i < gameWord.length(); i++)
{
    if (gameGuess == gameWord[i])
    {
        //Instead  of cout << gameGuess . I hope gameGuess is a 
       // a single character.
        userGuess[i] = gameGuess;    
    }
 
} //After the for loop display the changed string altogether
cout  << userGuess; 
//This will retain changes
guess(); //call member function of letter chosen ltrpos = gameWord.find(gameGuess); cout << "You have chosen the letter " << gameGuess << '.' << endl; if (ltrpos < gameWord.length()) //checks letter existance in chosen random word (correct choice) { for (i = 0; i < gameWord.length(); i++) { if (gameGuess == gameWord[i]) { //Instead of cout << gameGuess . I hope gameGuess is a // a single character. userGuess[i] = gameGuess; } } //After the for loop display the changed string altogether cout << userGuess; //This will retain changes
4. Search for any underscores remaining in the userGuess string.
If there are , the word is stilll incomplete. Loop again.

Edit:
@wildgoose: I think gameWord is the original string.

I can see how this will print out the word the way I have it now, but I'm not fully understanding how this will add new correctly guessed words to the string. I'll give it a whirl though. Thanks for your thoughts, I greatly appreciate it.

•

Join Date: Jun 2009

Posts: 830

Reputation: wildgoose is a name known to all

Solved Threads: 94

wildgoose

Offline

Practically a Posting Shark

Re: Correct Answers in Hangman

Jul 19th, 2009

Method of using a double character buffer!

['H']['a']['n']['g']['T']['e']['n'] <-- gameWord[]
['_']['_']['_']['_']['_']['_']['_'] <-- gameGuess[]

As each letter guessed, the letter is copied over to same index

Only display gameGuess, not gameWord gets displayed.

•

Join Date: Jul 2009

Posts: 4

Reputation: Cromarte888 is an unknown quantity at this point

Solved Threads: 0

Cromarte888 Cromarte888 is offline

Offline

Newbie Poster

Re: Correct Answers in Hangman

Jul 19th, 2009

•

Originally Posted by zalezog

Another approach could be
1.Make another userGuess string.
2. First , fill all the characters in userGuess string
with underscores( _ ).

I'm not clear on how this helps me if the word is a different length every time the game is played. That seems like it would be useful if all words were the same length.

Last edited by Cromarte888; Jul 19th, 2009 at 10:39 pm. Reason: needed to block close quote

•

Join Date: Jun 2009

Posts: 830

Reputation: wildgoose is a name known to all

Solved Threads: 94

wildgoose

Offline

Practically a Posting Shark

Re: Correct Answers in Hangman

Jul 19th, 2009

So, use a big buffer for both. Get string length of actual hangman string, fill guess string with '_' and set the ASCIIz terminator at the element corresponding to the string length.

So now they are the same length!

I said there were several ways of doing this. If you don't fully understand strings then go back and review!

•

Join Date: Oct 2008

Posts: 45

Reputation: zalezog is an unknown quantity at this point

Solved Threads: 11

zalezog

Offline

Light Poster

Re: Correct Answers in Hangman

Jul 20th, 2009

•

Originally Posted by Cromarte888

if the word is a different length every time the game is played. That seems like it would be useful if all words were the same length.

Then..

•

Originally Posted by wildgoose

So, use a big buffer for both....

I wouldn't agree.
A snippet which might help you if you can see the link.

 Help with Code Tags 
 C++ Syntax (Toggle Plain Text) 
#include<iostream>
#include<string>
 
int main()
{
    std::string word = "Hangman";
    std::string userGuess(word.length() , '+'); //<<
    std::cout <<"The original string is" << word;
    std::cout <<"\nThe contents of the userGuess string is" << userGuess;
    userGuess.erase(); //<<
    std::cout <<"\nAfter erasing" <<userGuess;
 
    word = "daniweb";
    std::cout <<"The original string is " << word;
    //And then again
    userGuess.resize(word.length(), '_'); //<<
    std::cout <<"The original string is" << word;
    std::cout <<"\nThe contents of the userGuess string is" << userGuess;
    return 0;
}
#include<iostream>
#include<string>

int main()
{
    std::string word = "Hangman";
    std::string userGuess(word.length() , '+'); //<<
    std::cout <<"The original string is" << word;
    std::cout <<"\nThe contents of the userGuess string is" << userGuess;
    userGuess.erase(); //<<
    std::cout <<"\nAfter erasing" <<userGuess;

    word = "daniweb";
    std::cout <<"The original string is " << word;
    //And then again
    userGuess.resize(word.length(), '_'); //<<
    std::cout <<"The original string is" << word;
    std::cout <<"\nThe contents of the userGuess string is" << userGuess;
    return 0;
}

Last edited by zalezog; Jul 20th, 2009 at 11:24 am.

•

Join Date: Jun 2009

Posts: 830

Reputation: wildgoose is a name known to all

Solved Threads: 94

wildgoose

Offline

Practically a Posting Shark

Re: Correct Answers in Hangman

#10

Jul 20th, 2009

Like I posted. There are several ways of doing this! In my case, it was based upon a fixed length buffer that gets reused. In Zalezog's case, they're promoting the use of a dynamic buffer. You're having trouble with simple ASCII buffers so I felt the fixed length buffer was a good starting place. The method of using variable length buffer's such as using the string class is the more advanced but better solution. You are using C++ in your code!

Last edited by wildgoose; Jul 20th, 2009 at 11:40 am.