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 trialAllen Emejuobi
5,009 PointsJava Objects IllegalArgumentException Challenge
I can't figure out how to work this code right. Can someone please help me out?
class GoKart {
public static final int MAX_BARS = 8;
private String color;
private int barCount;
private int lapsDriven;
public GoKart(String color) {
this.color = color;
}
public String getColor() {
return color;
}
public void charge() {
barCount = MAX_BARS;
}
public boolean isBatteryEmpty() {
return barCount == 0;
}
public boolean isFullyCharged() {
return MAX_BARS == barCount;
}
public void drive() {
drive(1);
}
public void drive(int laps) {
lapsDriven += laps;
barCount -= laps;
if(barCount < 0){
throw new IllegalArgumentException("Not enough bars for another lap!");
}
}
}
1 Answer
Joseph Wasden
20,406 PointsYou are pretty close!
you are on the right track with your if statement, but we are more concerned if the barcount is going to be less then zero, and not so much if it already is, right? We really ought to compare the bar count to the number of laps attempted to drive, and then throw the exception BEFORE we actually register any laps driven or any bars of energy spent in our goKart.
So, if you change the if statement to compare the requested number of laps, and if you only increment our member variables of laps driven and bar count AFTER checking to make sure the number of laps can actually be driven, you've got a goKart even Mario would be proud to drive.
See the code below.
public void drive() {
drive(1);
}
public void drive(int laps) {
if (barCount < laps) {
throw new IllegalArgumentException();
} else {
drive(laps);
}
lapsDriven += laps;
barCount -= laps;
}
Allen Emejuobi
5,009 PointsAllen Emejuobi
5,009 PointsThank you, it worked! I thought I needed to do try/catch.