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

Mark Miller
Mark Miller
45,831 Points
Posted by Mark Miller
Mark Miller
45,831 Points

I have no question except, "Why does the compiler not accept my code?"

public int getTileCount() { int count = 0; for (char tile: mHand.indexOf(tile) >= 0) { count += 1; } return count; }

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 count = 0;
    for (char tile: mHand.indexOf(tile) >= 0) {
      count += 1;
    }
    return count;
  }
}

2 Answers

Geovanie Alvarez
Geovanie Alvarez
21,500 Points
Geovanie Alvarez
Geovanie Alvarez
21,500 Points

In your method getTileCount() need to accepts a char then the for loop (for (char tile: mHand.indexOf(tile) >= 0) {) i think is invalid, try this code

public int getTileCount(char tile) { // accepts a char
    int count = 0;
    for (char tile1: mHand.toCharArray()) { // for loop
      if(tile1 == tile){
        count += 1;
      }
    }
    return count;
  }
Mark Miller
Mark Miller
45,831 Points
Mark Miller
Mark Miller
45,831 Points

Thanks for the help. I did not realize that the first declaration inside the 'for each' parenthesis is nothing more than a labeling of the items following the colon. That enables further working inside the curly braces, such as the test for equality that you are showing me in your code. I see now. Thanks.

There is no counter in this method. It is really just a flag of a Boolean type. It can be either zero or one, but will not add up because only one char can be passed in, and count is initialized to zero every time.