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

Java Java Objects Creating the MVP Scrabble Tiles

Posted by trae rhodes
trae rhodes
6,008 Points

char

Okay great, now can you fix the hasTile method for me, right now it always returns false.

Correct the existing hasTile method to return true if the tile is in the tiles field, and false if it isn't. You can solve this a few ways, however, I'd like you to practice returning the result of the expression that uses the index of a char in a String.

Bummer! There is a compiler error. Please click on preview to view your syntax errors!

ScrabblePlayer.java 
public class ScrabblePlayer {
  // A String representing all of the tiles that this player has
  private String tiles;

  public ScrabblePlayer() {
    tiles = "";
  }

  public String getTiles() {
    return tiles;
  }

  public void addTile(char tile) {
    // TODO: Add the tile to tiles
    tiles += tile;


  }

  public boolean hasTile(char tile) {
    // TODO: Determine if user has the tile passed in
 if (tiles.indexOf(tile) >= 0) {
 return true; 
 }else{ 
 return false;
    }
   }
 }

1 Answer

Pedro Cabral
Pedro Cabral
33,586 Points
Pedro Cabral
Pedro Cabral
33,586 Points

Your code is fine, but the challenge wants you to follow what the error message says: While you could definitely solve this using an if statement, try returning the result of the expression.. Remember that the if block itself evaluates to true or false. What you have at the moment is something like: "If this condition is true, then return true, else return false" when you should have something like: "Return whatever this condition evaluates to". Hope this sort of analogy makes it more clear.