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

Posted by constantind
constantind
1,853 Points

What am I doing wrong with my counter? It counts 8 instead of 1...

public int getTileCount(char tile) { int counter = 0; for (char letter : mHand.toCharArray()) { if(mHand.indexOf(tile) >= 0) { counter++; } } return counter; }

2 Answers

Dhruv Patel
Dhruv Patel
8,287 Points
Dhruv Patel
Dhruv Patel
8,287 Points

Instead of using indexOf, what we are trying to do is check for a match. Here's what i did.

public int getTileCount(char tile) { int count = 0; for(char myTile: getHand().toCharArray()) { if(myTile == tile) { count++; } } return count; }

constantind
constantind
1,853 Points
constantind
constantind
1,853 Points

Thanks a lot. It worked :)