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

Petter Jonasson
Petter Jonasson
808 Points
Posted by Petter Jonasson
Petter Jonasson
808 Points

Help pls.

Hi i need some help. I made it work but then it said that you could to it in this way but they wanted me to do in another way.

Okay great, now can you fix the hasTile method for me, right now it always returns false. 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 isHit = tiles.indexOf(tile) != -1;
    if (isHit) {
      System.out.println("True");
      return true;
    } else {
      System.out.println("False");
      return false;
    }
  }

}

1 Answer

Yanuar Prakoso
Yanuar Prakoso
15,196 Points
Yanuar Prakoso
Yanuar Prakoso
15,196 Points

Hi Petter

You can just be straightforward like:

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

String.indexOf(ch) if you look it up in the java doc returns an int. However, the whole tiles.indexOf(tile)!=1 statement is a boolean statement like you commonly used in IF statement. So it will returns boolean just like what the method wants.

I hope this help