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 trialAbhishek Patil
3,329 PointsCan someone please help me solve this?
Correct the existing hasTile method to return true if the tile is in the tiles field, and false if it isn't. You can solve this a few ways, however, I'd like you to practice returning the result of the expression that uses the index of a char in a String.
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) {
// TODO: Add the tile to tiles
tiles += tile;
}
public boolean hasTile(char tile) {
// TODO: Determine if user has the tile passed in
boolean hasTile = tiles.indexOf(tile) >= 0;
if(hasTile) {
tiles += tile;
}
return true;
return false;
}
}
3 Answers
Patricia Hector
42,901 Points public boolean hasTile(char tile) {
// TODO: Determine if user has the tile passed in
boolean hasTile = tiles.indexOf(tile) >= 0;
return hasTitle;
}
Or even shorter
public boolean hasTile(char tile) {
// TODO: Determine if user has the tile passed in
return tiles.indexOf(tile) >= 0;
}
Abhishek Patil
3,329 PointsThank you Patricia!
Mei Lee
3,339 PointsYou can simply return tiles.indexOf(tile) >= 0;
ygh5254e69hy5h545uj56592yh5j94595682hy95
7,934 Points// You were so close
// the return True needs to be inside the if statement, like this
public boolean hasTile(char tile) {
// TODO: Determine if user has the tile passed in
boolean hasTile = tiles.indexOf(tile) >= 0;
if(hasTile) {
tiles += tile;
return true;
}
return false;
}
Patricia Hector
42,901 PointsPatricia Hector
42,901 PointsBe aware that in your method hasTile(..) you have two returns, one next to the other, which means that your function will always exit from the first return, being your function always TRUE, and shadowing the second return because that one will never run.