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

Posted by priyamvad acharya
priyamvad acharya
Courses Plus Student 246 Points

How to solve this code challenge

So back to that ScrabblePlayer. I found that it's not enough to know if they just have a tile of a specific character. We need to know how many they actually have. Can you please add a method called getTileCount that uses the for each loop you just learned to loop through the characters and increment a counter if it matches? Return the count, please, thanks!

ScrabblePlayer.java 
public class ScrabblePlayer {
  private String mHand;
  char tile;

  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 int getTileCount() {

    for(char count:mHand.toCharArray()) {
      if(count==tile){
        count++;
      return count;
    }
    }
    return 0;
}
}

1 Answer

Paul Ryan
Paul Ryan
4,584 Points
Paul Ryan
Paul Ryan
4,584 Points 
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 int getTileCount(char tile){
    int counter = 0;
    for(char c: mHand.toCharArray()){
      if(c==tile){
        counter++;
      }
    }
    return counter;
  }
}

Couple points you are incrementing count++ but you declared count as a char. You need to declare an integer to keep track of the matches.

You also seem to be trying to return the count (which is a char) through each iteration of the loop. This should not be returned till the count has finished (also returned as an int).

Anymore questions, just ask.