Correct the existing hasTile method to return true if the tile is in the tiles field, and false if it isn't. You can sol

Question

whats wrong in my code

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) {
    tiles+=tile;

  }

  public boolean hasTile(char tile) {
    boolean isTile=tiles.indexOf(tile);
    if(isTile>=0){
      return isTile;
    }
}

Answer 1 · 2018-08-14T15:25:44Z

August 14, 2018 3:25pm

First off, it looks like your hasTile method is missing its closing brace. Secondly I get the following error:

./ScrabblePlayer.java:19: error: incompatible types: int cannot be converted to boolean
    boolean isTile=tiles.indexOf(tile);
                                ^
./ScrabblePlayer.java:20: error: bad operand types for binary operator '>='
    if(isTile>=0){
             ^
  first type:  boolean
  second type: int
2 errors

This is because you're trying to set a boolean to an integer value. Remember that the indexOf method will return -1 if an item is not found and >= 0 if it finds an item. So instead we can evaluate the boolean as such:

public boolean hasTile(char tile) {
    return (tiles.indexOf(tile) != -1);
}

What this says is this: If the index of tile char in the tiles String is not equal to -1, it must be in the tiles String. It has the tile so we return true. Otherwise, we return false because -1 means that the tile char is not present in the tiles String.

Hope that helps!

Welcome to the Treehouse Community

Looking to learn something new?

omkar chendwankar

omkar chendwankar

Correct the existing hasTile method to return true if the tile is in the tiles field, and false if it isn't. You can sol

1 Answer

Cam Richardson

Cam Richardson