Need help for that exercise

Question

I just wrote lines 21 - 29 in the for loop exercise, but the systems keeps asking me if I forgot to create the getTileCount method, but I created it... and there is no compile errors, so I assume that the syntax is correct.

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 () {
    int number = 0;
    for (char letter: mHand.toCharArray()) {
      if (mHand.indexOf(letter) >=0) {
      number ++;
      }
    }
    return number;
  }
}

Answer 1 · 2015-09-24T23:16:23Z

September 24, 2015 11:16pm

Hey Pierre,

Your code right now is doing the following: for each of the letters we have right now, match it to the letters that we have, and increase the count. What your doing is basically comparing your mHand to your mHand.

for (char letter: mHand.toCharArray()) {
      if (mHand.indexOf(letter) >=0) {
      number ++;
      }

In case you still cant see it, your looping through all the characters of your mHand in the first line and then comparing it to your mHand in the second line.

What the challenge is asking is that it wants to know the count of a specific tile in your hand. That would mean you have to pass in a char into that method. Therefore your method parameter should look like this:

public int getTileCount (Char tile)

We will call that method and put a letter in it and then check if we have that letter in your mHands. I believe you just read the question wrong as that what happened to me when I first did this. In case you dont know what to do next, I will show you.

So you got everything pretty much correct except lets remove this line:

if (mHand.indexOf(letter) >=0)

We actually dont need this. When we loop through all the characters in our mHands, we just have to check if it matches the letter that we passed in. We can do this simple by using ==

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

The above is the complete answer to the challenge. I hope that made sense and happy coding!

Kevin

Answer 2 · 2015-09-24T23:42:11Z

September 24, 2015 11:42pm

Hey thank you very much, your explanations are very clear. You are right, I might have misunderstood the question. All is alright now, thanks. Pierre.

Welcome to the Treehouse Community

Looking to learn something new?

Pierre-Alexandre GROSS

Pierre-Alexandre GROSS

Need help for that exercise

2 Answers

Kevin Faust

Kevin Faust

Pierre-Alexandre GROSS

Pierre-Alexandre GROSS

Kevin Faust

Kevin Faust