[Geschlossen] kurze hilfe JAVA gesucht

instanceof

http://www.dpunkt.de/java/Die_Sprache_Java/Objektorientierte_Programmierung_mit_Java/48.html

public void nurCDsAusgeben()
{
for (Medium alle : medien)
{
if( alle instanceof CD)
alle.auflisten();
System.out.println();
}

Ich glaube mein Ansatz ist ziemlich dirty:

if( alle instanceof type)

wie genau instanceof geschrieben wird sagt dir eclipse

#edit

too slow

okay, das es irgendwie über instanceof gehen muss, hab ich mir gedacht.

aber so bräuchte ich ja wieder mehrere methoden, quasi für jede klasse (cd, dvd,..) eine ausgabenfunktion, oder?

ich hätte am liebsten eine ausgabefunktion in der klasse Katalog, wo ich als parameter eingebe: cd und es wird alles ausgegeben, was von der klasse cd ist.

btw danke schonmal

also in der aufgabenstellung steht, die mthode soll irgendwie so aussehen:

private void ausgeben (Medium medium), aber damit krieg ich es nicht auf die reihe

quasi so:

public void nurXXXAusgeben(xxx)
{
for (Medium alle : medien)
{
if( alle instanceof xxx)
alle.auflisten();
System.out.println();
}

versuch das mal:

private void ausgeben(Medium m) {
for (Medium x : medien) {
if (x.getClass() == m.getClass()) {
System.out.println(x.name);
}
}
}

okay, das funktioniert schon fast.
problem ist, dass ich hier als parameter (Medium m) hier nicht die klasse CD eingeben kann und er mir alle CDs ausgibt, sondern ich gebe hier ein Objektnamen ein, beispielsweise CD1 und dann gibt er mir alle CDs richtig aus.

ich möchte aber nicht ein spezielles Objekt eingeben, sondern direkt die Klasse

so sieht es jetzt aus:

ich hab jetzt mal 2 cds, eine dvd und ein spiel. jetzt möchte ich als methode im katalog gern, dass ich "CD" eingebe und dann CD1 und CD2 ausgegeben werden, bzw. ich gebe "Videospiele" ein und es wird Spiel1 ausgegeben.

jetzt ist es so, dass ich "CD1" eingeben muss und CD1 und CD2 ausgegeben werden.

jetzt bekommt ausgeben einen string übergeben wenn der so ist wie die klasse heißt gibt er die elemente aus

wenn deine Klasse Cd heißt musst du medien.ausgeben("Cd"); aufrufen.

private void ausgeben(String m) {
	for (Medium x : medien) {
		if (x.getClass().getName().equals(m)) {
			System.out.println(x.name);
		}
	}
}

ja okay danke, über string gehts natürlich auch.

Poste doch mal die Klasse Medium. Vielleicht steht da schon eine Eigenschaft drin, über die man feststellen kann, was es ist.

Laut dieser Vorgabe hier musst Du aber ein Objekt wie cd1 übergeben:

private void ausgeben (Medium medium)

Ist dieser methodenkopf vorgegeben?
Wenn ja, ist die Lösung mit getClass() dock ok.

Ich meine, wie soll denn die Klasse Katalog wissen, welche Medienarten alles möglich sind? Das soll ja offen bleiben, wenn in Zukunft was Neues hinzu kommt. daher wäre eine Überprüfung mit instanceof schon dirty.

Wobei wirklich sinnig finde ich es auch nicht, ihm da ein Objekt rein zu geben.

Ich hätte in Medium eine Eigenschaft
int medientyp
erstellt, die dann im Konstruktor von den spezifischen geerbten Medienarten gesetzt wird. Z.B. 1 für CD, 2 für DVD etc.

In der Methode ausgeben(int medienart) könntest du dann in der schleife diesen wert abprüfen, um nur einen medientyp auszugeben.
mit einem if fragt man das dann einfach ab.
hier könnte man durch eingabe einer 0 zb auch alle ausgeben lassen. indem man folgendes prüft:

private void ausgeben(int m)
{
   for (Medium x : medien) 
   {
        if (x.getMedienart() == m or m == 0) 
        {    
              System.out.println(x.getName());
        }
    }  
}

Um sich nicht die ganzen Zahlen merken zu müssen, kann man auch im Katalog statische, unveränderbare Eigenschaften einführen, die sich auch unkompliziert erweitern lassen.

public static final int ALLE = 0;
public static final int CD = 1;
public static final int DVD = 2;
usw.

beim Aufrufen kannst Du dann schreiben

katalog.ausgeben(Katalog.CD);

edit: hatte beim code einen fehler. ist verbessert

cool danke

ich frag morgen mal beim dozenten nach, wie genau er das mit ausgeben (Medium medium) gemeint hat. wie schon gesagt, muss man hier eigentlich immer ein objekt übergeben, was aber nicht so wirklich sinnvoll ist.

Sonst probier doch mal:

public static final Medium = new Medium(); // wenn nicht abstract!
public static final CD = new CD();
public static final DVD = new DVD();
...

//dann hier..

public void ausgeben(Medium m){

     if(m instanceof Medium){
        //hier alle ausgeben
       return; //wichtig!
    }else{
       //hier feinere Unterteilung 
    }
}

Wenn es so sein soll sonst ist die int-Variante eleganter

Der Code mit getClass() war doch schon fast was du haben willst... Wenn dir das nicht gefällt, dass du ne Instanz brauchst:

private void ausgeben(Class typ) {
  for (Medium x : medien) {
    if (x.getClass() == typ) {
      ...
    }
  }
}

Aufgerufen wird das dann zum Beispiel ausgeben(CD.class)

.

Du könntest auch in Medium die Methoden

public boolean isCD(){
  return false;
}

public boolean isDVD(){
  return false;
}

...

...

erstellen und dann in der jeweiligen Klasse die passende Methode wieder überschreiben.

Bsp. für die Klasse CD:

@Override
public boolean isCD(){
  return true;
}

Und dann kannst du in der ausgeben Methode fragen

if (medium.isCD()){
   print("WHATEVER");
}

Objektorientierter Entwurf ftw! Ich empfehle dann aber, zur Zukunftssicherheit noch die Methoden isNewSubtypeIWillAddEventually1 bis 9 sowie isNewSubtypeIWantToAddTooButRanOutOfDigits hinzuzufügen.

Zur Übersetzung: Die von B4sti vorgeschlagene Methode funktioniert, bricht aber mit der Objektorientierung, da da die Oberklasse genauere Kenntnisse über ihre Unterklassen haben müsste. Dann kann man auch direkt alles da reinbauen.

Joa, des stimmt wohl.
Aber über getClass() finde ich es immer nicht so schön gelöst.

Dann könnte man es noch so machen:

In Medium

public String ausgeben(String parameter);

in z.B. CD:

public String ausgeben(String parameter){
  if(paramter.equals("CD") || parameter.equals("")){
    return this.getName();
  }
  return "";
}

Dann würde mit

medium.ausgeben("CD");

nur was zurückgegeben werden, wenn es auch wirklich eine CD ist.
So wird der Name der CD nur ausgegeben wenn keine Einschänkungen über den Parameter übergeben werden, oder wenn explizit CD als Einschränkung angegeben wird.

Allerdings muss der Aufrufer dann irgendwoher wissen, dass mann "CD" eingeben muss, wenn man CDs sehen will. Aber das kann man ja in der Doku verewigen ♦

Willst du dann jeder Klasse ne getName-Methode verpassen oder doch getClass().getName() verwenden?

Naja, die Methode muss halt das liefern was gewünscht ist.
Man kann da ja auch

getKuenstler() + getTitle()

ausgeben.
Was auch immer gefordert ist.
Den Klassennamen auszugeben ist wahrscheinlich nicht gewünscht