Zugriff auf Methode einer Subklasse

[sushi_2k]

Hallo Community,
ich stehe gerade leider total auf dem Schlauch und weiss nicht weiter.
Ich habe 3 Klassen geschrieben, für eine Flächenberechnung von Rechtecken und Quadraten.
Es funktioniert auch alles wie es soll, ausser das ich nicht weiss, wie ich von der Geo_Test, die Methode volumen von der classe Quader aufrufen soll. Da ich bei dem Testprogramm, noch das Volumen ausgeben möchte.

Klasse Rechteck:

package geometrische_figuren;

public class Rechteck {

	double laenge;
	double breite;
	
	Rechteck()
	{
		laenge = 1;
		breite = 1;
	}
	
	Rechteck(double l, double b)
	{
		if (l <= 0.0 || b <= 0.0)
		{
			laenge = 1;
			breite = 1;
		}
		else
		{
			laenge = l;
			breite = b;
		}
	}
	
	double flaeche()
	{
		return (laenge * breite);
	}
	
	void druckeKantenlaengen()
	{
		System.out.println("Laenge: " + laenge);
		System.out.println("Breite: " + breite);
	}
	
}

Quader, von Rechteck abgeleitet

package geometrische_figuren;

public class Quader extends Rechteck {

	double tiefe;
	
	Quader()
	{
		laenge = 1;
		breite = 1;
		tiefe = 1;
	}
	
	Quader(double b, double l, double t)
	{
		breite = b;
		laenge = l;
		tiefe = t;
		
		if (b <= 0.0)
			b = 1.0;
		else 
			breite = b;
		
		if (l <= 0.0)
			l = 1.0;
		else
			laenge = l;
		
		if (t <= 0.0)
			t = 1.0;
		else
			tiefe = t;
	}
	
	double flaeche()
	{
		return (2 * (breite * laenge + breite * tiefe + tiefe * laenge));
	}
	
	double volumen()
	{
		return (breite * laenge * tiefe);
	}
	
	void druckeKantenlaengen()
	{
		System.out.println("Laenge: " + laenge);
		System.out.println("Breite: " + breite);
		System.out.println("Tiefe: " + tiefe);
	}
}

und die Testanwendung

package geometrische_figuren;

public class Geo_Test {
	
	
	public static void main(String[] args) 
	{	
		Rechteck [] geo_feld = new Rechteck[5];
		
		geo_feld[0] = new Quader();
		geo_feld[1] = new Rechteck(10.0, 2.0);
		geo_feld[2] = new Rechteck();
		geo_feld[3] = new Quader(7.0, 3.0, 4.0);
		geo_feld[4] = new Rechteck(-5.0, 9.0);	

		int zaehler_rechteck = 1;
		int zaehler_quader = 1;
		
		for (int i=0; i < geo_feld.length; i++)
		{
			String klasse;
			
			System.out.println(geo_feld[i].getClass().getName());
			
			klasse = geo_feld[i].getClass().getName();
			
			if (klasse.equals("geometrische_figuren.Rechteck"))
			{
				System.out.println("Rechteck " + zaehler_rechteck);
				System.out.println("Flaeche: " + geo_feld[i].flaeche());
				geo_feld[i].druckeKantenlaengen();
				System.out.println("");
				zaehler_rechteck++;
			}
			else
			{
				System.out.println("Quader " + zaehler_quader);
				System.out.println("Flaeche:" + geo_feld[i].flaeche());
				//System.out.println("Flaeche:" + geo_feld[i].volumen());   <== Geht nicht...
				geo_feld[i].druckeKantenlaengen();
				System.out.println("");
				zaehler_quader++;
			}	
				
		}
	}

}

 

[Murray]

1. Besser nicht die Klassennamen vergleichen; lieber instanceof verwenden
2. durch expliziten Cast auf spezialisierte Methoden zugreifen

         if (geo_feld[i] instanceof Rechteck)) {
            System.out.println("Rechteck " + zaehler_rechteck);
            System.out.println("Flaeche: " + geo_feld[i].flaeche());
            geo_feld[i].druckeKantenlaengen();
            System.out.println("");
            zaehler_rechteck++;
         } else if (geo_feld[i] instanceof Quader) {
            Quader q = (Quader)(geo_feld[i]);
            System.out.println("Quader " + zaehler_quader);
            System.out.println("Flaeche:" + quader.flaeche());
            System.out.println("Flaeche:" + quader.volumen());
            quader.druckeKantenlaengen();
            System.out.println("");
            zaehler_quader++; 
       }

PS: die Zähler sollte man besser auf 0 initialisieren und dann gleich zu Beginn der Verzweigungen hochzählen, sonst hat man am Ende immer einen zuviel.

 

[L-ectron-X]

Randbemerkung:
Die eigentliche Vererbungslogik erscheint mir schon nicht korrekt. Ein Rechteck ist eine Fläche, ein Quarder ist ein Körper. Aber ein Quarder hat Seitenflächen, deren Flächeninhalt man errechnen kann.
Also müsste der Quarder eher die Methoden von Rechteck benutzen, nicht überschreiben. Vererbung hat hier meiner Meinung nach, nichts zu suchen.

 

