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 trialShubham Modi
5,799 PointsI dont know what to do
Are you sure you threw the IllegalArgumentException if the requested amount of laps would make the battery less than zero?
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(MAX_BARS);
}
public void drive(int laps) {
// Other driving code omitted for clarity purposes
int newLaps = laps - mBarsCount;
if(newLaps < mBarsCount)
{
throw new IllegalArgumentException("Not Enough battery remains");
}
mBarsCount = newLaps;
laps -=mBarsCount;
}
public void charge() {
while (!isFullyCharged()) {
mBarsCount++;
}
}
public boolean isBatteryEmpty() {
return mBarsCount == 0;
}
public boolean isFullyCharged() {
return mBarsCount == MAX_BARS;
}
}
1 Answer
Allan Clark
10,810 PointsWhen we drive a number of laps we want to take the number of bars left and subtract the laps from it. It looks like you have the reverse operation, also subtracting again at the bottom. As a suggestion we can simplify your code by first comparing the number of laps we want to drive, if it is too many then we throw the exception, else do the subtraction. Something like this:
'''java
if(laps > mBarsCount) { //throw your exception }
mBarsCount -= laps;
'''