Welcome to the Treehouse Community

The Treehouse Community is a meeting place for developers, designers, and programmers of all backgrounds and skill levels to get support. Collaborate here on code errors or bugs that you need feedback on, or asking for an extra set of eyes on your latest project. Join thousands of Treehouse students and alumni in the community today. (Note: Only Treehouse students can comment or ask questions, but non-students are welcome to browse our conversations.)

Looking to learn something new?

Treehouse offers a seven day free trial for new students. Get access to thousands of hours of content and a supportive community. Start your free trial today.

Java Java Objects Creating the MVP Scrabble Tiles

Can someone please help me solve this?

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.

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
    boolean hasTile = tiles.indexOf(tile) >= 0;
    if(hasTile) {
      tiles += tile;
    }
    return true;


    return false;

  }

}
Patricia Hector
Patricia Hector
42,901 Points

Be aware that in your method hasTile(..) you have two returns, one next to the other, which means that your function will always exit from the first return, being your function always TRUE, and shadowing the second return because that one will never run.

3 Answers

Patricia Hector
Patricia Hector
42,901 Points
 public boolean hasTile(char tile) {
    // TODO: Determine if user has the tile passed in
    boolean hasTile = tiles.indexOf(tile) >= 0;
    return hasTitle;
}

Or even shorter

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

Thank you Patricia!

You can simply return tiles.indexOf(tile) >= 0;

imo
imo
7,902 Points
// You were so close
// the return True needs to be inside the if statement, like this
 public boolean hasTile(char tile) {
    // TODO: Determine if user has the tile passed in
    boolean hasTile = tiles.indexOf(tile) >= 0;
    if(hasTile) {
      tiles += tile;
      return true;
    }
    return false;
  }