[Jango]

Du kannst in der Klasse Geo_Test nicht auf die Methode volumen der Klasse Quader zugreifen, weil die Methode private deklariert ist. Und warum auch? Sie tut doch auch garnichts, außer sich zu beenden. Wenn du in der Methode mittels return etwas zurückgeben möchtest, musst du das bei der Deklaration auch bekannt geben.

 

[Murray]

Du kannst in der Klasse Geo_Test nicht auf die Methode volumen der Klasse Quader zugreifen, weil die Methode private deklariert ist. Und warum auch? Sie tut doch auch garnichts, außer sich zu beenden. Wenn du in der Methode mittels return etwas zurückgeben möchtest, musst du das bei der Deklaration auch bekannt geben.

Das stimmt so nicht; die Methode hat keine expliziten Access-Modifier ist ist daher "package-protected", also für alle Klassen aus dem gleichen Package sichtbar.

 

[Guest]

1. Besser nicht die Klassennamen vergleichen; lieber instanceof verwenden

Das ganze ist eine Übung in der Programmierenvorlesung. Und wir sollen, laut Aufgabenstellung über getClass().getName() entscheiden ob es eine Rechteck oder Quader ist. Instanceof wäre sicherlich besser, aber kommt erst in der nächsten Übung ;-)

PS: die Zähler sollte man besser auf 0 initialisieren und dann gleich zu Beginn der Verzweigungen hochzählen, sonst hat man am Ende immer einen zuviel.

Der Zähler zählt mir nur die Anzahl der Rechtecke und Quader hoch die Ausgegeben werden. Hat also nichts mit dem eigentlichen Programm zu tun.
Also Quader 1, Rechteck 1, Quader 2....
Ansonsten natürlich 0, wenn ich auf die Felder direkt zugreife.

 

[sushi_2k]

Randbemerkung:
Die eigentliche Vererbungslogik erscheint mir schon nicht korrekt. Ein Rechteck ist eine Fläche, ein Quarder ist ein Körper. Aber ein Quarder hat Seitenflächen, deren Flächeninhalt man errechnen kann.
Also müsste der Quarder eher die Methoden von Rechteck benutzen, nicht überschreiben. Vererbung hat hier meiner Meinung nach, nichts zu suchen.

Wie gesagt, das ganze ist eine Aufgabenstellung in der Programmierenvorlesung. Und laut Aufgabenstellung soll eine Superklasse Rechteck erstellt werden und eine Subklasse Quader.

 

[sushi_2k]

Gibt es jetzt eigentlich noch eine andere Möglichkeit auf die Methode Volumen von der Subklasse Quader zuzugreifen, ausser der von Murray?
Also diese:

Quader q = (Quader)(geo_feld[i]);

Das bedeutet q ist jetzt ein Objekt der Klasse Quader und bekommt die Werte von dem aktuellen geo_feld?

 

[Marco13]

Das ist nur eincast. Es wird KEIN neues Objekt angelegt. Man überfprüft (mit instanceof, oder durch Abfragen des Klassennamens) OB das Objekt ein Quader ist, und WENN es ein Quader ist, dann bewirkt diese Zeile, dass man es auch als "Quader q" verwenden kann. (Wenn das Objekt in Wirklichkeit "nur" ein Rechteck wäre, würde es dort einen Fehler (ClassCastException) geben).

Also anstelle von

           Quader q = (Quader)(geo_feld[i]);
           System.out.println("Volumen:" + q.volumen());

köntest du auch schreiben

           System.out.println("Volumen:" + ((Quader)(geo_feld[i])).volumen());

das ist aber unübersichtlich, deswegen macht man den Cast einmal am Anfang, und spart sich nachher diesen Klammerwust.

Aber ich muß L-ectron-X zustimmen: Diese Vererbung macht eigentlich keinen Sinn. Zum Lernen und Verstehen der eigentlichen Idee, die hinter der Objektorientierten Programmierung und Vererbung steckt, ist dieses Programm imho sch...lecht: Jede Typabfrage (egal ob mit instaceof oder über den Klassennamen) ist streng genommen ein Modellierungsfehler (und so imho auch in diesem Fall). Vielleicht würde sie in bestimmten Zusammenhängen Sinn machen (Dann würde ich aber eine Hierarchie wie etwa
Punkt -> Linie -> Rechteck -> Quader ->
erwarten :roll: und selbst das könnte man schöner machen). Wenn das eine Schul-Hausaufgabe wäre, würde ich sagen, dass der verantwortliche vermutlich einfach ein inkompetenter Lehrer ist (Wie... "Tautologie"? :wink: ) aber mal nicht vorschnell urteilen. Vielleicht kommt da ja noch was ganz schlaues hinterher... :roll:

 

[sushi_2k]

Ok, danke. Ihr wart mir eine grosse Hilfe, wieder was gelernt ;-)
Mal schauen was die nächste Übung bringt...