getter/setter Garage

[Vladyslav]

Hi,
könnt mir vll. jemand sagen was ich da falsch gemacht habe ?
Ich soll zwei Klassen erstellen, in der einen sind die Objekte für ein Auto, mit entsprechendem Konstruktor.
In der anderen die eigentliche Garage. Der Hubraum der Autos muss verglichen und das mit dem größten soll ausgegeben werden.

Soweit so gut.
Ich soll in der Methode setHubraum(int xy) als auch im Konstruktor sicher stellen, dass keine ungültigen Werte übergeben werden, nur ( x >=0 ).
Im Konstruktor soll im Falle einer ungültigen Aufgabe der Hubraum auf 1600 hoch gesetzt werden.

Ich dachte ich hätte es so gelöst, nur kommt bei mir in der Klasse Garage keine solche Fehlermeldung raus, bin gerade überfragt. Hätte jemand vll. ein Tipp wie ich es bewerkstellige, das in der Methode setHubraum() und im Konstrukt sicherstelle, dass die Eingabe gültig ist und im Falle einer falschen Eingabe der Hubraum in Konstruktor auf 1600 gesetzt wird ?

mfG

public class Auto {
    
    private String marke;
    private String farbe;
    private int ps;
    private int hubraum;

    
    public Auto(String marke, String farbe, int ps, int hubraum) // Konstruktor
    {
        this.marke = marke;
        this.farbe = farbe;
        this.ps    = ps;
        if (hubraum <= 0)
        {
            hubraum = 1600;
        }
        else
        {
        this.hubraum = hubraum;
        }
    }
    
    //getter - lesender Zugriff
    
    public String getMarke ()
    {
        return marke;
    }
    public String getFarbe()
    {
        return farbe;
    }
    public int getPS()
    {
        return ps;
    }
    public int getHubraum()
    {
        return hubraum;
    }
    
    //setter - schreibender Zugriff
    
    public void setMarke (String marke)
    {
        this.marke = marke;
    }
    public void setFarbe (String farbe)
    {
        this.farbe = farbe;
    }
    public void setPS (int ps)
    {
        this.ps = ps;
    }
    public void setHubraum (int hubraum)
    {
        if (hubraum <= 0)
        {
            System.out.println("Falsche Eingabe!");
            this.hubraum = hubraum;
        }
        else
        {
        this.hubraum = hubraum;
        }
    }
    
    
}

 

[LimDul]

Im Konstruktor:

public Auto(String marke, String farbe, int ps, int hubraum) // Konstruktor
    {
        this.marke = marke;
        this.farbe = farbe;
        this.ps    = ps;
        if (hubraum <= 0)
        {
            hubraum = 1600;
        }
        else
        {
        this.hubraum = hubraum;
        }
    }

Welche Variable wird im Fall kleiner gleich 0 auf 1600 gesetzt? Es ist nicht das Feld, sondern der übergebene Parameter. Sprich, dein Hubram vom Auto bleibt 0, da du das nicht setzt. Da fehlt ein this davor: this.hubraum = 1600;

Im Setter

public void setHubraum (int hubraum)
    {
        if (hubraum <= 0)
        {
            System.out.println("Falsche Eingabe!");
            this.hubraum = hubraum;
        }
        else
        {
        this.hubraum = hubraum;
        }
    }

Was machst du im Falll einer ungültigen Eingabe?
Erstens: Du gibst aus: "Falsche Eingabe". Das ist schon mal gut
Zweitens: Dann übernimmst du die falsche Eingabe. Warum? Die Zeile unter dem System.out muss weg, du willst die Eingabe ja in dem Fall eben nicht übernehmen.

 

[VfL_Freak]

Moin,

meinst Du dies "System.out.println("Falsche Eingabe!")" mit Fehlermeldung oder kommt was in der Javakonsole ??
Wenn ich es richtig sehe (und dies der gesamte Code ist) rufst Du "setHubraum" auch nicht auf !!

VG Klaus

 

[Vladyslav]

Vielen Danke !

 

[Vladyslav]

@ VfL_Freak nein es ist nicht der gesamte Code, habe noch ein Klasse mit dem Namen Garage.

