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 Abhishek Patil
Abhishek Patil
3,329 Points

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,902 Points
Patricia Hector
Patricia Hector
42,902 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,902 Points
Patricia Hector
Patricia Hector
42,902 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;
}
Abhishek Patil
Abhishek Patil
3,329 Points

Thank you Patricia!

Mei Lee
Mei Lee
3,339 Points
Mei Lee
Mei Lee
3,339 Points

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

ygh5254e69hy5h545uj56592yh5j94595682hy95
ygh5254e69hy5h545uj56592yh5j94595682hy95
7,934 Points
ygh5254e69hy5h545uj56592yh5j94595682hy95
ygh5254e69hy5h545uj56592yh5j94595682hy95
7,934 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;
  }