Swing Duplikate im Jcombobox vermeiden

[nag]

Hallo
ich habe ein Textdatei mit diesem Format team1;team2;resutlt ausgelesen und mit split getrennt und in array gespeichert. jetzt möchte ich die team1 imJcomboBox speichern. da team 1 team behalten mit demselben Name, bekomme ich im Jcombobox zwei mal derselbe Team. weiß jemanden wie ich diese vermeiden kann. hier ist meine Code :

for (int i = 0; i < max_games; i++) {
                    String [] parts = I[I].split(";");
                   round.add(i, parts[0].trim());
                    home_team.add(i, parts[1].trim());
                    foreign_team.add(i, parts[2].trim());
                    result.add(i, parts[3].trim());
                    String list = parts[1];
                    teamList.addItem(list);

danke im voraus[/I]

 

[Robat]

Du könntest mit einer if-Abfrage prüfen ob das Team schon in der ComboxBox vorhanden ist:

for(int i = 0; i < max_games; i++)
{
   ...
   if(teamList.getIndexOf(list) == -1)
   {
        teamList.addItem(list);
   }
}

Erklärung:
Die Methode getIndexOf(String item); liefert -1 wenn der Parameter nicht in der Liste vorhanden ist.

Gruß
Robert

 

[Robat]

Du könntest mit einer if-Abfrage prüfen ob das Team schon in der ComboxBox vorhanden ist:

for(int i = 0; i < max_games; i++)
{
   ...
   if(teamList.getIndexOf(list) == -1)
   {
        teamList.addItem(list);
   }
}

Erklärung:
Die Methode getIndexOf(String item); liefert -1 wenn der Parameter nicht in der Liste vorhanden ist.

Gruß
Robert

 

[nag]

Danke Robat aber das fonktioniert nicht.

 

[Robat]

Sorry hab den wichtigsten Teil wieder als selbstverständlich angesehen.

Die Methode getIndexOf() kann man natürlich nur auf ein ComboBoxModel anwenden.
Du kannst daher also ein DefaultComboxModel verwenden und deine Elemente dort hinzufügen.
Das könnte so aussehen:

JFrame frame = new JFrame();
frame.setSize(500,500);
frame.setResizable(false);
frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);

DefaultComboBoxModel<String> model = new DefaultComboBoxModel<>();
JComboBox<String> combobox = new JComboBox<>(model);

final String[] content = { "Foo", "Foobar", "Foo1", "Foo", "Foobar", "Foo1" }; // wie du siehst gibt es hier doppelte Einträge

for(String name : content)
{
    if(model.getIndexOf(name) == -1)
        model.addElement(name);
}

frame.add(combobox);

frame.setVisible(true);

Gruß
Robert

 

[Enceladus271]

Du kannst auch einfach die Strings die du der ComboBox hinzufügen möchstest erst zu einem TreeSet hinzufügen. In diesem werden Duplikate nicht aufgenommen. Erst danach fügst du die Elemente des TreeSets der ComboBox hinzu. Hat hat auch den schönen Nebeneffekt, dass durch das TreeSet die Einträge alle sortiert sind (falls das nicht gewünscht ist ein HashSet nehmen).

 

[nag]

vielleicht soll ich deutlich meine idee erklären. ich habe in einem zipdatei ordner mit nummer von 2006 bis 2015 und in jeweiligen ordner textdatei 19 bis 34.txt mit inhalt round; team1;team2; result.Das Auslesen derTextdatei funktioniert. jetzt möchte ich wenn der benutzer im JTextField 2006 und 19 gibt sollt ich Jcombobox die Team angezeigt werden damit der Benutzer ein Team auswählen kann und mit weitere andere Button die Ergebnis berechnen. ich habe jetzt angefangen zu programmieren insofern bin nicht so fit im java

 

[nag]

Enceladus 271 ich habe mit Hashset probiert aber ich habe immer noch doppelte einträgen. hier ist die Code. was mache ich falsch
String list = parts[1];
HashSet set = new HashSet();
set.add(list);
teamList.addItem(set);

 

[Robat]

ich habe in einem zipdatei ordner mit nummer von 2006 bis 2015 und in jeweiligen ordner textdatei 19 bis 34.txt mit inhalt round; team1;team2; result.

Sorry aber ich verstehe kein Wort

Das Auslesen derTextdatei funktioniert

Hier speicherst du doch die Daten bestimmt in ein Array / eine ArrayList - was auch immer.

Dann kannst du doch auch wunderbar mit einem Model arbeiten - wie oben beschrieben. Wenn das nicht funktioniert dann poste bitte eine genauere Fehlerbeschreibung
als:

das fonktioniert nicht.

-----

was mache ich falsch
String list = parts[1];
HashSet set = new HashSet();
set.add(list);
teamList.addItem(set);

Du fügst ja auch nur 1 Item hinzu?

PS:
Poste deinen Code bitte in [code=Java]MY CODE[/code][plain] Tags.

Gruß

 

[nag]

