Java Java Objects Creating the MVP Scrabble Tiles

Abhishek Patil
Abhishek Patil
2,708 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,900 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,900 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;
}

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

Stivan Radev
Stivan Radev
7,012 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;
  }