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
Richard Groot
4,432 PointsNot understanding why this throw exception isn't working?
I just need help figuring out why this doesn't work and what I can do to, to make it work. When I run this I just get "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(10);
}
public void drive(int laps) {
// Other driving code omitted for clarity purposes
//mBarsCount -= laps;
int newAmount = mBarsCount + laps;
if (newAmount > MAX_BARS) {
throw new IllegalArgumentException("Not enough battery remains");
}
mBarsCount = newAmount;
}
public void charge() {
while (!isFullyCharged()) {
mBarsCount++;
}
}
public boolean isBatteryEmpty() {
return mBarsCount == 0;
}
public boolean isFullyCharged() {
return mBarsCount == MAX_BARS;
}
}
2 Answers
Rob Bridges
Full Stack JavaScript Techdegree Graduate 35,467 PointsHey Richard, your exception is fine you just need to clean the logic up to check and see if the number passed in for laps is higher is great than the current mBarsCount, next the only thing you need to do next is decrements the mBarsCount by the number of laps.
It would look something like below:
public void drive(int laps) {
if (laps > mBarsCount) {
throw new IllegalArgumentException("Not enough battery remains");
}
mBarsCount -= laps;
}
Richard Groot
4,432 PointsThank you so much, I understand why it needs to be done this way. Thanks again.