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

Edwin Amaya
PLUS
Edwin Amaya
Courses Plus Student 809 Points
Posted by Edwin Amaya
Edwin Amaya
Courses Plus Student 809 Points

Getting a weird error "java.lang.reflect.invation.target.exception error

this is all the code i added to the challenge code and i keep getting this error , the workspace shows nothing when i go search for the error , no line number or anything... so i am at a loss. i have reviewed the code and it all seems like it should work..so maybe a second set of eyes would help here

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(char tile)
  {
    int counter = 0; 
    for(char tileCount : mHand.toCharArray())
    {

      if(mHand.charAt(tileCount) == tile)
       counter++;

    }
    return counter;
  }

}

1 Answer

Dan Johnson
Dan Johnson
40,533 Points
Dan Johnson
Dan Johnson
40,533 Points

Using the for each loop, each pass your variable tileCount is being assigned the next character in the String. So all you need to do to check if you need to increment is this:

if(tileCount == tile)
    counter++;

Since you're checking against the type of tile, not it's position. You may want to change the variable name to reflect that as well.