Vererbung - instanceof

[086033]

Ich habe ein Verleihsystem u.a. mit den Kategorien CD's, DVD's und BlueRay. Folgende Klassen sind enthalten:
- Datenbank
- Medium (Superklasse)
- Digital (Subklasse von Medium)
- Analog (Subklasse von Medium)
- DVD, BlueRay, CD, ... (Subklassen von Digital)
- Buecher (Subklasse von Analog)

Ich konnte soweit die Aufgabenstellung implementieren bis auf einen Punkt:

"Es soll eine Methode

ausgeben(Medium medium)

implementiert werden, welche alle Datenfelder aller Medien der Kategorie medium ausgibt."

Folgendes habe ich versucht:

 public void ausgeben(Medium medium)
    // bsp. für CD... 
    {   for(Medium m : medien) {
            if (m instanceof CD) { 
            System.out.println(m);
            }
        }
    }

Das Problem ist nun folgendes:
1. Als Parameter kann ja nur etwas vom Typ Medium eingegeben werden. CD (als Klasse) wird nicht akzeptiert. cD1 (Name einer Instanz der Klasse CD) würde akzeptiert, das bringt mir aber nicht viel.

2. "m" sollte eigentlich durch medium ersetzt werden, nur kann ich die Klasse dann nicht mehr kompilieren.

Ich wäre froh, wenn ihr mir Ansatzpunkte geben könntet, ich glaube nämlich, dass dieser Aufgabenteil überhaupt nicht schwer sein sollte.

Besten Dank!

 

[Michael...]

Das Problem ist nun folgendes:
1. Als Parameter kann ja nur etwas vom Typ Medium eingegeben werden. CD (als Klasse) wird nicht akzeptiert. cD1 (Name einer Instanz der Klasse CD) würde akzeptiert, das bringt mir aber nicht viel.

