Java Java Objects Creating the MVP Counting Scrabble Tiles

kian christensen
kian christensen
744 Points

What am i doing wrong

this one is driving me crazy

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){
   for (char count : tiles.toCharArray()) {
    if (tiles.indexOf(count) != -1) {
      count = letter;
      }

   }
    return 0;
  }
}
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');
  }
}

1 Answer

Steven Parker
Steven Parker
200,698 Points

There are a few issues here, here are some hints:

  • you need a separate variable to keep the count from the one used for the tiles
  • you won't need "indexOf" for this, just compare each tile to the "letter"
  • for each match, add one to the count
  • at the end, return the count instead of 0
kian christensen
kian christensen
744 Points

thank you, yeah i got stuck trying to over complicate it

Steven Parker
Steven Parker
200,698 Points

kian christensen — Glad to help. You can mark a question solved by choosing a "best answer".
And happy coding!