Beginner Droid Project

[Bp Parker]

Hello again coderanch. I've been using instructions to write this
Droid code & I can't seem to get a few things right. I went over
it a couple times & can't figure out where or what i'm doing wrong.
Here's the code

public class Droid {
  
  int batteryLevel;
  
  public Droid() {
    
    batteryLevel = 100;
    
  }
  
  public void activate() {
    System.out.println("Activated. How can I help you?");
  }
  
  batteryLevel = batteryLevel - 5;
  System.out.println("Battery level is " + batteryLevel + "Percent!");
}

  public void chargeBattery(int hours) {
    if (batteryLevel > 100) {
      batteryLevel = 100;
      System.out.println(batteryLevel);
    }
    else {
    System.out.println("Droid Charging...");
  }
 
      batteryLevel = batteryLevel + hours;

  }

   public int checkBatteryLevel() {
     System.out.println("The Battery Level is " + batteryLevel + "Percent!");
   }
return batteryLevel;

   public void hover(int feet) {
     if (feet > 2) {
       System.out.println("Error! cannot hover above 2 feet.");
     }
     else {
       System.out.println("Hovering...");
     }
     batteryLevel = batteryLevel - 20;
     System.out.println(batteryLevel);
   }

    public static void main(String[] args) {
      
      Droid sample = new Droid();
      sample.activate;
      sample.chargeBattery(5);
      sample.hover(1);
    }
}

Any help would be greatly appreciated. Thanks!

[Tushar Goel]

What problem you are facing? It is easy to start with that instead of guessing..

[Bp Parker]

Tushar Goel wrote:What problem you are facing? It is easy to start with that instead of guessing..

It's solved now Tushar Goel, but earlier I kept getting multiple line errors
on the code. Come to find out everytime I told the system to

System.out.println();

I would close it out with a

 } 

wich kept closing my methods.

[Liutauras Vilda]

Welcome to the Ranch.

Please post your fixed code, so we could make sure you actually got it fixed, as at the moment I'm finding more problems than you pointed out.

[Bp Parker]

Working Code:

public class Droid {
  int batteryLevel;
  
  public Droid() {
    batteryLevel = 100;
  }
  
  public void activate() {
    System.out.println("Activated. How may I help you?");
    batteryLevel = batteryLevel - 5;
    System.out.println("Battery Level is: " + batteryLevel + " Percent!");
  }
  
  public void chargeBattery(int hours) {
    if (batteryLevel > 100) {
      batteryLevel = 100;
      System.out.println(batteryLevel);
    }
    else {
    System.out.println("Droid charging...");
    batteryLevel = batteryLevel + hours;
 
  }
  }
 
   public int checkBatteryLevel() {
     System.out.println("The Battery Level is " + batteryLevel + "Percent!");
 
return batteryLevel;
   }
 
   public void hover(int feet) {
     if (feet > 2) {
       System.out.println("Error! cannot hover above 2 feet.");
     }
     else {
       System.out.println("Hovering...");
     
     batteryLevel = batteryLevel - 20;
     System.out.println(batteryLevel + "%");
   }
   }
 
    public static void main(String[] args) {
       
      Droid sample = new Droid();
      sample.activate();
      sample.chargeBattery(5);
      sample.hover(1);
    }
}

[Liutauras Vilda]

1. Keep batteryLevel as private variable. Always like that, otherwise someone can drain your battery by directly accessing instance variable.
2. method chargeBattery. Need to keep phone for 100 hours to get it charged from 0 to 100?
3. same method as above, hmm, it doesn't look good to me. More than 100% could be...

public void chargeBattery(int hours) {
    if (batteryLevel > 100) {
        batteryLevel = 100;
        System.out.println(batteryLevel);
    }
    else {
        System.out.println("Droid charging...");
        batteryLevel = batteryLevel + hours;
    }
}