Warum nicht? Wenn CD von Medium erbt, kann überall wo ein Objekt vom Typ Medium erwartet wird jede beliebige Subklasse - auch CD - übergeben werden

 public void ausgeben(Medium medium)
    // bsp. für CD... 
    {   for(Medium m : medien) {

Das macht keinen Sinn es wird ein einzelnes Objekt vom Typ Medium übergeben, wie soll man da in einer Schleife drüber iterieren?

 

[AmunRa]

nachdem du ja geprüft hast ob dein Medium eine CD ist kannst du diese Ja in eine CD casten, Ist aber ein nicht ganz schöner Weg,

Der Zweite wäre, du hast in deinem Interface zum Beispiel eine Methode printDetails, und diese Methode überschreibst du jetzt in all deinen Subclassen.

Durch Dynamische Bindung wird nun wenn du m.printDetails() aufrufst im Falle einer CD die Mehtode der CD Klasse aufgerufen und bei einer DVD die der DVD-Klasse

Hoff das hat dir weitergeholfen

 

[Gastredner]

Oder einfach die toString()-Methode überschreiben und dann einfach die Ausgabe mit [c]System.out.println(medium)[/c] ausführen.

 

[086033]

Vielen Dank an euch!

public void ausgeben(Medium medium)
    // bsp. für CD... 
    {   for(Medium m : medien) {
            if (m instanceof CD) { 
            System.out.println(m);
            }
        }
    }

Warum nicht? Wenn CD von Medium erbt, kann überall wo ein Objekt vom Typ Medium erwartet wird jede beliebige Subklasse - auch CD - übergeben werden

Das ist mir eigentlich auch "klar", die Methode ist mir aber vorgegeben mit ausgeben(Medium medium). Diese lässt nur zu, dass ich eine Instanz der Subklassen angeben kann, also eben z.B. cd1, dvd1, ... aber nicht CD als Klasse. Das verunsichert mich.

Das macht keinen Sinn es wird ein einzelnes Objekt vom Typ Medium übergeben, wie soll man da in einer Schleife drüber iterieren?

Macht nun die Schleife an sich keinen Sinn, oder die Parameter? ( also eigentlich das, was ich oben erwähnt habe?)

Zudem macht die Methode so an sich keinen Sinn, weil der Parameter gar nie aufgegriffen wird. Irgendwie stehe ich auf dem Schlauch.

Betreffend toString(), diese Methode habe ich in derselben Klasse bereits mit folgender Methode genutzt (toString wird in den einzelnen Subklassen jeweils überschrieben):

    /**
     * Gib eine Liste aller aktuell gespeicherten Medien
     * auf der Konsole aus.
     */
    public void ausgeben()
    {
        for(Medium medium : medien)
        {
             System.out.println(medium);
        }
    }

toString möchte ich eigentlich auch wieder bei der "Problem"-Methode nutzen, aber mein Problem ist zurzeit, dass ich nicht begreife, wie ich nur die Daten einer spezifischen Kategorie ausgebe. Es muss mit der Methode

ausgeben(Medium medium);

umgesetzt werden.

 

[kidsos]

Macht nun die Schleife an sich keinen Sinn, oder die Parameter?

Die Schleife macht keinen Sinn. Wo keine Collection ist, kann man auch nicht drüber iterieren. Ideal wäre, wenn du deine Medien in einer Liste (z.B. Arraylist) vorhälst. Dann kannst du problemlos mit der for-Schleife über jeden Eintrag gehen und diesen dann ausgeben. Dazu musst du aber die Methode umbauen, was du nicht darfst.

Mit der Methode "ausgeben(Medium medium)", so wie sie dir vorgegeben ist, übergibst du ja nur ein Objekt. Das kann mit einer for-Schleife nicht funktionieren. Ich würde mir an deiner Stelle die Vorgehensweise von AmunRa näher anschauen. Das könnte klappen.

 

[Michael...]

die Methode ist mir aber vorgegeben mit ausgeben(Medium medium). Diese lässt nur zu, dass ich eine Instanz der Subklassen angeben kann, also eben z.B. cd1, dvd1, ... aber nicht CD als Klasse. Das verunsichert mich.

Wieso willst Du die Klasse übergeben? Willst Du sie per Reflection analysieren?
Dann müsste der Methodenrumpf aber eher so aussehen:

public void ausgeben(Class<? extends Medium) c)

 

[086033]

Ideal wäre, wenn du deine Medien in einer Liste (z.B. Arraylist) vorhälst. Dann kannst du problemlos mit der for-Schleife über jeden Eintrag gehen und diesen dann ausgeben.

Mit der Methode "ausgeben(Medium medium)", so wie sie dir vorgegeben ist, übergibst du ja nur ein Objekt. Das kann mit einer for-Schleife nicht funktionieren. Ich würde mir an deiner Stelle die Vorgehensweise von AmunRa näher anschauen. Das könnte klappen.

Danke! Nun, die Medien habe ich ja alle in einer ArrayList.

public class Datenbank
{
    private ArrayList<Medium> medien;

    /**
     * Erzeuge eine leere Datenbank.
     */
    public Datenbank()
    {
        medien = new ArrayList<Medium>();
    }

    /**
     * Erfasse das angegebene Medium in dieser Datenbank.
     * @param medium das erfasst werden soll.
     */
    public void erfasseMedium(Medium medium)
    {
        medien.add(medium);
    }

    /**
     * Gib eine Liste aller aktuell gespeicherten Medien
     * auf der Konsole aus.
     */
    public void ausgeben()
    {
        for(Medium medium : medien)
        {
             System.out.println(medium);
        }
    }

Die letzte Methode funktioniert auch, aber eben nicht die Methode ausgeben(Medium medium) - welche hier jetzt nicht enthalten ist. Wie du gesagt hast, übergebe ich mit der Methode ja nur ein Objekt und demnach keine Kategorie (Klasse).

Ich verstehe die Aufgabenstellung

"Es soll eine Methode

ausgeben(Medium medium)

implementiert werden, welche alle Datenfelder aller Medien der Kategorie medium ausgibt."

so, dass eben alle Datenfelder der Kategorien "CD, DVD, etc" ausgegeben werden, abhängig vom Parameter den ich übergebe. Aber wie kann ich durch übergabe eines Objekts, die ganzen Datenfelder der Objekte einer spezifischen Kategorie ("Klasse") ausgeben?

oder verstehe ich die Aufgabenstellung falsch? (bzw. versteht ihr das auch so )

Sorry, aber alles andere war eigentlich komplizierter (mehr Punkte), aber ich stehe da auf dem Schlauch..

 

[086033]

Wieso willst Du die Klasse übergeben? Willst Du sie per Reflection analysieren?
Dann müsste der Methodenrumpf aber eher so aussehen:

public void ausgeben(Class<? extends Medium) c)

Ich glaube, dass ich nicht verstehe, wie ich mit der Übergabe eines Objekts, alle Datenfelder einer Kategorie - ich verstehe das als "alle Datenfelder einer Subklasse, z.B. CD - ausgeben kann.

siehe auch Antwort um 12:35

 

[Michael...]

oder verstehe ich die Aufgabenstellung falsch? (bzw. versteht ihr das auch so )

Ich verstehe die Aufgabenstellung, so wie sie da steht, überhaupt nicht. Geht's hier um Reflection??

 

[086033]

Ich verstehe die Aufgabenstellung, so wie sie da steht, überhaupt nicht. Geht's hier um Reflection??

sollte nicht.. habe noch nie etwas von Reflection gehört.

 

[Michael...]

Dann muss ich passen, verstehe die Aufgabenstellung überhaupt nicht. Sorry...

 

[086033]

hättest du nicht so viele Beiträge, würde ich nicht nervös werden

es muss etwas mit instanceof zu tun haben.. also

(medium instanceof CD)

Trotzdem Danke! Hoffentlich kann mir sonst jemand helfen.

 

[Schumi]

Wenn alle Deine von Medium erbenden Klassen toString implementieren ist es doch gar kein Problem:

public void ausgeben(Medium medium)
            System.out.println(medium);
    }

