Collection Buch

  • Collection Buch

    Hallo Liebe Java freunde!!! :)

    Ich habe hier ein kleines Problem mit Collections also :

    Ich möchte eine Bibliothek erzeugen die beliebig viele Bücher speichern kann...

    Die Bibliothek hat folgende Methoden

    einfügen (Buch b) : boolean
    sucheNachAutor(String autor): Collection <Buch>
    sucheNachISBN(String isbn): Buch
    bestandSortierenNach(Comparator<Buch> buch):Collection<Buch>

    Das Buch ist folgendermaßen aufgebaut
    String autor
    String isbn
    String title

    (hinter dem Doppelpunkt steht immer der rückgabe wert der Methode)

    So nun hab ich das ganze mal angefangen :
    erst mal die Klasse Buch:

    Quellcode

    1. public class Buch {
    2. protected String autor;
    3. protected String isbn;
    4. protected String title;
    6. public Buch(String autor, String isbn, String title){
    7. this.autor=autor;
    8. this.isbn=isbn;
    9. this.title=title;
    10. }
    11. }
    Alles anzeigen


    als nächste die Klasse Bibliothek:

    Quellcode

    1. package Bibliothek;
    2. import java.util.*;
    3. public class Bibliothek {
    4. //instanzvariablen
    5. LinkedList <Buch> bib;
    6. Iterator <Buch> iter = bib.iterator();
    7. ArrayList <Buch> autor;
    9. public Bibliothek(){
    10. erzeugen der List
    11. this.bib=new LinkedList();
    13. }
    15. // fügt ein Object vom Typ Buch in die List ein wenn es geklappt hat true sonst false
    16. public boolean einfügen(Buch b){
    18. while(iter.hasNext()){
    20. if(iter.next() == null){
    22. bib.add(b);
    23. return true;
    24. }
    26. }
    27. return false;
    28. }
    29. //sucheNachAutor erstellt eine Collection und liefert alle bücher des Autors zurück
    30. public Buch sucheNachAutor(String autor){
    32. this.autor=new ArrayList();
    33. for(Buch b: bib ){
    35. if(iter.next().autor == autor){
    37. this.autor.add(iter);
    38. return this.autor;
    39. }
    40. }return null;
    42. }
    43. //sucht nach ISBN nummer und liefert das entsprechende Buch zurück
    44. public Buch sucheNachISBN(String isbn){
    45. for(Buch b: bib){
    47. if(iter.next().isbn==isbn){
    48. return b;
    50. }
    51. }return null;
    52. }
    53. //Sortiert Die Collection nach parameter comp
    54. //public Buch bestandSortierenNach(){
    57. //}
    58. }
    Alles anzeigen


    So nun zu meinen Fragen:
    1. Geht das hier überhaupt:

    Quellcode

    1. public boolean einfügen(Buch b){
    3. while(iter.hasNext()){
    5. if(iter.next() == null){


    Hat der Iterator hier überhaupt schon ein Element denn hier füge ich ja erst noch Bücher ein ?

    2. Hauptsächlich habe ich mit der Collection sucheNachAutor probleme :

    Quellcode

    1. this.autor=new ArrayList();
    2. for(Buch b: bib ){
    4. if(iter.next().autor == autor){
    6. this.autor.add(iter);
    7. return this.autor;
    8. }
    9. }return null;


    Denn hier möchte ich ja folgendermaßen vorgehen ich suche nach einem Autor durchlaufe die Liste wenn ich nun ein Buch mit dem Autor gefunden habe füge ich dieses zur Collection ArrayList hinzu so nun ist aber die Frage wie sag ich das denn ?? also wie kann ich mit dem iterator auf das Element zugreifen von dem ich gerade Festgestellt habe es hat den gleichen autor ???? hoffe man versteht die frage

    3. habe ich das gleiche Probemle mit der sucheNachISBN denn hier weiß ich auch nicht was ich denn jetzt returnen muss eigentlich das Objekt auf dem der Iterator gerade steht und für das gilt isbn== isbn ????

    4. bie bestandSortiereNach habe ich gar keine Idee deshalb wäre ein lösungsansatz oder ähnliches sehr hilfreich ????

    Ich danke schon mal im Vorraus für eure Hilfe !!!

    Mit freundlichen Grüßen

  • the_old_Pirate schrieb:


    So nun zu meinen Fragen:
    1. Geht das hier überhaupt:

    Quellcode

    1. public boolean einfügen(Buch b){
    3. while(iter.hasNext()){
    5. if(iter.next() == null){


    Nein, geht nicht. Die ArrayList hat eine Funktion Add, die solltest du nehmen, oder eine eigene verkettete Liste bauen.

    the_old_Pirate schrieb:


    2. Hauptsächlich habe ich mit der Collection sucheNachAutor probleme :

    Quellcode

    1. this.autor=new ArrayList();
    2. for(Buch b: bib ){
    4. if(iter.next().autor == autor){
    6. this.autor.add(iter);
    7. return this.autor;
    8. }
    9. }return null;


    Denn hier möchte ich ja folgendermaßen vorgehen ich suche nach einem Autor durchlaufe die Liste wenn ich nun ein Buch mit dem Autor gefunden habe füge ich dieses zur Collection ArrayList hinzu so nun ist aber die Frage wie sag ich das denn ?? also wie kann ich mit dem iterator auf das Element zugreifen von dem ich gerade Festgestellt habe es hat den gleichen autor ???? hoffe man versteht die frage

    Auch hier benötigst du den Iterator nicht, mach eine for-Schleife und benutze die get(index i)-Funktion der Arraylist. Oder einfach nur das foreach und guck dann mit b.autor. Evtl musst du es von protected auf public ändern, oder Kapseln.

    the_old_Pirate schrieb:


    4. bie bestandSortiereNach habe ich gar keine Idee deshalb wäre ein lösungsansatz oder ähnliches sehr hilfreich ????

    Da gibt es sehr viele tolle Möglichkeiten, mein Favorit wäre hier ein InsertSort, da du die Liste eh duplizieren musst. Vielleicht kann man das Suchen sogar als Strategiepattern mit mehreren Suchalgorithmen implementieren.
    ~ mfg SeBa

    Ich beantworte keine PMs zu Computer-/Programmierproblemen. Bitte wendet euch an das entsprechende Forum.

    [Blockierte Grafik: http://i.creativecommons.org/l/by-sa/3.0/80x15.png]

  • Danke Seba für deine Antwort!

    Allerdings komm ich immernoch nicht so recht mit der Collection zurecht:

    also habe jetzt die LinkedList in eine Arraylist geändert

    Quellcode

    1. ArrayList <Buch> bib;

    so nun habe ich auch meine einfügen ohne iterator gemacht :

    Quellcode

    1. public boolean einfügen(Buch b){
    3. for (Buch a: bib){
    4. System.out.println("hier ist die Schleife");
    5. if(bib.isEmpty()){
    6. System.out.println("Ich bin hier");
    7. bib.add(b);
    8. return true;
    9. }
    11. }return false;
    13. }
    Alles anzeigen

    so nun hab ich mal diese Ausgaben eingefügt um zu testen ob er überhaupt was tut dabei ist mir aufgefallen das er nicht mal in die foreach reinläuft blos warum ??? finde einfach den Fehler nicht
    außerdem ist mir auf gefallen das das ding mit isEmpty() auch nicht gehen kann ich bin nämlich davon ausgegangen das er die Collection durchläuft und an jedem platz schaut ob dieser voll ist wenn nicht fügt er das buch ein => also hab ich wohl die Arraylist nicht richtig verstanden ;) denn er fügt ja immer das Element an das Ende der Liste und vergrößert sie wenn nötig habe also nun das ganze zeugs weggelassen und einfach folgendes gemacht:

    Quellcode

    1. public boolean einfügen(Buch b){
    2. bib.add(b);
    3. return true;
    4. }

    nun ist nur noch die frage wieso ist in der Aufgabe vorgegeben das diese methode einfügen den rückgabewert boolean hat weil das einfügen sollte doch immer klappen oder nicht ? und auch würde mich trotzdem interessieren warum er nicht in die Schleife läuft also es ist hier nicht nötig aber ich würde es trotzdem gerne wissen ???

    auch wenn die einfügen nun funktioniert tut es die sucheNachautor noch nicht ;)
    die sieht nun folgendermaßen aus :

    Quellcode

    1. public ArrayList<Buch> sucheNachAutor(String autor){
    3. this.suche=new ArrayList();
    4. for(Buch b: bib ){
    5. System.out.println("Hier bin ich in der Schleife");
    6. if(b.autor.equals(autor)){
    7. System.out.println("jetzt bin ich auf der suche");
    8. this.suche.add(b);
    9. return this.suche;
    10. }
    11. }return null;
    13. }
    Alles anzeigen

    HIer ist schlicht und einfach das problem das er nichts findet und einfach null zurückgibt jedoch sollte er eigentlich was finden was ist hier falsch ????

    Dieser Beitrag wurde bereits 1 mal editiert, zuletzt von the_old_Pirate ()

  • the_old_Pirate schrieb:


    Quellcode

    1. public boolean einfügen(Buch b){
    3. for (Buch a: bib){
    4. System.out.println("hier ist die Schleife");
    5. if(bib.isEmpty()){
    6. System.out.println("Ich bin hier");
    7. bib.add(b);
    8. return true;
    9. }
    11. }return false;
    13. }
    Alles anzeigen

    Hier kommt er wahrscheinlich nie in die foreach Schleife rein, da niemals ein Buch in die Liste eingefügt wird. Wenn eine Liste leer ist und man versucht sie per foreach Schleife zu durchlaufen passiert garnichts, da es ja kein Element gibt, was er "rausnehmen" könnte. Daher kommt er bei ner leeren Buchliste nicht in die foreach-Schleife rein. Wenn er aber nicht in die Schleife geht, wird auch nie ein Buch eingefügt, also bleibt die Liste leer. Daher ist auch die if-Abfrage if(bib.isEmpty()) widersprüchlich, da diese innerhalb der Schleife niemals true sein kann, da er sonst garnicht in die Schleife kommen würde.
    Abgesehen davon mag es Java nicht, wenn man innerhalb einer foreach-Schleife das Objekt, durch das man iteriert verändert. (also elemente entfernt/hinzufügt etc.) Die Elemente in der Liste kann man aber selbstverständlich verändern.