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 (Retired) Creating the MVP For Each Loop

Scrabble problem

So, I am supposed to create a method that returns how many tiles are in the Hand, using a for each loop, and to the best of my knowledge, I have done that. But I guess I am doing something wrong. Any help would be appreciated.

ScrabblePlayer.java
public class ScrabblePlayer {
  private String mHand;

  public ScrabblePlayer() {
    mHand = "";
  }

  public String getHand() {
   return mHand;
  }

  public void addTile(char tile) {
    // Adds the tile to the hand of the player
    mHand += tile;
  }

  public boolean hasTile(char tile) {
   return mHand.indexOf(tile) > -1;
  }
  public String getTileCount(){
    int tileCount = 0;
      for(char tile: mHand.toCharArray()){
        if (mHand.hasTile){
          tileCount ++;
        }
      }
    return tileCount;
  }
}

2 Answers

For starters you have your method returning a String when it supposed to return an int. You also needed to add a char parameter to your method. Here is what it should look like:

public int getTileCount(char tile){
    int tileCount = 0;
      for(char letter: mHand.toCharArray()){
        if (letter == tile){
          tileCount++;
        }
      }
    return tileCount;
  }

You need to be able to pass in a tile so you can loop through your mHand and count how many of those you have. Then you need to return the number of tiles that you have. so your method heading should be public int getTileCount(char tile){ return number of tiles passed in}

Thanks!