Da müsstest Du nicht mal zwischen den einzelnen Klassen unterscheiden, da das ausgeben der jeweiligen Klasse überlassen wird. Sprich, Du brauchst keine instanceof Abfrage. In der toString() Methode der Klasse musst Du dann nur sicher gehen, dass Du auch alle Inhalte dieser Klasse mit ausgibst.

 

[Antoras]

Ich fasse die Aufgabe so auf, dass an die Methode ein Medium übergeben wird und dass alle Medien, die den gleichen Typ wie das übergebene Medium haben, ausgegeben werden sollen.

Und die Lösung dafür hast du ja schon beschrieben in Post 5.

 

[Schumi]

Wenn es wirklich das ist, dann steht die Lösung ja sogar schon in Post 1:

Folgendes habe ich versucht:

 public void ausgeben(Medium medium)
    // bsp. für CD... 
    {   for(Medium m : medien) {
            if (m instanceof CD) { 
            System.out.println(m);
            }
        }
    }

Das Problem ist nun folgendes:
1. Als Parameter kann ja nur etwas vom Typ Medium eingegeben werden. CD (als Klasse) wird nicht akzeptiert. cD1 (Name einer Instanz der Klasse CD) würde akzeptiert, das bringt mir aber nicht viel.

Fragt sich nur, warum das nicht viel bringt?

Ah ne, man muss natürlich vorher noch die medien-Klasse prüfen. Oder eben bei jedem m schauen, ob die Klasse gleich der des übergebenen Mediums ist.

 

[Suinos]

Möglichkeit 1:
Du sollst nur die gemeinsamen Werte eines Mediums ausgeben.
Dann muss

Medium

aber getter-Methoden vorgeben, welche du aufrufen kannst:

System.out.println("Autor: " + medium.getAuthor());
System.out.println("Länge: " + medium.getLength());

Möglichkeit 2:
Du sollst herausfinden, welches Objekt sich hinter der Referenz befindet, auf eine neue Referenz casten und dann darauf spezielle Methoden aufrufen:

if (medium instanceof CD)
{
	CD cd = (CD) medium;
	
	System.out.println("Anzahl an Songs: " + cd.getSongsCount());
}

---

Das mit der

toString

Methode kann doch nicht sein, da man dann einfach diese Zeile Code:

System.out.println(medium);

, mit dieser hier ausgetauscht hat:

ausgeben(medium);

Das würde sich kaum lohnen.

 

[Schumi]

