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
Justin Covell
1,148 PointsMake sure you throw the exception before changing the mBarsCount field.
I keep getting an error telling me to throw the exception before changing the mBarsCount field. Im sure Im doing that. What have I done 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(1);
}
public void drive(int laps) {
// Other driving code omitted for clarity purposes
int enoughCharge = mBarsCount -= laps;
if (enoughCharge < 0) {
throw new IllegalArgumentException("Not enough battery remains");
}
mBarsCount = enoughCharge;
}
public void charge() {
while (!isFullyCharged()) {
mBarsCount++;
}
}
public boolean isBatteryEmpty() {
return mBarsCount == 0;
}
public boolean isFullyCharged() {
return mBarsCount == MAX_BARS;
}
}
2 Answers
Ken Alger
Treehouse TeacherJustin;
You have one minor syntax issue with your code. Why are you using an assignment operator on mBarsCount when you are initializing your enoughCharge variable? That is what is giving you the error message.
Happy coding,
Ken
ISAIAH S
1,409 PointsYou're getting that message because the challenge doesn't need a new variable, it wants you to throw the IllegalArgumentException. So instead of int enoughCharge = mBarsCount -= laps; mBarsCount = enoughCharge;, try
if(laps > mBarsCount) {
throw new IllegalArgumentException("Not enough battery remains");
}
mBarsCount -= laps;