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 kumbirai mangwanda
kumbirai mangwanda
1,628 Points

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

3 Answers

KRIS NIKOLAISEN
KRIS NIKOLAISEN
54,971 Points
KRIS NIKOLAISEN
KRIS NIKOLAISEN
54,971 Points

The challenge wants you to return a single expression. This should be easy enough to accomplish with what you have already:

return tiles.indexOf(tile) !=-1;

and eliminate the variable and if block does the same thing.

Steven Parker
Steven Parker
231,248 Points
Steven Parker
Steven Parker
231,248 Points

The error message gives you a clue: "Bummer: While you could definitely solve this using an if statement, try returning the result of the expression."

You are probably closer than you think. You've actually solved it on just one line, but you have added a good bit of extra code around it that isn't needed. Take a close look at your work and see if you can eliminate the "if" and meet the objective.

Bonus hint: you also won't need to create any variables.

Prince Chinyadza
Prince Chinyadza
6,333 Points
Prince Chinyadza
Prince Chinyadza
6,333 Points

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 return false; }

}