Um noch einmal Antoras Verständnis der Aufgabe aufzugreifen:

 public void ausgeben(Medium medium)
    {   for(Medium m : medien) {
            if (m.getClass() == medium.getClass()) { 
            System.out.println(m);
            }
        }
    }

würde es bei implementierten toString tun.

 

[086033]

edit: das ist dasselbe Ergebnis wie das von Schumi, oder? (mit der Ausnahme, dass seines viel eleganter ist )

Hab mal folgendes versucht und es funktioniert anscheinend. Aber... wieso sollte ich ein Objekt angeben müssen um alle Datenfelder der Kategorie (Klasse) auszugeben? Ist es richtig, dass es mit der Methode ausgeben(Medium medium) keine Möglichkeit gibt, die Klasse anzugeben?

public void ausgeben(Medium medium)
    {
        if (medium instanceof CD) {
            for(Medium m : medien) {
                if (m instanceof CD) {
                    System.out.println(m);
                }
            }
        }
        
        else if (medium instanceof DVD) {
            for(Medium m : medien) {
                if (m instanceof DVD) {
                    System.out.println(m);
                }
            }
        } 
       
        else if (medium instanceof BlueRay) {
            for(Medium m : medien) {
                if (m instanceof BlueRay) {
                    System.out.println(m);
                }
            }
        }
        
        else if (medium instanceof Videospiele) {
            for(Medium m : medien) {
                if (m instanceof Videospiele) {
                    System.out.println(m);
                }
            }
        }
        
    }

 

[086033]

"Es soll eine Methode ausgeben(Medium medium) implementiert werden, welche alle Datenfelder aller Medien der Kategorie medium ausgibt."

"Kategorie medium" das kann doch nicht sein... unsere Lösung ist nun:

"Es soll eine Methode ausgeben(Medium medium) implementiert werden, welche alle Datenfelder aller Medien mit derselben Kategorie wie der von medium ausgibt."

oder? sorry, tut mir leid!

noch ein Nachtrag:

Im Buch "java lernen mit BlueJ" steht:

9.10 Der Operator instanceof

"... Doch was wäre, wenn wir nur die CDs oder DVDs aus der Liste herausziehen möchten? ... Nach dem Einsatz von instanceof-Operators folgt oft direkt eine Typumwandlung der Objektreferenz in den identifizierten Typ - wie in dem folgenden Code, der alle CDs aus einer Medienliste herausfiltert und iner eigenen Liste speichert:"

ArrayList<CD> cds = new ArrayList<CD>();
for(Medium medium : medien) {
if(medium instanceof CD) {
cds.add((CD) medium);
}}

Kann es mit diesem Code etwas zu tun haben? Muss nicht, oder? Da die ArrayList ja schon besteht, allerdings mit allen Medien.

 

[Gastredner]

ArrayList<CD> cds = new ArrayList<CD>();
for(Medium medium : medien) {
if(medium instanceof CD) {
cds.add((CD) medium);
}}

Das ist genau das, was du benötigst. Du ermittelst die Klasse des übergebenen Mediums, suchst dir alle Medien vom gleichen Typ zusammen und gibst diese dann aus. Ersteres kannst du so wie in dem obrigen Code machen (ich würde es aber anders machen, folgt gleich), Letzteres damit, dass du in allen von Medium erbenden Klassen die Methode toString() überschreibst.

Das Herausfiltern würde ich statt mit instanceof eher mit einem Vergleich der Klassen erledigen, dies ist flexibler. Außerdem kann man sich das Speichern in einer separaten Liste sparen, wenn man die Ausgabe direkt erledigt. Konkret hieße das also:

public void ausgeben(Medium medium) {
	Class<? extends Medium> clazz = medium.getClass();
	for (Medium m: medien) {
		if (m.getClass().equals(clazz)) {
			System.out.println(m);	// Hier wird von println() auf m die Methode toString() aufgerufen - die du zuvor überschrieben haben musst.
		}
	}
}

Du musst dann halt nur noch die toString()-Methode in allen Unterklassen von Medium überschreiben. Durch den Abgleich der Klassen musst du außerdem keine neuen instaceof-Abfragen einbauen, wenn du eine weitere Klasse von Medium erben lässt und diese in das Programm integrieren willst.