Counting Scrabble tiles HELP

Question

Why I can't pass this lesson with my code? What is wrong with him?

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 mCount = 0;
    char [] letters = tiles.toCharArray();
    mCount = letters.getCountOfLetter(letter);
   return mCount;  
  }


}

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-09T21:21:10Z

March 9, 2017 9:21pm

you are not using a loop. they want the enhanced for loop (type variable : collection) and then check that variable == letter in the loop. also don't put the charArray in a variable, loop directly through it (where i put collection in my syntax example).

Answer 2 · 2017-03-09T21:56:39Z

March 9, 2017 9:56pm

You are calling getCountOfLetter within itself. There is no need for this.

public int getCountOfLetter(char letter) {
    int mCount = 0;
    char [] letters = tiles.toCharArray();
    //LOOP through the char array while checking IF the tile is EQUAL to the inputted letter.
    //INCREMENT the count each time this is TRUE
   return mCount;  
  }

Start from here.

Welcome to the Treehouse Community

Looking to learn something new?

Rafael Almeida

Rafael Almeida

Counting Scrabble tiles HELP

2 Answers

james south

james south

Adrian Orta

Adrian Orta