Frage zu Typ

[cz3kit]

Hallo, ich hab eine Frage was die Typenbeschreibung betrifft. ICh hab hier ein Programm, Mitarbeiter Verwaltung. Da gibt es fünf KLassen, eine Klasse Mitarbeiter, die Abstrakt ist und dann weitere vier Klassen, Azubi, Arbeiter, Angestellter und Geschäftsführer und diese erben von Mitarbeiter. Diese Klassen werden später in einer HashMap gespeichert. Wenn ich die Klassen aus der HashMap heraushole, dann weiß ich, dass Sie von Typ Mitarbeiter sind, aber nicht ob es ein Azubi oder ein Angestellter ist. Deswegen überprüft ich was genau für ein Typ das ist und das hab ich so geschrieben:

if (m instanceof Azubi){
    		System.out.println("Verguetung des Azubi");
    		((Azubi) m).setAusbildungsverguetung(this.Doubleaendern());
    	}

m ist vom Typ Mitarbeiter. Meine Frage richtet sich jetzt an die dritte Zeile. Ich versteh nicht genau was es bedeutet soll. Ich vermute, dass es sowas bedeutet wie "wenn m von Typ Azubi", bin mir aer nicht sicher, wegen den Klammer.
Könnte mir das jemand kurz erklären oder hat dazu eine Quelle?

Danke

 

[faetzminator]

Es wird nach Azubi gecastet. Du sagst dort, dass [c]m[/c] eine Instanz von Azubi ist. Ob das wirklich so ist, wird auf Zeile eins mit [c]instanceof[/c] geprüft. [c]instanceof[/c] ist aber unschön und wird oft verschmäht. Man hintergeht damit eigentlich Richtlinien von OOP.

 

[cz3kit]

Ahso okej, aber wie sollte man es dan am Besten überprüfen?

 

[faetzminator]

Nunja, in diesem Beispiel würde ich sagen, dass alle Mitarbeiter eine Vergütung haben, egal ob dies eine "Ausbildungsvergütung" ist oder nicht. Also sollte die Klasse Mitarbeiter eine Methode [c]setVerguetung()[/c] (ich würde es sowieso in englisch schreiben) besitzen, welche du aufrufst. Zusätzlich kann man z.B. mit einem Enum einen Typen festlegen, falls man ausserhab den konkreten Typen kennen muss (z.B. Grösse des Salärs). Bei schöner Objektorientierung sollte aber auch das nicht notwendig sein.

 

[cz3kit]

Das ist gerade das Problem, das nur Azubi eine Vergütung hat und die anderen haben dan einen Fixgehalt oder einen StundenLohn. Der Mitarbeiter hat zB nut getBrutto(), wobei man es bei den anderen Klassen dann halt aus Stunden Lohn und Arbeitszeit oder Provision berechnet.

 

[faetzminator]

Wie wärs damit:

setStunden(int);
setFixverguetung(int);
setStundenlohn(int);

int getBrutto() {
    return getStunden() * getStundenlohn() + getFixverguetung();
}

Da kannst du den Stundenlohn und/oder die Stunden des Azubis einfach auf 0 setzen (bzw. ist der Standardwert eines int's als Instanzvariable sowieso 0). Du setzt einfach die Fixvergütung. Umgekehrt natürlich bei den "normalen" Mitarbeitern.

 

[cz3kit]

Ich glaub meine Aussage war nicht eindeutig. Hier ist das UML DIagramm dazu, vielleicht wird dann klar was das Problem ist:

 

[faetzminator]

Natürlich, laut dem UML Diagramm stimmt dein Code schon. Es ist allerdings kein schönes OOP.

 

[cz3kit]

Okej gut zu wissen, dann liegt es einfach an dem schlechten UML. ICh danke dir für die Mühe.