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 trialjatinder kumar
Courses Plus Student 4,886 Pointswhats wrong with this code, its not getting passed
if(laps>mBarsCount) { throw new IllegalArgumentException("Not enough battery remains"); }
public class GoKart {
public static final int MAX_BARS = 8;
private String mColor;
private int mBarsCount;
public GoKart(String color) {
mColor = color;
mBarsCount = 0;
}
public String getColor() {
return mColor;
}
public void drive() {
drive(1);
}
public void drive(int laps) {
// Other driving code omitted for clarity purposes
mBarsCount -= laps;
if(laps>mBarsCount)
{
throw new IllegalArgumentException("Not enough battery remains");
}
}
public void charge() {
while (!isFullyCharged()) {
mBarsCount++;
}
}
public boolean isBatteryEmpty() {
return mBarsCount == 0;
}
public boolean isFullyCharged() {
return mBarsCount == MAX_BARS;
}
}
1 Answer
Jennifer Nordell
Treehouse TeacherHi there! It's not so much that the code is wrong but rather that it's a little out of order. In fact, you're so close here I'm not even going to give the direct answer. I'm just going to tell you how the logic of that is working. Right now when the function executes the first thing it does is reduce the mBarsCount
. It does this before it even checks if there's enough battery Try putting that reduction to mBarsCount
in the proper place. You've got this!