Welcome to the Treehouse Community

The Treehouse Community is a meeting place for developers, designers, and programmers of all backgrounds and skill levels to get support. Collaborate here on code errors or bugs that you need feedback on, or asking for an extra set of eyes on your latest project. Join thousands of Treehouse students and alumni in the community today. (Note: Only Treehouse students can comment or ask questions, but non-students are welcome to browse our conversations.)

Looking to learn something new?

Treehouse offers a seven day free trial for new students. Get access to thousands of hours of content and a supportive community. Start your free trial today.

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!