Welcome to the Treehouse Community

The Treehouse Community is a meeting place for developers, designers, and programmers of all backgrounds and skill levels to get support. Collaborate here on code errors or bugs that you need feedback on, or asking for an extra set of eyes on your latest project. Join thousands of Treehouse students and alumni in the community today. (Note: Only Treehouse students can comment or ask questions, but non-students are welcome to browse our conversations.)

Looking to learn something new?

Treehouse offers a seven day free trial for new students. Get access to thousands of hours of content and a supportive community. Start your free trial today.

Java Java Objects (Retired) Harnessing the Power of Objects Incrementing

public boolean isFullyCharged(){while(!isFullyCharged){mBarCount++;} return isFullyCharged();

the compiler is giving an error. what I'm making wrong with this part of the program thk guys

GoKart.java
public class GoKart {
  public static final int MAX_ENERGY_BARS = 8;
  private String mColor;
  private int mBarsCount;

  public GoKart(String color) {
    mColor = color;
    mBarsCount = 0;
  }

  public String getColor() {
    return mColor;
  }

  public void charge() {
    mBarsCount = MAX_ENERGY_BARS;
  }

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

  public boolean isFullyCharged() {
  // charge == false;
    while( !isFullyCharged()){ mBarsCount++;}   return isFullyCharged();
  }


}

3 Answers

You don't need to return anything, so the return type should be void and there should be no return statement. Further, you shouldn't change the name of the method. The rest is fine:

  public void charge() {
    while (!isFullyCharged()) {
      mBarsCount++;
    }
  }
Aaron Kaye
Aaron Kaye
10,948 Points

You should try and follow what the loop isFullyCharged would do. You are never returning a boolean but you are probably getting a stack overflow. It sounds like you should be checking if the battery is fully charged, not trying to charge it. Look at the isBatteryEmpty() method for a clue!

Paul Cox
Paul Cox
12,671 Points

It seems you are recursively calling the isFullyCharged function within isFullyCharged leading to an infinite loop that will lead to a stack overflow.