Welcome to the Treehouse Community
Want to collaborate on code errors? Have bugs you need feedback on? Looking for an extra set of eyes on your latest project? Get support with fellow developers, designers, and programmers of all backgrounds and skill levels here with the Treehouse Community! While you're at it, check out some resources Treehouse students have shared here.
Looking to learn something new?
Treehouse offers a seven day free trial for new students. Get access to thousands of hours of content and join thousands of Treehouse students and alumni in the community today.
Start your free trial![](https://ecs-static.teamtreehouse.com/assets/views/marketing/shared/community-banner-white-47072046c51352fe6a69f5e691ff5700b28bb11d45197d7bdf066d9ea3f72d0c.webp)
![Yashvir Sangha](https://secure.gravatar.com/avatar/e904339da5b186a814cff34a0e504499?s=96&d=https%3A%2F%2Fecs-static.teamtreehouse.com%2Fassets%2Fcontent%2Fdefault_avatar-445fbbabfc8dc9188fb5967fe43322ee0c3e0dd1e10f378bf8343784af5a13eb.webp&r=pg)
Yashvir Sangha
620 PointsChecking isHit incorrectly causing incorrect looping
No matter what correct answer I input the output is always tree----e, even if I type h, o, u or s. Here is my code:
-------Game.java:
class Game {
private String answer;
private String hits;
private String misses;
public Game(String answer){
this.answer = answer;
hits = "";
misses = "";
}
public boolean applyGuess(char letter){
boolean isHit = answer.indexOf(letter) != -1;
if (isHit) {
hits += "letter";
} else {
misses += "letter";
}
return isHit;
}
public String currentGuesses(){
String progress = "";
for (char letter : answer.toCharArray()){
char display = '-';
if(hits.indexOf(letter) != -1){
display = letter;
}
progress += display;
}
return progress;
}
}
--------Prompter.java:
import java.util.Scanner;
class Prompter {
private Game game;
public Prompter(Game game) {
this.game = game;
}
public boolean promptForGuess() {
Scanner scanner = new Scanner(System.in);
System.out.print("Enter a letter:");
String guessInput = scanner.nextLine();
char guess = guessInput.charAt(0);
return game.applyGuess(guess);
}
public void displayProgress() {
System.out.printf("Try to solve; %s%n",
game.currentGuesses());
}
}
------ Hangman.java:
public class Hangman {
public static void main(String[] args) {
// Your incredible code goes here...
Game game = new Game("treehouse");
Prompter prompter = new Prompter(game);
prompter.displayProgress();
boolean isHit = prompter.promptForGuess();
if (isHit) {
System.out.println("We got a hit");
} else{
System.out.println("You missed!");
}
prompter.displayProgress();
}
}
2 Answers
![Mark Sebeck](https://uploads.teamtreehouse.com/production/profile-photos/6569772/micro_mark.jpg)
Mark Sebeck
Treehouse Moderator 37,777 PointsHi Yashvir. So close but a tiny typo can make the results so weird it's hard to track down.
In your game class look at the applyGuess method
public boolean applyGuess(char letter){
boolean isHit = answer.indexOf(letter) != -1;
if (isHit) {
hits += "letter";
} else {
misses += "letter";
}
return isHit;
}
hits += "letter"; - will add the string "letter" to hits. Instead you want to add what is stored in the variable letter. Just delete the quotes and it will work as expected.
public boolean applyGuess(char letter){
boolean isHit = answer.indexOf(letter) != -1;
if (isHit) {
hits += letter;
} else {
misses += letter;
}
return isHit;
}
In the string "letter" you have matching e, r and t. So that is why you are getting the results you are. Good luck and keep going to finish the game.
![Yashvir Sangha](https://secure.gravatar.com/avatar/e904339da5b186a814cff34a0e504499?s=96&d=https%3A%2F%2Fecs-static.teamtreehouse.com%2Fassets%2Fcontent%2Fdefault_avatar-445fbbabfc8dc9188fb5967fe43322ee0c3e0dd1e10f378bf8343784af5a13eb.webp&r=pg)
Yashvir Sangha
620 PointsThanks so much! I spent so much time trying to figure it out.