Kapselung override Setter

[der_roemer]

Halli hallo,

ich wurschtel mich gerade durch die Grundlagen von Vererbung und Kapselung.
Nun habe ich hier ein paar Klassen, welche vererbt werden:

Tier --> HundeArtigesTier --> Wolf, Hund
Tier --> KatzenArtigesTier --> Löwe, Hauskatze

Ich hab unter Tier das Gewicht deklariert, PRIVATE,
dazu hab ich einen Setter angelegt, PUBLIC,
der ein Gewicht zwischen 1 und 500 kg zulässt.

public class Tier {
    private int _gewicht;
    private int _groesse;
    
    public void setGewicht(int gewicht) {
        if (gewicht < 500 & gewicht > 1) {
            _gewicht = gewicht;    
        } else {
               System.out.println("Mit dem Gewicht des Tieres stimmt was nicht, sie wiegt "+ gewicht + " kg ???");
        }
        
    }
    
    public int getGewicht() {
        if (_gewicht > 0) {
            return _gewicht;    
        } else {
            System.out.println("Fehler beim Auslesen des Gewichtes");
            return 0;
        }
    }

Jetzt will ich den Setter in "Hauskatze" überschreiben,
weil eine solche sicher nicht 500 kg wiegen wird.
das Problem ist, ich komm nicht mehr an das PRIVATE aus Tier ran:

    @Override
    public void setGewicht(int gewicht) {
        // Setter für das Gewicht der Katze überschreiben
        if (gewicht < 15 & gewicht > 1) {
            _gewicht = gewicht;    
        } else {
            System.out.println("Mit dem Gewicht der Katze stimmt was nicht, sie wiegt "+ gewicht + " kg ???");
        }
        
    }

Was kann ich tun?

Vielen Dank schon mal


P.S.: ich hab schon reichlich gesucht aber nichts zum Thema gefunden,
hab ich den falschen Suchbegriff benutzt?

 

[faetzminator]

Ein Ansatz könnte sein, in [c]Tier[/c] ein [c]minGewicht[/c] und [c]maxGewicht[/c] zu implementieren. Etwa so:

public abstract class Tier {

    private final int minGewicht;
    private final int maxGewicht;
    private int gewicht;

    public Tier(int minGewicht, int maxGewicht) {
        this.minGewicht = minGewicht;
        this.maxGewicht = maxGewicht;
    }
    
    public void setGewicht(int gewicht) {
        if (gewicht < minGewicht || gewicht > maxGewicht) {
            throw new IllegalArgumentException("Das Gewicht " + gewicht + " liegt ausserhalb der Gewichtsgrenzen");
        }
        this.gewicht = gewicht;
    }
}

public class Hauskatze extends Tier {

    public Hauskatze() {
        super(1, 10);
    }
}

1. nenn die Variablen nicht _xy
2. Verwende kein sysout sondern werfe eine Exception
3. ich würd [c]Tier[/c] abstrakt machen

 

[der_roemer]

Hallo faetzminator,

Danke für die Antwort.

Ja mit Konstruktoren geht das wohl,
hab mich da erst einarbeiten müssen, kannte ich noch nicht.

Aber warum soll ich da eine Exception werfen?

throw new IllegalArgumentException("Das Gewicht " + gewicht + " liegt ausserhalb der Gewichtsgrenzen");

Dann bricht ja mein Programm ab, sofern da ein Wert
ausserhalb der Grenzen eingegeben wird.
Der Sinn erschließt sich mir nicht...


Gruß roman.


P.S.:

die Notation mit dem Unterstrich "int _xy" z.B.
hatte ich mir überlegt für private Variablen,
hatte das mal irgendwo gehört,
aber das galt nur für lokale in Methoden,
weiß nicht mehr genau...

 

[Marco13]

Höm... einfach super?

    @Override
    public void setGewicht(int gewicht) {
        // Setter für das Gewicht der Katze überschreiben
        if (gewicht < 15 & gewicht > 1) {
            super.setGewicht(gewicht); <------------------------- Da?!
        } else {
            System.out.println("Mit dem Gewicht der Katze stimmt was nicht, sie wiegt "+ gewicht + " kg ???");
        }
        
    }

!?

 

[faetzminator]

Aber warum soll ich da eine Exception werfen?

throw new IllegalArgumentException("Das Gewicht " + gewicht + " liegt ausserhalb der Gewichtsgrenzen");

Dann bricht ja mein Programm ab, sofern da ein Wert
ausserhalb der Grenzen eingegeben wird.

Natürlich. 2 Möglichkeiten:

• Es wurde durch einen Programmierfehler verursacht. Werfe diese IllegalArgumentException.
• Es wurde durch eine Fehleingabe vom User verursacht. Da hast du mehrere Möglichkeiten
  • Du machst davor eine Validierung
  • Du wirfst nicht eine RuntimeException (was IllegalArgumentException ist), sondern eine Checked Exception, welche man bearbeiten/weiterwerfen muss
  • Du wirfst eine RuntimeException aber hast irgendwo im Programmfluss eine Fehlerverarbeitung, welche dem User (auf diese IllegalArgumentException) irgendwas sinnvolles ausgibt

 

[der_roemer]

@Marco13:
Das sieht echt gut aus
Ja da fehlt mir die Erfahrung,
hatte mich schon gefragt, wie ich eine katzentypische Meldung werfe;
einfach genauso wie beim Konstruktor mit super... durchreichen

@faetzminator:
das muss ich mir mal in Ruhe anschauen,
Danke für die Anregung!



Ich geh aber ins Bett erstmal

Schöne Nacht noch!

roman.