Geschachtelte klassen

[sh33p]

public class CdListe {

   String archivTitel;
   int maxAnzahl;

   public CdListe(String archivTitel, int maxAnzahl){
      this.archivTitel = archivTitel;
      this.maxAnzahl = maxAnzahl;

   }

   public void cdHinzuefuegen(String cdTitel,String kuenstler, int jahr){
      Cd cd = new Cd(cdTitel,kuenstler,jahr);
   }
   
   public void listeAnzeigen(){
      System.out.println(Cd.toString());
   }

   public class Cd{
      String cdTitel;
      String kuenstler;
      int jahr;

      public Cd(String cdTitel, String kuenstler, int jahr){
         this.cdTitel = cdTitel;
         this.kuenstler = kuenstler;
         this.jahr = jahr;

      }

      public String toString(){
        return  archivTitel+" " + maxAnzahl + " " + cdTitel + " " + kuenstler + " "+ jahr;
      }

   }

}

ich möchte über die Methode listeAnzeigen() die gesamte Liste der im archiv hinzugefügten cd's ausgeben. dies möchte ich über die toString methode realisieren.nur das funktioniert nicht ganz

 

[javimka]

was genau funktioniert denn nicht?

 

[sh33p]

die methode listeAnzeigen()

 

[Landei]

Du brauchst erst mal eine Datenstruktur, die die ganzen Cd-Einträge hält. Nur weil Cd eine "geschachtelte" (besser: innere) Klasse ist, kennt CdListe deshalb noch lange nicht "automatisch" alle erzeugten Cds. (Andersrum kennt ein Cd-Objekt aber die erzeugende CdListen-Instanz, zumindest wenn Cd nicht statisch ist)

Aus der Variable maxAnzahl schließe ich mal, du willst dafür ein Array verwenden (Listen können beliebig viele Einträge aufnehmen). Das ginge dann ungefähr so (ungetestet):

public class CdListe {
  String archivTitel;
  int index = 0;
  private Cd[]  cds;

  public CdListe(String archivTitel, int maxAnzahl){
    this.archivTitel = archivTitel;
    cds = new Cd[maxAnzahl];
  }

  public void cdHinzuefuegen(String cdTitel,String kuenstler, int jahr){
     if(index = cds.length) {
        throw new IllegalArgumentException("Liste ist voll!");  
     }
     cds[index] = new Cd(cdTitel,kuenstler,jahr);
     index ++;
  }

  public void listeAnzeigen(){
    for(Cd cd : cds) {
       if(cd != null) {
          System.out.println(cd);
       }
    }
  }

  public static class Cd{
    String cdTitel;
    String kuenstler;
    int jahr;
 
    public Cd(String cdTitel, String kuenstler, int jahr){
      this.cdTitel = cdTitel;
      this.kuenstler = kuenstler;
      this.jahr = jahr;
    }
    
    @Override public String toString(){
      return  archivTitel+" " + maxAnzahl + " " + cdTitel + " " + kuenstler + " "+ jahr;
    }
  }
}

 

[L-ectron-X]

Im Moment gibts einen Compilerfehler:

CdListe.java:21:24: non-static method toString() cannot be referenced from a static context
  System.out.println(Cd.toString());
                       ^

Du rufst die Methode als Klassenmethode auf. Dazu müsste sie aber statisch sein.
Ich habe deinen Code mal etwas umgebaut, weil einiges gefehlt hat, um das Programm sinnvoll zum Laufen zu bringen.

public class CdListe {
   private java.util.List<Cd> cdList;
   private String archivTitel;

   public CdListe(String archivTitel){
      cdList = new java.util.ArrayList<Cd>();
      this.archivTitel = archivTitel;
   }

   public void cdHinzufuegen(String cdTitel,String kuenstler, int jahr){
      Cd cd = new Cd(cdTitel,kuenstler,jahr);
      cdList.add(cd);
   }
   
   public void listeAnzeigen(){
      for(int i = 0, j = cdList.size(); i < j; i++) {
         System.out.println(cdList.get(i));
      }
   }
   
   public String gibListenTitel() {
      return archivTitel;
   }

   public class Cd {
      String cdTitel;
      String kuenstler;
      int jahr;

      public Cd(String cdTitel, String kuenstler, int jahr){
         this.cdTitel = cdTitel;
         this.kuenstler = kuenstler;
         this.jahr = jahr;

      }

      public String toString(){
         return cdTitel + "\t" + kuenstler + "\t"+ jahr;
      }

   }
   
   public static void main(String[] args) {
      CdListe cdl = new CdListe("Kinderlieder");
      cdl.cdHinzufuegen("Tausend Sterne", "Nena", 2002);
      cdl.cdHinzufuegen("Himmel, Sonne, Wind und Regen", "Nena", 2008);
      System.out.println(cdl.gibListenTitel()) ;
      cdl.listeAnzeigen();
   }

}

Edit: OK, nun hast du schon zwei Varianten.

 

[sh33p]

danke euch beiden

ich werds mir morgen mal genauer anschauen

 

[sh33p]

okay aber bei deiner version l-ectron- x wird die toString-methode der inneren klasse nicht verwendet?!

 

[L-ectron-X]

Und trotzdem wird die Ausgabe genau so gemacht, wie von der toString()-Methode definiert.
Warum?

Die println()-Methode gibt automatisch immer die Rückgabe einer implementierten toString()-Methode eines Objektes aus.
Daher sollte auch immer in eigenen Objekten die toString()-Methode überschrieben werden und darin eine formatierte Ausgabe des Objektinhaltes zurückgegeben werden.

 

[sh33p]

Das heisst, das wenn ich in einer klasse eine toString Methode definiere, von dieser Klasse ein
Objekt erzeuge und anschließend in einer Testklasse die println Methode Aufrufe, das println automatisch die toString Methode aufruft?

 

[L-ectron-X]

Ja.

Nehmen wir an du hast eine Klasse Buch mit einer implementierten toString()-Methode.

In der Methode gibst du einen String mit Daten über ein Exemplar (konkretes Objekt) eines Buches zurück.

Dann wird bei...

Buch buch = new Buch(....);
System.out.println(buch);

...die

println()

-Methode automatisch die

toString()

-Methode von buch aufrufen und den darin aufgebauten Rückhgabewert des Objektes zurück geben.