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 trialJeremy Nash
Courses Plus Student 2,250 Pointsillegal argument exceptions
Okay, so let's throw an IllegalArgumentException from the drive method if the requested number of laps cannot be completed on the current battery level. Make the exception message "Not enough battery remains".
Bummer! Are you sure you threw the IllegalArgumentException if the requested amount of laps would make the battery less than zero?
I've also added the drive(10); to my iae sting and gotten a bummer try again im not quite sure where i've gone wrong
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(10);
}
public void drive(int laps) {
// Other driving code omitted for clarity purposes
int numberOfLaps = mBarsCount + laps;
if (numberOfLaps > MAX_BARS) {
throw new IllegalArgumentException("Not enough battery remain.");
}
mBarsCount = numberOfLaps;
}
public void charge() {
while (!isFullyCharged()) {
mBarsCount++;
}
}
public boolean isBatteryEmpty() {
return mBarsCount == 0;
}
public boolean isFullyCharged() {
return mBarsCount == MAX_BARS;
}
}
1 Answer
Chase Marchione
155,055 PointsHi Jeremy,
Currently, you're checking if the number of laps is greater than the number of bars if the battery is fully charged. What you'll want to do is rewrite that if statement so that what's actually being checked is the amount of bars remaining.
Thus,
if (numberOfLaps > MAX_BARS) {
should be rewritten as:
if (numberOfLaps > mBarsCount) {
Hope this helps!
Jeremy Nash
Courses Plus Student 2,250 PointsJeremy Nash
Courses Plus Student 2,250 Pointsthanks for the help i ended up figuring it out yesterday