public class Garage {
   
    public static void main (String []args )
    {
        int anzahlAutos = 3;
        int groessterHubraum = 0;
        String autoGroessterHubraum = "";
        Auto []autos = new Auto[3];
        autos[0] = new Auto ("Auto1","blau",50,5500);
        autos[1] = new Auto ("Auto2","rot",100,2000);
        autos[2] = new Auto ("Auto3","gelb",150, 1000);
       
        for ( int i = 0 ; i < anzahlAutos; i ++)
        {
            if (autos[i].getHubraum()>groessterHubraum)
            {
                groessterHubraum = autos[i].getHubraum();
                autoGroessterHubraum = autos[i].getMarke();
            }
        }
        System.out.println(autoGroessterHubraum+" mit "+ groessterHubraum+ "ccm");
    }
}

 

[VfL_Freak]

Ganz wundervoll
Bleibt immer noch die Frage, was Du mit "Fehlermeldung" meinst .....
Und auch in "Garage" rufst Du "setHubraum" auch nicht auf !!
VG Klaus

 

[Vladyslav]

Mit Fehlermeldung, war nur die Ausgabe mit System.out.println gemeint.

Das mit dem "setHubraum".
Mir ist unklar, wie ich es einbauen soll. setHubraum ist doch lediglich der lesende Zugriff und mit get kann ich die Werte doch ggf. verändern oder?

 

[VfL_Freak]

setHubraum ist doch lediglich der lesende Zugriff und mit get kann ich die Werte doch ggf. verändern oder?

nee, gernau andersrum !!
set setzt einen Wert
get liest einen Wert

 

[Vladyslav]

Ok.
Verstehe ich es richtig?
Um setHubraum richtig in der Klasse Java richtig zu implementieren, damit es Werte einließt. Würde ich eine Scanner in der Klasse Auto bei "public void setHubraum (int hubraum) " einsetzen.
In der Klasse Garage müssten, dann in der Konsole die Hubraumwerte manuell eingegeben werden?

 

[VfL_Freak]

Um setHubraum richtig in der Klasse Java richtig zu implementieren, damit es Werte einließt.

um Werte zu lesen benutzt du doch schon getHubraum() in der for-Schleife !!
mit setHubraum würdest Du die Werte setzen/schreiben können, die Du dann mit getHubraum() wieder ausliest

 

[Vladyslav]

danke

 

[temi]

Eine geschickte Lösung, die auch unnötige Codeverdoppelung vermeidet wäre diese:

Schreibe die Methode setHubraum(), im Prinzip so, wie du es bereits fast richtig gemacht hast.

Pseudocode: Wenn der Wert des Parameters kleiner oder gleich Null ist, dann setze die Instanzvariable hubraum auf 1600, ansonsten setze die Instanzvariable auf den Wert des Parameters.

Im Konstruktor rufst du einfach die Methode setHubraum() auf, als Aufrufparameter der Methode verwendest du den Parameter hubraum des Konstruktors. Damit wird die Grenze überwacht und der Wert korrekt gesetzt.

 

[LimDul]

Eine geschickte Lösung, die auch unnötige Codeverdoppelung vermeidet wäre diese:

Schreibe die Methode setHubraum(), im Prinzip so, wie du es bereits fast richtig gemacht hast.

Pseudocode: Wenn der Wert des Parameters kleiner oder gleich Null ist, dann setze die Instanzvariable hubraum auf 1600, ansonsten setze die Instanzvariable auf den Wert des Parameters.

Im Konstruktor rufst du einfach die Methode setHubraum() auf, als Aufrufparameter der Methode verwendest du den Parameter hubraum des Konstruktors. Damit wird die Grenze überwacht und der Wert korrekt gesetzt.

Dann sollte die setMethode aber nicht public sein - public Methoden würde ich vermeiden aus dem Konstruktor aufzurufen.

Ich würde - wenn wie normalerweise - die set Methode public sein soll, eine Methode setHubraumInternal schreiben, die private ist und die Logik wie oben beschrieben hat. Der Konstruktor und der public setter rufen dann nur diese Methode auf, delegieren also dahin.