Creating a "Battery Charger"

[Hyperrr]

Hey everyone! First post here at java forums! As im sure you have all been as frustrated as I am right now, lets get into it.
What I am doing is I have a "battery" class and a "batterycharger" class. The idea is to be able to put these battery objects into the batterycharger. My question is, is the way i use the method "putBattery" correct? And how would I create the method "removeBattery" to hold the reference of what is it battery charger. Oh and the battery charger can only hold one battery at a time.

public class BatteryCharger {
	public Battery bat, removeBattery;
	public boolean ok, putBattery;
	public boolean occupied;	
	public static int hours;
	public int numberOfHours;
		
	public BatteryCharger(){
		int inUse;
		occupied = false;
		if(occupied = true){
			bat.currentLevel = bat.currentLevel + 50;
			inUse = 1;}
	else{
			inUse = 0;
			bat = null;
				}
	}	
	public static void passTime(int numberOfHours){
		String input = JOptionPane.showInputDialog(null, "How many hours should pass?").trim();
		numberOfHours = Integer.parseInt(input);
		numberOfHours = hours;
	}
	public Battery removeBattery(){
			boolean ok;
			if(bat == ){
				
			}
	}

	public  boolean putBattery(Battery theBat){
		boolean ok;
		if(bat == null){
			bat = theBat;
			ok = true;
		}
		else{
			ok = false;}
		return ok;
	}

	public String getDescription(){
		System.out.println("This batter charger");
	}

	public void charge(){
		if(occupied = false){
			;}
		else{
			if(occupied = true){
			numberOfHours = numberOfHours + 1;
			}
		}
			
		
	}
}

Appreciate the help!
-Hyper

[Junky]

The removeBattery method should return the Battery object (actually a reference to the Battery object) and set the instance variable to null. However, you cannot set the instance variable to null after the return statement as it will not be reachable (think about that). You also cannot set the instance variable to null before the return statement otherwise it will return null. So how do you fix this conundrum? Use a temp variable.

Now for some nitpicks.

public Battery bat, removeBattery;
public boolean ok, putBattery;

removeBattery and putBattery are methods. Why have you declared them as instance variables? Do you plan on using these variables later in your code? If so then use different names as it leads to confusion if you give classes, methods and variabels the same name.

if(occupied = true)

Contrast this if statement to the one where you compare bat to null. In any case when dealing with booleans there is no need to use comparison.

if( bool ) {
if( ! bool ) {

[Hyperrr]

appreciate the speedy reply! The reason I have

if(occupied = true)

is because the charger can be in use or not, and if it is in use, it adds +50 power to the battery in it. Am I following correctly? I am very new to java so still learning. Also, what is this?

if( bool ) {
if( ! bool ) {

-Mike

[Junky]

And the reason I mentioned it was because you are using assignment (one equals) and not comparison (two equals). Also the code snippet I posted is the preferred way of using booleans in conditional statements. A boolean is already true or false so there is no need to use them in a conditional statement, just use them.