Robat hier mein Vorgang. ich habe eine class Reading der die Zip datei ausliest

package infpp.meisterschaftsproblem;

import java.io.FileInputStream;
import java.util.Scanner;
import java.util.zip.ZipEntry;
import java.util.zip.ZipInputStream;

public class Reading {
    public String[] read(String season, String matchday, String[] I) {
        try {
            FileInputStream fis = new FileInputStream("E:/Projektgruppe2017/MyprojektOR/Bundesliga.zip");
            ZipInputStream zis = new ZipInputStream(fis);
            StringBuilder s = new StringBuilder();
            ZipEntry ze;
            int i = 0;
            while ((ze = zis.getNextEntry()) != null) {
                if (ze.getName().contains(season + "/" + matchday + ".txt")) {
                    Scanner sc = new Scanner(zis);
                    while (sc.hasNextLine()) {
                        I[i] = sc.nextLine().toString();
                        i++;
                    }

                }
                zis.closeEntry();
            }
            System.out.println(s);
            zis.close();
        } catch (Exception e) {
        }

        return I;
    }
}

und jetzt ich rufe in Gui diese classe auf

 int s0= Integer.parseInt(saison.getText());
                  int m0= Integer.parseInt(spieltag.getText());
                  String s1 = String.valueOf(s0);
                  String m1 = String.valueOf(m0);
               
          
                int max_games = 306;
                String[] I = new String[max_games];
                Reading r = new Reading();
                I = r.read(s1, m1, I);
for (int i = 0; i < max_games; i++) {
                    String [] parts = I[i].split(";");
String list = parts[1];
                    HashSet set = new HashSet();
                     set.add(list);
                    teamList.addItem(set);
                  
                
                  
                  
                }

ich greife mit parts[1] nur der zweite zeile im Text datei round;team1;team2;result zu oder. im textdatei 19.txt sind zum beispiel solche Eintrag 1;Hamburg ; Bayen München;U
2; VfL Bochum VfB ;Stuttgart N

 

[Robat]

teamList.addItem(set);

Wenn du deine JComboBox "ordentlich" initialisiert hast, dann dürfte das hier nicht gehen.
Hast du deiner ComboBox mittels Generics den Typ der Elemente die sie enthält mitgegeben??

Was du machen kannst wäre folgendes: (ist allerdings nicht die schönste Lösung..)

HashSet set = new HashSet();
for(int i = 0; i < max_games; i++)
{
    String[] parts = I[i].split(";");
    String list = parts[1];
    set.add(list);
}

for(String entry : set)
{
    teamList.addItem(entry);
}

Kurz zur Erklärung:

- Du fügst deiner HashSet erst die Einträge hinzu, die du ausgelesen hast. Eine HashSet vermeidet doppelte Einträge - daher funktioniert das.
- Dann durchläufst du alle Einträge der HashSet und fügst sie deiner JComboBox hinzu.


Off-Topic mal noch was zu deinem Code:
- Verwende ordentliche Variablennamen. I sagt nichts aus und ist für ein Array ganz schlecht gewählt
- Halt dich an die Java-Namenskonventionen. Klassennamen UpperCammelCase, den rest lowerCammelCase. Konstanten werden GROSS geschrieben.

 

[nag]

danke Robat für den Typ aber ich habe einen Fehler bei der for schleife. for(String entry :set) .zeigt Type mismatch

 

[nag]

hat das jetzt funktioniert ich habe
Set<String> set = new HashSet<String>(); benutzt. vielen dank Robert

 

[nag]

und noch eine Frage gibt es ein ActionListener für JTextField ?. ich habe zwei JTextField einmal saison und Spieltag. ich möchte dass,wenn der Benutzer im Feld saison 2006 und im Feld Spieltag 19 gibt. die Team in diese datei im Jcombobox automatique angezeigt werden. hat jemand eine idee wie ich diese implementieren kann.

 

[Robat]

Da würde ich dir eigentlich zu einem DocumentListener raten. Dieser hat 3 Methode changedUpdate() - wird ausgeführt sobald du was in deine Tastatur reinhackst - removeUpdate() - wenn du was löschst - insertUpdate wenn du mittels Copy&Paste etwas reinkopierst.

Du müsstest also nach bestimmten Begriffen suchen.

 

[Neumi5694]

Falls das Auslesen einmalig passiert, dann verwende

setModel(new DefaultComboBoxModel(allItems))

anstatt mehrerer Aufrufe von

addItem(oneItem)

. Wenn ich mich recht erinnere, löst

addItem

beim ersten Aufruf auch einen Event aus (da das erste Element gleich auch gesetzt wird, falls die Box vorher leer war), was recht unpraktisch ist, wenn die Liste noch nicht vollständig ist.

Für das Model verwende eine eindeutige Liste, der Weg über das Set ist der einfachste.
Ich hab mir ein ComboBox-Model geschrieben, das jede Collection fassen kann, aber mit dem Standardmodell (Array oder Vector) kommst du natürlich auch gut weiter