Need help for creating class

Question

i m designing a class named Fan to represent a fan. the class contains:
Three constants name slow, medium, and fast with values 1, 2 and 3 to denote the fan speed.
an int data field named speed that specifies the speed of fan(default slow)
an boolean data field named on that specifies whether the fan is on(default false)
a double data field named radius that specifies the radius of the fan(default 5)
a string data field named color that specifies the color of the fan(default blue)
so here is what i want, a method named toString(that returns the fan speed, color and radius in one combined string.if the fan is not on. the method returns fa n color and radius along with string "fan is off" in one combined string.
and a test program creates two fan objects assign maximum speed, radius 10, color yellow, and turn it on to the first object. assign medium speed, radius 5, color blue, and tun it off to the second object.
after i created my two objects this is my output
speed1 is slow, the fan is off, the radius is5.0Blue
speed2 is slow, the fan is off, the radius is5.0Blue
its not the result i want, i know there r somethin wrong with the if statement and the method..but i cannot fix it.
Thank everyone who can give me some tips or help.
here is my class
* To change this template, choose Tools | Templates
* and open the template in the editor.
package javaapplication1;
* @author Administrator
public class Fan {
    private int speed;
    private boolean on;
    private double radius;
    private String color;
    Fan(int speen, boolean on, double raidus, String color)
        this.speed = 1;
        this.on = false;
        this.radius = 5.0;
        this.color = "Blue";
    public int getSpeed()
        return speed;
    public void setSpeed(int speed)
        this.speed = speed;
    public boolean isOn()
        return on;
    public void setIsOn(boolean on)
        this.on = on;
    public double getRadius()
        return radius;
    public void setRadius(double radius)
        this.radius = radius;
    public String getColor()
        return color;
    public void setColor(String color)
        this.color = color;
    public String fanSpeed()
        String output = "";
       if(speed == 1)
           return output = "slow";
       else if(speed == 2)
           return output = "Medium";
       else
           return output = "Fast";
    public String IsFanon()
        String output = "";
        if(on == false)
            return output = "the fan is off";
        else
            return output = "the fan is on";
}Here is my main class
* To change this template, choose Tools | Templates
* and open the template in the editor.
package javaapplication1;
* @author Administrator
import javax.swing.JOptionPane;
public class testFan {
     * @param args the command line arguments
    public static void main(String[] args) {
        Fan fan1 = new Fan(3, true, 10.0, "yellow");
        Fan fan2 = new Fan(2, false, 5.0, "blue");
        System.out.println("speed1 is" + "  " +fan1.fanSpeed() + ",  " + fan1.IsFanon() + ", " + "the radius is" +fan1.getRadius() + fan1.getColor());
        System.out.println("speed2 is" + "  " + fan2.fanSpeed() + ",  " + fan2.IsFanon() + ", " + "the radius is" + fan2.getRadius() + fan2.getColor());
}

Answer 1

Peter__Lawrey wrote:
I would not suggest using a variable like "onoff". What would true mean, is it on or off or onoff (alternating)? e.g. an on-off relationship is one which is not clearly on or off for long.
You could just use an IDE to generate the constructor which would be simpler, however if there is no good reason to have two names for parameters and fields I would make them the same.I was wondering what a "speen" is.