Please, find the problem in this program.

Question

I have been trying to solve this exercise for more than an hour. For some reason, the compiler says:

Expected 0 but getting 1.

Please, check if there is any error in my program. Your help would be really appreciated. Thanks in advance.

ScrabblePlayer.java

public class ScrabblePlayer {
  private String mHand;
  private int mCount;

  public ScrabblePlayer() {
    mHand = "";
    mCount = 0;
  }

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

Answer 1 · 2016-06-04T19:20:27Z

June 4, 2016 7:20pm

Your function will work as written ... exactly one time. You have your mCount variable declared as a property of the ScrabblePlayer, so think about what will happen if you call the getTileCount() function twice. (This is probably how the challenge is being evaluated, because again, it will work correctly once.) The first time it will count how many matching tiles there are, and increment mCount by that much. So say you have two matches, after getTileCount() is run, mCount will be equal to 2.

Now when you run the function again on a new letter, the function starts but mCount is already = 2. So you will actually be adding however many matches occur in this run of the function to the amount of matches in the last function run.

You can fix this by simply adding the line

mCount = 0;

to the start of the function. However, there really is no reason to have mCount declared outside the scope of the function, so it is preferable to just declare and initialize mCount inside the function. Look at my solution below:

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

Hope that helps, let me know if you don't understand anything!

Answer 2 · 2016-06-05T08:20:03Z

June 5, 2016 8:20am

Thank you, again. Have a nice day.

Welcome to the Treehouse Community

Looking to learn something new?

Naveed Sarwar

Naveed Sarwar

Please, find the problem in this program.

2 Answers

Katherine Marino

Katherine Marino

Naveed Sarwar

Naveed Sarwar

Katherine Marino

Katherine Marino

Naveed Sarwar

Naveed Sarwar

Katherine Marino

Katherine Marino