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

Java Java Objects (Retired) Harnessing the Power of Objects Throwing Exceptions

Posted by Shree Devi
Shree Devi
2,121 Points

How am I throwing the exception wrong?

help.

GoKart.java 
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

    if(mBarsCount<0){
             mBarsCount -= laps;
      throw new IllegalArgumentException("Not enough battery remains") ;   

  }
  }

  public void charge() {
    while (!isFullyCharged()) {
      mBarsCount++;
    }
  }

  public boolean isBatteryEmpty() {
    return mBarsCount == 0;
  }

  public boolean isFullyCharged() {
    return mBarsCount == MAX_BARS;
  }

}

2 Answers

Jay Wellings
Jay Wellings
4,593 Points
Jay Wellings
Jay Wellings
4,593 Points

You have the logic as little muddled.

You need to decrease the battery when we have enough batter left, if we dont then we throw the exception,

Right now you code works as such: If i have more than 0 laps, remove a battery section and throw an error.

if(mBarsCount == 0){ throw new IllegalArgumentException("Not enough battery remains") ;
} else { mBarsCount --; }

Phillip Hurst
Phillip Hurst
4,204 Points
Phillip Hurst
Phillip Hurst
4,204 Points 
if(mBarsCount<0){
             mBarsCount -= laps;
      throw new IllegalArgumentException("Not enough battery remains") ;   

  }

My translation of your logic: If battery is dead, subtract a lap count from the battery charge and give error message.

How I would say it: If battery has power, subtract a charge count. if battery does not have power, give error message

if (mBarsCount > 0) {
    mBarsCount--;
} else {
    throw new IllegalArgumentException("Not enough battery remains");
}