Hjælp til construcor

public Heater() {
    temperature = 15;
increment = 5;
    }

Okay, jeg har gjort følgende:
Er det korrekt?
public class Heater
{
private int temperature;
private int min;
private int max;
private int increment;


    public Heater() {
    temperature = 15;
    }

    public Heater(int newMin, int newMax) {
        min = newMin;
        max = newMax;
    }
   
    public Heater(int newIncrement) {
    increment = 5;
    }
   
    /**
    * The temperature is 15.
    */
    public void setTemperature(int newTemperature) {
    temperature = newTemperature;
    }

    /**
    * The warmer increase the temperature.
    */
    public void warmer()
    { temperature = (temperature + increment );
    if(temperature > max) {temperature = max;}
    }

    /**
    * The cooler decrease the temperature.
    */
    public void cooler()
    { temperature = (temperature - increment);
    if(temperature < min) {temperature = min;}
    }

    /**
    * return the value of the temperature.
    */
    public int getTemperature()  {
    return temperature;
    }

    public void setIncrement(int newIncrement) {
        if(increment >= 0) {increment = newIncrement; }
    }

}

Når, jeg skal jo bare sætte temperatur = 15 og increment = 5 i constoctorne right?

ja ifølge det du sagde du ville have.
Det er hvad jeg har gjort i min kommentar:)

Tak for hjælpen :)

Skal de indsættes i

public Heater(int newMin, int newMax) {
        min = newMin;
        max = newMax;
    }

altså temperatur og increment?

du kan jo have 2 konstruktører.

så den du har lavet der og den her

public Heater() {
    temperature = 15;
increment = 5;
    }

Hvis meningen er at temperature og increment skal have værdierne 15 og 5 som default vil jeg anbefale:

public class Heater
{
private int temperature = 15;
private int min;
private int max;
private int increment = 5;


    public Heater() {
        // hvis du benytter denne constructor har min og max ingen værdi.
        // ? bør de evt også have en default ?
    }

    public Heater(int newMin, int newMax) {
        min = newMin;
        max = newMax;
        // nu har temperature og increment også en værdi hvis du instantierer
        // klassen med denne constructor.
    }
...

mvh JakobA

Som jeg læser spørgsmålet, SKAL temperature og increment sættes gennem constructor, og så er der vel kun én løsning

private int temperature;
private int min;
private int max;
private int increment;

    public Heater() {
        temperature = 15;
        increment = 5;
    }

    public Heater(int newMin, int newMax) {
        this();
        min = newMin;
        max = newMax;
    }

fassosa, Eksperten er baseret på princippet hjælp-til-selvhjælp. Jeg vil gerne hjælpe dig, men jeg vil ikke lave dine lektier for dig. Du skal m.a.o. selv levere noget - og så kan vi tage den derfra.

I øvrigt kan du tage et kig på de spørgsmål ole8 har gang i:

http://www.eksperten.dk/spm/797587
http://www.eksperten.dk/spm/797735
http://www.eksperten.dk/spm/797741
http://www.eksperten.dk/spm/797768

Det er vist ret sikkert at I går på det samme hold. :^)

Hov, fik postet et forkert sted...