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 Henrique Teodoro
Henrique Teodoro
815 Points

What i did wrong here?

The introduction to the challenge was:

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;

  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) {
    String letters = "";
    int numberOfTiles = 0;

    for (char tilesLetters : mHand.toCharArray()) {
      letters += tilesLetters;

      if (letters.indexOf(tile) >= 0) {
        numberOfTiles += 1;
      }
    }

    return numberOfTiles;

  }
}

1 Answer

Christopher Augg
Christopher Augg
21,223 Points
Christopher Augg
Christopher Augg
21,223 Points

Henrique,

I will try and give you some hints and see if that helps first.

       public int getTileCount(char tile) { //notice we are passing in a character here named tile.
       String letters = ""; //This String is not needed as we do not need to rebuild the String
       int numberOfTiles = 0; 

       for (char tilesLetters : mHand.toCharArray()) {
            letters += tilesLetters; //This concatenation is not needed. See above.

           //We need to compare the tile being passed into the method with the tile in tilesLetters (tile == tilesLetters)     
           if (letters.indexOf(tile) >= 0) { 
              numberOfTiles += 1; //this is fine but you can also use numberOfTiles++;
           }
        }

       return numberOfTiles;

     }

Hope this helps.

Regards,

Chris

Henrique Teodoro
Henrique Teodoro
815 Points

Thank you Chris! it worked pretty well :)