ArrayList wird doppelt ausgegeben

  • ArrayList wird doppelt ausgegeben

    Hey Leute,
    bin noch neu und habe folgendes Problem:
    Mein Lehrer hat mir zur Übung fürs mündliche Abi ein Projekt zur Übung gegeben welche ich von einem Projekt mit objektorientierter Datenbank zu einem Projekt mit ArrayList umschreiben soll. Schön und gut, habe mich daran versucht und bissle herumexperimentiert, da wir das Thema schon vor nem halben Jahr hatten und deswegen ich mich auch wieder reintasten musste und hab es am Ende endlich geschafft das alles genauso ausgegeben wird, wie es im alten Projekt war. Nun gut, da dachte ich, ich hätte alles richtig gemacht, bis ich auf das Problem mit meinem Button "Neue Zuordnung"(heißt hier btnLeeren) gestolpert bin, welches meine Spieler plötzlich doppelt in der Combobox anzeigt.
    Ich hoffe mal ihr könnt mir sagen was mein Fehler ist, da ich grade ziemlich auf dem Schlauch stehe und ich meinen Lehrer erst ende der Woche wieder sehe.

    Quellcode

    1. public Hauptfenster()
    2. {
    4. m1 = new Mannschaft();
    5. m2 = new Mannschaft();
    6. aktMannschaft = new Mannschaft();
    7. m1.setBezeichnung("Real Madrid");
    8. m2.setBezeichnung("FC Barcelona");
    9. aktSpieler = new Spieler();
    10. s1 = new Spieler("Messi", "Lionel");
    11. s2 = new Spieler("Ronaldo", "Cristiano");
    12. s3 = new Spieler("Sanchez", "Alexis");
    13. s4 = new Spieler("Casillas", "Iker");
    14. s5 = new Spieler("Özil", "Mesut");
    15. s6 = new Spieler("Iniesta", "Andres");
    16. Spielerliste();
    17. Mannschaftsliste();
    19. btnLeeren.addMouseListener(new MouseAdapter()
    20. {
    21. @Override
    22. public void mouseClicked(MouseEvent e)
    23. {
    24. taAusgabe.removeAll();
    25. Spielerliste();
    26. Mannschaftsliste();
    27. }
    28. });
    29. btnLeeren.setBounds(248, 175, 130, 33);
    31. jContentPane.add(btnLeeren);
    34. private void Spielerliste()
    35. {
    36. String zwischen;
    37. cboSpieler.removeAllItems();
    38. aktMannschaft.setSpielerliste(s1);
    39. aktMannschaft.setSpielerliste(s2);
    40. aktMannschaft.setSpielerliste(s3);
    41. aktMannschaft.setSpielerliste(s4);
    42. aktMannschaft.setSpielerliste(s5);
    43. aktMannschaft.setSpielerliste(s6);
    44. spielerliste = aktMannschaft.getSpielerliste();
    45. for (int i = 0; i < spielerliste.size(); i++)
    46. {
    47. zwischen = spielerliste.get(i).getVorname() + " "
    48. + spielerliste.get(i).getName();
    49. cboSpieler.addItem(zwischen);
    50. }
    51. }
    53. private void Mannschaftsliste()
    54. {
    55. String zwischen;
    56. cboMannschaften.removeAllItems();
    57. aktSpieler.setMannschaftsliste(m1);
    58. aktSpieler.setMannschaftsliste(m2);
    59. mannschaftsliste = aktSpieler.getMannschaftsliste();
    60. for (int i = 0; i < mannschaftsliste.size(); i++)
    61. {
    62. zwischen = mannschaftsliste.get(i).getBezeichnung();
    63. cboMannschaften.addItem(zwischen);
    64. }
    65. }
    67. }
    Alles anzeigen



    Schonmal danke im vorraus für eure Hilfe :)

  • Ja, ich habs in Zeile 16 gemacht, damit die Combobox mit den jeweiligen Objekten aus der ArrayList gefüllt wird und in der Zeile 25, damit ich die Combobox wieder mit den Objekten fülle, da diese mit ner anderen Methode aus der Combobox gelöscht wird.
    Da fällt mir ein, das ich diese Methode vllt hätte oben erwähnen sollte:

    Quellcode

    1. btnZuordnen.addMouseListener(new MouseAdapter()
    2. {
    3. @Override
    4. public void mouseClicked(MouseEvent e)
    5. {
    6. String zwischen;
    7. if(cboSpieler.getItemCount() > 0)
    8. {
    9. zwischen = (String) cboSpieler.getSelectedItem();
    10. cboSpieler.removeItem(cboSpieler.getSelectedItem());
    11. int index = zwischen.indexOf(" ");
    12. aktSpieler1 = new Spieler(null, null);
    13. aktSpieler1.setVorname(zwischen.substring(0, index + 1));
    14. aktSpieler1.setName(zwischen.substring(index + 1,
    15. zwischen.length()));
    16. spielerliste = m1.getSpielerliste();
    17. spielerliste = m2.getSpielerliste();
    18. if(cboMannschaften.getSelectedItem().equals("Real Madrid"))
    19. {
    20. m1.setBezeichnung("Real Madrid");
    22. m1.setSpielerliste(aktSpieler1);
    23. taAusgabe.add(aktSpieler1.getVorname() + " "
    24. + aktSpieler1.getName() + " gehört zu: "
    25. + m1.getBezeichnung());
    27. }
    28. else
    29. {
    30. m2.setBezeichnung("FC Barcelona");
    31. m2.setSpielerliste(aktSpieler);
    32. taAusgabe.add(aktSpieler1.getVorname() + " "
    33. + aktSpieler1.getName() + " gehört zu: "
    34. + m2.getBezeichnung());
    35. }
    36. }
    37. else
    38. {
    39. JOptionPane.showMessageDialog(null,
    40. "Alle Spieler sind verteilt");
    41. }
    42. }
    43. });
    44. btnZuordnen.setBounds(248, 131, 130, 33);
    46. jContentPane.add(btnZuordnen);
    47. btnLeeren.addMouseListener(new MouseAdapter()
    48. {
    49. @Override
    50. public void mouseClicked(MouseEvent e)
    51. {
    52. taAusgabe.removeAll();
    53. // Spielerliste();
    54. // Mannschaftsliste();
    55. }
    56. });
    Alles anzeigen


    Hab mir iwie gedacht, dass ich die Objekte vllt aus der Arraylist erst lösche und dann mit der jeweiligen Methode dann wieder schreibe, aber bin mir nicht so ganz sicher und ich ich kenn auch nicht den Befehl um diese Idee zu testen.


    und @shureg
    was meinst du mit "ausserdem kann ich dir empfehlen man den debugger zu bemühen, das gehört auch zur programmiereung und fehlersuche "?