Incrementing a counter

Question

Now in your new method, have it return a number representing the count of tiles that match the letter that was passed in to the method. Make sure to check Example.java for some example uses.

I can not seem to get my counter to work.... please help me see what I'm doing wrong.

ScrabblePlayer.java

public class ScrabblePlayer {
  // A String representing all of the tiles that this player has
  private String tiles;

  public ScrabblePlayer() {
    tiles = "";
  }

  public String getTiles() {
    return tiles;
  }

  public void addTile(char tile) {
    tiles += tile;
  }

  public boolean hasTile(char tile) {
    return tiles.indexOf(tile) != -1;
  }

  public int getCountOfLetter(char letter){
   int count=0;
    for(char tile:tiles.toCharArray()){
        if(tiles.indexOf(letter)!=-1){
          count+=count;
        }
      return count;
    }

  }

}

Example.java

// This code is here for example purposes only
public class Example {

  public static void main(String[] args) {
    ScrabblePlayer player1 = new ScrabblePlayer();
    player1.addTile('d');
    player1.addTile('d');
    player1.addTile('p');
    player1.addTile('e');
    player1.addTile('l');
    player1.addTile('u');

    ScrabblePlayer player2 = new ScrabblePlayer();
    player2.addTile('z');
    player2.addTile('z');
    player2.addTile('y');
    player2.addTile('f');
    player2.addTile('u');
    player2.addTile('z');

    int count = 0;
    // This would set count to 1 because player1 has 1 'p' tile in her collection of tiles
    count = player1.getCountOfLetter('p');
    // This would set count to 2 because player1 has 2 'd'' tiles in her collection of tiles
    count = player1.getCountOfLetter('d');
    // This would set 0, because there isn't an 'a' tile in player1's tiles
    count = player1.getCountOfLetter('a');

    // This will return 3 because player2 has 3 'z' tiles in his collection of tiles
    count = player2.getCountOfLetter('z');
    // This will return 1 because player2 has 1 'f' tiles in his collection of tiles
    count = player2.getCountOfLetter('f');
  }
}

Answer 1 · 2017-03-26T10:30:08Z

March 26, 2017 10:30am

The correct answer is:

  public int getCountOfLetter(char letter){
   int count=0;
    for(char tile: tiles.toCharArray()){
        if(tile == letter){ //change made here
          count++; //equivalent to count += 1;
        }  
    }
    return count; //moved this outside loop
  }

You needed to return count outside the loop, needed to increment by 1 (not count), and your test was probably a mistake as it doesn't actually use the tile variable.

Welcome to the Treehouse Community

Looking to learn something new?

Nikolas Tamvakis

Nikolas Tamvakis

Incrementing a counter

1 Answer

Simon Coates

Simon Coates